[PHP & MYSQL] LoginScript?

[Blakkeight]

Ok dann muss ich es mal anders erklären.


Ich will die daten aus der DB in einen array speichern und nach und nach vergleichen mit einer schleife, bis einer gleich ist und man eingelogt würd.

Zurzeit mache ich das so:

PHP-Code:

    if($_REQUEST['send']){
        $sql=mysql_query("SELECT ID, user, passwort, email FROM ".$db."_userlog;") or die(mysql_error());
        while($row=mysql_fetch_array($sql)){
            if($_REQUEST['name']==$row['user']){
                if(md5($_REQUEST['passwort'])==$row['passwort']){
                    session_register('session_name');
                    session_register('session_pass');
                    session_register('session_email');
                    session_register('session_logonoff');
                    $session_name=$row['user'];
                    $session_pass=$row['passwort'];
                    $session_email=$row['email'];
                    $session_logonoff=1;
                    if($session_name==$row['user']){
                        if($session_pass==$row['passwort']){
                            header("location: acp/admin_index.php?acp=status");
                        }
                    }
                }
            }
        } 

aber da nimmt er imo irgend einen aber leuft das scheinbar nicht durch bis er den richtigen gefunden hat.
Wie mach ich das, das er es so macht wie ich meine?

ps: ob der code nun zu lang oder komisch aussieht ist egal ich will damit keinen BlumenTopf gewinnen ist nur für mich selber.

[Manni]

PHP-Code:

    if($_REQUEST['send']){
        $sql=mysql_query("SELECT ID, user, passwort, email FROM ".$db."_userlog;") or die(mysql_error());
        while($row=mysql_fetch_array($sql)){
            if($_REQUEST['name']==$row['user']){
                if(md5($_REQUEST['passwort'])==$row['passwort']){
                    session_register('session_name');
                    session_register('session_pass');
                    session_register('session_email');
                    session_register('session_logonoff');
                    $session_name=$row['user'];
                    $session_pass=$row['passwort'];
                    $session_email=$row['email'];
                    $session_logonoff=1;
                    if($session_name==$row['user']){
                        if($session_pass==$row['passwort']){
                            header("location: acp/admin_index.php?acp=status");
                        }
                    }
                }
            }
        } 

...

Mal ein paar Tipps

• session_register("bla") und $bla = "foo" ist veraltet. Benutz stattdessen session_start() und $_SESSION['bla'] = "foo".
• Die Schleife ist unnötig. Wähl doch einfach direkt in der MySQL-Query mit "... WHERE `name` = ".$_POST['name']." ..." gleich den richtigen Benutzer aus.
• Schreib am besten die Namen der Felder in der MySQL-Query in ` und `. Damit vermeidest du Doppeldeutigkeiten und es sieht besser aus (imho)