PDO用戶驗證

Question

count($existuser)函數總是返回1.即使沒有具有該名稱或電子郵件的用戶。

下面是代碼：

function registerUser($username, $password, $passwordagain, $email, $mcname) { 
    include $_SERVER['DOCUMENT_ROOT'] . "/config/config.php"; 
    $conn   = new PDO('mysql:host=' . $ip . ';dbname=' . $database, $username, $password); 
    $validusername = "/^[a-z0-9]+$/"; 
    $validpassword = "/^[A-Za-z0-9]+$/"; 
    $validemail = "/^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/"; 
    $validmcname = "/^[A-Za-z0-9]+$/"; 
    $error   = 0; 

    if (strlen($username) < 4 || strlen($username) > 24) { 
     $error = 1; 
    } 
    if (strlen($password) < 6 || strlen($password) > 24) { 
     $error = 1; 
    } 
    if (strlen($mcname) < 4 || strlen($mcname) > 24) { 
     $error = 1; 
    } 
    if (!preg_match($validusername, $username)) { 
     $error = 1; 
    } 
    if (!preg_match($validpassword, $password)) { 
     $error = 1; 
    } 
    if (!preg_match($validemail, $email)) { 
     $error = 1; 
    } 
    if (!preg_match($validmcname, $mcname)) { 
     $error = 1; 
    } 
    if ($password != $passwordagain) { 
     $error = 1; 
    } 
//test 
    $userquery = $conn->query('SELECT * FROM users WHERE username="' . $username . '"'); 
    $existuser = $userquery->fetch(); 
    echo count($existuser); 
//test 
    if (count($existuser)) { 
     $error = 1; 
     echo "<div class='erroralert'>Username already exists!</div>"; 
    } 
//test 
    $emailquery = $conn->query('SELECT * FROM users WHERE email="' . $email . '"'); 
    $existemail = $emailquery->fetch(); 
//test 
    if (count($existemail)) { 
     $error = 1; 
     echo "<div class='erroralert'>E-mail already exists!</div>"; 
    } 
    if ($error != 1) { 
     $encryptedpassword = hash('sha512', $password); 
     $registeruser  = $conn->query("INSERT INTO users(username, password, email, mcname) VALUES ('$username', '$encryptedpassword', '$email', '$mcname')"); 
     echo "<div class='successalert'>Succesfully registred</div>"; 
    } 
} 

Answer 1

該查詢可能無法執行，這就是爲什麼你總是得到「1」，這可能是錯誤報告。

通過您的代碼閱讀，我建議你試試這個：

在下面的查詢，你是其中附有用戶名」，但是，這並不總是支持，而是應該使用單引號'並用「包圍字符串本身。所以下面的一行

$userquery = $conn->query('SELECT * FROM users WHERE username="' . $username . '"'); 

應該

$userquery = $conn->query("SELECT * FROM users WHERE username='" . $username . "'"); 

做同樣的具有相同問題的其他問題。

Answer 2

你爲什麼要使用PDO如果你這樣做：

$userquery = $conn->query('SELECT * FROM users WHERE username="' . $username . '"'); 
$existuser = $userquery->fetch(); 

你應該有以下邏輯：

$userquery = $conn->prepare('SELECT * FROM users WHERE username = ?'); 
$userquery->execute(array($username)); 
if ($userquery->rowCount()) { 
    // found user 
} else { 
    // user not found 
} 

Answer 3

您使用count()錯在這裏。它會返回1爲任何正常變量和一個數組，它將返回元素的數量。

重要的部分是第一部分。如果沒有行，PDOStatement::fetch()將返回FALSE，其計數1這是truthy：

count(FALSE);  # 1 
count($existuser); # 1 when there is no user, when there is a user at least 2 
        # for default fetchmode PDO::FETCH_BOTH 

所以你剛纔檢查什麼。取而代之的測試，它是不FALSE：

if ($existuser === FALSE) { 
    // error. 
}