PHP/SQL问题：点击按钮插入$row['id']时误插入所有行ID而非单个

嘿，我来帮你搞定这两个问题！先拆解下根源，再给你修正后的代码：

问题1：只有最后一张优惠券的按钮能正常触发 #

这是两个原因叠加导致的：

• 逻辑顺序错误：你把处理优惠券使用的SQL插入代码写在了while循环之前，这时候$row变量还没有被赋值（PHP每次请求都是独立的，之前循环的$row不会保留）。
• 参数使用错误：你明明在链接里通过del_task传递了对应优惠券的ID，却错误地用了$row['id']来插入数据——这会导致要么插入失败（PHP报错），要么在循环结束后$row保留最后一条数据的ID，所以不管点击哪个按钮，最终操作的都是最后一张优惠券。

问题2：点击按钮后所有优惠券图片都被移除 #

这其实是问题1的连锁反应：当你点击任何按钮，后台都插入了最后一张优惠券的ID到deleted_coupons表，再加上原代码里第一个<td>未闭合导致HTML结构错乱，会让页面渲染异常，看起来像是所有优惠券都被移除了；错误的插入逻辑也可能导致数据库记录不符合预期，进而影响查询结果。

修正后的完整代码 #

<?php
// 先处理优惠券使用的请求（添加参数校验避免非法请求）
if (isset($_GET['del_task']) && is_numeric($_GET['del_task'])) {
    // 使用预处理语句防止SQL注入，这是必须的安全操作！
    $stmt = mysqli_prepare($link, "INSERT INTO deleted_coupons (coupon_id, user_id) VALUES (?, ?)");
    mysqli_stmt_bind_param($stmt, "ii", $_GET['del_task'], $_SESSION['act']);
    mysqli_stmt_execute($stmt);
    mysqli_stmt_close($stmt);
}

// 查询当前用户未使用的优惠券
$coupons = mysqli_query($link, "
    SELECT coupons.* 
    FROM coupons 
    LEFT JOIN deleted_coupons 
        ON deleted_coupons.coupon_id = coupons.id 
        AND deleted_coupons.user_id = {$_SESSION['act']} 
    WHERE deleted_coupons.coupon_id IS NULL
");
?>

<table>
    <?php while ($row = mysqli_fetch_array($coupons)) { ?>
    <tr>
        <!-- 修复：闭合td标签，避免HTML结构错乱 -->
        <td><?php echo $row['id'] ?></td>
        <td class="task">
            <?php echo '<img src="data:image/jpeg;base64,'.base64_encode( $row['name'] ).'"/>'; ?>
        </td>
        <td class="delete">
            <a href="welcome.php?del_task=<?php echo $row['id'] ?>">Use Coupon</a>
        </td>
    </tr>
    <?php } ?>
</table>

关键修改点说明 #

1. 调整逻辑顺序：把处理优惠券使用的代码移到查询优惠券之前，确保插入操作完成后，查询的是最新的未使用优惠券数据。
2. 正确使用请求参数：用$_GET['del_task']作为要插入的优惠券ID，同时加上is_numeric()校验参数合法性，避免非法值插入。
3. 使用预处理语句：直接拼接用户输入到SQL里会导致严重的SQL注入风险，预处理语句能彻底避免这个问题。
4. 修复HTML结构错误：给第一个<td>加上闭合标签，解决页面渲染异常导致的交互问题。

这样修改后，每个按钮都会正确触发对应优惠券的使用操作，点击后也只会隐藏对应的优惠券图片啦！

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