[PHP5] Rückgabewert von Methoden

[Poscht]

Hallo, ich mal wieder

Wie würdet ihr in dem folgenden Fall verfahren?
Ich habe folgende Beispiel-Klasse

class test
{
 function test()
 {
   if($irgendeine_aktion==TRUE)
   {
        return $ergebniss_array;
   }
   else
   {
        return FALSE;
   }
 }
} 

Diese prüfe ich dann beim Aufruf wie folgt.

$t = new test;
if($t->test())
{
  // mach was
} 

Problem hierbei ist, dass mal ein Array und mal ein Boolean zurückgegeben wird und das nicht ganz sauber ist ...
(Jedenfalls bemängelt das das Zend Studio)
Wie seht ihr dies bzw. würdet ihr das ändern? Würdet ihr überhaupt etwas ändern?

Poscht

[jahlives]

Quick and Dirty

class test
{
 function test()
 {
   if($irgendeine_aktion==TRUE)
   {
        return $ergebniss_array;
   }
   else
   {
        return array(FALSE);
   }
 }
} 

Gruss

tobi

[unset]

Ich wüsste nicht, was daran unsauber sein soll. Eine ganze Reihe von PHP-Funktionen gibt auch false oder beispielsweise eine Ressource zurück.

[H2O]

PHP selber hat genug Funktionen, die im Fehlerfall false und sonst einen andern Wert zurück geben. Aber jahlives ist sicher schöner

[PHP-Desaster]

Wenn der eigentliche Rückgabewert ein array, würde ich im Fehlerfall null zurückgeben. So wäre es auch in anderen Sprachen! false halte ich durchaus für unsauber (auch bei den php-Funktionen!)

[combie]

Neee.... Das ist ein ganz übles Beispiel!!!

Seit wann haben Konstruktoren einen Rückgabewert?
Und wenn, dann bestenfalls eine Referenz auf eine Objektinstance!

[PHP-Desaster]

Neee.... Das ist ein ganz übles Beispiel!!!

Seit wann haben Konstruktoren einen Rückgabewert?
Und wenn, dann bestenfalls eine Referenz auf eine Objektinstance!

Glaube eher, das war nicht beabsichtigt. Hat ja von uns auch keiner als Konstruktor gesehen gehabt, weil es aus der Fragestellung heraus eigentlich klar war, das es sich um eine Methode handeln soll!

[combie]

Hat ja von uns auch keiner als Konstruktor gesehen gehabt,

Und genau darum habe ich es angesprochen. Ich möchte auch niemandem Absicht unterstellen. Allerdings denke ich schon, dass man nur anhand von testbarem Code solche Angelegenheiten besprechen kann.

(Jedenfalls bemängelt das das Zend Studio)

Das zeugt auf jeden Fall schon mal von dem Versuch es laufen zu lassen.

Man sollte den TE nicht in der Falle sitzen lassen.

[Poscht]

Ok, das Beispiel war nicht ganz sauber.
Die Methode test ist dummerweise auch der Konstruktor.
Es war nur eine simple Methode aus der Klasse gemeint.
Der Konstruktor wird ja auch eher über __construct definiert.

Die Klasse, in der ich das Ergebniss in der Zend-Umgebung hatte, ist ziemlich umfangreich und als Beispiel nicht wirklich geeignet.

Poscht

[unset]

Original geschrieben von Poscht
Die Klasse, in der ich das Ergebniss in der Zend-Umgebung hatte, ist ziemlich umfangreich und als Beispiel nicht wirklich geeignet.

Es reicht ja, wenn du die Methode postest.

[Poscht]

Original geschrieben von unset
Es reicht ja, wenn du die Methode postest.

Die Methode macht doch auch nichts weiter als entweder ein Array oder FALSE zurückzugeben.
Was in der Methode selbst passiert ist doch auch für die Frage nicht wirklich relevant oder?

Meine Frage ist eigentlich mit den ganzen Antwortpostings beantwortet, da fast alle die Rückgabe eines leeren Arrays der Rückgabe von FALSE vorziehen würden.

Vielen Dank an alle,
Poscht

[unset]

Ich würde false allerdings klein schreiben. Um das letzte Wort zu haben ...

[combie]

Obwohl einige gewichtige Gründe dagegensprechen, schreibe ich auch immer FALSE und TRUE.
Das schiebt sie in die Nähe von Konstanten.

[PHP-Desaster]

Meine Frage ist eigentlich mit den ganzen Antwortpostings beantwortet, da fast alle die Rückgabe eines leeren Arrays der Rückgabe von FALSE vorziehen würden.

Soweit ich das sehe gab es nur die Vorschläge false, array(false) und null...

[combie]

Mein Vorschlag:

Alter Stil:
Funktion OK und liefert Daten -->> gefülltes Array
Funktion OK und hat keine Daten -->> leeres Array
Funktion undurchführbar -->> FALSE und wirft Warning(error)

Neuer Stil:
Funktion OK und liefert Daten -->> gefülltes Array
Funktion OK und hat keine Daten -->> leeres Array
Funktion undurchführbar -->> wirft Exeption