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:[
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:[