如何修改删除功能实现指定用户删除而非批量删除用户列表？

你当前的问题出在页面渲染时就执行了删除方法：在循环渲染用户列表时，<a href="<?php $user->delete()?>">Delete</a>这段代码里，$user->delete()会被直接执行，而不是等到点击链接才触发。所以页面加载时，循环遍历每个用户都会调用一次delete方法，导致所有用户被批量删除。

1. 修改删除链接，传递用户ID参数 #

把原来的删除链接改成传递删除动作和用户ID，而不是直接执行删除方法：

<?php foreach($users as $user): ?>
      <tr>
         <th><?= $user->name ?></th>
         <td><?= $user->email ?></td>
         <!-- 修改这里的href，传递删除动作和用户ID -->
         <td><a href="?action=delete&id=<?= $user->id ?>">Delete</a></td>
         <td><a href="">Update</a></td>
       </tr>
    <?php endforeach?>

2. 在页面顶部添加删除逻辑处理 #

在用户列表页面的最开头（渲染HTML之前），判断是否有删除请求，若有则执行删除操作：

<?php
// 假设你已经初始化了数据库连接和User类
if(isset($_GET['action']) && $_GET['action'] === 'delete' && isset($_GET['id'])){
    $userId = (int)$_GET['id']; // 强制转为整数，避免SQL注入
    // 方式一：通过User实例删除
    $user = new User();
    $user->id = $userId;
    $user->delete();

    // 或者方式二：直接调用数据库删除（如果不需要User类的封装）
    // $db->delete("users", "id={$userId}");

    // 删除后跳回列表页，避免刷新重复提交
    header("Location: {$_SERVER['PHP_SELF']}");
    exit;
}
?>

3. 修复SQL注入风险（重要） #

你当前的Database delete函数使用字符串拼接WHERE $where，存在严重的SQL注入风险。建议修改为预处理语句：

public function delete(string $table, string $whereColumn, $whereValue, int $limit=1){
    // 例如：$whereColumn是'id'，$whereValue是用户ID
    $stmt = $this->connection->prepare("DELETE FROM $table WHERE $whereColumn = ? LIMIT $limit");
    $stmt->execute([$whereValue]);
    return $stmt->rowCount();
}

对应的User类delete方法也需要调整：

public function delete(){
    // 调用修改后的delete方法，传递列名和值
    $result = $this->db->delete("users", "id", $this->id);
    return $result;
}

修改后，页面加载时只会渲染用户列表和带参数的删除链接；当用户点击"Delete"链接时，浏览器会发起一个带action=delete和id=用户ID的请求，后端接收到请求后才会执行对应该ID用户的删除操作，最后跳回列表页面，避免重复提交。

内容的提问来源于stack exchange，提问作者flaw