🚀 go-pugleaf

On Thu, 03 Sep 2009 16:32:32 +0200, emila <warunek@poczta.onet.eu> wrote:

> Mariusz Chwalba pisze:
>> On Thu, 03 Sep 2009 15:34:57 +0200, dziobacz <aaabbbcccdaabb@gmail.com>  
>> wrote:
>>  [...]
>>> session_start();
>>> $koszyk = klasaKoszyk::getInstance();
>>  To chyba nie jest kompletny kod, prawda? W tym miejscu za każdym razem
>> inicjowałbyś sesję ponownie.
> w którym miejscu??

W getInstance(). PHP sam z siebie jest bezstanowy, więc private static
$koszyk_na_zakupy przy każdym wywołaniu skryptu będzie pusty. Wywoła
zatem konstruktora, który zaś nadpisze $_SESSION['koszyk'].

On Thu, 03 Sep 2009 16:12:43 +0200, dziobacz <aaabbbcccdaabb@gmail.com>  
wrote:

> Nie, ten kod jest kompletny - tzn. mam pare drobnych rzeczy jeszcze,
> ale sa one nieistotne.  Odnośnie session_register() - to nie
> korzystam, nie wiem nawet co to jest, ale z tego co wlasnie
> przeczytalem to nie zaleca sie tego uzywac, wiec chyba dobrze robieTak  
> czy siak nie rozumiem dlaczego dostaje ten blad.

W jakiś sposób przypisujesz $koszyk do $_SESSION['koszyk']. Jeżeli nie
sam, to PHP robi to za Ciebie. Będę strzelał w register_globals w php.ini
ustawiony na "true". Przy drugim wywołaniu skryptu przypisuje on ci do
$kosztyk automatycznie $_SESSION['koszyk']. Gdy Twój getInstance zwraca
nowy klasaKoszyk, przypisujesz go zatem nie do zwykłej zmiennej, ale do
zmiennej sesyjnej. Stąd przy następnym wywołaniu w $_SESSION['koszyk'] masz
nie tablicę, a obiekt.

Dla apache register_globals wyłączysz tworząc plik .htaccess (z kropką) o
treści:

php_flag register_globals off

HTH,
-- 
Mariusz 'koder' Chwalba