【发布时间】:2018-03-04 06:53:35
【问题描述】:
我正在尝试为购物车编写代码,该代码正在运行。但是,当我刷新页面时,它会自动将产品添加到我的购物车中。
例如,我的网站上有 3 个产品,它们是 Apple, Banana, Orange。
我点击 Apple,它被添加到我的购物车,数量为 1,并且 URL 显示
`mydomain.com/my-cart?action=addcart&p_id=FylvGt6Yyb6n%2BzTXcJHwjBawOY%2Fw3QSZxF7rdUJLqhA%3D#`
现在,如果我刷新页面,它会将另一个 Apple 添加到我的购物车(数量 2)。然后,如果我再次刷新页面,它会添加另一个 Apple(数量 3)等等。我不知道为什么会这样。它正在添加到 SESSION。
你会在这方面帮助我吗?
下面是我的购物车代码。
$action = isset($_GET['action'])?$_GET['action']:"";
$p_id=$conn->real_escape_string($_GET['p_id']);
$decrypted_p_id = decryptIt($p_id);
//Add to cart
if($action=='addcart') {
//Finding the product by code
$query = "SELECT p_unique_id, p_images,p_name, p_currentprice FROM products WHERE p_id=?";
if ($stmt = $conn->prepare($query)) {
$stmt->bind_param("i", $decrypted_p_id);
$stmt->execute();
$stmt->bind_result($p_unique_id,$p_images, $p_name, $p_currentprice);
$stmt->fetch();
}
$currentQty = $_SESSION['products'][$decrypted_p_id]['qty']+1; //Incrementing the product qty in cart
$_SESSION['products'][$decrypted_p_id] =array(
'qty'=>$currentQty,
'p_unique_id'=>$p_unique_id,
'p_images'=>$p_images,
'p_name'=>$p_name,
'p_currentprice'=>$p_currentprice
);
$product='';
// header("Location:cart.php");
}
展示产品
<?php if(!empty($_SESSION['products'])):
foreach($_SESSION['products'] as $key=>$product):?>
/*some code here*/
endforeach;?>
<?php endif;?>
此处编辑的代码由 ADyson 建议
if (isset($_POST['submit'])) {
$action=$conn->real_escape_string($_POST['action']);
$decrypted_p_id=$conn->real_escape_string($_POST['p_id']);
// whole code here
【问题讨论】:
-
是的,因为如果您只是单击刷新完全相同的页面,则 action=addcart 等仍在 URL 中,因此它会再次运行该操作。老实说,像这样的“添加”操作最好作为 POST 请求完成,部分原因是语义原因(它是“发送”数据而不是“获取”数据),部分原因是为了避免这样的烦恼。理想情况下,GET 请求不应导致应用程序中的任何状态更改。
-
另外,您的代码容易受到 SQL 注入攻击。您应该使用参数化查询和准备好的语句来帮助防止攻击者通过使用恶意输入值来破坏您的数据库。 bobby-tables.com 给出了风险解释,以及如何使用 PHP / mysqli 安全地编写查询的一些示例。切勿将未经处理的数据直接插入 SQL。转义字符串本身并不安全 - 该函数从未打算防止注入攻击(尽管在某些情况下可以,但并非总是如此,也不是设计使然)。
-
@ADyson,现在我在准备好的语句中更新了我的代码。现在可以了吗?
-
@ADyson,你能分享一些购物车中 POST 请求的例子吗?
-
好吧,我不知道你是如何首先向 URL 发出请求的,但如果它是通过 HTML 表单完成的,那么将其“方法”属性更改为“POST”,并且在 PHP 中寻找 $_POST 变量而不是 $_GET。您也可以在网上找到大量简单的 POST 请求示例。
标签: javascript php jquery ajax session