[PHP]foreach a if, pętla wykonuje zarówno if jak i else Opcje

[matyskiewicz]

Witam
Napisałem funkcję:

[PHP] pobierz, plaintext 
<?php
function addToCart($zamId, $zamIlosc)
{
    $item = isset($_SESSION['cart']) ? $_SESSION['cart'] : null; //deklaracja
    
    if($item == null) //dodajemy do tablicy $_SESSION['cart'][0] dane
    {
        $_SESSION['cart'][] = array('id' => $zamId, 'ilosc' => $zamIlosc); 
    }
    else //sprawdzamy czy id istnieje już w tablicy jeśli tak to zwiększamy o jeden, jeśli nie to dodajemy $_SESSION['cart'][n+1]
    {
        foreach($_SESSION['cart'] as $key => $value) 
        {
               if ($value['id'] == $zamId) 
               {
                   $_SESSION['cart'][$key]['ilosc']++;  
                 echo'zwiększono o 1'; 
                 break;                 
               }
 
               else
               {
                $_SESSION['cart'][] = array('id' => $zamId, 'ilosc' => $zamIlosc);
                echo'dodno nowy';                   
                        }
        }    
    }    
}
?>
[PHP] pobierz, plaintext 

Funkcja działa następująco
1.dodajemy dane do tablicy
a) jeśli chcemy zwiększyć wartość ilość - dodaje
(IMG:http://forum.php.pl/style_emoticons/default/cool.gif) jeśli dodamy nowe dane - dodaje
2. jeśli w bazie tabeli mamy kilka elementów i chcemy zwiększyć ilość powiedzmy w 1 to foreach wykona zarówno to co jest w if jak i w else... Proszę o wskazówkę gdzie popełniłem błąd.Pozdrawiam

EDIT przykład tablicy po dodaniu nowego elementu, a następnie aktualizacji poprzedniego:

[PHP] pobierz, plaintext 
<?php
Array
(
    [0] => Array
        (
            [id] => 6
            [ilosc] => 4
        )
 
    [1] => Array
        (
            [id] => 23
            [ilosc] => 1
        )
 
    [2] => Array
        (
            [id] => 6
            [ilosc] => 1
        )
 
)
?>
[PHP] pobierz, plaintext 

Ten post edytował matyskiewicz 19.05.2009, 11:19:11

[nospor]

podpowiedź:
w foreach iterujesz po wszystkich elementach. Dopoki nie natrafisz na ten wlasciwy, to dodajesz do tablicy jak leci. Dopiero jak natrafisz na wlasciwy to zwiekszasz licznik i konczysz iteracje.
No i tu jest pies pogrzebany. pomysl jak to naprawic (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[matyskiewicz]

iterujesz

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[nospor]

foreach + iterowac = przechodzic przez kolejne elementy tablicy przy pomocy foreach

[Largo]

Witam,

Iteracja

PS. nospor był szybszy o parę sekund :-)

Pozdrawiam,
Largo

Ten post edytował Largo 19.05.2009, 11:29:26

[matyskiewicz]

Czyli teoretycznie odwrócenie instrukcji powinno pomóc

[PHP] pobierz, plaintext 
<?php
foreach($_SESSION['cart'] as $key => $value) 
        {
               if ($value['id'] !== $zamId) 
               {
                $_SESSION['cart'][] = array('id' => $zamId, 'ilosc' => $zamIlosc);
                echo'dodno nowy';            
               }
               else
               {
                   $_SESSION['cart'][$key]['ilosc']++;  
                 echo'zwiększono o 1'; 
                 break;
               }
        }
?>
[PHP] pobierz, plaintext 

ale problem pojawia się ponownie po operacji:
1. dodaje 1 wartosc
2. podaje 2 wartosc
3. aktualizuje 1 wartosc
4. aktualizuje 2 wartość:

[HTML] pobierz, plaintext 
Array
(
    [0] => Array
        (
            [id] => 6
            [ilosc] => 2
        )
 
    [1] => Array
        (
            [id] => 14
            [ilosc] => 2
        )
 
    [2] => Array
        (
            [id] => 14
            [ilosc] => 1
        )
[HTML] pobierz, plaintext 

PS.
Dziękuję za definicję (myślałem że popełniłeś literówkę)

Ten post edytował matyskiewicz 19.05.2009, 11:45:19

[nospor]

Czyli teoretycznie odwrócenie instrukcji powinno pomóc

PRzeciez to jest dokladnie to samo tylko inaczej zapisane....

dobra, niech ktos inny przejmie paleczke (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Dziękuję za definicję (myślałem że popełniłeś literówkę)

I ze niby co tam mialo byc zamiast slowa "iterować" ? Ciekawi mnie z jakiego wyrazu mialaby byc ta literowka (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[matyskiewicz]

[PHP] pobierz, plaintext 
<?php
foreach($_SESSION['cart'] as $key => $value) 
        {
               if ($value['id'] == $zamId) 
               {
                $_SESSION['cart'][$key]['ilosc']++;          
               }
        }
?>
[PHP] pobierz, plaintext 

ok, aktualizuje lecz nie dodaje nowego wpisu. W jaki sposób więc mogę sprawdzić czy pętla coś zwróciła?

[erix]

Yyyy...?

[PHP] pobierz, plaintext 
<?php
$jest = false;
for/foreach/while(){
  $jest = true;
  //...
}
?>
[PHP] pobierz, plaintext 

Zwracać może funkcja, a nie pętla... Sprecyzuj.

[matyskiewicz]

Opornie ale się udało:

[PHP] pobierz, plaintext 
<?php
function addToCart($zamId, $zamIlosc)
{
    $item = isset($_SESSION['cart']) ? $_SESSION['cart'] : null;
    
    if($item == null)
    {
        $_SESSION['cart'][] = array('id' => $zamId, 'ilosc' => $zamIlosc);
    }
    else
    {
        foreach($_SESSION['cart'] as $key => $value) 
        {
               if ($value['id'] == $zamId) 
               {
                   $updateId = $key;
                $update = true;    
               }
        }
                
        if($update == false)
        {
            $_SESSION['cart'][] = array('id' => $zamId, 'ilosc' => $zamIlosc);
        }
        else
        {
            $_SESSION['cart'][$updateId]['ilosc']++;
            $update = false;    
        }
    }        
    
    header("location: index.php?module=shoppingCart");
}
?>
[PHP] pobierz, plaintext 

[PrinceOfPersia]

Opornie ale się udało:

[PHP] pobierz, plaintext 
<?php
foreach($_SESSION['cart'] as $key => $value) 
        {
               if ($value['id'] == $zamId) 
               {
                   $updateId = $key;
                $update = true;    
               }
        }
?>
[PHP] pobierz, plaintext 

i break po tym. Po co ma dalej przechodzić jak już znalazł.

$updateId = $key;
$update = true;
break;

[erix]

Nie wiem, za co dostałem pomógła, ale Twój kod ustawia zmienną update, a i tak jej nie zwraca poza funkcję. ;p