Warum sind Zuordnungen unter Bedingungen schlecht?

Question

Ich benutze NetBeans für PHP 6.5.

In meinem Code verwende ich häufig den folgenden Befehlstyp:

if (($row = $db->get_row($sql))) { 
     return $row->folder; 
    } else { 
     return FALSE; 
    } 

Netbeans sagt mir, dass ich nicht in der IF-Anweisung mit Zuweisungen werden.

Warum?

Answer 1

Sie sind nicht schlecht, aber sie können zu gefährlichen Fehlern führen.

In c wie Sprachen, in denen eine Zuordnung ein Ausdruck ist, (unterstützen beispielsweise a = b = c = 1;) ein häufiger Fehler ist:

if (a = 1) { .. } 

Aber Sie wollten

haben

if (a == 1) { .. } 

haben Einige Entwickler gelernt

if (1 == a) { .. } 

um einen Fehler zu erzeugen, wenn ein ‚=‘ vergessen zu geben. Aber ich denke, dass es die Lesbarkeit nicht verbessert.

jedoch moderne Compiler, eine Warnung, wenn Sie

if (a = 1) { .. } 

schreiben, die ich denke, eine bessere Lösung ist. In diesem Fall müssen Sie prüfen, ob Sie das wirklich wollten.

Answer 2

Es ist wahrscheinlich versuchen Sie die gefürchteten Tippfehler vermeiden helfen:

if(a = b) 
    //logic error 

Obwohl ich eine Umgebung intelligent genug, erwarten Sie, dass, zu warnen, auch klug genug zu sein, „oh haben, Mach dir keine Sorgen über diesen Fall "Bedingungen.

Answer 3

Ich benutze sie die ganze Zeit, mit Schlaufen (nicht sicher, warum das einen Unterschied machen würde), wie:

$counter = 0; 
while($getWhateverDataObj = mysql_fetch_object($sqlResult)) { 
    $getWhateverObj->firstName[$counter] = $getWhateverDataObj->firstName; 
    $getWhateverObj->lastName[$counter] = $getWhateverDataObj->lastName; 
    $counter++; 
} 

Und es funktioniert gut.

Answer 4

In Sprachen, die durchweg Wert auf Zuweisungen zurückkehren, es ist nicht schlecht (ich glaube, es in funktionalen Sprachen durchaus üblich ist), aber (wie andere allready gesagt haben, während ich dies getippt) ist es in der Regel, da Sie oder jemand soll vermieden werden sonst könnte es für einen Vergleich verwechseln. Der Compiler sollte normalerweise darüber warnen, aber er kann ignoriert werden, wenn Sie sicher sind, was Sie tun ...

Answer 5

Bedingungsfaktoren enthalten oft Kurzschlussoperatoren. Also, dieses Beispiel gegeben:

if (a=func(x) && b=func(y)) 
{ 
    // do this 
} 

Es kann nicht ohne weiteres ersichtlich sein, aber die zweite Zuordnung erfolgt nur dann, wenn die ersten >0 zurückgekehrt, und wenn func(y) andere Nebenwirkungen haben, dass Sie erwartet haben, würden sie auch nicht passieren .

Kurz gesagt, wenn Sie wissen, was Sie tun und die Nebenwirkungen verstehen, dann ist nichts falsch daran. Sie müssen jedoch die Möglichkeit in Erwägung ziehen, dass jemand anders Ihren Code beibehält, wenn Sie nicht mehr in der Lage sind. Möglicherweise sind Sie nicht so erfahren wie Sie.

if (a==func(x) && b==func(y)) ... 

Wenn sie „reparieren“ Ihr Code, brechen sie es tatsächlich: die folgenden soll

Auch Sie zukünftige Maintainer vielleicht denken.

Answer 6

Wie würde ein Code aussehen, wenn Sie den $ -Zeilenwert im Loop-Zustand nicht zuweisen das wäre viel komplizierter ich denke ... obwohl nicht so gut für einige Betreuer zu lesen, nicht? gut Sie können es tun, wie

$next = mysql_fetch_assoc($result) 
do{ 
... 
... 
... 

$next = mysql_fetch_assoc($result) or break; 
}while ($next)