2013-10-24 2 views
0

У меня есть массив корзин для небольшого сайта электронной коммерции, который я создаю, и столкнулся с циклом, который я не могу понять. Если у меня есть 3 разных продукта (не уверен, что количество продуктов имеет значение более 2) в моем массиве корзин (с разными ID #), и я пытаюсь обновить количество второго элемента, это вызывает бесконечный цикл и попытки постоянно добавлять продукт снова в качестве нового продукта, а не обновлять существующий indentical продукт.PHP Infinite Loop UGH

<?php 
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////// 
//  Section 3 (if user chooses to adjust item quantity) 
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////// 
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) { 
       $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 
} 
?> 

Я просто хочу, чтобы он обновил количество существующего продукта, а не добавлял его в качестве другого. Заранее благодарим за помощь!

+0

Это своего рода трудно определить причину .. но один ключ к сведению, попробуйте использовать Еогеасп вместо цикла при использовании Еогеасп оптимизирован для итерации по коллекции. Также я не чувствую этот массив (массив (.. вещь. – xlordt

ответ

1

Когда вы достигаете команды array_splice, возможно, вы сбросите указатель на массив, поэтому, когда foreach выполняет итерацию над следующим элементом, он фактически начинается с первого элемента снова.

Что я предлагаю вам сделать, устанавливается флаг после array_splice и выходит из цикла while. Затем проверьте флаг перед следующей итерацией foreach и выйдите из него, если он установлен.

т.е.

array_splice($_SESSION["cart_array"], $i-1, 1, array(array("item_id" => $item_to_adjust, "quantity" => $quantity))); 
    $breakflag=true; 
    break; 
} 
if($breakflag){ 
    break; 
}