Fonction PHP ne renvoyant pas la chaîne

Question

J'ai construit une fonction qui prend un nom de fichier, et incrémente un compteur dans le nom de fichier et le renvoie, cependant, tout est correct, sauf que le retour ne retourne pas le nom du fichier.

Toute aide, s'il vous plaît?

Mon code:

$filename = join("", array_reverse($date)); 
$filename .= ".xml"; 
$dir = "../gigs"; 
$file = $dir."/".$filename; 

function getNewFileName($filename, $dir) { 
if (is_file("$dir/$filename")) { 
    if (strpos($filename, "_") === false) { 
     $filename = str_replace(".xml","_1.xml",$filename); 
     getNewFileName($filename, $dir); 
    } 
    else { 
      $pos = strpos($filename, "_"); 
      $counter = (int)substr($filename, $pos+1,1); 
      $counter++; 
      $filename = substr($filename,0, $pos)."_".$counter.".xml"; 
      getNewFileName($filename, $dir); 
     } 
    } else { 
       // echoing HERE shows that the string is manipulated correctly 
     return (string)$filename; // but returning here is not working 
    } 
} 

echo getNewFileName($filename, $dir); // <- this last line prints nothing out 

Merci à l'avance.

Answer 1

La ligne:

getNewFileName($filename, $dir); 

a besoin d'un return:

return getNewFileName($filename, $dir); 

Answer 2

C'est ce que votre fonction devrait ressembler à:

function getNewFileName($filename, $dir) { 
    if (is_file("$dir/$filename")) { 
     if (strpos($filename, "_") === false) { 
      $filename = str_replace(".xml","_1.xml",$filename); 
      return getNewFileName($filename, $dir); 
     } 
     else { 
       $pos = strpos($filename, "_"); 
       $counter = (int)substr($filename, $pos+1,1); 
       $counter++; 
       $filename = substr($filename,0, $pos)."_".$counter.".xml"; 
       return getNewFileName($filename, $dir); 
     } 
    } 
    return (string)$filename; 
} 

echo getNewFileName($filename, $dir); // <- this last line prints nothing out 

Answer 3

Tout d'abord, s'il vous plaît essayer de formater votre code pour que les tirets sont lisibles. Deuxièmement, vous êtes tout simplement pas revenir des appels récursifs à getNewFileName():

function getNewFileName($filename, $dir) { 
    if (is_file("$dir/$filename")) { 
    if (strpos($filename, "_") === false) { 
     $filename = str_replace(".xml","_1.xml",$filename); 
     return getNewFileName($filename, $dir); // here 
    } else { 
     $pos = strpos($filename, "_"); 
     $counter = (int)substr($filename, $pos+1,1); 
     $counter++; 
     $filename = substr($filename,0, $pos)."_".$counter.".xml"; 
     return getNewFileName($filename, $dir); // and here 
    } 
    } else { 
    return (string)$filename; 
    } 
} 

en supposant que est votre intention.

Answer 4

function getNewFileName($filename, $dir) { 
    if (is_file("$dir/$filename")) { 
    if (strpos($filename, "_") === false) { 
     $filename = str_replace(".xml","_1.xml",$filename); 
     return (string)$filename; 
    } else { 
     $pos = strpos($filename, "_"); 
     $counter = (int)substr($filename, $pos+1,1); 
     $counter++; 
     $filename = substr($filename,0, $pos)."_".$counter.".xml"; 
     return (string)$filename; 
    } 
    } else { 
    return (string)$filename; 
    } 
} 

Votre fonction avait une boucle à l'infini.