0

所有的作品都完美无缺,但是当购物车中有更多的物品时...以及任何物品的数量(列表中最后一个物品除外)被更改时,下面的代码进入无限循环,我通过在其中放置print_r语句来验证它。php foreach进入无限循环,数组存储在会话中

即进入无限循环的代码的一部分:

if (isset($_POST['item_to_adjust']) && $_POST['item_to_adjust'] != "") { 
    // execute some code 
    $item_to_adjust = $_POST['item_to_adjust']; 
    $quantity = $_POST['quantity']; 
    $quantity = preg_replace('#[^0-9]#i', '', $quantity); // filter everything but numbers 
    if ($quantity >= 100) { $quantity = 99; } 
    if ($quantity < 1) { $quantity = 1; } 
    if ($quantity == "") { $quantity = 1; } 
    $i = 0; 
    foreach ($_SESSION["cart_array"] as $each_item) { 
       print_r($each_item); 
       $i++; 
       while (list($key, $value) = each($each_item)) { 
        if ($key == "item_id" && $value == $item_to_adjust) { 
         // That item is in cart already so let's adjust its quantity using array_splice() 
         array_splice($_SESSION["cart_array"], $i-1, 1, array(array("item_id" => $item_to_adjust, "quantity" => $quantity))); 
        } // close if condition 
       } // close while loop 
    } // close foreach loop 
} 

P.S.

这是在添加第1项时如何初始化数组。

$_SESSION["cart_array"] = array(0 => array("item_id" => $pid, "quantity" => 1)); 

如果需要其他任何细节,请让我知道..

更新:假设有在购物车中的三个项目。而且我改变了第三项的数量。这行得通。 但是,如果我更改第二个项目的数量,脚本将达到最大执行时间,第二个和第三个项目会在购物车中无限重复。

+3

对正在循环的数组进行更改是一个非常糟糕的主意! – GordonM 2012-04-19 06:58:02

+0

我有一种感觉,它可能会影响for循环,如果我添加/删除项目..但我只是更换循环内的项目..认为它不应该成为一个问题。 – 2012-04-19 07:06:16

+0

在循环到达结尾之前,您正在将这些项目添加到数组中。 – fragmentedreality 2012-04-19 07:06:25

回答

2
foreach ($_SESSION["cart_array"] as $item_key => $each_item) { 
    if ($item_to_adjust == $each_item["item_id"]) { 
     $_SESSION["cart_array"][$item_key]["quantity"] = $quantity; 
    } 
} 

这仍然修改循环内的数组(但不酷),但它不会混淆索引。

+0

谢谢,我会试试看。 但我在这里尝试了原始代码.. http://tinyurl.com/c5qh3ha 它确实按预期工作.. – 2012-04-19 07:11:46

+0

它确实工作:) 谢谢! – 2012-04-19 07:26:36