Forum PHP.pl

Robię sondę a dokładniej jej edycję. Mam zrobione tak, że przy dodawaniu mam 10 inputów, jeśli są pełne to je dodaje. Mam dwie tabele - sonda i odpowiedzi. I teraz przy edycji wypisuję odpowiedzi w których pole SONDA jest równe ID sondy. Taki powiązanie. Ale teraz jest problem taki, że gdy chcę zedytować odpowiedzi to robię dla każdej odpowiedzi (foreach) UPDATE odpowiedzi WHERE id="$idanswer". No właśnie... wyciągnąłem id tych rekordów ale mam problem właśnie z foreach. Najlepiej jak pokażę to co zrobiłem:

[PHP] pobierz, plaintext 
// $answer to tablica odpowiedzi
// $idanswer to ID dla odpowiedzi, też w tablicy 
 
foreach ($answer as $a) {
	if (!empty($a)) {
		foreach ($idanswer as $ida) {
			mysql_query("UPDATE answers SET answer='$a' WHERE id='$ida'");
		}
	}
}
[PHP] pobierz, plaintext 

I tutaj plik formularza edycji:

[PHP] pobierz, plaintext 
<input type='text' name='answer[]' value='".$row0['answer']."'><input type'hidden' name='idanswer[]' value='".$row0['id']."'>
[PHP] pobierz, plaintext 

Pozdrawiam

Z tego co zrozumiałem to wynika, że odpowiedź z $anwser będzie przypisana tylko do jednego ID z $idanwser i w tym kodzie co napisałeś to ta odpowiedź jest wpisywana do wszystkich ID z każdą następną pętlą wynik się nadpisuje tak?

wydaj mi się, że chciałeś uzyskać coś takiego :

[PHP] pobierz, plaintext 
foreach ($answer AS $key -> $a) 
{
	if(!empty($a)) 
	{
		$ida = $idanswer[$key];
		$query = mysql_query("UPDATE answers SET answer='$a' WHERE id='$ida'") or die(mysql_error());
	}
}
[PHP] pobierz, plaintext 

Wyskakuje taki błąd:

[PHP] pobierz, plaintext 
Notice: Undefined variable: on line 16
Fatal error: Cannot access empty property  on line 16
[PHP] pobierz, plaintext 

Czyli chodzi o linię:

[PHP] pobierz, plaintext 
	if(!empty($a)) 
[PHP] pobierz, plaintext 

widzę, że tam zamiast = dałem - , spróbuj z tym:

[PHP] pobierz, plaintext 
foreach ($answer AS $key => $a) 
{
	if(!empty($a)) 
	{
		$ida = $idanswer[$key];
		$query = mysql_query("UPDATE answers SET answer='$a' WHERE id='$ida'") or die(mysql_error());
	}
}
[PHP] pobierz, plaintext 

Kombinujesz w złą stronę. Pętla w pętli a w niej zapytanie do bazy? Ile tych zapytań do bazy będzie w sumie?

Możesz stworzyć taką tablicę:

[PHP] pobierz, plaintext 
$sonda[1][1]='';
$sonda[2][3]='';
$sonda[3][2]='';
[PHP] pobierz, plaintext 

- pierwszy indeks to numer pytania, drugi to odpowiedź a ewentualny tekst to np. "inne, napisz jakie" - taką tablicę da się łatwo pobrać z tabeli "sonda", która wygląda np. tak:
id_sondy|id_pytania|id_odpowiedzi|tekst_odpowiedzi

Nie wiem jaka to ma być sonda i jak to ma działać, potem wystarczy stworzyć plik o podobnej strukturze i dodać od razu całą sondę do bazy jednym zapytaniem LOAD DATA INFILE.

Rzeczywiście, też tego nie zauważyłem. Teraz działa.

Co jest złego w takim rozwiązaniu? Zapytań będzie max.8... wydaje mi się, że taka metoda jest do przyjęcia przy takiej ilości? Pewnie przy większej ilości mogłaby być niewydajna.

No a ja nie wiem, czy 8 czy więcej, po prostu przywykłem do robienia takich rzeczy jednym zapytaniem i wbiłem sobie do głowy, że zapytanie wewnątrz pętli jest złe. Poza tym jeśli będzie korzystało z tego n użytkowników wykonujących x operacji to n*x*8 a n*x*1 to różnica. Ktoś potem rozbuduje ankiete do dwóch stron i się zdziwi, ale każdy robi jak uważa, jak będziesz potrzebował audytu wydajności czy optymalizacji to za odpowiednie wynagrodzenie jestem do usług