Kan denne kode skrives på en anden måde?

I stedet for 'or' bruger jeg '||'. Og der skal ikke være ';' efter '}'.

Det her:
if (isset($_REQUEST["user"]) or isset($_REQUEST["pass"])){
    if ($_REQUEST["user"] == "" or $_REQUEST["pass"] == ""){
        if ($_REQUEST["user"] == ""){ $status = "Indtast venligst brugernavn!<br />"; };
        if ($_REQUEST["pass"] == ""){ $status = $status."Indtast venligst kodeord!<br />"; };
    } else {
        if ($rowLogin["password"] != $md5password or $_REQUEST["user"] != $rowLogin["user"]){ $status = $status."Brugernavn eller kodeord er forkert!<br />"; };
    };
};

kan du ændre til:

if (isset($_REQUEST["user"]) || isset($_REQUEST["pass"])){
    if ($_REQUEST["user"] == "" || $_REQUEST["pass"] == ""){
        if ($_REQUEST["user"] == ""){ $status = "Indtast venligst brugernavn!<br />"; }
        if ($_REQUEST["pass"] == ""){ $status = $status."Indtast venligst kodeord!<br />"; }
    } elseif ($rowLogin["password"] != $md5password || $_REQUEST["user"] != $rowLogin["user"]){ $status = $status."Brugernavn eller kodeord er forkert!<br />"; }
    }
}

Det var nu ikke så meget det jeg tænkte på :) Men ellers tak :)

Jeg tænkte på, at det var urimelig mange forgreninger, som jeg måske tænkte at hard-core folk kunne finde en smartere og måske rigtigere måde at formulere...??

Jeg ved at der er noget med cases osv... Men det er måske ikke i en situation som denne??

Jeg ville da overveje at kigge på switch muligheden ;)

mccookie -> Kender du et sted med en god tutorial til switch??

Mads

Prøv at se her:

http://spamfighter.com/Lang_DA/reseller_list.asp

Den er ligetil at gå til ;)

Hvad har det med sagen at gøre?? Spamfighter??

lukker