Pętla foreach zagnieżdżona w for - problem z indeksami Opcje

[jurcio6]

Witam.

Mam pewien problem z indeksami. Kod:

[PHP] pobierz, plaintext 
//$answers to dwuwymiarowa tablica rekordów pobranych z bazy - załóżmy, że tych rekordów ma tam 3
for($i=0; $i<3; $i++)
{
        //jakiś kod do wykonania
	$j = 1;
	foreach($answers as $row)
	{				
	       //jakiś kod do wykonania - $row coś tam - wiadomo, ale problem poniżej:
               echo $j.", ";
               $j++;
	}
}
[PHP] pobierz, plaintext 

I teraz problem mam następujący: wynikiem takiej pętli jest: "1, 2, 3, 4, 5, 6, 7, 8, 9" - jakim cudem, skoro przy każdej iteracji pętli zewnętrznej for ustawiam licznik $j =1? Czy wynikiem nie powinno być "1, 2, 3, 1, 2, 3, 1, 2, 3"? Jak uzyskać taki efekt?

Proszę o pomoc! Z góry dzięki.

Ten post edytował jurcio6 14.02.2011, 11:59:29

[Piogola]

Jesteś pewien, że masz 3 rekordy, a nie 9 ?

[darko]

Pętla foreach ma to do siebie, że iteruje po całej tablicy bez żadnych ograniczeń,natomiast Ty ustawiasz wartość zmiennej $j już poza nią. To co robi teraz ten kod, to trzykrotny foreach po całej tablicy. Jeśli chciałbyś wyciągnąć co trzeci rekord to należałoby zrobić małą modyfikację związaną z wykorzystaniem reszty z dzielenia (sprawdzasz czy modulo % 3 z $j == 0) i wszystko w jednym foreachu.

[jurcio6]

Pętla działa w porządku - tzn. zgodnie z moim zamierzeniem. Wykorzystuję ją przy ankietach i służy ona konkretnie do wypisania wszystkich odpowiedzi do danego pytania - po pytaniach iteruje for, po odpowiedziach foreach - tu wszystko z pewnością jest dobrze. Wyciąłem to po prostu z kontekstu i dlatego wygląda to tak, jakby iterowała 3 razy po tym samym - w rzeczywistości przy każdej iteracji zmienia się argument $answers.

Problem jest taki, że chciałem nazywać odpowiedzi dla każdego pytania 1, 2, 3, 4.. , a tutaj nie resetuje się ten index $j i mi to uniemożliwia. Konkretnie chodzi o atrybuty value dla inputów. Nie chcę korzystać z $row['question_id'], który jest wyciągany z bazy i po czasie może przyjmować dosyć wysokie wartości. Cały kod wygląda mniej więcej tak:

[PHP] pobierz, plaintext 
echo "<table align=\"center\">";
for($i=0; $i<$nq; $i++)
{
	echo "\n<tr><td style=\"padding: 15px 0px 5px 0px;\"><b>Pytanie ";
	echo $i+1 .": <i>".$questions[$i]['question_text']."</i></b>";
	if($questions[$i]['multiple_choice']==1)
		echo " [wybór wielokrotny]";
	echo "</td></tr>\n<tr><td style=\"padding-left: 30px;\">";
	$j = 1;
	foreach($answers as $row)
	{
		if($questions[$i]['question_id'] == $row['question_id'])
			if($questions[$i]['multiple_choice']==1)
				echo "\n<label name=\"".$row['question_id']."\"><input type=\"checkbox\" name=\"question_".$i."[]\" value=\"".$j."\" />".$row['answer_text']."</label><br />";
			else	
				echo "\n<label name=\"".$row['question_id']."\"><input type=\"radio\" name=\"question_".$i."\" value=\"".$j."\" />".$row['answer_text']."</label><br />";
		$j++;
	}
	echo "</td></tr>";
}
echo "\n</table>";
[PHP] pobierz, plaintext 

Wersja mocno robocza (IMG:style_emoticons/default/smile.gif)

//edit: oczywiście chodzi o wartości value dla inputów, a nie o name.

Ten post edytował jurcio6 14.02.2011, 12:23:28

[darko]

Dodaj dla porządku odpowiednie nawiasy klamrowe do instrukcji warunkowych if i sprawdź efekt.

[jurcio6]

Niestety nic się nie zmieniło. Poszczególne pola <input.. /> nadal przyjmują wartości value="" od 1 do 23 (tyle jest rekordów).

[PHP] pobierz, plaintext 
echo "<table align=\"center\">";
for($i=0; $i<$nq; $i++)
{
	echo "\n<tr><td style=\"padding: 15px 0px 5px 0px;\"><b>Pytanie ";
	echo $i+1 .": <i>".$questions[$i]['question_text']."</i></b>";
	if($questions[$i]['multiple_choice']==1)
	{
		echo " [wybór wielokrotny]";
	}
	echo "</td></tr>\n<tr><td style=\"padding-left: 30px;\">";
	$j = 1;
	foreach($answers as $row)
	{
		if($questions[$i]['question_id'] == $row['question_id'])
		{
			if($questions[$i]['multiple_choice']==1)
			{
				echo "\n<label name=\"".$row['question_id']."\"><input type=\"checkbox\" name=\"question_".$i."[]\" value=\"".$j."\" />".$row['answer_text']."</label><br />";
			}
			else	
			{
				echo "\n<label name=\"".$row['question_id']."\"><input type=\"radio\" name=\"question_".$i."\" value=\"".$j."\" />".$row['answer_text']."</label><br />";
			}
		}
		$j++;
	}
	echo "</td></tr>";
}
echo "\n</table>";
[PHP] pobierz, plaintext 

Efekt:

[HTML] pobierz, plaintext 
<table align="center">
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 1: <i>Czy wolisz kolory jasne czy ciemne?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="92"><input type="radio" name="question_0" value="1" />jasne</label><br />
<label name="92"><input type="radio" name="question_0" value="2" />ciemne</label><br />
<label name="92"><input type="radio" name="question_0" value="3" />jest mi to obojętne</label><br /></td></tr>
 
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 2: <i>Czy wolisz odcienie stonowane czy jaskrawe?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="93"><input type="radio" name="question_1" value="4" />jaskrawe</label><br />
<label name="93"><input type="radio" name="question_1" value="5" />stonowane</label><br />
<label name="93"><input type="radio" name="question_1" value="6" />jest mi to obojętne</label><br /></td></tr>
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 3: <i>Jakiego koloru koszulki lubisz nosić?</i></b> [wybór wielokrotny]</td></tr>
<tr><td style="padding-left: 30px;">
<label name="94"><input type="checkbox" name="question_2[]" value="7" />białe</label><br />
 
<label name="94"><input type="checkbox" name="question_2[]" value="8" />czarne</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="9" />zielone</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="10" />czerwone</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="11" />żółte</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="12" />niebieskie</label><br /></td></tr>
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 4: <i>Czy lubisz odcienie szarości?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="95"><input type="radio" name="question_3" value="13" />tak</label><br />
<label name="95"><input type="radio" name="question_3" value="14" />nie</label><br /></td></tr>
 
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 5: <i>Jaki jest twój ulubiony kolor?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="96"><input type="radio" name="question_4" value="15" />pomarańczowy</label><br />
<label name="96"><input type="radio" name="question_4" value="16" />fioletowy</label><br />
<label name="96"><input type="radio" name="question_4" value="17" />niebieski</label><br />
<label name="96"><input type="radio" name="question_4" value="18" />żółty</label><br />
<label name="96"><input type="radio" name="question_4" value="19" />zielony</label><br />
<label name="96"><input type="radio" name="question_4" value="20" />czerwony</label><br />
<label name="96"><input type="radio" name="question_4" value="21" />biały</label><br />
 
<label name="96"><input type="radio" name="question_4" value="22" />czarny</label><br />
<label name="96"><input type="radio" name="question_4" value="23" />turkusowy</label><br /></td></tr>
</table>
[HTML] pobierz, plaintext 

Pożądany efekt:

[HTML] pobierz, plaintext 
<table align="center">
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 1: <i>Czy wolisz kolory jasne czy ciemne?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="92"><input type="radio" name="question_0" value="1" />jasne</label><br />
<label name="92"><input type="radio" name="question_0" value="2" />ciemne</label><br />
<label name="92"><input type="radio" name="question_0" value="3" />jest mi to obojętne</label><br /></td></tr>
 
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 2: <i>Czy wolisz odcienie stonowane czy jaskrawe?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="93"><input type="radio" name="question_1" value="1" />jaskrawe</label><br />
<label name="93"><input type="radio" name="question_1" value="2" />stonowane</label><br />
<label name="93"><input type="radio" name="question_1" value="3" />jest mi to obojętne</label><br /></td></tr>
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 3: <i>Jakiego koloru koszulki lubisz nosić?</i></b> [wybór wielokrotny]</td></tr>
<tr><td style="padding-left: 30px;">
<label name="94"><input type="checkbox" name="question_2[]" value="1" />białe</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="1" />czarne</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="1" />zielone</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="1" />czerwone</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="1" />żółte</label><br />
<label name="94"><input type="checkbox" name="question_2[]" value="1" />niebieskie</label><br /></td></tr>
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 4: <i>Czy lubisz odcienie szarości?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="95"><input type="radio" name="question_3" value="1" />tak</label><br />
<label name="95"><input type="radio" name="question_3" value="2" />nie</label><br /></td></tr>
 
<tr><td style="padding: 15px 0px 5px 0px;"><b>Pytanie 5: <i>Jaki jest twój ulubiony kolor?</i></b></td></tr>
<tr><td style="padding-left: 30px;">
<label name="96"><input type="radio" name="question_4" value="1" />pomarańczowy</label><br />
<label name="96"><input type="radio" name="question_4" value="2" />fioletowy</label><br />
<label name="96"><input type="radio" name="question_4" value="3" />niebieski</label><br />
<label name="96"><input type="radio" name="question_4" value="4" />żółty</label><br />
<label name="96"><input type="radio" name="question_4" value="5" />zielony</label><br />
<label name="96"><input type="radio" name="question_4" value="6" />czerwony</label><br />
<label name="96"><input type="radio" name="question_4" value="7" />biały</label><br />
 
<label name="96"><input type="radio" name="question_4" value="1" />czarny</label><br />
<label name="96"><input type="radio" name="question_4" value="2" />turkusowy</label><br /></td></tr>
</table>
[HTML] pobierz, plaintext 

//ignore na pola checkbox - je wiem jak obsłużyć (IMG:style_emoticons/default/smile.gif)

Nadal nie udało mi się uporać z tym problemem - jakieś pomysły? ;/

Ten post edytował jurcio6 14.02.2011, 12:53:54

[vermis]

Wykonałem ten twój kod i działa poprawnie. Jesteś pewien, że u ciebie wykonuje się dokładnie to co wkleiłeś?

[jurcio6]

Dla pewności wykonałem kod jeszcze raz i niestety jest to, co było, czyli value w inputach przyjmują wartości od 1 do 23, a nie tak, jak chcę. Naprawdę nie wiem czemu.

Pracuję na wamp server, PHP 5.3.0 - może przez średnio aktualną wersję? Choć szczerze mówiąc teraz nie jest najlepszy czas na zmiany oprogramowania - muszę szybko skończyć ten projekt.

[TheTester]

I ten pierwszy, najprostszy przykład też Ci nie działa? A mógłbyś dla tego uproszczonego przykładu zrobić print_r testowej tablicy jaką tam podajesz?

[greycoffey]

Btw. optymalizacja pierwszego przykładu:

[PHP] pobierz, plaintext 
<?php
// ...
$j = 0; // zamiast $j=0
foreach($answers as $row)
{
//jakiś kod do wykonania - $row coś tam - wiadomo, ale problem poniżej:
echo ++$j.", "; //dodajemy preinkrementacje
// usuawmy $j++;
}
?>
[PHP] pobierz, plaintext 

[donpablo]

teraz problem mam następujący: wynikiem takiej pętli jest: "1, 2, 3, 4, 5, 6, 7, 8, 9" - jakim cudem, skoro przy każdej iteracji pętli zewnętrznej for ustawiam licznik $j =1? Czy wynikiem nie powinno być "1, 2, 3, 1, 2, 3, 1, 2, 3"? Jak uzyskać taki efekt?

Jak się pisze bzdury to się ma problemy. Proponuje na poczatek zapoznać się z podstawami działania pętli. Poczatkowy fragment skryptu robi dokładnie to co powinien robić - bo tak napisałeś. To samo dotyczy tablic wielowymiarowych.
By uzyskac efekt 1,2,3 trzeba to napisać - chocby tak:

[PHP] pobierz, plaintext 
foreach ($answers as $id => $row )
{
   for($i=0, $j=1; $i<sizeof($row); $i++, $j++) //lub zamiast $i<sizeof($row) -> $i<3 o ile ilośc komórek równa jest 3
  {
      echo $j;
  }
}
[PHP] pobierz, plaintext 

[jurcio6]

Siadłem ze świeżym umysłem i od razu znalazłem błąd - na który nikt w tym topicu nie wpadł...

Donpablo - być może wiesz dużo, ale nie wpadłeś na najprostsze rozwiązanie. Może i nie znam na tyle dobrze mechanizmów PHP, ale jeśli bym znał, to bym nie potrzebował pomocy na tym forum ani gdziekolwiek indziej. W każdym razie w tym wypadku nie tu leżał problem... pętle działały tak, jak było to przez mnie zamierzone - jedyną rzeczą, którą zrobiłem źle, to inkrementacja $j - powinna ona być tylko wtedy, gdy warunek jest spełniony (czyli gdy znajdę odpowiednie pole), a było zrobione tak, że wykonuje się zawsze (niezależnie od spełnienia warunku). Dodatkowo zmieniłem pętle for na foreach - dla własnej wygody. Dzięki wszystkim za chęci (IMG:style_emoticons/default/smile.gif)

Rozwiązanie:

[PHP] pobierz, plaintext 
foreach($questions as $q_row)
{
	echo "\n<tr><td style=\"padding: 15px 0px 5px 0px;\"><b>Pytanie ";
	echo $i+1 .": <i>".$q_row['question_text']."</i></b>";
	if($q_row[$i]['multiple_choice']==1)
	{
		echo " [wybór wielokrotny]";
	}
	echo "</td></tr>\n<tr><td style=\"padding-left: 30px;\">";
	$j = 0;
	foreach($answers as $a_row)
	{
		if($q_row['question_id'] == $a_row['question_id'])
		{
			if($q_row['multiple_choice']==1)
			{
				++$j;   //PROBLEM ROZWIĄZANY
				echo "\n<label name=\"".$a_row['question_id']."\"><input type=\"checkbox\" name=\"question_".$i."[]\" value=\"".$j."\" />".$a_row['answer_text']."</label><br />";
			}
			else	
			{
				++$j;   //PROBLEM ROZWIĄZANY
				echo "\n<label name=\"".$a_row['question_id']."\"><input type=\"radio\" name=\"question_".$i."\" value=\"".$j."\" />".$a_row['answer_text']."</label><br />";
			}
		}
	}
        echo "</td></tr>";
}
echo "\n</table>";
 
[PHP] pobierz, plaintext 

Ten post edytował jurcio6 19.02.2011, 10:59:52