购物车删除所有数量而不是一个

时间:2015-08-28 09:26:58

标签: php shopping-cart

我正在尝试用PHP创建一个简单的购物车。我已设法将项添加到会话中。如果您添加相同的项目,数量会更新。

但是,当我删除数量超过1的项目时,它会删除该项目的所有内容,而不是从数量中删除1。

我想知道是否有人可以检查我可能做错了什么?

获取产品的ID:

<a href="checkout.php?id=<?php echo $earthProducts -> id; ?>"> Order Now </a>

我有一个Item类:

<?php
Class Item{
var $id;
var $name;
var $price;
var $quantity;
}
?>

在结帐页面上,它将显示当前购物车中的所有商品:

require 'item.php';

    if (isset($_GET['id'])) {
        $result = mysqli_query($con, 'SELECT * FROM earthProducts WHERE id=' . $_GET['id']);
        $earthProducts = mysqli_fetch_object($result);
        $item = new Item();
        $item->id = $earthProducts->id;
        $item->name = $earthProducts->name;
        $item->price = $earthProducts->price;
        $item->quantity = 1;

        // Check product is existing in cart
        $index = -1;
        $cart = unserialize(serialize($_SESSION['cart']));
        for ($i = 0; $i < count($cart); $i++)
            if ($cart[$i]->id == $_GET['id']) {
                $index = $i;
                break;
            }
        if ($index == -1) {
            $_SESSION['cart'] [] = $item;
        } else {
            $cart[$index]->quantity++;
            $_SESSION['cart'] = $cart;
        }
    }
    ?>

然后我用一个按钮打印购物车以删除该项目:

<table cellpadding="2" cellspacing="2" border="1">
                    <tr>
                        <th>Option</th>
                        <th>Id</th>
                        <th>Name</th>
                        <th>Price</th>
                        <th>Quantity</th>
                        <th>Sub Total</th>
                    </tr>
                    <?php
                    $cart = unserialize(serialize($_SESSION['cart']));
                    $s = 0;
                    $index = 0;
                    for ($i = 0; $i < count($cart); $i++) {
                        $s += $cart[$i]->price * $cart[$i]->quantity;
                        ?>
                        <tr>
                            <td> <a href="checkout.php?index=<?php echo $index; ?>" onclick="return confirm('Are you sure?')">Delete</td>
                            <td><?php echo $cart[$i]->id; ?></td>
                            <td><?php echo $cart[$i]->name; ?></td>
                            <td><?php echo $cart[$i]->price; ?></td>
                            <td><?php echo $cart[$i]->quantity; ?></td>
                            <td><?php echo $cart[$i]->price * $cart[$i]->quantity; ?></td>
                        </tr>
                        <?php
                        $index++;
                    }
                    ?>
                    <tr>
                        <td colspan="4" align="right">Sum</tr>
                    <td align="left"> <?php echo $s ?></td>
                </table>
                <br>
                <a href="earth_products.php"> Continue Shopping </a>

                <br>
                <br>
                <?php
                print_r($cart);
                ?>

我删除购物车中商品的代码(哪个错误):

// Delete product in cart
    if (isset($_GET['index'])) {
        $cart = unserialize(serialize($_SESSION['cart']));
        unset($cart[$_GET['index']]);
        $cart = array_values($cart);
        $_SESSION['cart'] = $cart;
    }

因此,如果我有数量为1的第1项,我按删除将删除它。如果我的物品2的数量为2,它将删除这两个数量并从购物车中删除第2项。

如果有人可以提供帮助,请提前感谢您。

2 个答案:

答案 0 :(得分:1)

你需要在取消设置之前检查数量,这样的事情应该有效:

if (isset($_GET['index'])) {
    $cart = unserialize(serialize($_SESSION['cart']));
    if ($cart[$_GET['index']]->quantity == 1){
       unset($cart[$_GET['index']]);
    }else{
       $cart[$_GET['index']]->quantity--;
    }
    $cart = array_values($cart);
    $_SESSION['cart'] = $cart;
}

答案 1 :(得分:0)

您需要减少商品的数量,而不是仅仅删除它,因为当数量大于1时,您的代码只会从购物车中完全删除商品。

此外,在SQL代码中直接使用$ _GET变量是非常危险的,这使得很容易进行SQL注入并转储数据库。