Question

当用户点击“添加到购物车”时，会发出ajax请求并将该项目添加到购物车。检索产品ID，然后从数据库中提取所需的详细信息并将其放入购物车会话数组中。一切正常。

public function AddToCart()
{
    if($_SERVER['REQUEST_METHOD'] == 'POST') {
        $quantity = 1;
        $prod_id = $_POST['prod_id'];
        $prod_data = $this->ProductFendModel->GetProductByIdCart($prod_id);

        $data = [
            'prod_name' => $prod_data->prod_name,
            'sale_price' => $prod_data->sale_price,
            'sp_name' => $prod_data->sp_name    
        ];

        $response = array();
        $message = '';    
        $key = "{$prod_id}";

        if (empty($_SESSION['cart_array'][$key])) {
            $_SESSION['cart_array'][$key] = array(
                'prod_id' => $prod_id,
                'quantity' => $quantity,
                'prod_name' => $data['prod_name'],
                'sale_price' => $data['sale_price'],
                'sp_name' => $data['sp_name']
            );
        }
        else {
            $_SESSION['cart_array'][$key]['quantity'] += $quantity;
        }

        $response['success'] = true;
        $response['message'] = 'Product added to cart';

        echo json_encode($response);
    }
}

但是，当我尝试更新购物车时，它会使用正确的数量复制现有商品，而不是仅仅更新购物车中已有商品的数量。例如，如果我在购物车中有2个商品，我现在得到4.前2个具有原始数量，新添加的2具有正确的数量。

public function UpdateCart()
{
    if($_SERVER['REQUEST_METHOD'] == 'POST') {
        foreach($_POST['prod_id'] as $key => $id)
        {
            $qty = $_POST['quantity'][$key];
            $_SESSION['cart_array'][$key]['quantity'] = $qty;
        }
    }
}

Answer 1

我看到的一件事是在UpdateCart中您将$ _POST ['prod_id']作为关联数组访问，但是作为AddToCart中的整数。 foreach($_POST['prod_id'] as $key => $id) VS $prod_id = $_POST['prod_id'];。另外，在生产中使用它之前，不要忘记过滤POST变量！这是一个很好的做法，即使您只是将数据写入会话，甚至可以允许您按类型进行清理，以便您可以更确定在运行时具有该控件。

如果这不能指向正确的方向，您可以考虑发布$ _SESSION和$ _POST转储。此外，如果您还没有这样做，请使用PHPUnit。使用它并不是很复杂，即使有最好的使用方法。

对于请求变量，您可以将数组传递给函数或类，而不是直接从函数内访问状态变量。这样做可以避免使逻辑过于复杂，并允许您将大部分应用程序设计保留在短期工作内存中。

更新购物车功能添加新项目而不是更新现有项目

1 个答案: