PHP mysqli Skript mit AND-Operator funktioniert nicht wie erwartet

Ich schreibe weiterhin Abfragen, die scheinen nie so zu arbeiten, wie ich es möchte. Wenn der Benutzer, der versucht, sich einzuloggen, in diesem Skript user_level 'Admin' hat, sollte er direkt zu adminhome.php sein. Elseif user hat user_level 'Employee' er sollte an employeehome.php gerichtet werden. Ich habe diese beiden Benutzer Amanda mit user_level Admin und Hugo mit user_level 'Employee' erstellt, um das Skript zu testen, aber egal wen ich mich anmelde, es feuert $error.PHP mysqli Skript mit AND-Operator funktioniert nicht wie erwartet

login.php

<?php 
if($_SERVER["REQUEST_METHOD"] == "POST") { 

    $myusername = mysqli_real_escape_string($db,$_POST['username']); 
    $mypassword = $_POST['password'];   


    $hashedPasswordQry = "SELECT password FROM users WHERE username = '$myusername'"; 

    $userLevel = mysqli_query($db, "SELECT user_level FROM users WHERE username='".$myusername."'"); 

    $result = mysqli_query($db,$hashedPasswordQry); 
    $row = mysqli_fetch_array($result,MYSQLI_ASSOC); 

    $count = mysqli_num_rows($result); 

    // If result matched $myusername and $mypassword, table row must be 1 row 
    if($count == 1 && (password_verify($mypassword, $row['password'])) && $userLevel == 'Admin') { 

     $_SESSION['login_user'] = $myusername; 


     header("location: user/adminhome.php"); 

    }elseif($count == 1 && (password_verify($mypassword, $row['password'])) && $userLevel == 'Employee'){ 

      $_SESSION['login_user'] = $myusername; 


     header("location: user/employeehome.php"); 
    } 
else{ 
    $error = '<h5 style="text-align: center;" class="alert alert-danger" >Your username or password is invalid</h5>'; 

} 
} 
?>

Quelle

2017-02-16 Sebastian Farham

Warum verwenden Sie zwei SELECT-Abfragen in derselben Tabelle mit derselben WHERE-Klausel für zwei verschiedene Spalten? Wenn Sie nicht wussten, können Sie mehrere Spalten aus einer einzigen Abfrage ziehen "SELECT password, user_level" ... – cteski

@cteski: in Ordnung danke Kumpel. Es klappt. –

@cteski: Sie wollen das Cred oder ... Ich lasse meine eigene Antwort? –

Sie sind zwei sehr ähnliche wählt verwenden, wenn man nur notwendig ist. Sie können mehrere Spalten in einer Abfrage auswählen, indem Sie sie durch ein Komma trennen. SELECT this_column, that_column FROM ... Und während Sie beide Abfragen ausgeführt haben, greifen Sie nur auf die Ergebnisse der Kennwortabfrage und nicht auf das Ergebnis $userLevel zu. Eine korrektere Version würde folgendermaßen aussehen:

$query = "SELECT password, user_level FROM users WHERE username = '$myusername'"; 

$result = mysqli_query($db, $query); 
$row = mysqli_fetch_array($result, MYSQLI_ASSOC); 

$count = mysqli_num_rows($result); 

if($count == 1 && (password_verify($mypassword, $row['password'])) && $row['user_level'] == 'Admin') 
{ 
    // do stuff 
} elseif ($count == 1 && (password_verify($mypassword, $row['password'])) && $row['user_level'] == 'Employee') 
{ 
    // do stuff 
}

Quelle

2017-02-16 23:39:56 cteski

PHP mysqli Skript mit AND-Operator funktioniert nicht wie erwartet

Antwort

Verwandte Themen