[PHP] Pętlę (lub co innego), ucinają jeden wynik Opcje

[thomson89]

Witam!

Kombinuję już od 3h nad tym i już nie mogę. Proszę o pomoc!

Niżej podam kod, który zwraca wartości pobrane z bazy i przetwarza je. Tworzę z danych z bazy, kwadraciki z danymi produktów. Kwadratów jest 12 na stronę - 4 linijki po 3 takie "elementy". Problem polega na tym, że ostatni "element" z ostatniej linijki nie jest wyświetlany. Próbowałem już z poprawianiem pętli, zmianami numerków i liczb. Ale to na nic. Wyświetlając także zmienną $zew_licznik, wychodziło na to, że kod dla 12 elementu się wykonuje.

[PHP] pobierz, plaintext 
$polaczenie = new baza;
			$query = 'SELECT * FROM `produkty` LIMIT '.$start.', '.$this -> ile_na_stronie;
			$wynik1 = mysql_query($query);
			$query = 'SELECT * FROM `produkty`';
			$wynik2 = mysql_query($query);
 
			$out = array();
			while($row = mysql_fetch_assoc($wynik1)) {
				$out[] = $row;
			}
 
			$textResult .= '</p><p class="zdjecia">
			<table class="produkty">';
 
			$ile = mysql_num_rows($wynik1);
			$ile = ceil($ile/3);
			$zew_licznik = 1;
			$i = 1;
			while($i <= $ile)
			{
				$textResult .= '<tr>';
				$c = 1;
				while($c <= 3)
				{
					if($out[$zew_licznik] != '')
					{
						$plik = file_get_contents($this -> szablon_produkt);
						$plik = str_replace('<cena>', $out[$zew_licznik]['cena'], $plik);
						$plik = str_replace('<symbol>', $out[$zew_licznik]['nazwa'], $plik);
						$plik = str_replace('<opis>', $out[$zew_licznik]['opis'], $plik);
 
						//obrazki					
						$nazwa = $out[$zew_licznik]['nazwa'];
						$nazwa = preg_replace('/[^a-z ]+/i', '', $nazwa);
						$nazwa = str_replace(' ', '', $nazwa);
						$nazwa = strtolower($nazwa);
						$plik = str_replace('<obrazek>', 'http://www.gl-art.pl/sklep/produkty/'.$nazwa.'.jpg', $plik);					
 
						$textResult .= $plik;						
 
 
					}
					$c++;
					$zew_licznik++;
 
				}
				$textResult .= '</tr>';
				$i++;
			}
 
			$textResult .= '</table>';
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
<td><img height="150" src="<obrazek>"><strong>Cena:</strong> <cena><br><strong>Nazwa:</strong> <symbol><br><strong>Opis:</strong> <opis><br><br><span class="linki"><a href="<link>">Więcej ?</a></span></td>
[HTML] pobierz, plaintext 

Ten post edytował thomson89 6.08.2010, 22:26:51

[vokiel]

Po pierwsze, to te drugie zapytanie jest tu chyba niepotrzebne.

Po drugie po co zliczasz ilość zwróconych rekordów, skoro masz tablicę $out, w której masz te produkty. Podlicz liczbę wierszy w tej tablicy. W końcu to z tych danych później korzystasz.

Pokaż jaką wartość ma zmienna $ile z linii 15 i linii 16.

W ogóle to co to za dziwne konstrukcje. Po co Ci tutaj aż dwie pętle while? Masz tablicę elementów, chcesz co trzy elementy przechodzić do nowej linii, to weź zrób to coś jak tak (warunek z if może trzeba będzie na końcu umieścić, nie sprawdzałem, a późno już):

[PHP] pobierz, plaintext 
$i = 0;
foreach ($out as $item){
    if ($i%3==0){
        // trzeci element czyli np
        echo '</tr><tr>';
    }
    // tutaj Twoje wyświetlanie, z tym, że zamiast $out[$zew_licznik]['cena'] używasz $item['cena']
    $i++;
}
[PHP] pobierz, plaintext 

edit: Można zacząć licznik od 1, albo dodać w warunku:

[PHP] pobierz, plaintext 
if ($i%3==0 && $i!=0){}
[PHP] pobierz, plaintext 

Ten post edytował vokiel 6.08.2010, 22:48:44

[glh]

Tylko powierzchownie przyglądnąłem się temu, nie testowałem nic i będę strzelał

Zmienna out jest tablicą, a tablice liczone są od zera.
Tymczasem u Ciebie pętla zaczyna od elementu o indeksie 1. Może tu tkwi problem - w zmiennej zew_licznik.

[thek]

I po co tak męczyć się? Nie prościej zrobić zabawę z modulo? Na starcie sprawdzanie czy licznik osiągnął określona wartość. Jeśli tak to zamknij dotychczasowy <tr> i otwórz nowy. Kod skrócisz o minimum kilkanaście linii.

EDIT: Vokiel mnie ubiegł... I IF ma być na samym początku. Musi jednak wyeliminować pierwsze przejście pętli bo mu wstawi całkowicie pusty <tr> bez jakichkolwiek <td> wewnątrz.

Powód edycji: [thek]: Uzupełnienie

[thomson89]

glh, rozwiązał problem. Oczywiście dziękuje za propozycje zmian.