dongtanliefang8765 2013-06-26 15:15
浏览 27
已采纳

PHP条件使用其他值不正确

I am writing a conditional in PHP to check if certain user credentials have been used, and am using an if....elseif....else conditional method to do it. In each statement, I am defining the value of a variable if that statement is true. No matter what, the else value is always being used.

Also, if you want brownie points, would this be easier to do with a switch statement?

$username = $_POST['username'];
$password = strong_crypt($_POST['password'] . $username );
$name = $_POST['name'];
$email = $_POST['email'];

while($row = mysqli_fetch_array($query))
{
    $allEmails = $row['email'];
    $allUnames = $row['uname'];
}

if($allEmails === $email)
{
    $message = "That email already exists. You can only have one account per email address.";
}
elseif($allUnames === $username)
{
    $message = "That username has already been taken. Please choose a different one.";
}
elseif($username === null || $email === null || $password === null || $name === null) 
{
    $message = "Please fill out all fields to register";
}
else
{
    mysqli_query($conn, "INSERT INTO login (name, uname, pword, email, lat, lon, it) VALUES ('$name', '$username', '$password', '$email', '$lat', '$lon', '$it')");
    $message = "goodtogo";
}
  • 写回答

4条回答 默认 最新

  • doy57007 2013-06-26 15:20
    关注

    It appears the issue here is that you're doing the checks outside of the while loop, so that the check will always be done against the values from the LAST row of your query. Perhaps this code is more what you were aiming for?

    $username = $_POST['username'];
    $password = strong_crypt($_POST['password'] . $username );
    $name = $_POST['name'];
    $email = $_POST['email'];
    $message = '';
    
    if($username === null || $email === null || $password === null || $name === null) 
    {
        $message = "Please fill out all fields to register";
    }
    else
    {
        while($row = mysqli_fetch_array($query))
        {
            $allEmails = $row['email'];
            $allUnames = $row['uname'];
    
            if($allEmails === $email)
            {
                $message = "That email already exists. You can only have one account per email address.";
                break;
            }
            else if($allUnames === $username)
            {
                $message = "That username has already been taken. Please choose a different one.";
                break;
            }
        }
    }
    if (!$message)
    {
        mysqli_query($conn, "INSERT INTO login (name, uname, pword, email, lat, lon, it) VALUES ('$name', '$username', '$password', '$email', '$lat', '$lon', '$it')");
        $message = "goodtogo";
    }
    
    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论
查看更多回答(3条)

报告相同问题?

悬赏问题

  • ¥15 unity第一人称射击小游戏,有demo,在原脚本的基础上进行修改以达到要求
  • ¥15 买了个传感器,根据商家发的代码和步骤使用但是代码报错了不会改,有没有人可以看看
  • ¥15 关于#Java#的问题,如何解决?
  • ¥15 加热介质是液体,换热器壳侧导热系数和总的导热系数怎么算
  • ¥100 嵌入式系统基于PIC16F882和热敏电阻的数字温度计
  • ¥15 cmd cl 0x000007b
  • ¥20 BAPI_PR_CHANGE how to add account assignment information for service line
  • ¥500 火焰左右视图、视差(基于双目相机)
  • ¥100 set_link_state
  • ¥15 虚幻5 UE美术毛发渲染