Mysql不断删除错误的表行

Question

用户应该能够删除特定行，但程序会继续删除第一行。我找到了答案here 试过了，没用。我想知道我做错了什么。

cart.php

 <form id='updateCartForm' action="update_cart.php" method="get">
             <input name="cart_item_name" type = "hidden"  id ="cart_item_name" value='<?=$product['prod_name'];?>'>
 <input name="cart_item_id" type = "hidden"  id ="cart_item_id" value='<?=$product['id'];?>'>
 </form>

update_cart.php

<?php
require_once "core/db.php"

//get data from form
$cart_item_name = $_GET['cart_item_name'];
$cart_item_id = $_GET['cart_item_id'];

//execute query
$sql = "DELETE FROM cart WHERE id =" .$cart_item_id;  
$db->query($sql);

//flash success message
$domain =($_SERVER['HTTP_HOST'] != 'localhost')?'.'.$_SERVER['HTTP_HOST']:false;
$_SESSION['success_flash'] = $cart_item_name. ' was deleted from your cart.'; 

Answer 1

设计对吗？
在您之前的问题中，我也评论了您的设计是否正确。在使用 ehre 时使用产品 ID 作为购物车项目 ID 是没有意义的。

<form id='updateCartForm' action="update_cart.php" method="get">
             <input name="cart_item_name" type = "hidden"  id ="cart_item_name" value='<?=$product['prod_name'];?>'>
 <input name="cart_item_id" type = "hidden"  id ="cart_item_id" value='<?=$product['id'];?>'>
 </form>

使用此 cart_item_id 删除实际上意味着从产品为 $product['id'] 的所有用户中删除购物车中的所有商品

这让我想到了第二点。为什么表格中同时包含产品名称和产品 ID？只需要产品 ID。开始删除。

结论，你有错误的表设计或错误的查询设计。

第一行是什么？

程序不断删除第一行。

您的查询中没有排序依据。所以这里没有第一排这样的东西。实际发生的情况可能就是我在第一段中所描述的。

SQL 注入
现在是一个重大的安全问题。

$sql = "DELETE FROM cart WHERE id =" .$cart_item_id;  
$db->query($sql);

这是一种可怕的查询方式。有一天，您会发现不仅删除了一行，而且删除了整个数据库。阅读 PDO 准备好的语句。

Answer 2

如果没有 WHERE 子句，删除查询将删除所有行， 否则它会删除与 WHERE 原因匹配的行。

如果它没有按预期工作，请检查包含以下 id 的变量：

echo $cart_item_id;
exit;

看看里面有什么。

Answer 3

决定采用更简单的方式使用 href。

cart.php

<a href="update_cart.php?productcartid=<?=$product['id'];?>">&times</a>

update_cart.php

<?php
ob_start();
require_once 'core/db.php';

//delete from cart

           $productcartid =$_REQUEST['productcartid'];

           // sending query
           $del = "DELETE FROM cart WHERE id='$productcartid'";
           $db->query($del);
           header("Location: cart.php");