[PHP] KOSZYK SKLEPU, dodawanie produktu do koszyka Opcje

[kielich]

Witam,
Tworzę koszyk i mam błąd który nie wiem jak obejść mianowicie o to kod:

[PHP] pobierz, plaintext 
        $koszyk = new Zend_Session_Namespace('koszyk'); //Zmienna sesyjna koszy 
 
        $dane['id'] = $id;
        $dane['sztuk'] = 1;
        $koszyk->dane[] = $dane;
 
 
       if($id==1): //sprawdzamy czy juz jest to id (ja sprawdzałem tylko na ID 1 - dla większego komfortu  )
          foreach($koszyk->dane as $dane => $k):
             $koszyk->dane[$dane]['sztuk'] ++; //zwiekszamy liczbe sztuk o jeden 
          endforeach;
      endif;
 
           print_r($koszyk->dane);
[PHP] pobierz, plaintext 

i błąd polega na tym że dodaje tę 1 sztukę więc tak jak ma dodawać ale dodatkowo tworzy nowe rekordy w tablicy a ja bym chciał aby zmienił tylko ilość w danym produkcie.
Gdzie się zakręciłem

Ten post edytował kielich 8.08.2010, 21:23:06

[nospor]

$koszyk->dane[] = $dane;
przeciez ten kod zawsze dodaje do tablicy kolejny wpis.

[kielich]

tak wiem . Wypisałem tak

[PHP] pobierz, plaintext 
echo $koszyk->dane['0']['sztuk'];
[PHP] pobierz, plaintext 

I jest OK jeśli mam 1 produkt i kilka razy dodany wiec sztuki się zwiększaja ale jest problem kiedy jest wiecej niz 2 produkty wtedy nie wiem jak znaleźć dany rekord


Więc jak mogę to rozwiązać?

[dr_bonzo]

Do koszyka wstawiaj strukture/obiekt ktor bedzie zawieraj ID produktu i jego ilosc. Wtedy przegladajac zawartosc koszyka mozesz sprawdzic jaki produkt akurat przegladasz i do niego dodac kolejna sztuke itp.

[kielich]

napisałem tak

[PHP] pobierz, plaintext 
      $koszyk = new Zend_Session_Namespace('koszyk');
 
      if(isset($id)):
         $ok = true;
             foreach ($koszyk->dane as $kosz => $opcja):
                echo 'To jest ID: '.$opcja['id'].' ma sztuk: '.$opcja['sztuk'].'<br />';
                    if($opcja['id']==$id):
                        $koszyk->dane[$kosz]['sztuk']++;
                        $ok = false;
                    endif;
            endforeach;
 
            if($ok):
                $dane['id'] = $id;
                $dane['sztuk'] = 1;
                $koszyk->dane[]= $dane;
            endif;
      endif;
[PHP] pobierz, plaintext 

I działa to już znacznie lepiej ale jest gdzieś błąd ponieważ wpisuje w url id 4 i enter i dodaje mi do tablicy klikam jeszcze raz dodaje mi następną sztukę czyli tak jak chce
Wpisuje id 6 klikam i dodaje się do tablicy z tym że dodatkowo dodaje jedna sztukę do poprzedniej czyli do ID 4 ale kiedy odświeżę to normalnie dodaje do 6 jak należy .
Co jest tu nie tak ?

Moim zdaniem problem leży w if' w foreach ponieważ zawsze będzie spełniony a muszę sprawdzić czy id jest już w tablicy analogicznie mogę użyć array_key_exists lecz kiedy zrobie tak

[PHP] pobierz, plaintext 
if(array_key_exists($id,$opcja['id'])):
[PHP] pobierz, plaintext 

wywala błędy array_key_exists() expects parameter 2 to be array, integer given

KURDE NIKT NIE JEST W STANIE MI POMÓC ;:(

[zegarek84]

zwróć uwagę jak wstawiasz do koszyka kolejne produkty:
$koszyk->dane[]= $dane;
tutaj kolejne produkty będą miały kolejne klucze... w sumie tak też może być ale mogło być np.
$koszyk->dane[$id_produktu]= $dane; - okej, pomińmy to idźmy dalej... w $opcja['id'] trzymasz ten identyfikator, niech i tak będzie, ale funkcja array_key_exists($id,$opcja['id']) dokładnie sprawdza czy istnieje klucz w tablicy (która u Ciebie nie jest tablicą bo to identyfikator...) $opcja['id'] o nazwie $id... czyli prawie jakby to samo co sprawdzać czy istnieje zmienna w lokalizacji $opcja['id'][$id]...

I działa to już znacznie lepiej ale jest gdzieś błąd ponieważ wpisuje w url id 4 i enter i dodaje mi do tablicy klikam jeszcze raz dodaje mi następną sztukę czyli tak jak chce smile.gif
Wpisuje id 6 klikam i dodaje się do tablicy z tym że dodatkowo dodaje jedna sztukę do poprzedniej czyli do ID 4 ale kiedy odświeżę to normalnie dodaje do 6 jak należy .
Co jest tu nie tak questionmark.gif?

nie napisałeś jak wyświetlasz sobie ten koszyk... założę się, że dodajesz kolejny produkt od razu wyświetlając zawartość koszyka i zapewne robisz coś na styl:
$i=2;
echo $i++; // ile się wyświetli - 2 a zmienna ma 3, czyli niżej
echo $i; // 3, zaś:
echo ++$i; // 4 i zmienna jest 4

[kielich]

Dziękuje za opowiedz ale szczerze mówiąc do niczego mnie to nie nakierowało

[zegarek84]

pokaż gdzieś ale w całości skrypt którym dodajesz produkty i którym wyświetlasz zawartość koszyka - z tego co pisałeś na 99% jest to ten sam skrypt gdzie wszystko się dzieje w jednej akcji...

[kielich]

O to cały kod

[PHP] pobierz, plaintext 
     public function koszykAction()
     {
         $filtr = new Application_Model_statyczne_filtry();
         $id = $filtr->filtrowanie_id($this->getRequest()->getParam('id'));
         $koszyk = new Zend_Session_Namespace('koszyk');
         $koszyk->setExpirationSeconds(7200);
 
         if($id):
 
      if(isset($id)):
         $ok = true;
             foreach ($koszyk->dane as $kosz => $opcja):
                echo 'To jest ID: '.$opcja['id'].' ma sztuk: '.$opcja['sztuk'].'<br />';
                    if($opcja['id']==$id):
                        $koszyk->dane[$kosz]['sztuk']++;
                        $ok = false;
                    endif;
            endforeach;
 
            if($ok):
                $dane['id'] = $id;
                $dane['sztuk'] = 1;
                $koszyk->dane[]= $dane;
            endif;
      endif;
 
endif;
[PHP] pobierz, plaintext 

Jeśli nie pracujesz z zendem a chcesz mi pomóc do zapodaj to sobie normalnie na localu zmien tylko $id na GET oraz pozostałe zmienne

Ten post edytował kielich 9.08.2010, 11:31:56

[zegarek84]

O to cały kod...

Prawie, jednak nie cały bo i metoda nie domknięta a i nie ma przykładu użycia tej metody - ale wystarczy

...Jeśli nie pracujesz z zendem a chcesz mi pomóc do zapodaj to sobie normalnie na localu zmien tylko $id na GET oraz pozostałe zmienne

w ogóle przy komputerach nie pracuję ^^ - a do zenda czasem zaglądam coby nieraz podpatrzeć zależności klas - jednak z samego zenda jakoś nigdy nie korzystałem - ale jestem wzrokowcem to i tego kodu nie musiałem odpalać ...

skoro to prawie działa to zamień ten foreach:

[PHP] pobierz, plaintext 
foreach ($koszyk->dane as $kosz => $opcja):
    echo 'To jest ID: ' . $opcja['id'] . ' ma sztuk: ' . $opcja['sztuk'] . '<br />'; //zauważ, że w pierwszej kolejności wyświetliłeś a dopiero potem dodałeś do koszyka - nie miałeś tego na ekranie to tego nie zauważyłeś...
    if ($opcja['id'] == $id):
        $koszyk->dane[$kosz]['sztuk']++;
        $ok = false;
    endif;
endforeach;
[PHP] pobierz, plaintext 

na ten:

[PHP] pobierz, plaintext 
foreach ($koszyk->dane as $kosz => $opcja):
    if ($opcja['id'] == $id):
        $koszyk->dane[$kosz]['sztuk']++;
        $ok = false;
    endif;
    echo 'To jest ID: ' . $opcja['id'] . ' ma sztuk: ' . $opcja['sztuk'] . '<br />';
endforeach;
[PHP] pobierz, plaintext 

oczywiście od tak tylko piszę, że ten ciąg powinien być w tej metodzie zwrócony a nie wyświetlony od razu na ekran ...
PS.
chyba dawniej sporo pisałeś w czymś w stylu VBA ... - a i przy okazji w językach skryptowych szybsze jest ++$i od $i++, w językach kompilowalnych w zasadzie to różnicy nie robi... a różnica jest taka, że przy $i++ tworzona jest dodatkowa zmienna a dopiero po użyciu starej zmiennej bądź w następnych linijkach kodu zastępowana przez nową wartość...

[kielich]

No jednak to nic nie dało kiedy dodam do koszyk wyjde wybieram nowy produkt i klikam do koszyk to dodaje ale wartość poprzednio dodanego zwiększa o jeden

[zegarek84]

to podaj gdzieś cały kod - zmienić się zmieniło bo wcześniej echo wyświetlało Ci zawartość koszyka przed dodaniem +1, teraz echo się wyświetla po dodaniu +1...

podaj cały kod z przykładem urzycia gdzieć - czy to na wklej czy gdziekolwiek... podaj w komentarzu u góry jeszcze adres koszyka przykładowy i przykładowy adres produktu - może metodę tą masz także w określonym produkcie użytą - gdyż jeśli tak to tam też dodajesz produkt o ile jest ta zmienna w adresie...

[kielich]

Napisałem cały kod już jest po prostu użyty poprzez kontroler np. localhost/koszyk/5
gdzie 5 do ID

[zegarek84]

czy na pewno zamieniłeś sobie te pętle foreach??... nie masz tam czasem redirecta gdzieś [np. po dodaniu pierwszego produktu, gdyz kod podałeś bez domknięcia...], co jest zwracane tutaj [czy na pewno liczba??]:
$id = $filtr->filtrowanie_id($this->getRequest()->getParam('id'));
dlaczego niby twierdzisz??:

Moim zdaniem problem leży w if' w foreach ponieważ zawsze będzie spełniony

wpisz może var_dump($id) zaraz po przypisaniu jej wartości?? - czy to string czy liczba...

[kielich]

$id jest liczba(integer) jak widać jest filtrowane i jest to id produktu dodane do koszyka a o to cały kod

[PHP] pobierz, plaintext 
public function koszykAction()
     {
         $filtr = new Application_Model_statyczne_filtry();
         $id = $filtr->filtrowanie_id($this->getRequest()->getParam('id'));
         $koszyk = new Zend_Session_Namespace('koszyk');
         $koszyk->setExpirationSeconds(7200);
if($id):
         $ok = true;
             foreach ($koszyk->dane as $kosz => $opcja):
                 if ($opcja['id'] == $id):
                   $koszyk->dane[$kosz]['sztuk']++;
                   $ok = false;
                 endif;
     echo 'To jest ID: ' . $opcja['id'] . ' ma sztuk: ' . $opcja['sztuk'] . '<br />';
              endforeach;
 
            if($ok):
                $dane['id'] = $id;
                $dane['sztuk'] = 1;
                $koszyk->dane[]= $dane;
            endif;
      endif;
 
 
     }
[PHP] pobierz, plaintext 

Odpal to sobie jest możesz ponieważ kompilujesz wzrokowo i może coś pominąłeś Może jakieś inne działające rozwiązanie . Kurcze od kilku dni się z tym meczę a błąd pewnie banalny .

Ten post edytował kielich 9.08.2010, 13:46:24

[zegarek84]

zamień tą metodę na inną - trochę mi tu coś nie gra - nie wiem, czy masz gdzieś redirekta czy co [akurat nie w metodzie], jeśli już to wcześniej zobaczysz skok o 2 [zamieniłem też tutaj strukturę tabeli koszyka]:

[PHP] pobierz, plaintext 
public function koszykAction() {
    $filtr = new Application_Model_statyczne_filtry();
    $id = (integer) $filtr->filtrowanie_id($this->getRequest()->getParam('id'));
    $koszyk = new Zend_Session_Namespace('koszyk');
    $koszyk->setExpirationSeconds(7200);
 
    if (isset($id) && $id !== 0):
        if (!isset($koszyk->dane[$id])) { //DODAWANIE
            $dane = array('sztuk'=>1);
            $koszyk->dane[$id] = $dane;
        } else { //ZWIĘKSZANIE LICZBY
            ++$koszyk->dane[$id]['sztuk'];
        }
        // WYŚWIETLANIE
        foreach ($koszyk->dane as $id_key => $info_produkt):
            echo 'To jest ID: ' . $id_key . ' ma sztuk: ' . $info_produkt['sztuk'] . '<br />';
        endforeach;
    endif;
}
// KOSZYK O STRUKTURZE
//$koszyk->dane=array(
//    INTEGER_klucz_produktu => ARRAY_dodatkowe_dane_produktu
//);
[PHP] pobierz, plaintext 

a wogóle to bym to podzielił na 2 części razem z redirektem gdyż różne rzeczy o indexacji słyszałem...

[PHP] pobierz, plaintext 
class koszyk {
 
    private $koszyk;
 
    public function  __construct() {
        $this->koszyk = new Zend_Session_Namespace('koszyk');
        $this->koszyk->setExpirationSeconds(7200);
    }
 
    public function dodajAction() {
        $filtr = new Application_Model_statyczne_filtry();
        $id = (integer) $filtr->filtrowanie_id($this->getRequest()->getParam('id'));
        $this->koszyk = new Zend_Session_Namespace('koszyk');
        $this->koszyk->setExpirationSeconds(7200);
 
        if (isset($id) && $id !== 0):
            if (!isset($this->koszyk->dane[$id])) { //DODAWANIE
                $dane = array('sztuk'=>1);
                $this->koszyk->dane[$id] = $dane;
            } else { //ZWIĘKSZANIE LICZBY
                ++$this->koszyk->dane[$id]['sztuk'];
        }
        endif;
        // PRZEKIEROWUJEMY NA KOSZYK COBY WYŚWIETLIĆ ZAWARTOŚĆ
        header('Location: ' . $ADRES_DO_KOSZYKA);
    }
 
    public function koszykAction() {
        // WYŚWIETLANIE ZAWARTOŚCI
        $str = '';
        foreach ($this->koszyk->dane as $id_key => $info_produkt):
            $str .= 'To jest ID: ' . $id_key . ' ma sztuk: ' . $info_produkt['sztuk'] . '<br />';
        endforeach;
        return $str;
    }
}
 
//a tak najlepiej to by było dane przekazywać metodą POST gdzie jeszcze dodatkowo mógłbyś zdefiniować ile sztuk ktoś chce kupić
class koszyk2 {
 
    private $koszyk;
 
    public function  __construct() {
        $this->koszyk = new Zend_Session_Namespace('koszyk');
        $this->koszyk->setExpirationSeconds(7200);
    }
 
    public function dodajAction() {
        $id = (integer) $_POST['id'];
        $ile = (integer) $_POST['ile'];
// PASUJE SPRAWDZIĆ ALBO TUTAJ ALBO PRZY WYŚWIETLANIU CZY JEST W BAZIE PRODUKT O TYM ID - PROPONUJĘ PRZY WYŚWIETLANIU I JEŚLI W BAZIE NIE MA TO TAKIE INDEKSY POUSUWAĆ
// TRZEBA JESZCZE PAMIĘTAĆ, BY KTOŚ NIE ZAMÓWIŁ WIĘCEJ NIŻ JEST DOSTĘPNYCH PRODUKTÓW
        // CENY NIE PRZECHOWUJ W SESJI ZWŁASZCZA JESLI KOSZYK CHOĆ KILKA GODZIN MA ŻYWOTNOŚCI - POBIERAJ PRZY WYŚWIETLANIU Z BAZY...
        if ($id !== 0 && $ile !== 0):
            if (!isset($this->koszyk->dane[$id])) { //DODAWANIE
                $dane = array('sztuk'=>$ile);
                $this->koszyk->dane[$id] = $dane;
            } else { //ZWIĘKSZANIE LICZBY - o tyle ktoś dokupuje
                $this->koszyk->dane[$id]['sztuk'] += $ile;
        }
        endif;
        // PRZEKIEROWUJEMY NA KOSZYK COBY WYŚWIETLIĆ ZAWARTOŚĆ
        header('Location: ' . $ADRES_DO_KOSZYKA);
    }
 
    public function koszykAction() {
        if (reset($this->koszyk->dane) === false) return false; //PUSTY KOSZYK I NIC NIE MA - ZWRACAMY JAKIŚ WIDOK PUSTKI...
        // POBIERAMY AKTUALNE INFO z bazy w stylu "SELECT * FROM produkty WHERE `id` IN (...)
        $in = implode(', ', array_keys($this->koszyk->dane));
        // $sql='SELECT * FROM `produkty` WHERE `id` IN ('.$in.')';
        //...
        //ODPYTUJEMY BAZE O AKTUALNE PRODUKTY;
        $produkty_z_bazy = array();//NIE PUSTE JUŻ... I NAJLEPIEJ TAK JUŻ UŁOŻONY JAK W KOSZYKU PO ID PRODUKTU
        // CENĘ W BAZIE PRZECHOWYWAĆ JAKO INTEGER A POTEM PODZIELIĆ PRZEZ 100..
        // JAK NIE MUSZE TO NIE UŻYWAM FOREACH - ALE RÓŻNICY TO NIE ROBI
        $to_del = array();
        $bufor = '';
        $razem = 0;
        do {
            $id_key = key($this->koszyk->dane);
            if (!isset($produkty_z_bazy[$id_key])) {
                $to_del[] = $id_key;
                continue; // POMIJAMY NIE POPRAWNE WIERSZE
            };
            // TRZA BY ZROBIĆ JESZCZE JEDNEGO IFA CZY PRODUKTU W BAZIE NIE MA MNIEJ NIŻ W KOSZYKU I JESLI TAK TO W KOSZYKU ZMIENIĆ LICZBĘ NA MNIEJSZĄ
            $sztuk = $this->koszyk->dane[$id_key]['sztuk'];
            $za_produkt_razem = $sztuk * $produkty_z_bazy[$id_key]['cena'];//cena dalej jest 100 razy większa.
            $razem += $za_produkt_razem;
            $bufor .= 'To jest ID: ' . $id_key . ' ma sztuk: '
                    . $this->koszyk->dane[$id_key]['sztuk'] . ' o łącznej wartości: '
                    . floor($za_produkt_razem / 100) . ',' . $za_produkt_razem % 100 . '<br />';
        } while (next($this->koszyk->dane) !== false);
        if (isset($to_del[0]))
            for ($i = 0, $count = count($to_del);$i < $count;++$i)
                unset($this->koszyk->dane[$i]);
        $bufor .= 'RAZEM: ' . floor($razem / 100) . ',' . $razem % 100;
        return $bufor;
    }
}
[PHP] pobierz, plaintext 

[kielich]

Czy ty na prawdę pisałeś to specjalnie dla mnie Kurcze jak to zobaczyłem to przez 30 sek nie wierzyłem ponieważ nie wszyscy mają takie chęci jak ty. Pierwszy kod (mój przerobiony) działa jak należy oczywiście dodałem jeszcze sprawdzanie czy jest tyle w bazie itd. itd. ale działa jak należy .BARDZO TOBIE DZIĘKUJE na prawdę dzięki.

[zegarek84]

Pierwszy kod (mój przerobiony) działa jak należy oczywiście dodałem jeszcze sprawdzanie czy jest tyle w bazie itd. itd. ale działa jak należy...

tylko i tak jak wspomniałem pasowało by ten pierwszy kod uszczuplić a wyświetlanie przerzucić na inną akcję choćby z tego względu, iż user gdy będzie w innej części sklepu to może będzie chciał zajrzeć do koszyka??... albo idąc dalej jak już będziesz dorabiał zarządzanie koszykiem [czyli usuwanie produktów itp.] to też potem pasuje to wyświetlić... a wystarczyło by bez wyświetlania dodawać/usuwać zawartość koszyka a po całej operacji redirektem przekierować na stronę z akcją wyświetlania koszyka - tzn. jego zawartości... w zasadzie przy zarządzaniu koszykiem tam też można by zwiększać i zmniejszać liczbę produktów a czy się zwiększa czy zmniejsza można rozpoznać po odpowiednim klikniętym przycisku formularza... i tą operację usuwania można by dać do metody dodawania tylko bardziej to rozbudować [choć dla wygody można by wydzielić do innej metody a dopiero tamtą metodę usuwania wywołać w metodzie dodawania]...

tak tylko ogólnie piszę, ale z tego co się orientuję to wielu jeśli akcja była przez linki zakupów z botami miało problemy - choćby z indeksacją koszyka w google...

ale w sumie to też pisze tak ogólnie i w zasadzie zrób jak uważasz