Javascript gibt false in if-Anweisungen zurück

Question

Ist es eine gute Vorgehensweise, "return false" zu verwenden? im Grunde sagen, nichts in einer if-Anweisung tun? Zum Beispiel:

if (navigator.userAgent.match(/iPad/i) != null) { 
    return false; 
} else { 
    //Usual script here 
} 

Ich frage mich nur, ob es irgendwelche Rückschläge zu diesem gibt. Ich kann die if-Anweisung ohne das else verwenden, aber ich möchte nur einen Einblick bekommen. Ich habe ein Plugin, das ich nicht auf dem iPad laufen lassen will und so wickle ich es in den bedingten. irgendwelche Kommentare würden geschätzt werden!

Answer 1

Meiner Erfahrung nach nur, wenn Sie tatsächlich nach dem Falschen suchten, um zurückgegeben zu werden.

Answer 2

Sie sollten nur einen Wert zurückgeben, wenn ein Aufrufer etwas mit diesem Wert tun wird. Wenn du in einer if-Anweisung "nichts" machen willst, ist das ein Zeichen dafür, dass deine Logik falsch ist. Ändern Sie Ihre Erklärung dazu:

if (navigator.userAgent.match(/iPad/i) == null) { 
    //Usual script here 
} 

Auf diese Weise brauchen Sie nicht zu „brechen“ aus Ihrer Funktion mit der Rückkehr (keine gute Praxis in diesem Szenario).

Answer 3

Sie können es tatsächlich weiter vereinfachen, wie folgt aus:

if (navigator.userAgent.match(/iPad/i) != null) return false; 
//Usual script here 

Ist es "good practice" ... sicher, ob es für Sie und Ihr Team arbeitet. Wenn man aus einer Funktion aussteigt, sobald man nichts mehr zu tun hat, kann man sehr effizient etwas tun, solange es für jeden, der daran arbeitet, einfach zu verstehen und zu pflegen ist.

Ob Sie speziell false möchten, hängt von der Situation ab. Wenn Sie beispielsweise zurückkehren möchten, aber nicht verhindern, dass andere Ereignishandler später ausgeführt werden, können Sie stattdessen return true; verwenden.

Answer 4

Gruppe 1 wird sagen, dass es eine schreckliche Übung ist, da es schwer zu folgen ist.

Gruppe 2 wird sagen, tun Sie es.

Gruppe 3 wird sagen, es tun, aber in 1 Zeile

Gruppe 4 wird sagen Sie das sonst nicht verwenden

Gruppe 5 sagen, nicht zu tun, um die Rückkehr zu verwenden, benutzen Sie einfach die, wenn um den Code du willst laufen. Alias:

if (navigator.userAgent.match(/iPad/i) === null) { 
    //Usual script here 
} 

Answer 5

Ich stimme mit snkmchnb, sonst nur die Bedingung negieren. Sie können langen Ausdruck mit diesen negieren:

!(a && b) = !a || !b 
!(a || b) = !a && !b 

Und verwenden Sie diese mehrmals, um zu bekommen, was Sie wollen. Zum Beispiel negiert einen langen Ausdruck

!((a && b || c) && (d || e) || f) = 
    !((a && b || c) && (d || e)) && !f = 
    (!(a && b || c) || !(d || e)) && !f = 
    (!(a && b) && !c || !d && !e) && !f = 
    ((!a || !b) && !c || !d && !e) && !f 

Das jetzt hässlich aussieht, aber die meiste Zeit negiert bedeutet nicht zu verkomplizieren. Zum Beispiel negiert "< =" ergibt ">"

Also nie benutzen!(Long_expression) weder:

if (long expression) 
{ 
} 
else 
{ 
    //do stuff here 
} 

Answer 6

erstens ist es eine sehr gute Übung, nehmen Sie dieses Beispiel

var window.__page_loaded__; 
var Loadpage = function() 
{ 
    if(window.__page_loaded__ != undefined) 
    { 
     return; //The page has already laoded 
    } 

    //Proceed to load the page 
} 

durch die Verwendung von return; Sie das gleiche zu tun wie mit einer else Aussage aber ohne den zusätzlichen Block, und da Loadpage() normalerweise keine Daten zurückgibt, ist es vollkommen in Ordnung, den Code zu kürzen.