IF-Konstrukt kürzen

[dedlfix]

Hi!

if($aktiv == "1")
{
        $var = $_GET['var']; //mit Absicht nicht geprüft

Was soll denn diese Umkopiererei?

if($var = "test")

Das ist hoffentlich nur ein Tippfehler und soll wohl ein == sein.

Wenn du mal genau hinsiehst merkst du, dass $var nur gesetzt wird wenn $aktiv == 1 ist.

Mit gesetzten oder nicht gesetzten Variablen als Entscheidungskriterium zu arbeiten, ist im Allgemeinen nicht besonders sinnvoll, wenn es sich nicht gerade im solche handelt, auf die men selbst keinen Einfluss hat, also $_GET/$_POST/etc.-Inhalte.

In deinem Fall ist es so, dass du mit $aktiv schon ein Entscheidungskriterium für eine bedingte Auswertung von $_GET['var'] hast, da brauchst du nicht noch ein gesetztes oder nicht gesetztes $var.

Auch meine Empfehlung lautet, die Bedingungen zu verknüpfen. Im Zweifelsfall helfen Wahrheitstabellen, wie sie Gunnar gezeigt hat.

Lo!