[MySQL][PHP]Analiza kodu - gdzieś błąd, Edycja - formularz Opcje

[kvachu]

Witam!
Już kompletnie nie mam pojęcia co może być nie tak w poniższym kodzie. Siedzę nad tym chyba z 5 godzin i już brak mi pomysłów. Wszystko przebiega jakby normalnie do czasu UPDATE w "mysql_query("UPDATE wars SET ..." i nic się nie edytuje... Wprawne oko profesjonalisty powinno wychwycić co jest nie tak =(.
"error_reporting(E_ALL);" mówi tylko o nie zdefiniowanych zmiennych "$zapis" i "$przejdz_do_edycji", które są w liniach 5 i 17, a one są definiowane prawie na samym dole.
A oto ten feralny kod:

[PHP] pobierz, plaintext 
<?php
// error_reporting(E_ALL);
if(isset($_COOKIE['ID_my_site'])) {
	include("admincw.php");
	if($zapis == "1") {
	if(!$tag) { 
		echo 'Nie wpisano przeciwnika!<br>'; 
		$error = "1";
		echo $gracz1."<BR>".$gracz2."<BR>".$gracz3."<BR>".$gracz4;
	}
	if($error != "1") {
			mysql_query("UPDATE wars SET tag='$tag', dzien='$dzien', miesiac='$miesiac', rok='$rok', tryb='$tryb', typ='$typ', gracz1='$gracz1', gracz2='$gracz2', gracz3='$gracz3', gracz4='$gracz4', flaga='$flaga', runda1='$runda1', r1my='$r1my', r1oni='$r1oni', runda2='$runda2', r2my='$r2my', r2oni='$r2oni', runda3='$runda3', r3my='$r3my', r3oni='$r3oni' WHERE id='$id'");
 
			header("Location: index.php?link=cws_all");
	}
} else {
	if($przejdz_do_edycji) {
		$result = mysql_query("SELECT * FROM wars WHERE id='$id'");
		while($rekord = mysql_fetch_array($result)) {
 
		$tag     = $rekord['tag'];
		$dzien   = $rekord['dzien'];
		$miesiac = $rekord['miesiac'];
		$rok     = $rekord['rok'];
		$tryb    = $rekord['tryb'];
		$typ     = $rekord['typ'];
		$gracz1  = $rekord['gracz1'];
		$gracz2  = $rekord['gracz2'];
		$gracz3  = $rekord['gracz3'];
		$gracz4  = $rekord['gracz4'];
		$flaga   = $rekord['flaga'];
		$runda1  = $rekord['runda1'];
		$r1my    = $rekord['r1my'];
		$r1oni   = $rekord['r1oni'];
		$runda2  = $rekord['runda2'];
		$r2my    = $rekord['r2my'];
		$r2oni   = $rekord['r2oni'];
		$runda3  = $rekord['runda3'];
		$r3my    = $rekord['r3my'];
		$r3oni   = $rekord['r3oni'];
 
// FORM ACTION
		echo '<form action="index.php?link=edytujcw" method="post">';
		echo '<br><CENTER><table cellspacing="1" cellpadding="1" width="508">';
		echo '<tr><td colspan="2" class="addcw" align=center><BR>Edytowanie klanówki.<BR><BR></td></tr>';
		echo '<tr><td class="addcw" width="100">Klan:</td><td class="addcw">';
 
// TAG
		echo '<input name="tag" value="'.$tag.'"></td></tr>';
		echo '<tr><td class="addcw">Kraj:</td><td class="addcw">';
 
// FLAGA
		echo '<input name="flaga" value="'.$flaga.'" size="10"></td></tr>';
		echo '<tr><td class="addcw">Grany dnia:</td><td class="addcw">';
 
// DZIEN
		echo '<select name="dzien" size="1">';
		$i = 1;
		while($i <= 31) {
			if($i == $dzien) {
				if($i <=9) 
					echo "<option value=\"0$i\" selected>0$i</option>";
				else echo "<option value=\"$i\" selected>$i</option>";
			} else {
				if($i <=9) echo "<option value=\"0$i\">0$i</option>";
				else echo "<option value=\"$i\">$i</option>";
			}
			$i++;
		}
 
// MIESIAC
		echo '</select>.';
		echo '<select name="miesiac" size="1">';
		$i = 1;
		while($i <= 12) {
			if($i==$miesiac) {
				if($i <=9) echo "<option value=\"0$i\" selected>0$i</option>";
				else echo "<option value=\"$i\" selected>$i</option>";
			} else {
				if($i <=9) echo "<option value=\"0$i\">0$i</option>";
				else echo "<option value=\"$i\">$i</option>";
			}
			$i++;
		}
 
// ROK
		echo '</select>.<input name="rok" maxlength="4" size="5" value="'.$rok.'"></td></tr>';
 
// TYP
		echo '<tr><td class="addcw">Typ meczu:</td><td class="addcw"><input name="typ" value="'.$typ.'" size="10"></td></tr>';
 
// RUNDA 1
		echo '<tr><td class="addcw">Runda 1:</td><td class="addcw">';
		echo '<input name="runda1" value="'.$runda1.'" size="10">';
		echo '<input name="r1my" value="'.$r1my.'" size="1">';
		echo '<input name="r1oni" value="'.$r1oni.'" size="1"></td></tr>';
 
// RUNDA 2
		echo '<tr><td class="addcw">Runda 2:</td><td class="addcw">';
		echo '<input name="runda2" value="'.$runda2.'" size="10">';
		echo '<input name="r2my" value="'.$r2my.'" size="1">';
		echo '<input name="r2oni" value="'.$r2oni.'" size="1"></td></tr>';
 
// RUNDA 3
		echo '<tr><td class="addcw">Runda 3:</td><td class="addcw">';
		echo '<input name="runda3" value="'.$runda3.'" size="10">';
		echo '<input name="r3my" value="'.$r3my.'" size="1">';
		echo '<input name="r3oni" value="'.$r3oni.'" size="1"></td></tr>';
 
// SKLAD
		echo '<tr><td class="addcw">Skład:</td><td colspan="2" class="addcw">';
 
// GRACZ 1
	echo '<select name="gracz1"><option></option>';
		$result2 = mysql_query("SELECT username FROM users ORDER BY username ASC");
		while($users = mysql_fetch_array($result2)) {
			if($rekord['gracz1'] === $users['username']) {
				echo '<option value="'.$users['username'].'" selected>'.$users['username'].'</option>';
		    } else {
			    echo '<option value="'.$users['username'].'">'.$users['username'].'';
		    }
        }
		echo '</select>&nbsp;&nbsp;';
 
// GRACZ 2
		echo '<select name="gracz2"><option></option>';
		$result2 = mysql_query("SELECT username FROM users ORDER BY username ASC");
		while($users = mysql_fetch_array($result2)) {
			if($rekord['gracz2'] === $users['username']) {
				echo '<option value="'.$users['username'].'" selected>'.$users['username'].'</option>';
		   } else {
			    echo "<option value=\"".$users['username']."\">".$users['username'];
		   }
		}
		echo '</select>&nbsp;&nbsp;';
 
// GRACZ 3
		echo '<select name="gracz3"><option></option>';
		$result2 = mysql_query("SELECT username FROM users ORDER BY username ASC");
		while($users = mysql_fetch_array($result2)) {
			if($rekord['gracz3'] === $users['username']) {
				echo '<option value="'.$users['username'].'" selected>'.$users['username'].'</option>';
			} else {
				echo '<option value="'.$users['username'].'">'.$users['username'].'';
			}
        }
		echo '</select>&nbsp;&nbsp;';
 
// GRACZ 4
		echo '<select name="gracz4"><option></option>';
		$result2 = mysql_query("SELECT username FROM users ORDER BY username ASC");
		while($users = mysql_fetch_array($result2)) {
			if($rekord['gracz4'] === $users['username']) {
				echo '<option value="'.$users['username'].'" selected>'.$users['username'].'</option>';
			} else {
				echo '<option value="'.$users['username'].'">'.$users['username'].'';
			}
        }
		echo '</select>&nbsp;&nbsp;';
 
		echo '</td></tr>';
		echo '<tr><td colspan="2" valign="bottom" class="addcw" align="center"><BR>';
 
// SUBMIT
		echo '<input type="hidden" value="edit" name="admin">';
		echo '<input type="hidden" value="1" name="zapis">';
		echo '<input type="hidden" value="$id" name="id">';
		echo '<input type="submit" value="Zapisz zmiany" style="color: #000000;">';
		echo '<BR><BR></td></tr></table></CENTER>';
		echo '</form>';
}
} else {
 
// WYBOR KLANOWKI
	echo '<BR><CENTER><form action="index.php?link=edytujcw" method=post>';
	echo '<table width="508" cellspacing="0" cellpadding="0">';
	echo '<tr><td colspan="2" class="addcw" width="508" align="center"><BR>Edytowanie klanówki.<BR><BR></td></tr>';
	echo '<tr><td></td><td class="addcw" width="508" align="center">';
 
// KLANOWKI
	echo '<select name="id" size="20">';
	$result = mysql_query("SELECT * FROM wars ORDER BY rok DESC, miesiac DESC, dzien DESC, id DESC");
	while($rekord = mysql_fetch_array($result)) {
		echo "<option value=\"".$rekord['id']."\"> klanowka&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;>>>".$rekord['tag']."<<<&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ".$rekord['dzien'].".".$rekord['miesiac'].".".$rekord['rok']." -- ".$rekord['typ'] . $rekord['runda1']."-".$rekord['runda2']."-".$rekord['runda3'];
	}
	echo '</select></td>';
	echo '</tr><tr><td colspan="2" class="addcw" width="508" align="center"><BR>';
	echo '<input type="hidden" value="1" name="przejdz_do_edycji">';
	echo '<input type="submit" value="Edytuj" style="color:#0600hg;"><BR><BR>';
	echo '</td></tr></table>';
	echo '</form></CENTER>';
}
}
} else {
	echo '<BR><BR>dZo byź zjaD????????';
}
?>
[PHP] pobierz, plaintext 

Z góry dzięki za pomoc =(

Ten post edytował kvachu 22.11.2009, 23:00:55

[Meares]

Sprawdź $tag i inne za pomocą var_dumpa.

[kvachu]

Zrobiłem, choć do końca nie wiem o co ci chodzi, bo ja jestem początkujący tak jakby. $rekord['gracz1'] zwraca NULL, a $gracz1 zwraca coś jakby STRING(9) "Janek".
Mi to za bardzo nic nie mówi.
Zrobiłem dwa var_dump. Jeden w linii 182 i tam zwracane są wartości z bazy, a var_dump z linii 10 zwraca już zedytowane wartości jak ma to być. Jakby problem był przy:

[PHP] pobierz, plaintext 
mysql_query("UPDATE wars SET tag='$tag', dzien='$dzien', miesiac='$miesiac', rok='$rok', tryb='$tryb', typ='$typ', gracz1='$gracz1', gracz2='$gracz2', gracz3='$gracz3', gracz4='$gracz4', flaga='$flaga', runda1='$runda1', r1my='$r1my', r1oni='$r1oni', runda2='$runda2', r2my='$r2my', r2oni='$r2oni', runda3='$runda3', r3my='$r3my', r3oni='$r3oni' WHERE id='$id'");
[PHP] pobierz, plaintext 

Tylko, że ja tu nie widzę nic złego...

Ten post edytował kvachu 22.11.2009, 23:45:32

[darko]

"error_reporting(E_ALL);" mówi tylko o nie zdefiniowanych zmiennych "$zapis" i "$przejdz_do_edycji", które są w liniach 5 i 17, a one są definiowane prawie na samym dole.

No właśnie, pokaż, w którym miejscu jest "na samym dole", bo nie widzę, poza tym jeśli odwołujesz się do jakiejś zmiennej w kodzie, to wypadałoby najpierw ją zadeklarować przed użyciem.

[cojack]

deklaracja != definicji
definicja == deklaracji

nauczcie się tego. Jak możesz zadeklarować coś co nie było jeszcze zdefiniowane?

[kvachu]

No właśnie, pokaż, w którym miejscu jest "na samym dole", bo nie widzę, poza tym jeśli odwołujesz się do jakiejś zmiennej w kodzie, to wypadałoby najpierw ją zadeklarować przed użyciem.

$zapis jest w linii 189, natomiast $przejdz_do_edycji w linii 211. No chyba, że to nie jest definicja. Nie zawracam sobie za bardzo tym głowy ponieważ w starszej wersji tego skryptu było tak samo, a jedyne co się zmieniło to nazwa tablicy i przy graczach były inputy zamiast selecty i skrypt śmigał elegancko, edytowało aż miło. Ale, że nie interesuje mnie zwykły input, a select to przerabiam...

[darko]

$zapis jest w linii 189,

to chyba tylko u Ciebie, bo tutaj jest echo '<input type="submit" value="Edytuj" style="color:#0600hg;"><BR><BR>';

natomiast $przejdz_do_edycji w linii 211.

jw. z tym, że z tego, co podałeś kod kończy się na linii 197

deklaracja != definicji
definicja == deklaracji

nauczcie się tego. Jak możesz zadeklarować coś co nie było jeszcze zdefiniowane?

purysta (IMG:style_emoticons/default/winksmiley.jpg) ale ma rację
(tak na marginesie tak: $zmienna; (IMG:style_emoticons/default/smile.gif) )

No chyba, że to nie jest definicja.

W językach programowania o słabym typowaniu (jak php) nie ma potrzeby deklarowania zmiennych, wystarczy definicja $zmienna = "wartość"; będąca równocześnie deklaracją. Oddaję, co powinienem. Jednakże przed użyciem jakiejkolwiek zmiennej należy ją zdefiniować, aby można było odwołać się do jej wartości.

[kvachu]

Kolejno linie 166 i 188. Wcześniej podawałem linie z EditPlus, w którym pracuję, a kod tutaj widocznie wciął kilka enterów.

Wiele z tego co tutaj piszecie słyszę pierwszy raz. A, że potrzebuję trochę pomocy to postuję tutaj, ponieważ jest to PRZEDSZKOLE:

"Tematyka i zasady panujące na forum Przedszkole
Forum przeznaczone jest dla osób zaczynających "zabawę" w tematyce WWW."

To właśnie ja, dopiero zaczynam.
Spostrzegłem, że wielu z was nawet nie analizuje tutaj kodów, a próbuje udzielać pomocy okrężną drogą(+1?).
Anyway, dzięki za pomoc... Dam sobie z tym spokój narazie i wezmę się za łatwiejsze rzeczy, które muszę zrobić na stronie. Za rzeczy na poziomie żłobka.