Camping_RIDER: Problem mit Rückgabewert

Beitrag lesen

Aloha ;)

Ich würde statt dieser Zeile gleich true zurückgeben:
if ...
else if ...
else return true;

return false; // wenn nicht return true

Oder auch gleich:

  
If (Fehler1) {  
    echo 'Fehler1'; return false;  
}  
  
If (Fehler2) {  
    Echo 'Fehler2'; return false;  
}  
  
...  
  
return true;  

Das erfüllt denselben Zweck (aufgrund der else-if's wird auch beim ursprünglichen Code maximal eine Fehlermeldung ausgegeben), vermeidet aber diese unglaublich riesige if-else-if-Struktur und ist dadurch lesbarer und weniger Fehleranfällig. Sollen alle Fehler ausgegeben werden würde ich sowas empfehlen:

  
$ok = true;  
  
If (Fehler1) {  
    echo 'Fehler1'; $ok = false;  
}  
  
If (Fehler2) {  
    Echo 'Fehler2'; $ok = false;  
}  
  
...  
  
return $ok;  

Auch übersichtlicher als die ursprüngliche Variante ;)

Grüße,

RIDER

--
Camping_RIDER a.k.a. Riders Flame a.k.a. Janosch Zoller
ch:? rl:| br:> n4:? ie:% mo:| va:) js:) de:> zu:) fl:( ss:| ls:[