损坏的 PHP if-elseif 语句

Question

我正在处理一个个人项目，我正在尝试创建一个 php 脚本来显示一个关注按钮，如果登录的会话用户尚未关注显示的用户..

这就是我所拥有的：

<?php
      $query = query("SELECT*FROM following, users WHERE followed_id=" .escape_string($_GET['id'])." 
      AND user_id=".escape_string($_GET['id'])." AND follower_id =".escape_string($_SESSION['user_id'])."");
      confirm($query);
      while($row = fetch_array($query)):

       if(mysqli_num_rows($query)==0){

         $followbutton=<<<DELIMITER

       <form method="post" action="">
         <input type="hidden" name="followed" value="{$row['user_id']}">
         <input type="hidden" name="follower" value="{$_SESSION['user_id']}">
         <input class="btn btn-info" type="submit" name="submit" value="follow">
       </form>

DELIMITER;
       echo $followbutton;

    }elseif($row['follower_id'] == $_SESSION['user_id']){
      $unfollowbutton=<<<DELIMITER

    <form method="post" action="unfollow.php?id={$row['user_id']}">
      <input class="btn btn-info" type="submit"  value="Unfollow">
    </form>

DELIMITER;
    echo $unfollowbutton;

  }
endwhile;    ?>

查询、escape_string、fetch_array 和 Confirm 的作用如下：

function redirect($location){
header("Location: $location");
}

function query ($sql){
global $connection;
return mysqli_query($connection,$sql);
}

function confirm($result) {
global $connection;
if(!$result){
    die("QUERY FAILED ".mysqli_error($connection));
}
}

function escape_string($string){
global $connection;
return mysqli_real_escape_string($connection,$string);
 }

 function fetch_array($result) {
return mysqli_fetch_array($result);
}

“以下”表只有两列； follow_id 和 follower_id。当用户单击关注按钮时，它会将登录的 user_id 作为 follower_id 传递，并将被关注人的 user_id 作为 follower_id 传递。由于如果用户 A 尚未关注用户 B，则不会有任何条目，因此我需要仅在查询未返回任何结果时才显示该按钮。

Answer 1

如果我正确理解您的问题，您要么希望显示 follow 按钮或 unfollow 按钮。在这种情况下，您可以使用简单的 if/else 语句。 目前，您的 elseif 语句仅在返回一行时为真，因为您正在检查其是否存在：if ($row['follower_id'] == $_SESSION['user_id'])。 我现在看到您的 elseif 正在检查用户是否存在。我倒过来了。我的答案中的其他所有内容仍然有效。

不管怎样，您确实应该使用准备好的语句来防止 SQL 注入攻击。另外，我认为您的代码可以简化一点以使其更清晰。

• 不知道为什么要从following 和users 两种能力中进行选择。您只需要检查following 表中是否有您（登录用户）和您要检查的人，不是吗？
• 不需要while 循环。您可以简单地计算行数。如果有 > 0 行，您就知道您已经在关注此人。理论上我猜，应该只有 1 行。

这是我如何编写此内容的示例。我在这里使用PDO 只是因为那是我的API，但您可以继续使用mysqli。

<?php
$stmt = $pdo->prepare("SELECT count(*) FROM following WHERE follower_id = ? AND followed_id = ?");
$stmt->execute([
    $_SESSION['user_id'], // logged-in user
    $_GET['id']           // who we are following
]);

// How to count when using PDO - https://phpdelusions.net/pdo_examples/count
$count = $stmt->fetchColumn();

if ($count > 0) {
    // show `unfollow` form
} else {
    // show `follow` form
}