isset 函数提交 while 循环 PHP 中的所有按钮

Question

我遇到了这个问题，while 循环内的提交按钮重新执行 while 循环。

PHP：

$query1 = $con->query("SELECT * FROM room");
if($query1->num_rows > 0) {
    while($row1 = $query1->fetch_array()) {
        $idroom = $row1['idroom'];

        echo "<tr>";
        echo "<td> $idroom </td>";
        echo "<td> <form method='POST'> <input type='submit' name='delete' value='DELETE'> </form></td>";
        echo "</tr>";

        if(isset($_POST['delete'])) {
            $query2 = $con->query("DELETE FROM room WHERE idroom='$idroom'");
        }
    }
}

表格： [1]：https://i.stack.imgur.com/dujps.png

问题是，当我单击删除按钮时，它只会删除所有房间而不是我要删除的房间。我相信程序认为我一次按下了所有的删除按钮，因为 'isset' 功能。

我尝试过的事情：

• 将 isset 中的 $query2 行替换为 echo $idroom;，输出为 room1room2room3。这意味着程序认为我一次按下了所有按钮。

Answer 1

这段代码有很多问题，尤其是它非常不安全并且容易被注入。在继续之前，您应该阅读准备好的陈述。教程和答案没有尽头，所以我不会在这里介绍它们。为了帮助新程序员（不久前我还是新手），我将在这里指出我认为出了什么问题：

1. $query1 = $con->query("SELECT * FROM room");

每次脚本运行时都会运行。这是要实现的关键。如果您加载页面，或者如果您提交帖子删除，这种情况总是会发生。这导致：

1. if($query1->num_rows > 0) { while($row1 = $query1->fetch_array()) { ...

您开始循环，循环通过every 记录。注意every。

因此，对于每条记录，然后检查：

1. if(isset($_POST['delete'])) {...

这是您的记录被删除的地方，也是 CBroe 在他的评论中指出的地方。您相信（我认为）您检查了该个人记录的删除，但是您只检查了 isset($_POST['delete']) 并且您为 EVERY 记录执行此操作。请记住，POST 变量一直存在到脚本结束或被删除。因此，通过单击删除并提交 _POST 值，您可以为循环中的每条记录传递此条件。

• 在继续之前，您应该先消化一下，因为这是新程序员的常见错误。请记住，计算机按顺序读取和操作脚本，记住它也会执行EXACTLY 你所要求的，仅此而已。经典的例子是描述制作一杯咖啡。如果我对你说“打开水壶，将咖啡放入杯中，加满水，加牛奶”，你会煮咖啡，但如果你对电脑这么说，你最终会得到，充其量是带着电脑一个水壶寻找一个杯子来放咖啡并告诉你牛奶不是一个数字。

所以解决方案。好吧，我不会为你写，有不同的选择。您应该将某种标识符传递给特定于您想要的记录的帖子，然后您需要检查它，例如 isset POST[delete] && isset(POST['room_id'])。然后你需要决定最好的地方，在脚本的开头，在不同的脚本中，可能不在循环中（这很少是伟大编程的标志）。如果您要删除记录，您可能应该在为其创建输出之前执行此操作。 （为什么要收集记录然后删除它，效率高吗？）。

如果您确实必须在循环中执行此操作，那么您需要在运行删除之前检查房间的 id 是否与 post 值的 id 相同。

希望这会有所帮助，但请确保您紧急调查并开始使用准备好的陈述 - 2020 年真的没有理由不这样做

Answer 2

在您的代码中，您没有遵循基本结构。使用 JS 进行编辑/删除的点击事件并使用 ajax 调用来执行操作。

这是基于您粘贴的代码的一种解决方案，它不是标准代码，但可以帮助您解决问题。

$query1 = $con->query("SELECT * FROM room");
if($query1->num_rows > 0) {
    while($row1 = $query1->fetch_array()) {
        $idroom = $row1['idroom'];

        $tr = "<tr>";
        $tr .= "<td> $idroom </td>";
        $tr .= "<td><form method='POST'>";
        $tr .= "<input type='hidden' name='room_id' value='".$idroom."'>";
        $tr .= "<input type='submit' name='delete' value='DELETE'> </form></td>";
        $tr .= "</tr>";

        echo $tr;
    }
}

if(isset($_POST['delete'])) {
  $roomId = $_POST['room_id'];
  $query2 = $con->query("DELETE FROM room WHERE idroom='$roomId'");
}