0

すべてが完全に機能しますが、カートに複数のアイテムがあり、アイテムの数量 (リストの最後のアイテムを除く) が変更されると、以下のコードが無限ループに入ります。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
}

PS

これは、最初のアイテムが追加されたときに配列が初期化される方法です。

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

他の詳細が必要な場合はお知らせください..

更新: カートに 3 つのアイテムがあるとします。そして、3番目の商品の数量を変更します。それは動作します。しかし、2 番目のアイテムの数量を変更すると、スクリプトが最大実行時間に達し、2 番目と 3 番目のアイテムがカート内で無限に繰り返されます。

4

1 に答える 1

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;
    }
}

これはまだループ内の配列を変更しています(クールではありません)が、インデックスを混乱させることはありません。

于 2012-04-19T07:10:51.433 に答える