Vernüftige Abfrage erstellen

[Wotan]

Wie mache ich das als vernüftige Abfrage:

//Passwörter Da?
if($_SESSION[passwd1] == NULL || $_SESSION[passwd2] == NULL)
    {
    $error    = 1;
    $_SESSION[FNR_pass]    = "Passwörter sind leer";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000\""; //leer
    }
//Passwörter Zulässig?
if(!preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd1]) ||
   !preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd2]))
    {
    $error    = 1;
    $_SESSION[FNR_passwd1]    = "Passwörter enthält unzulässige Zeichen<br>oder sind nicht mindestens 6 Zeichen lang";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";//Zeichen
    }
// Passwörter gleich?
if($_SESSION[passwd1] != $_SESSION[passwd2])
    {
    $error    = 1;
    $_SESSION[FNR_passwd1]    = "Die Passwörter stimmen nicht überein!";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";//Zeichen
    }

else
    {
    $passwd = $_SESSION[passwd1];
    } 

momentan sieht es so aus:

if($_SESSION[passwd1] == NULL || $_SESSION[passwd2] == NULL)
    {
    $error    = 1;
    $_SESSION[FNR_pass]    = "Passwörter sind leer";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000\""; //leer
    }
else
    {
    if(!preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd1]) ||
       !preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd2]))
        {
        $error    = 1;
        $_SESSION[FNR_passwd1]    = "Passwörter enthält unzulässige Zeichen<br>oder sind nicht mindestens 6 Zeichen lang";
        $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";//Zeichen
        }
    else
        {
        if($_SESSION[passwd1] != $_SESSION[passwd2])
            {
            $error    = 1;
            $_SESSION[FNR_passwd1]    = "Die Passwörter stimmen nicht überein!";
            $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";//Zeichen
            }
        else
            {
            $passwd = $_SESSION[passwd1];
            }
        }
    } 

und hier läst er mir ungleiche Passwörter durch.

[Abraxax]

ohne garantie

if($_SESSION[passwd1] == NULL || $_SESSION[passwd2] == NULL)
    {
    $error    = 1;
    $_SESSION[FNR_pass]    = "Passwörter sind leer";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000\""; //leer
    }
else
    {
    if($_SESSION[passwd1] != $_SESSION[passwd2])
        {
        $error    = 1;
        $_SESSION[FNR_passwd1]    = "Die Passwörter stimmen nicht überein!";
        $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";//Zeichen
        }
    else
        {
        if(!preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd1]) ||
           !preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd2]))
            {
            $error    = 1;
            $_SESSION[FNR_passwd1]    = "Passwörter enthält unzulässige Zeichen<br>oder sind nicht mindestens 6 Zeichen lang";
            $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";//Zeichen
            }
        else
            {
            $passwd = $_SESSION[passwd1];
            }
        }
    } 

[Wotan]

Bliebt dabei. Ungleich Paswörter werden durchgelassen.

[TobiaZ]

macht man das echt mit vertiefungen bis zum geht nicht mehr??

hab aktuell mal gemacht mit 2 oder 3 abfragen.

aber ansonsten kann man das auch mit ner $error oder so regeln, oder?

[Wotan]

Poste mal deine Lösung. Bevor ich das mit Session gemacht habe ging es ja. Aber jetzt nach dem ich es umgestellt habe auf Session geht es nicht mehr.

[TobiaZ]

$error = FALSE;
if(!bed) $error = TRUE;

[Wotan]

Original geschrieben von TobiaZ
$error = FALSE;
if(!bed) $error = TRUE;

Du meinst einzelen Abfrage und dann noch mal zum Schluß deine Abfrage?

[TobiaZ]

alternativ kommt in die nächste abfrage if($error || !bed) $error=TRUE;

usw. o.s.ä.

[Wotan]

Hatte im ganze einen Denkfehler drin.
Jetzt geht es aber!

Man sollte schon darauf prüfen ob != 1 oder == 1
Und dann die Zweige richtig bestücken.

if($_SESSION[passwd1] != $_SESSION[passwd2])
    {
    $pass_ungleich = 1;
    $_SESSION[FNR_passwd1] = "Die Passwörter stimmen nicht überein!";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";// Ungleich
    }
else
    {
    $_SESSION[FNR_pass_1] = "style=\"background: #B9C5E5; color: #000000\"";// Ungleich
    }

if($_SESSION[passwd1] == NULL || $_SESSION[passwd2] == NULL)
    {
    $pass_leer = 1;
    $_SESSION[FNR_pass] = "Passwörter sind leer";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000\""; // leer
    }
else
    {
    $_SESSION[FNR_pass_1] = "style=\"background: #B9C5E5; color: #000000\"";// leer
    }

if(!preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd1]) ||
   !preg_match("/([a-z0-9_]{6,})/i",$_SESSION[passwd2]))
    {
    $pass_zeichen = 1;
    $_SESSION[FNR_passwd1] = "Passwörter enthält unzulässige Zeichen<br>oder sind nicht mindestens 6 Zeichen lang";
    $_SESSION[FNR_pass_1] = "style=\"background: #FF0000; color: #FFFFFF\"";// Zeichen
    }
else
    {
    $_SESSION[FNR_pass_1] = "style=\"background: #B9C5E5; color: #000000\"";// Zeichen
    }

if($pass_ungleich == 1 || $pass_leer == 1 || $pass_zeichen == 1)
    {
    $error = 1;
    }
else
    {
    $passwd = $_SESSION[passwd1];
    } 

[TobiaZ]

na, solche sind mir ja immer die besten!