Php добавить в корзину проблемы - PullRequest
Новая
       15

Php добавить в корзину проблемы

2 голосов
/ 09 мая 2011

Я пытаюсь создать функцию php, которая добавляет товар в корзину. я хочу, чтобы он проверял массив, чтобы увидеть, есть ли уже там товар, увеличит ли он количество, если не создаст товар в корзине.

Вместо этого он добавляет элемент, он будет работать в первый раз (если элементы уже есть, он просто увеличит количество), но если вы добавите другой элемент, он продолжит создавать новые экземпляры этого элемента. в корзине * например 1003 *

позиция 1 - количество 4 позиция 2 - количество 1 позиция 2 - количество 1 пункт 2 - количество 1 ... и так далее ...

ниже код, который у меня есть? 

function add_item ($id, $qty)
    {
        $count=$this->countItems;
        echo  "uytfdgghjkl;kj<br>";
        $added = false;
        if($count>0)
        {
            $i=0;
            while($added == false)
            {
                echo "fghjkl<br>";
                $tid = $this->items[$i]->getId();
                echo "new ID: ".$tid."<br>";
                echo "old ID: ".$id."<br>";
                echo $i;
                if($tid == $id)
                {
                    $amount = $this->items[$i]->getQty();
                    $this->items[$i]->setQty($amount+1);
                    $added = true;
                    //$i++;
                    //break;
                }
                if($added == true)
                {
                    break;
                }
                else //if($added == false)
                {
                    $this->items[$this->countItems] = new OrderItem($id, $qty);
                    //$this->total = $total+ ($qty *$price);
                    $this->countItems++;
                    $added = true;
                    //break;
                }
                //else break;
                $i++;
            }

        }
        else
        {
            $this->items[$this->countItems] = new OrderItem($id, $qty);
            //$this->total = $total+ ($qty *$price);
            $this->countItems++;
        }
    }

Ответы [ 2 ]

0 голосов
/ 09 мая 2011

Логика ошибочна.Код будет увеличивать количество товара, только если это будет первый товар в корзине.В противном случае он добавит элемент.Свидетельство здесь: 

if($added == true) // if this cart item's quantity was incremented
{
    break;
}
else // add item
{
    $this->items[$this->countItems] = new OrderItem($id, $qty);
    // etc...
}

Вместо этого вы должны удалить new OrderItem($id, $qty) из цикла и проверить значение $added после прохождения всех элементов корзины.Чтобы это работало, вам нужно выполнить цикл через for (вместо времени), используя $count.

0 голосов
/ 09 мая 2011

Проблема в том, что вы сначала не просматриваете весь массив, чтобы увидеть, присутствует ли в нем элемент.Приведенный ниже код должен работать, но я, возможно, сделал опечатку или что-то еще, поэтому убедитесь, что вы дважды проверили его. 

function add_item ($id, $qty)
    {
        $count=$this->countItems;
        echo  "uytfdgghjkl;kj<br>";
        $added = false;
        if($count>0)
        {
            for($i=0; $i < $count; $i++)
            {
                echo "fghjkl<br>";
                $tid = $this->items[$i]->getId();
                echo "new ID: ".$tid."<br>";
                echo "old ID: ".$id."<br>";
                echo $i;
                if($tid == $id)
                {
                    $amount = $this->items[$i]->getQty();
                    $this->items[$i]->setQty($amount+1);
                    $added = true;
                    break;
                }
            }

        }
        if(!$added)
        {
            $this->items[$this->countItems] = new OrderItem($id, $qty);
            //$this->total = $total+ ($qty *$price);
            $this->countItems++;
        }
    }

Еще лучшим вариантом будет использование словаря

, т.е.. 

$arr = array();
$arr['item_id'] = new OrderItem(...);

Затем вы можете проверить, находится ли элемент в массиве, используя: 

if(isset($arr[$id])){
    ...
}
...