php foreach 进入无限循环，数组存储在会话中答案

【问题标题】：php foreach goes into infinite loop, array stored in sessionphp foreach 进入无限循环，数组存储在会话中
【发布时间】：2012-04-30 16:00:42
【问题描述】：

一切正常，但是当购物车中有多个物品时..并且任何物品的数量（列表中的最后一项除外）发生更改，下面的代码进入无限循环，我已经通过放置来验证它print_r 语句。

进入无限循环的部分代码：

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
}

附言

这是添加第一项时数组的初始化方式。

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

如果需要任何其他细节，请告诉我..

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

【问题讨论】：

对正在循环的数组进行更改是一个非常糟糕的主意！
我有一种感觉，如果我添加/删除项目，它可能会影响 for 循环。但我只是替换循环内的项目。我认为它不应该构成问题。
您在循环结束之前将项目添加到数组中。
@PrathameshGharat 替换数组元素将导致数组被重新索引。任何人都猜测这会对循环结构产生什么影响。
感谢 GordonM 感谢您的详细解释。现在有道理了，我想在 for 循环之后执行 array_splice 本来可以工作，但是直接访问数组要好得多，这要感谢 Fragmentedreality。

标签： php multidimensional-array foreach infinite-loop array-splice

【解决方案1】：

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

这仍然在修改循环内的数组（不酷），但它不会弄乱索引。

【讨论】：

谢谢，我试试。但我在这里尝试了原始代码..tinyurl.com/c5qh3ha 它确实按预期工作..