help rewrite code

[mrbell]

This is the first function I've ever written, but I don't understand how to call it later on...

function clipfilename()
{
$pos1 = strpos(basename($SERVER['PHP_SELF']), '-');
$pos2 = strpos(basename($SERVER['PHP_SELF']), '');
if($pos2===false) {
$clip = substr(basename($SERVER['PHP_SELF']),3,-4);
print $clip;
}
else {
$clip = substr(basename($_SERVER['PHP_SELF']),3,-3+$pos2);
print $clip;
}
}
$uselater = $clip;

I want my variable $uselater to be one of two $clip variables from my function, but I don't understand how to call it. This is where I want to call it on a different page...

$links=array("07-".$uselater ."selecting".".php" =>"Selecting",
"07-".$uselater ."listening".".php" =>"Listening",
"07-".$uselater ."_history".".php" =>"History"
);

I want to end up with a dynamic menu of links. If this makes any sense, please help me rewrite this code. Thanks!!

[eoghain]

Ok here is what you need to do. First your function needs to be accessable to the code that you want to use it, this means it either has to be in the same file, or has to be in a file that is included. I'm just going to put it in the same file to show you how to use it. Second you don't want to print out your $clip variable from your function you want to return it.

<?php
   function clipfilename() 
   { 
      $pos1 = strpos(basename($_SERVER['PHP_SELF']), '-'); 
      $pos2 = strpos(basename($_SERVER['PHP_SELF']), '_'); 
      if($pos2===false) { 
         $clip = substr(basename($_SERVER['PHP_SELF']),3,-4); 
      } else { 
         $clip = substr(basename($_SERVER['PHP_SELF']), 3, -3+$pos2); 
      } 
      return $clip; 
   } 

   $uselater = clipfilename();

   $links=array("07-".$uselater ."_selecting".".php" =>"Selecting",
                "07-".$uselater ."_listening".".php" =>"Listening",
                "07-".$uselater ."_history".".php" =>"History");
?>

So to call your function you just type the name with () after it, just like you'd call any builtin PHP functions. And since you've told your function to return $clip you will want to catch that and put it in a variable, which you've called $uselater. Now you can use that variable wherever you want and it will hold the information from your function.

[mrbell]

😃 Thanks for your help. I really appreciate it - works like a charm!