Forum PHP.pl

Witam, mam pewną pętlę foreach, w której muszę zwiększać parametr $this->i o jeden.. próbuję już ponad godzinę na różne sposoby i nie idzie za nic.. proszę o pomoc, kod:

[PHP] pobierz, plaintext 
 
$this->xtab = explode(",",$this->tab); 
			$this->i=0;
 
			foreach ($this->model_catalog_product->getProductOptions($this->xtab[$this->i]) as $option) { 
 
 
				if ($option['type'] == 'select' || $option['type'] == 'radio' || $option['type'] == 'checkbox' || $option['type'] == 'image') { 
					$option_value_data = array();
 
					foreach ($option['option_value'] as $option_value) {
						if (!$option_value['subtract'] || ($option_value['quantity'] > 0)) {
							if ((($this->config->get('config_customer_price') && $this->customer->isLogged()) || !$this->config->get('config_customer_price')) && (float)$option_value['price']) {
								$price = $this->currency->format($this->tax->calculate($option_value['price'], $product_info['tax_class_id'], $this->config->get('config_tax')));
							} else {
								$price = false;
							}
 
							$option_value_data[] = array(
								'product_option_value_id' => $option_value['product_option_value_id'],
								'option_value_id'         => $option_value['option_value_id'],
								'name'                    => $option_value['name'],
								'image'                   => $this->model_tool_image->resize($option_value['image'], 50, 50),
								'price'                   => $price,
								'price_prefix'            => $option_value['price_prefix']
							);
						}
					}
 
					$this->data['options'][] = array(
						'product_option_id' => $option['product_option_id'],
						'option_id'         => $option['option_id'],
						'name'              => $option['name'],
						'type'              => $option['type'],
						'option_value'      => $option_value_data,
						'required'          => $option['required']
					);					
				} elseif ($option['type'] == 'text' || $option['type'] == 'textarea' || $option['type'] == 'file' || $option['type'] == 'date' || $option['type'] == 'datetime' || $option['type'] == 'time') {
					$this->data['options'][] = array(
						'product_option_id' => $option['product_option_id'],
						'option_id'         => $option['option_id'],
						'name'              => $option['name'],
						'type'              => $option['type'],
						'option_value'      => $option['option_value'],
						'required'          => $option['required']
					);						
				}
			}		
[PHP] pobierz, plaintext 

Nie rozumiem Cię... Masz kod mniej więcej taki:

[PHP] pobierz, plaintext 
$tablica = explode(",", $array);
$i = 0;
 
foreach($cos AS $costam)
{
 
// bla bla
 
}
[PHP] pobierz, plaintext 

No i jakim cudem ma Ci się to $i zwiększać o jeden jak Ty tego nigdzie nie zdefiniowałeś...?

Musisz wewnątrz pętli dorzucić:

[PHP] pobierz, plaintext 
$i++;
[PHP] pobierz, plaintext 

czyli:

[PHP] pobierz, plaintext 
$tablica = explode(",", $array);
$i = 0;
 
foreach($cos AS $costam)
{
 
$i++;
 
// bla bla
 
}
[PHP] pobierz, plaintext 

Ten post edytował Michael2318 2.12.2013, 19:47:35

Wewnątrz foreach() zwiększanie nic nie da, niech doda pętlę for() przed foreach() na podstawie count($this->xtab)

Dać da - zliczy konkretną ilość "przelotów" przez pętlę. Nie wiem w jakim celu mu to dodawanie jest potrzebne bo tego nie napisał, aczkolwiek wykorzystuje ją w uwarunkowaniu samej pętli foreach() i w tym wypadku rzeczywiście to nic nie da. Możesz spróbować z tą pętlą for, tak jak zaproponował poprzednik np. tak:

[PHP] pobierz, plaintext 
for($i = 0; $i <= count($array); $i++)
{
    foreach($this->costam[$i] AS costam)
    {
        // bla bla
    }
}
[PHP] pobierz, plaintext 

Nie wiem dlaczego, ale to mi nie chce działać, bo nic mi nie wyświetla, a te zliczanie mi potrzebna (jak zauwazyles) zeby zmieniac wartosc w pętli foreach()

[PHP] pobierz, plaintext 
for($i = 0; $i <= count($array); $i++)
{
    foreach($this->costam[$i] AS costam)
    {
        // bla bla
    }
}
[PHP] pobierz, plaintext 

sa jakies inne rozwiazania? bo jak wpisze np

[PHP] pobierz, plaintext 
              	$this->xtab = explode(",",$this->tab);  
		foreach ($this->model_catalog_product->getProductOptions($this->tabx[1]) as $option) { 
[PHP] pobierz, plaintext 

to mi dziala, juz nie mam pojecia co jest

Ten post edytował miras 2.12.2013, 20:25:28

Pokaż cały ten kod, który Ci nie działa (jak go zedytowałeś).

[PHP] pobierz, plaintext 
$this->data['options'] = array();
 
			$xtab=array();
			$xtab = explode(",",$this->tab);  
			for($ii = 0; $ii < count($xtab); $ii++) { 
			foreach ($this->model_catalog_product->getProductOptions($xtab[$ii]) as $option) { 
 
				if ($option['type'] == 'select' || $option['type'] == 'radio' || $option['type'] == 'checkbox' || $option['type'] == 'image') { 
					$option_value_data = array();
 
					foreach ($option['option_value'] as $option_value) {
						if (!$option_value['subtract'] || ($option_value['quantity'] > 0)) {
							if ((($this->config->get('config_customer_price') && $this->customer->isLogged()) || !$this->config->get('config_customer_price')) && (float)$option_value['price']) {
								$price = $this->currency->format($this->tax->calculate($option_value['price'], $product_info['tax_class_id'], $this->config->get('config_tax')));
							} else {
								$price = false;
							}
 
							$option_value_data[] = array(
								'product_option_value_id' => $option_value['product_option_value_id'],
								'option_value_id'         => $option_value['option_value_id'],
								'name'                    => $option_value['name'],
								'image'                   => $this->model_tool_image->resize($option_value['image'], 50, 50),
								'price'                   => $price,
								'price_prefix'            => $option_value['price_prefix']
							);
						}
					}
 
					$this->data['options'][] = array(
						'product_option_id' => $option['product_option_id'],
						'option_id'         => $option['option_id'],
						'name'              => $option['name'],
						'type'              => $option['type'],
						'option_value'      => $option_value_data,
						'required'          => $option['required']
					);					
				} elseif ($option['type'] == 'text' || $option['type'] == 'textarea' || $option['type'] == 'file' || $option['type'] == 'date' || $option['type'] == 'datetime' || $option['type'] == 'time') {
					$this->data['options'][] = array(
						'product_option_id' => $option['product_option_id'],
						'option_id'         => $option['option_id'],
						'name'              => $option['name'],
						'type'              => $option['type'],
						'option_value'      => $option['option_value'],
						'required'          => $option['required']
					);						
				}
			}		
		}
[PHP] pobierz, plaintext 

no ja juz wariuje..
jak dam tak:

[PHP] pobierz, plaintext 
for($ii = 0; $ii <= count($xtab); $ii++)
[PHP] pobierz, plaintext 

to wywala mi błąd:

Notice: Undefined offset: 3 in ... on line 74


74 linijka to:

[PHP] pobierz, plaintext 
			foreach ($this->model_catalog_product->getProductOptions($xtab[$ii]) as $option) { 
[PHP] pobierz, plaintext 

A co w ogóle zwraca Ci count($xtab) ?

nie bardzo mam jak to sprawdzić, bo robię to w kontrolerze, ale jak tam numer tablicy 0 to nic się nie wyswietla, a jak dam 1 to jest dobrze (czyli w przy jednej pozycji się wyświetla, a przy drugiej nie i tak powinno być, a jak zrobię, w pętli (tak jak podałem w przykładzie) to w obu pozycjach wyświetla mi się to samo, a powinno w jednej się wyświetlić, a w drugiej nie... nie zrozumiecie pewnie, bo namotałem, ale w tablicy jest wszystko ok..

print_r($xtab) i wyjdzie szydło z worka

Array ( [0] => [1] => 51 [2] => 52 [3] => 51 [4] => 52 ) to mi zwróciło

Skoro wartości się powtarzają to i w foreach() się powtarza to samo. Poza tym masz pewnie przecinek na początku $this->tab

a jak się nazywała funkcja usuwająca powtarzające się i puste rekordy ?

Ten post edytował miras 2.12.2013, 22:15:26

$xtab = array_values(array_filter(array_unique($xtab)));
pierwsza przypisuje nowe klucze kolejno, druga usuwa puste, trzecia zwraca unikalne (IMG:style_emoticons/default/wink.gif)

Ten post edytował _Borys_ 2.12.2013, 22:35:02

[PHP] pobierz, plaintext 
Array ( [0] => [1] => 51 [2] => 52 [3] => 51 [4] => 52 )
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
array_filter(array_unique($xtab))
[PHP] pobierz, plaintext 

To mi zwraca.. czyli nadal coś nie tak.

no teraz dobrze.. ale nadal mi wyświetla w 2ch pozycjach, a powinno w jednej

Jak to robisz że tak samo wychodzi, podaj kod ?

Tam jest raczej dobrze wszystko, bo pod ten kod podpięte jest wyświetlanie tych danych na podstronie i jest wszystko.. ale podam kod:


najpierw kontroler:

[PHP] pobierz, plaintext 
$this->data['options'] = array();
 
			$xtab=array();
			$xtab = explode(",",$this->tab);  
			$xtab = array_values(array_filter(array_unique($xtab)));
 
			for($ii = 0; $ii < count($xtab); $ii++) { 
			foreach ($this->model_catalog_product->getProductOptions($xtab[$ii]) as $option) { 
 
				if ($option['type'] == 'select' || $option['type'] == 'radio' || $option['type'] == 'checkbox' || $option['type'] == 'image') { 
					$option_value_data = array();
 
					foreach ($option['option_value'] as $option_value) {
						if (!$option_value['subtract'] || ($option_value['quantity'] > 0)) {
							if ((($this->config->get('config_customer_price') && $this->customer->isLogged()) || !$this->config->get('config_customer_price')) && (float)$option_value['price']) {
								$price = $this->currency->format($this->tax->calculate($option_value['price'], $product_info['tax_class_id'], $this->config->get('config_tax')));
							} else {
								$price = false;
							}
 
							$option_value_data[] = array(
								'product_option_value_id' => $option_value['product_option_value_id'],
								'option_value_id'         => $option_value['option_value_id'],
								'name'                    => $option_value['name'],
								'image'                   => $this->model_tool_image->resize($option_value['image'], 50, 50),
								'price'                   => $price,
								'price_prefix'            => $option_value['price_prefix']
							);
						}
					}
 
					$this->data['options'][] = array(
						'product_option_id' => $option['product_option_id'],
						'option_id'         => $option['option_id'],
						'name'              => $option['name'],
						'type'              => $option['type'],
						'option_value'      => $option_value_data,
						'required'          => $option['required']
					);					
				} elseif ($option['type'] == 'text' || $option['type'] == 'textarea' || $option['type'] == 'file' || $option['type'] == 'date' || $option['type'] == 'datetime' || $option['type'] == 'time') {
					$this->data['options'][] = array(
						'product_option_id' => $option['product_option_id'],
						'option_id'         => $option['option_id'],
						'name'              => $option['name'],
						'type'              => $option['type'],
						'option_value'      => $option['option_value'],
						'required'          => $option['required']
					);						
				}
			}		
		}
[PHP] pobierz, plaintext 

widok:


http://wklej.org/id/1195397/

Wyświetla to samo dwa razy ?

no tak właśnie.. a z bazy powinien się jeden wyświetlać..

To pewnie gdzie indziej błąd. A od czego zależy, żeby raz wyświetlało ?
W tablicy masz dwie wartości, pętla for() przelatuje dwa razy czyli wyświetla dwa razy, może złe dane masz w tablicy.
Zakomentuj for(){} i ręcznie wpisz indeksy w foreach()
Raz:

[PHP] pobierz, plaintext 
foreach ($this->model_catalog_product->getProductOptions(51) as $option)
[PHP] pobierz, plaintext 

Potem drugi raz:

[PHP] pobierz, plaintext 
foreach ($this->model_catalog_product->getProductOptions(52) as $option)
[PHP] pobierz, plaintext 

Jak wyświetli za każdym razem to szukaj błędu w innym miejscu.