Forum PHP.pl

mam koszyk w którym dodawanie produktów wygląda następująco:

[PHP] pobierz, plaintext 
<?php
session_start();
$dodaj = $_POST['dodaj'][0];
 
if($dodaj){
	 if (!isset($_SESSION['koszyk'])){
		  $_SESSION['koszyk'] = array();
	 }
 
	 if(isset($_SESSION['koszyk'][$dodaj])){
		  if ($_SESSION['koszyk'][$dodaj] != 99){
			   $_SESSION['koszyk'][$dodaj]++;
		  }
	 }else{
		  $_SESSION['koszyk'][$dodaj] = 1;
	 }
}
?>
[PHP] pobierz, plaintext 

wszystko niby działa bez zarzutu ale co jakiś czas pojawia się taki błąd:

linia 266 wygląda tak:

[PHP] pobierz, plaintext 
<?php
foreach ($_SESSION['koszyk'] as $id_produktu => $ilosc){
?>
[PHP] pobierz, plaintext 

mi się ten błąd nie pojawił nigdy ale widzę w logach że klienci coś takiego dostają. O co tu może chodzić?

Zanim wywolasz swojego foreach'a, to musisz najpierw sprawdzic, czy $_SESSION['koszyk'] istnieje i czy jest tablicą.
Bo przeciez te wartosc ustawiasz dopiero gdy ktos dodaje do koszyka, a jesli jeszcze nic nie dodal to tej wartosci nie ma i dostajesz bledy

czy to chodzi o coś takiego:

[PHP] pobierz, plaintext 
<?php
if($_SESSION['koszyk'] && array_count_values($_SESSION['koszyk'])){
	 foreach ($_SESSION['koszyk'] as $id_produktu => $ilosc){
		  // tutaj zapisuję produky do bazy
	 }
}
?>
[PHP] pobierz, plaintext 

tylko problem w tym że to jest foreach którym pobieram produkty z koszyka i zapisuję w bazie (jako zamówienie), nie da się go wywołać jeżeli w koszyku nic nie ma (jeżeli koszyk jest pusty to nie ma linku który umożliwiłby złożenie zamówienia). wcześniej gdy wyświetlam zawartość koszyka mam zrobione tak jak tu napisałem.

Najwyraźniej jednak sie da, skoro masz takie logi

A z tym warunkiem to przekombinowales. zrob tak:

[PHP] pobierz, plaintext 
<?php
if(isset($_SESSION['koszyk']) && is_array($_SESSION['koszyk']))
?>
[PHP] pobierz, plaintext 

czyli kod zapisujący zamówienie powinien wyglądać tak:

[PHP] pobierz, plaintext 
<?php
if(isset($_SESSION['koszyk']) && is_array($_SESSION['koszyk'])){
	 foreach ($_SESSION['koszyk'] as $id_produktu => $ilosc){
		  // tutaj zapisuję produky do bazy
	 }
}else{
	 echo "przepraszamy ale wystąpił błąd, proszę spróbować ponownie";
}
?>
[PHP] pobierz, plaintext 

link kierujący do złożenia zamówienia wyświetlany jest pod zawartością koszyka (tylko jeśli coś się w nim znajduje), nie ma innej drogi na złożenie zamówienia
ta sytuacja wygląda tak że gdy klient kliknie "złóż zamówienie" i przejdzie do strony gdzie podaje swoje dane to zanim potwierdzi zamówienie to gubiona jest zawartość koszyka...

A jesli ktos zapamietal ten link i potem wklepal go bezposrednio do przeglądarki?

Teraz ten kod wyglada sensownie

znów mi się pojawił ten błąd, tym razem w tym foreach:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['aktualizacja'])){   
	 foreach ($_SESSION['koszyk'] as $id_produktu => $ilosc){
		  if($_POST[$id_produktu]=='0' || !is_numeric($_POST[$id_produktu])){
			   unset($_SESSION['koszyk'][$id_produktu]);
		  }else{
			   $_SESSION['koszyk'][$id_produktu] = $_POST[$id_produktu];
		  }
	}
}
?>
[PHP] pobierz, plaintext 

jest to kod który uaktualnia liczbę produktów w koszyku, czyli np. miałem 1, wpisuję że chcę 3 i wciskam przycisk "aktualizuj koszyk" który wywołuje ten kod i powraca do tego samego adresu czyli mojastrona.pl/koszyk
tym razem jeśli ktoś zapamiętałby adres i wszedł bezpośrednio na mojastrona.pl/koszyk to jest sprawdzenie czy istnieje $_SESSION['koszyk'] i czy jest tablicą więc co najwyżej dostałby tekst że koszyk jest pusty
w jaki sposób w powyższym kodzie wywołać taki błąd?

To jest dokladnie ten sam blad? Napewno wskazuje na ten kod? No i czy teraz caly kod tej petli?
Bo przy tym kodzie co podales, to trudno o ten blad, a przynajmniej mi sie nie udalo a probowalem
No i zakladam ze naprawde jesli nie ma sesji koszyk to ten kod sie nie wykona. Bo tu tego warunku nie widze

dodawanie, aktualizacja i wyświetlanie produktów wykonywane jest za pomocą tego kodu (wyrzuciłem tutaj część odpowiedzialną za wyświetlanie zawartości)

[PHP] pobierz, plaintext 
<?php
session_start();
$dodaj = $_POST['dodaj'][0];
 
if($dodaj){
	 if (!isset($_SESSION['koszyk'])){
		  $_SESSION['koszyk'] = array();
	 }
 
	 if(isset($_SESSION['koszyk'][$dodaj])){
		  if ($_SESSION['koszyk'][$dodaj] != 99){
			   $_SESSION['koszyk'][$dodaj]++;
		  }
	 }else{
		  $_SESSION['koszyk'][$dodaj] = 1;
	 }
}
 
if(isset($_POST['aktualizacja'])){   
	 foreach ($_SESSION['koszyk'] as $id_produktu => $ilosc){
		  if($_POST[$id_produktu]=='0' || !is_numeric($_POST[$id_produktu])){
			   unset($_SESSION['koszyk'][$id_produktu]);
		  }else{
			   $_SESSION['koszyk'][$id_produktu] = $_POST[$id_produktu];
		  }
	}
}
 
if($_SESSION['koszyk'] && array_count_values($_SESSION['koszyk'])){
	 //wyświetlanie zawartości koszyka
}else{
	 $output .= "<br><br><center>Koszyk jest pusty.</center>";
}
?>
[PHP] pobierz, plaintext 

ja nie widzę innej drogi żeby dostać się do tego foreach niż poprzez kliknięcie przycisku "aktualizuj koszyk", a jeśli tak to znaczy że jestem w koszyku w którym się coś znajduje czyli sejsa istnieje (ale oczywiście mogę się mylić)