Return wird nicht abgearbeitet

[asscareth]

Hallo!

Aufgabe ist hier herauszufinden ob ein Dateiname bereits vorhanden ist und wenn er das ist, dann soll der schon bestehende Dateiname verändert werden und die neue Datei unter dem veränderten Namen gespeichert werden. Die Methode soll so lange aufgerufen werden und der Dateiname verändert werden, bis ein Dateiname entstanden ist unter dem garantiert noch keine Datei gespeichert wurde.

Mein Problem ist dabei das ich am Punkt "testausgabe: " absolut keinen Return-Wert bekomme. In der Zeile return "testausgabe: " . $ouput also wird nichts zurückgegeben, was mich glauben lässt das return nicht aufgerufen wird. Wie man erkennen kann habe ich auch bereits versucht die Rückgabe an anderer Stelle einzubauen, also ganz am Ende der Methode. Aber auch das hat nicht funktioniert. Was mich dabei total verwirrt ist die Tatsache das wenn ich einen Dateinamen wähle der nocht nicht vorhanden ist, return $output normal funktioniert und der noch nicht vorhandene Dateiname ausgegeben wird. .. also im else-Teil, beim if-Teil gehts nicht :-( ). Ich bin mir aber absolut sicher das der if-Teil abgearbeitet wird, da die echo-anweisung direkt vor return "testausgabe: " . $output; zum richtigen Zeitpunkt ausgegeben wird.

Kann mir dabei bitte jemand helfen?

class file_check {
       var $index = 0;

      function file_c( $filename ) {
            if( file_exists( $filename ) ){
             $this->index++;
             $chunks = explode(".", $filename );
            $chunksnum = count( $chunks ) - 1;
                if( substr_count( $filename, "_") ){
               $filenr = "_" . substr( $chunks[ $chunksnum - 1 ], strrpos( $chunks[ $chunksnum - 1 ], "_") + 1 ); $filenr++;
               $prefile = "." . substr( $chunks[ $chunksnum - 1 ], 0, strrpos( $chunks[ $chunksnum - 1 ], "_"));
               $postfile = "." . $chunks[ $chunksnum ];
               $this->file_c( $prefile.$filenr.$postfile );
            } else {
               $filenr = "_" . $this->index;
               $postfile = "." . $chunks[ $chunksnum ];
               unset( $chunks[ $chunksnum ] );
               $prefile = "." . implode( $chunks );
               $this->file_c( $prefile.$filenr.$postfile );
            }
         } else {
             if( $this->index > 0 ){
                echo "<br />testausgabe: " . $filename;
                $output = $filename;
               return "output: " . $output;
            } else {
                $output = $filename;
               return $output;
            }
         }
         //echo $output;
// return "output: " . $output;
      }
   }
   $test = new file_check;
       $valid_file = $test->file_c( $_SESSION['uploaddir'] . $_FILES['newPic']['name'] );
        echo "<br /><br />valid_file: " . $valid_file; 

[mrhappiness]

Re: Return wird nicht abgearbeitet

Original geschrieben von asscareth
beim if-Teil gehts nicht

Da steht ja auch kein return

[asscareth]

ich sehe da 2 return-anweisungen und eine auskommentierte. ich habe es mit der auskommentierten bereits versucht. das funktionierte leider nicht. und es funktioniert leider nur die returnanweisung im else-teil der if-anweisung. mag sein das ich nicht verstanden habe was du meinst. wäre nett von dir wenn du den code-teil, den du meinst noch einmal quoten könntest. ich jedenfalls meine diesen hier:

         } else {
             if( $this->index > 0 ){
                echo "<br />testausgabe: " . $filename;
                $output = $filename;
               return "output: " . $output;
            } else {
                $output = $filename;
               return $output;
            }
         }
         //echo $output;
// return "output: " . $output; 

[onemorenerd]

Schau dir die Struktur der Funktion doch mal an!

function file_c() {
  if ( bedingung1 ) {
    // kein return hier
    if ( bedingung2 ){
      // auch hier KEIN RETURN
    } else {
      // hier auch nicht
    }
  } else {
    if( bedingung3 ){
      return "output: " . $output;
    } else {
      return $output;
    }
  }
  // return "output: " . $output;
} 

Man erreicht kein return, wenn bedingung1 = true. Genau das wird bei dir passieren.

Und nun komm mal ausm Mustopp und mach paar Debugging-Ausgaben!

[TobiaZ]

@asscareth: Bei nem 19" TFT habe ich einen Scrollbalken. Bitte die Regeln beachten!

[asscareth]

so passt mein post auf einen 19" er.

[TobiaZ]

Original geschrieben von asscareth
so passt mein post auf einen 19" er.

Na, dann hast du die Regel aber nicht richtig gelesen, oder? Da ist die Rede von einer 1024er auflösung! Bitte fürs nächste Thema berücksichtigen...