[php] $_POST Opcje

[Lethys]

Mam problem , pisze gre i chce zrobc skrypt karmienia

Tutaj kod karmienie.php

[PHP] pobierz, plaintext 
<?php include("config.php"); ?>
<?php
$stat = mysql_fetch_array(mysql_query("select * from players where email='$email' and pass='$pass'"));
?>
<form method=post action=karmienie.php?action=karmienie>
<center><br><br>Czym chcesz karmic ?<br><input type="radio" name=karmienie><option value=sila>Wodorosla ze Smoczej Przeleczy ( dodaje sily )</option><br><input type="radio" name=karmienie><option value=zrecznosc>Jajecznica z gryfinich jaj ( dodaje zrecznosc )</option></select><br><br> 
<input type=submit value=Nakarm>
</form>
 
<?php
if($_POST['karmienie'] == karmienie) {
 
 
	if ($stat['glod'] < 1) {
		print "<br><br>Twoja Kreatura nie jest glodna !";
		exit;
	}
	mysql_query("update players set glod=0 where id=$stat[id]");
	mysql_query("update players set $karmienie=$karmienie+1 where id=$stat[id]");
	print "<br><br>Twoja kreatura zyskala <b> 1 pkt</b> wybranej umiejetnosci.";
 
}
?>
[PHP] pobierz, plaintext 

Chodzi mi o to ze co reset glod ustawiany jest na 1 i kiedy sie karmi odejmujemy glod=glod-1 , czyli bedzie mozna karmic raz dziennie . Wydaje mi sie ze nie ma bledu w kodzie a mimo wszytsko nie dziala (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[jarrod]

Dodam od siebie że bez sensu jest robić kilka zapytań tego samego tympu do tej samej bazy...

[PHP] pobierz, plaintext 
<?php
$sql = "update players set glod=0,karmienie=karmienie+1,zrecznosc=zrecznosc+1  where id={$stat[id]}";
?>
[PHP] pobierz, plaintext 

Wówczas zamiast 3 zapytań masz tylko 1. OPTYMALIZACJA!

Jeśli uzywasz $stat['nazwa'], to stosuj fetch_assoc nie array.

Tu się zgodzę

Poaz tym '".$email."' jak już (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Nie widzę takiej potrzeby. Jest to kwestia użytych cudzysłowów. Jeżeli masz --> " <-- to parser sprawdza co jest wewnątrz i jeżeli jest to jakaś zmienna to próbuje ją przetworzyć. Zatem w tym zapytaniu nie ma blędu jeśli chodzi o zmienne. Jeżeli byłby to --> ' <-- to wówczas parser to pomija i w zapytaniu do bazy poszło by dokałdnie email='$email'. Dla pewności (w nawiasach podwójnych) można zmienną "opakować" w klamry co powie parserowi że jest ro zmienna do parsowania.

Ten post edytował jarrod 17.10.2006, 20:05:56

[D4rky]

Nie widzę takiej potrzeby. Jest to kwestia użytych cudzysłowów. Jeżeli masz --> " <-- to parser sprawdza co jest wewnątrz i jeżeli jest to jakaś zmienna to próbuje ją przetworzyć. Zatem w tym zapytaniu nie ma blędu jeśli chodzi o zmienne. Jeżeli byłby to --> ' <-- to wówczas parser to pomija i w zapytaniu do bazy poszło by dokałdnie email='$email'. Dla pewności (w nawiasach podwójnych) można zmienną "opakować" w klamry co powie parserowi że jest ro zmienna do parsowania.

Kwestia przyzwyczajen, aczkolwiek wydaje mi sie, ze metoda ktora podajesz nie zadziala np (int)$stat['id']. wiec lepiej sie przyzwyczajac do ". ." (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)