Question

我想在用户忘记用户名时返回用户名。我让它再次验证他们的电子邮件地址，但由于某种原因它只是一直说'错误：找不到用户名'。我希望使用正确的mysqli语法。

if (isset($_POST['user'])) {
    $email = mysqli_real_escape_string($con, $_POST['email']);
    $username = "";
    if (!eregi("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$", $email)) {
        $errmsg = 'Error: ' . $email . ' is not a valid email address';
    } else {
        $query = "SELECT email FROM admin WHERE email = '$email'";
        $results = mysqli_query($con, $query);

        $query2 = mysqli_fetch_array($results);

        if ($query2 == 0) {
            $errmsg = 'Error: ' . $email . ' is not found, please try again';
        }
        if (!errmsg) {
            $getuname = "SELECT * FROM admin WHERE email = '$email'";
            if (!mysqli_query($con, $getuname)) {
                die('Error: ' . mysqli_error($con));
            }
            $row = mysql_fetch_array($getuname);
        }
        $username = '<div class="registersuccess">Your username is: ' . $row['username'] . '</div>';
    }
    $username = '<div class="registererror">Error: cannot find username</div>';

    mysqli_close($con);
}

当谈到这一点时，我是一个菜鸟，但我很确定这是正确的。如果不是我哪里出错了？

Answer 1

这些是我注意到的一些错误：

你正在做if (!errmsg)，但没有这样的常数，而你想要if (!$errmsg)。
正如Marc B指出的那样，你正在做$row = mysql_fetch_array($getuname);，但你需要$row = mysqli_fetch_array($getuname);。

此外，您描述的具体问题可能是因为$username声明之前的mysqli_close声明。您正在重置$username的值，无论数据库查询结果如何，它都会始终回显相同的消息。

$username = "";
if(condition)
{
    # code ...
}
else
{
    # code ...
}
$username = '<div class="registererror">Error: cannot find username</div>';
mysqli_close($con);

这是代码中的许多逻辑错误之一。使用适当的缩进来构造if-else块，您不应该遇到此问题。

Answer 2

你的代码是一场灾难。您正在使用ereg，自石器时代以来已被弃用。您正在混合调用mysql（no i）和mysqli（带i）库。它们不可互换，彼此不兼容。

您需要切换到preg函数，并在SINGLE mysql库上进行标准化。由于mysql也已弃用，因此仅使用mysqli。

    $row = mysql_fetch_array($getuname);
               ^^^---note the LACK of an i

if (!errmsg){
    ^^^--note the lack of a $ sign, meaning this is undefined/undeclared constant.

Answer 3

更改此部分代码：

    if (!errmsg){
      $getuname = "SELECT * FROM admin WHERE email = '$email'";
      if (!mysqli_query($con,$getuname))
        {
          die('Error: ' . mysqli_error($con));
        }
      $row = mysql_fetch_array($getuname);
    }       
    $username = '<div class="registersuccess">Your username is: '.$row['username'].'</div>';
 }
 $username = '<div class="registererror">Error: cannot find username</div>';

为：

    if (!errmsg){
      $getuname = "SELECT * FROM admin WHERE email = '$email'";
      $uresult = mysqli_query($con, $getuname);
      if (!$uresult))
        {
          die('Error: ' . mysqli_error($con));
        }
      $row = mysqli_fetch_array($uresult);
      if ($row) {
        $username = '<div class="registersuccess">Your username is: '.$row['username'].'</div>';
      } else {
        $username = '<div class="registererror">Error: cannot find username</div>';
    }
  }

你的错误：

您正在呼叫mysql_fetch_array而不是mysqli_fetch_array。
您将SQL字符串传递给mysql_fetch_array，而不是mysqli_query的结果。
您没有检查是否返回了行，您只是无条件地设置$username - 首先是成功消息，然后是失败消息。

Answer 4

我在你的代码中发现了一些错误，你可以看到Mark B和Amal Murali的回答中的错误。让我让else块简单明了。您只能使用1个查询而不是2个查询。

else {
    $query = mysqli_query("SELECT * FROM admin WHERE email = '$email'");
    $row = mysqli_fetch_array($query);
    $numrows = mysqli_num_rows($query);
    if (!$errmsg){
        if($numrows == 0) {
            $errmsg = 'Error: '.$email.' is not found, please try again'; 
                // or here you can add message like Cannot find username
        }
        else
        {
            $username = '<div class="registersuccess">Your username is: '.$row['username'].'</div>';
        }
 }

不会返回用户名的MYSQLI结果

4 个答案: