logisches UND in IF

[devas]

Hallo,

kann mir jemand sagen ob das so richtig ist und wenn wie ich das besser machen kann?


if (($row['s']>=$_POST['s']) && ($_POST['x2'] != $xz) && ($_POST['y2'] != $yz) ){

[PHP-Desaster]

Was soll daran falsch sein? Die Klammerung der einzelnen Terme ist überflüssig, sonst passt das! Hoffe natürlich, deine POST-Werte sind auch gesetzt!

[jahlives]

Solche Konstrukte brauchst du nicht zu klammern, da Verküpfungsoperatoren (&& oder ||) nach den Vergleichsoperatoren vom Parser drangenommen werden. Also wird der Parser immer erst den Vergleich auswerten...
Ein Tipp noch isset() kann man auch mehrere Parameter übergeben z.B.

if (isset($_POST['s'],$_POST['x2'],$_POST['y2']) && $row['s']>=$_POST['s'] && $_POST['x2'] != $xz && $_POST['y2'] != $yz){ 

So würdest du auch allfällige Warnings des Parsers wegen nicht-definierten Vars verhindern.

Gruss

tobi