[PHP]Przeszkuwianie tablicy pod katem istniejącej zmiennej, Dane się dodają do tablicy pomimo warunku Opcje

[rozny]

Witam,

mój problem jest z goła błahy, ale nie zmienia to faktu, że się z nim pieszczę już zdecydowanie za długo, dlatego postanowiłem poszukać pomocy u Was

Temat - koszyk sklepowy
Problem - Dodaję towar do koszyka - trzymam to w dwuwymiarowej tablicy. Skrypt ma sprawdzić, czy taki towar już istnieje, jeżeli tak, to wyświetla info, jeżeli nie to dodaje do koszyka. Problem w tym, że nawet jeśli skrypt znajdzie już produkt w tablicy, to wyświetla info i dodaje kolejny raz ten sam produkt do tablicy.
Szukamy towaru po "$prod_id".

Tablica:

[XML] pobierz, plaintext 
array
  0 => 
    array
      'prod_id' => string '3' (length=1)
      'amount' => string '1' (length=1)
  1 => 
    array
      'prod_id' => string '1' (length=1)
      'amount' => string '1' (length=1)
  2 => 
    array
      'prod_id' => string '1' (length=1)
      'amount' => string '1' (length=1)
[XML] pobierz, plaintext 

Próbowałem na dwa sposoby chapnąć to zagadnienie, ale oba działają tak samo, dlatego stwierdzam że to problem mojego podejścia do tematu. Prosiłbym o przeanalizowanie tych skryptów i podzielenie się ze mną solucją jak to zjeść i nie zwrócić

Sposób 1:

[PHP] pobierz, plaintext 
                $product_id = trim(htmlspecialchars($_POST['product_id']));
       			$amount_of_product = trim(htmlspecialchars($_POST['amount_of_product']));
 
       			$tab = array('prod_id' => ''.$product_id.'', 'amount' => ''.$amount_of_product.'');
 
       			$iloscrekordow = count($_SESSION['sklep.koszyk']);
 
       			if ($iloscrekordow == 0) {
	           		$_SESSION['sklep.koszyk'][] = $tab;
		 	}
 
		 	else {
 
       				for($i = 0; $i<$iloscrekordow; $i++){
       					if (in_array($product_id, $_SESSION['sklep.koszyk'][$i])) {
	           				echo 'Produkt istnieje w koszyku';
 
		 			}
 
		 			else {
	     					$_SESSION['sklep.koszyk'][] = $tab;
	     					break;
	   				}
	   			}
 
  			}
[PHP] pobierz, plaintext 

Sposób 2:

[PHP] pobierz, plaintext 
                $product_id = trim(htmlspecialchars($_POST['product_id']));
       			$amount_of_product = trim(htmlspecialchars($_POST['amount_of_product']));
 
       			$tab = array('prod_id' => ''.$product_id.'', 'amount' => ''.$amount_of_product.'');
 
       			$iloscrekordow = count($_SESSION['sklep.koszyk']);
 
       			if ($iloscrekordow == 0) {
	           		$_SESSION['sklep.koszyk'][] = $tab;
		 	}
 
		 	else {
 
	   			$result=-1;
				foreach($_SESSION['sklep.koszyk'] as $key=>$val){
					if($val['prod_id']==$product_id){
						$result=$key;
					}
					else {
			    			$_SESSION['sklep.koszyk'][] = $tab;
			  		}
				}
				echo $result;
  			}
[PHP] pobierz, plaintext 

Będę wdzięczny za pomoc

[Tuminure]

O ile dobrze zrozumiałem kod i założenia, to w obydwóch sposobach na moje oko problemem jest pętla (for w pierwszym, foreach w drugim).

[PHP] pobierz, plaintext 
for($i = 0; $i<$iloscrekordow; $i++){
       					if (in_array($product_id, $_SESSION['sklep.koszyk'][$i])) {
	           				echo 'Produkt istnieje w koszyku';
 
		 			}
 
		 			else {
	     					$_SESSION['sklep.koszyk'][] = $tab;
	     					break;
	   				}
	   			}
[PHP] pobierz, plaintext 

Przypuśćmy, że zmienna $iloscrekordow jest równa 3, a poszukiwany produkt istnieje w koszyku...

1. ...w $_SESSION['sklep.koszyk'][2]. - Pętla wykonuje raz dla $i = 0. Warunek z if nie jest spełniony, więc produkt dodaje się do $_SESSION['sklep.koszyk']. Break kończy wykonywanie pętli, nie dostajesz informacji o tym, że produkt istnieje w koszyku.
2. ...w $_SESSION['sklep.koszyk'][1]. - Dzieje się wszystko dokładnie tak samo, jak w sytuacji powyżej.
3. ...w $_SESSION['sklep.koszyk'][0]. - Pętla wykonuje się dla $i = 0. Warunek z if tym razem jest spełniony i echo wyświetla tekst, po czym pętla wykonuje się dla $i = 1. Tym razem warunek nie jest spełniony i następuje dodanie do koszyka i zakończenie pętli.

Przed pętlą stworzyłbym zmienną $znaleziono (Trzymając się Twojego nazewnictwa) i przypisał do niej wartość false i zmieniał ją tylko gdy produkt zostanie znaleziony. Następnie sprawdzał, wartość $znaleziono i wyświetlał informację/dodawał do tablicy.

Ten post edytował Tuminure 28.11.2012, 11:09:13

[rozny]

Tak właśnie myślałem, że zrobiłem błąd w założeniu, ale jak się długo nad czymś siedzi to zazwyczaj logiczne postrzeganie zagadnienia ulatnia się z czasem

Dzięki za pomysł ze zmienną dla zdarzenia "found" Teraz wszystko działa jak należy.