php Rückgabe gemischter Datentypen - gut oder schlecht

Question

Wenn Sie Werte in PHP zurückgeben, wird es als gute oder schlechte Methode angesehen, gemischte Datentypen zurückzugeben. Ich arbeite an einem Projekt, wo ich ständig mit Methoden konfrontiert werde, die eine ID-Nummer oder null zurückgeben. Ich handle den null Wert, indem ich nach null überprüfe und -1 zurückgebe, wenn es null ist.

Eine andere Situation, in der ich mich oft befinde, ist, wo eine Methode etwas tun und eine Zeichenfolge zurückgeben soll. Manchmal ist es jedoch nicht möglich, die Zeichenfolge zurückzugeben, da sie nicht gefunden wurde oder eine Ausnahme aufgetreten ist. Was ist das Beste, was du hier machen kannst? Geben Sie eine Zeichenfolge wie "fehlgeschlagen" oder etwas zurück? Dies erzeugt dann eine String-Kopplung zwischen Methoden, denke ich, da die aufrufende Methode genau die String-Fehler-Nachricht kennen muss, um sie zu überprüfen.

EDIT: OK, es gibt schon ein paar verschiedene Meinungen. Ich mag die Idee, bei einem Fehler und dem tatsächlichen Ergebnis, unabhängig vom Datentyp, auf Erfolg zurückzukommen. Aber ... gibt es eine de facto Best Practice, wenn es darum geht? Ich meine, was machen Programmierer in anderen Sprachen, d. H. Java und C++ usw. in diesen Situationen?

Answer 1

Was ich normalerweise tun, ist, wenn die Methode funktioniert, den Wert zurückgeben, und wenn es fehlgeschlagen ist, zurückgeben FALSE. Das ist es, was viele von PHP eingebaute Methoden tun. Also, dann können Sie einfach überprüfen, ob die Funktion FALSE zurückgegeben hat oder nicht.

Answer 2

Ich denke, es ist eine schlechte Praxis, gemischte Datentypen zurückzugeben. Es ist möglich, wie Sie bereits gesagt haben, aber denken Sie an die Lesbarkeit und Wartbarkeit Ihres Codes. Stellen Sie sicher, dass Sie kommentieren, was Sie zurückgeben und warum, ich denke, dass das am wichtigsten sein wird. Wenn Sie ein int erwarten und -1 anstelle von null zurückgeben, kommentieren Sie das, damit Sie (oder jemand anderes) nicht verrückt wird, um herauszufinden, was Sie zu tun versuchten.

Answer 3

Null ist ein ziemlich häufiger Rückgabewert, der anzeigt, dass kein Rückgabewert vorhanden ist. Sie sollten null (nicht "fehlgeschlagen" oder -1) zurückgeben, wenn die Funktion überhaupt keine ID zurückgeben möchte.

Wenn es Ausnahme ist, dass eine ID nicht gefunden wurde, sollten Sie eine Ausnahme auslösen.

Answer 4

Eine Funktion, die gemischte Werte zurückgibt, wird nicht als schlecht angesehen. In der Tat ist dies die Schönheit von PHP, da es eine dynamische Sprache ist. Wenn also die Funktion korrekt ausgeführt wird, gibt false bei Fehler zurück .

if(false == ($data = do_something())) return false; 
else print_r($data); 

Answer 5

Ich stimme den obigen Antworten zu.

Wenn Sie jedoch ein ganzes System entwerfen, würde die "beste Vorgehensweise" darin bestehen, Ausnahmen zu verwenden: Geben Sie immer etwas Sinnvolles zurück und werfen Sie im Falle einer Anomalie eine Ausnahme. Der Anrufer kann dann mit den Situationen fertig werden, die er kennt, und jemand kann den Rest überholen.

Answer 6

Wiederkehrende gemischte Art ist schlecht, mindestens heute in 2013. Boom! Der Weg zu gehen, ist dies zu spalten:

BAD, gemischten Rückgabetyp Stil:

function checkResult($data) 
{ 
    if ($data) { 
     ... 
     return $stuff; 
    } else { 
     return false; 
    } 
} 

Menschen zusätzliche Logik benötigen checkRsult() zu arbeiten, und sie wissen nie genau, welche Art zurück.

GOOD, eindeutig festgelegt Rückgabetyp Stil:

Vielleicht ist das Beispiel nicht wirklich gut, aber es zeigt den Weg zu gehen.