[PHP]Problem z foreach, Pętla foreach Opcje

[lukasz_web]

Opisem wstępu robię w sklepie internetowym moduł szybkich zakupów w którym trzeba podać numer katalogowy i ilość zamawianego produktu

[PHP] pobierz, plaintext 
 
foreach($_POST['products_model'] as $products_model) {
                $thulequery="SELECT products_id FROM products WHERE products_model = '{$products_model}' ";
                $thule_products_id_query = mysql_query($thulequery) or die(mysql_error());
		$thule_products_id =  mysql_result($thule_products_id_query,0);
 
 
foreach($_POST['quantity'] as $ilosc_p) {
 
 
$cart->add_cart($thule_products_id, $cart->get_quantity($thule_products_id)+$ilosc_p);
 
}
}
[PHP] pobierz, plaintext 

Pierwsza pętlą foreach zachowuje się poprawnie dodaje mi numery katalogowe do koszyka w sklepie internetowym. Problem polega z drugą która ma dodawać ilości sztuk do koszyka.

Problem polega na tym że jak dodam taki input:
[nr. kat][ilość]
[0001][2]
[0002][4]

Do koszyka doda mi te dwa numery katalogowe ale ilości sumuje z inputów do pojedynczego numeru katalogowego w KOSZYKU:
[nr. kat][ilość]
[0001][6]
[0002][6]

A powinno dodać do numeru 0001 2 szt. a do numeru 0002 4szt.

[SmokAnalog]

Przeanalizuj taki kod:

[PHP] pobierz, plaintext 
$products = array();
 
$products_number = count($_POST['products_model']);
 
for($i = 0; $i < $products_number; $i += 1) {
	$products_model = $_POST['products_model'][$i];
	$quantity = $_POST['quantity'][$i];
 
	if(isset($products[$product_model])) {
		$products[$product_model]['quantity'] += $quantity;
	} else {
		$thulequery="SELECT products_id FROM products WHERE products_model = '{$products_model}' ";
		$thule_products_id_query = mysql_query($thulequery) or die(mysql_error());
 
		$products[$product_model] = array(
			'id'       => mysql_result($thule_products_id_query,0),
			'quantity' => $quantity,
		);
	}
}
 
foreach($products as $product) {									
	$cart->add_cart($product['id'], $cart->get_quantity($product['id']) + $product['quantity']);
}
[PHP] pobierz, plaintext 

[lukasz_web]

Dzięki ale nadal ten sam wynik:
[nr. kat][ilość]
[0001][6]
[0002][6]
A powinno dodać do numeru 0001 2 szt. a do numeru 0002 4szt.

Ale analizuje twój kod i może się uda na nim coś znaleźć by rozwiązać ten problem

[SmokAnalog]

To coś źle robisz. Pokaż formularz.

[lukasz_web]

Pełen kod z twoja zmianą:

[PHP] pobierz, plaintext 
case 'add_product_quick' :
		$query_price_to_guest_result = ALLOW_GUEST_TO_SEE_PRICES;	
		if ((($query_price_to_guest_result=='true') && !(tep_session_is_registered('customer_id'))) || ((tep_session_is_registered('customer_id')))) {
        if (isset($_POST['products_model'])) {
 
          	$products = array();
 
$products_number = count($_POST['products_model']);
 
for($i = 0; $i < $products_number; $i += 1) {
	$products_model = $_POST['products_model'][$i];
	$quantity = $_POST['quantity'][$i];
 
	if(isset($products[$product_model])) {
		$products[$product_model]['quantity'] += $quantity;
	} else {
		$thulequery="SELECT products_id FROM products WHERE products_model = '{$products_model}' ";
		$thule_products_id_query = mysql_query($thulequery) or die(mysql_error());
 
		$products[$product_model] = array(
			'id'       => mysql_result($thule_products_id_query,0),
			'quantity' => $quantity,
		);
	}
}
 
foreach($products as $product) {									
	$cart->add_cart($product['id'], $cart->get_quantity($product['id']) + $product['quantity']);
}
		}
 
      tep_redirect(tep_href_link($goto, tep_get_all_get_params(array('action'))));	
 
		} else {																		
        $navigation->set_snapshot();																		
        tep_redirect(tep_href_link(FILENAME_LOGIN, '', 'SSL'));																		
        }																		
        break;	
 
[PHP] pobierz, plaintext 

Formularz:

[HTML] pobierz, plaintext 
<?php echo '<label for="p_scnts"><input type="text" size="30" name="products_model['.$licznik_produktow.']" value="" placeholder="Wpisz numer katalogowy GRENE" /><input type="text"  name="quantity['.$licznik_produktow.']" value="" maxlength="3" size="5" placeholder="ilość" /></label>'; ?>
[HTML] pobierz, plaintext 

pola input powielam sobie przez jquery.

[SmokAnalog]

Tak myślałem, masz błąd w formularzu. Usuń indeksy z nazw pól:

[PHP] pobierz, plaintext 
<?php echo '<label for="p_scnts"><input type="text" size="30" name="products_model[]" value="" placeholder="Wpisz numer katalogowy GRENE" /><input type="text"  name="quantity[]" value="" maxlength="3" size="5" placeholder="ilość" /></label>'; ?>
[PHP] pobierz, plaintext 

Pola, których nazwy są zakończone otwartym i zamkniętym nawiasem kwadratowym [] są automatycznie indeksowane i tego chcemy. Ty powielasz pola, z tym samym indeksem, czyli na dobrą sprawę masz tylko jedno pole (jedną unikalną nazwę).

Poza tym dbaj o wcięcia, bo Twój kod jest nieczytelny.

[lukasz_web]

Za rady wielki + Ale nadal jest efekt ten sam, jeżeli uzupełnię jedno pole z numerem katalogowym i ilością to nie ma problemu dodaje ładnie do koszyka, ale jeżeli mam już dwa to ciągle to samo:
[nr. kat][ilość]
[0001][6]
[0002][6]
A powinno dodać do numeru 0001 2 szt. a do numeru 0002 4szt.

[SmokAnalog]

Jesteś pewien, że prawidłowo używasz tej metody dodającej do koszyka?

Możesz sobie zrobić test tego mojego fragmentu robiąc po nim:

[PHP] pobierz, plaintext 
var_dump($products);
[PHP] pobierz, plaintext 

Polecam wykonać taki test, może po drodze pomyliłeś jakieś wartości.

[lukasz_web]

To działa:

[PHP] pobierz, plaintext 
 case 'add_listing' :											
 
                              $query_price_to_guest_result = ALLOW_GUEST_TO_SEE_PRICES;																		
 
                              if ((($query_price_to_guest_result=='true') && !(tep_session_is_registered('customer_id'))) || ((tep_session_is_registered('customer_id')))) {					  													
 
                                if (isset($_POST['products_id'])) {																		
 
									$tablica = $_POST['products_id'];									
 
									$ilosc = $_POST['quantity'];									
 
								 	for($i = 0; $i < sizeof($tablica); $i++)									
 
								 	{									
 
										if ((int)$ilosc[$i] > 0) {								
 
											$attributes_tab = $_POST['atrib'];							
 
											if (isset($attributes_tab[$tablica[$i]])) {							
 
												$tabs = substr($attributes_tab[$tablica[$i]],0,strlen($attributes_tab[$tablica[$i]])-1);						
 
												$tabsa = explode(',',$tabs);						
 
												$attributes = array();						
 
												for($b = 0; $b < sizeof($tabsa); $b++) {						
 
													$podtablica = explode(':',$tabsa[$b]);					
 
													$attributes[$podtablica[0]] = $podtablica[1];					
 
												}						
 
												$cart->add_cart($tablica[$i], $cart->get_quantity(tep_get_uprid($tablica[$i], $attributes))+$ilosc[$i], $attributes);						
 
											} else {							
 
												$cart->add_cart($tablica[$i], $cart->get_quantity($tablica[$i])+$ilosc[$i]);						
 
											}							
 
										}								
 
								 	}                                    									
 
                                }																		
 
                                tep_redirect(tep_href_link($goto, tep_get_all_get_params(array('action', 'pid'))));																		
 
                              } else {																		
 
                                $navigation->set_snapshot();																		
 
                                tep_redirect(tep_href_link(FILENAME_LOGIN, '', 'SSL'));																		
 
                              }																		
 
                              break;
 
[PHP] pobierz, plaintext 

Ale dodaje po ID produktu (products_id) a nie po numerze katalogowym (products_model), próbowałem to przerobić by po products_model dodawał.

[SmokAnalog]

Zrób taki test jak napisałem powyżej i pokaż wynik.

[lukasz_web]

Zrobiłem test i nic mi nie zwraca.

Jesteś pewien, że prawidłowo używasz tej metody dodającej do koszyka?

Próbowałem przerobić ten kod co ostatnio dodałem: case 'add_listing' -> który dodaje prawidłowo tylko że po wpisaniu id produktu, ale klient posługuje się numerami katalogowymi a nie id produktu z bazy danych i próbuje to przerobić by działało z products_model

[SmokAnalog]

Jak to nic nie zwraca? Coś musi zwracać.

Przed tą linią:

[PHP] pobierz, plaintext 
foreach($products as $product) {
[PHP] pobierz, plaintext 

Daj:

[PHP] pobierz, plaintext 
var_dump($products); exit;
[PHP] pobierz, plaintext 

Jak to nic nie wypisze na ekranie to znaczy, że cały ten blok kodu w ogóle się nie wykonuje. Dałeś dość mały fragment kodu więc zgadywał nie będę, rób testy, o których mówię.

[lukasz_web]

Dokładnie w złym miejscu wpisałem sorki ale mam już wynik:

array(1) { [""]=> array(2) { ["id"]=> string(6) "918360" ["quantity"]=> int(7) } }

Po wpisaniu dwóch produktów do koszyka

Ten post edytował lukasz_web 30.10.2013, 12:13:39

[SmokAnalog]

I mamy buga z mojej strony. W całym moim fragmencie kodu zamień wszystkie frazy $product_model na $products_model, po czym uruchom test ponownie i pokaż wynik.

Ten post edytował SmokAnalog 30.10.2013, 12:19:06

[lukasz_web]

Działa! Jest idealny efekt

array(2) { ["1275-001005"]=> array(2) { ["id"]=> string(6) "918350" ["quantity"]=> string(1) "5" } ["1275-001012"]=> array(2) { ["id"]=> string(6) "918356" ["quantity"]=> string(1) "6" } }

Wielkie dzięki, chętnie bym cię zaprosił na piwo bo rozwiązanie rozwiązaniem ale wiele też się nauczyłem.

[SmokAnalog]

Jak jesteś z Krakowa to zaproszenie przyjęte

[lukasz_web]

Kurcze kawałek mam ja z wielkopolski podziękować