Programmierstil: Meinung/Verbesserungen?

[Kai345]

[latex]Mae  govannen![/latex]

if ($this->xmlobj) {

Wenn ich sowas sehe (und das ist nur einer von vielen ähnlichen Konstrukten), denke ich zuerst, du willst da was boolsches auswerten. Es ist nicht ersichtlich, dass du stattdessen auf Leerstring mit der Bedeutung "noch nicht initialisiert" prüfst. Nimm doch lieber null und mach die Prüfung mit is_null() oder === null.

<frust>
Ich habe meinen Code gerade speziell auf diese Art von Vergleichen hin abgeklopft und bin nicht unbedingt glücklich mit der durch eine spezifische Auswertung entstehenden Verwirrung. In dem obigen speziellen Fall habe ich nun null verwendet, aber teilweise komme ich mit den verschiedenen Rückgabearten nicht zurecht (was deren Sinn angeht). <grummel>Wieso zum Geier gibt z.B. simplexml_load_file() beim Mißerfolg false zurück? Es ist ein Objekt, da hat verdammt nochmal die Rückgabe null zu sein, wenn das Objekt nicht erzeugt werden konnte und nicht false. Das ist total widersinnig.</grummel> Es widerstrebt mir regelrecht, die Initialisierung mit private $xmlobj = false; durchzuführen, nur weil keine sinnvollen Rückgabewerte verwendet werden. Ich möchte jetzt eigentlich gar nicht mehr wissen, wie das bei anderen Funktionen aussieht...
</frust>

Cü,

Kai