Severin Kacianka: Problem mig Funktion

Beitrag lesen

Hallo,

if ($this->showold == "true")

Das sieht schon mal, nicht gut aus. true oder false sind boolsche Werte und entsprechen 0 und 1. Darum reicht es völlig wenn du schreibst:
$foo = true;
if($foo){
   //aktion
}
Du kannst auch if($foo == true) schreiben, aber das ist völlig unnötig.

Als nächstes solltest du  auch noch einmal deine Klammernsetzung überprüfen, es gehen drei auf aber nur zwei zu. Wenn du dir damit schwer tust, dann setz die Klammern gleich am Anfang. Zum Beispiel:
if(){
}
dann weiter:
if($foo){
    //aktion
}
Und du solltest deine Klammern auch schön verschachteln:
if($foo){
   while(true){
        //aktion
   }
}
Solltest du dennoch Probleme haben, kannst du auch immer Kommentare dazu schreiben:
if($foo){
   while(true){
        //aktionen
   }//ende while Schleife
}//ende if Bedingung

Solltest du weiter Probleme haben, dann frag einfach nach.

gruß,
Severin

--
Realität ist das, was nicht verschwindet, wenn man aufhört, daran zu glauben.
--Philip K. Dick