[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

[gszesiek]

pokaż błąd

[revyag]

Co to za temat topicu ? Proszę poprawić albo zamknę.

[PHP] pobierz, plaintext 
<?php
if($_POST['karmienie'] == karmienie)
?>
[PHP] pobierz, plaintext 

Co to jest ? Skad ta zmienna post, co to jest karmienie Zmienna, stała ?

[erix]

@revyag:

Skad ta zmienna post

[HTML] pobierz, plaintext 
<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>
[HTML] pobierz, plaintext 

@[Lethys]:
http://pl.php.net/manual/pl/language.types.string.php

[Lethys]

Bledu nie ma poprostu nic sie nie dzieje a co do nazwy tematu to nie wiem jak nazwac moze cos zaproponuj ...

[fx69]

sprobuj:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['karmienie'])) {
$karmienie = $_POST['karmienie'];
...
?>
[PHP] pobierz, plaintext 

Ten post edytował fx69 17.10.2006, 16:03:06

[Lethys]

Dziwne bo teraz wyskakuje "ze kreatura nie jest glodna"

A przeciez warunek jest = ($stat['glod'] < 1)

Specjalnie sprawdzalem jeszcze raz w bazie i jest rowny 1

[gszesiek]

napisz sobie przed if'em echo $stat['glod']; i będziesz wiedział co masz

[erix]

<?php
$stat = mysql_fetch_array(mysql_query("select * from players where email='$email' and pass='$pass'"));
?>

Co wywala mysql_error" title="Zobacz w manualu php" target="_manual?
Sprawdź jeszcze co znajduje się w $stat.

[Lethys]

Nie ma zadnego bledu a co do $stats to napewno dobrze bo w innych funkcjach dziala dobrze ..

A moze jest inny sposob na to co chce stworzyc.Chodzi mi o to zeby wybieralo sie jedna z 2 opcji ( sila lub zrecznosci ) i przy wykonywaniu akcji odejmowala $glod=$glod - 1 . No i zeby zabespieczyc jak $glod=0 to nie mozna nakarmic .

Tym razem zrobilem prosty kod karmienia

[PHP] pobierz, plaintext 
<?php  
 
include("config.php");
$stat = mysql_fetch_array(mysql_query("select * from players where email='$email' and pass='$pass'"));
 
if ($stat['glod'] == 0) {
		print "<br><br>Twoja Kreatura nie jest glodna !";
 
}
 
if ($stat['glod'] == 1) {
 
 
 
				mysql_query("update players set glod=0 where id=$stat[id]");
			mysql_query("update players set $stat[zrecznosc]=$stat[zrecznosc]+1 where id=$stat[id]");
				mysql_query("update players set $stat[nastroj]=$stat[nastroj]+1 where id=$stat[id]");
				print "<br><br>Twoj chowaniec zyskal jeden punkt zrecznosci!";
 
}
 
 
?>
[PHP] pobierz, plaintext 

I wyskakuje ten sam blad czyli ze chowaniec nie jest glodny a glod w bazie jest rowny 1 Juz naprawde nie wiem co jest grane ...

Ten post edytował [Lethys] 17.10.2006, 17:04:03

[erix]

No i zeby zabespieczyc jak $glod=0 to

bez komentarza

[PHP] pobierz, plaintext 
<?php
mysql_query("update players set glod=0 where id=$stat[id]");
			mysql_query("update players set $stat[zrecznosc]=$stat[zrecznosc]+1 where id=$stat[id]");
				mysql_query("update players set $stat[nastroj]=$stat[nastroj]+1 where id=$stat[id]");
?>
[PHP] pobierz, plaintext 

Zamiast nazw pól po SET podajesz wartość=wartość...

[Lethys]

To nie ma znaczenia

Btw . zmienilem dla sprawdzenia i ten sam blad ...

[D4rky]

Wy wszyscy pieprzycie PIERDOŁY, a błąd widać na kilometr
Na początek

[PHP] pobierz, plaintext 
<?php
$stat = mysql_fetch_array(mysql_query("select * from players where email='$email' and pass='$pass'"));
?>
[PHP] pobierz, plaintext 

Już tutaj masz 2 błedy w samym stylu pisania, i opieprzę cię tak dla pewności Jeśli uzywasz $stat['nazwa'], to stosuj fetch_assoc nie array. Poaz tym '".$email."' jak już

Dalej:

[PHP] pobierz, plaintext 
<?php
if($_POST['karmienie'] == karmienie) {
?>
[PHP] pobierz, plaintext 

Powinno ci wywalić undefined const karmienie. Nie dałeś "", więc php łapie karmienie jako stałą (define i te sprawy).

No i jeszcze

[PHP] pobierz, plaintext 
<?php
mysql_query("update players set glod=0 where id=$stat[id]");
	mysql_query("update players set $karmienie=$karmienie+1 where id=$stat[id]");
?>
[PHP] pobierz, plaintext 

Po pierwsze jak już mówiłem "'.$stat['id']."' (tutaj dodatkowo zgubiłeś ' ' w $stat), a drugi błąd polega na tym, że chcesz zmienić zmienna SQLa czyli karmienie, a łapiesz tam zmienną PHPową czyli $karmienie, której nigdzie tu nie widzę. Wywal $ w obu karmieniach i będize ok.

No i na drugi raz przyjdź najpierw do mnie, bo widzisz, że tu nikt pomóc nie chce

[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ż

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

[Lethys]

Niby wszystko poprawione a jak wlaczam to nic sie nie dzieje ( klikam na karmienie i nic )


Poprawiony kod

[PHP] pobierz, plaintext 
<?php include("config.php"); ?>
<?php
$stat = mysql_fetch_assoc(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='".(int)$stat[id]."'");
	mysql_query("update players set karmienie=karmienie+1 where id='".(int)$stat[id]."'");
	print "<br><br>Twoja kreatura zyskala <b> 1 pkt</b> wybranej umiejetnosci.";
 
}
?>
[PHP] pobierz, plaintext 

[jarrod]

hmmm....
wydawało mi się że pole typu radio trochę inaczej wygląda....

[HTML] pobierz, plaintext 
<form method="post" action="karmienie.php?action=karmienie">
<br/><br/><center>Czym chcesz karmic ?</center><br/>
<input type="radio" name="karmienie" value="sila" id="sila"/><label for="sila">Wodorosla ze Smoczej Przeleczy ( dodaje sily )</label><br/>
<input type="radio" name="karmienie" value="zrecznosc" id="zrecznosc"/><label for="zrecznosc">Jajecznica z gryfinich jaj ( dodaje zrecznosc )</label>
<br/><br/> <input type="submit" value="Nakarm"/></form>
[HTML] pobierz, plaintext 

Tamten formularz ZUPEŁNIE NIC NIE PRZENOSIŁ. Bo niby jak?

a całość:

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

Tyle że wpowyższym if-ie nie ma igdzie powiedziane czym ją karmisz...
To czym jest karmiona jest przechowywane w zmiennej:

[PHP] pobierz, plaintext 
<?php
$_POST['karmienie'];
?>
[PHP] pobierz, plaintext 

Ten post edytował jarrod 18.10.2006, 18:43:05

[Lethys]

No i teraz pisze ze nie jest glodna a wdl warunku jest Nie wiem czy kiedys to zrobie .....

[jarrod]

Zrób tak:

[PHP] pobierz, plaintext 
<?php
include("config.php"); 
$stat = mysql_fetch_assoc(mysql_query("select * from players where email='".$email."' and pass='".$pass."'"));
 
var_dump($stat);
var_dump($_POST);
?>
[PHP] pobierz, plaintext 

i przeklej to tutaj.

[Lethys]

Wyskakuje

bool(false) array(1) { ["karmienie"]=> string(4) "sila" } ?>" title="Zobacz w manualu php" target="_manual

[jarrod]

' date='18.10.2006, 20:57:06 ' post='302280']
Wyskakuje

bool(false) array(1) { ["karmienie"]=> string(4) "sila" } ?>" title="Zobacz w manualu php" target="_manual

I nic ci to nie mówi??

To zrób jeszcze tak:

[PHP] pobierz, plaintext 
<?php
$sql = "select * from players where email='".$email."' and pass='".$pass."'";
$res = mysql_query($sql);
 
echo mysql_error();
echo '<br/>'.$sql;
?>
[PHP] pobierz, plaintext 

i też przekopiuj tutaj.

[Lethys]

Teraz wyskakuje


select * from players where email='' and pass='' " title="Zobacz w manualu php" target="_manual


Niestety nie wiem za bardzo co jest zle poniewaz od niedawna ucze sie php metada prob i bledow ale tutaj juz mi rece opadaja

Jak mozesz i wiesz jak to napisz mi ja ma byc zeby bylo dobrze , z gory dzieki.

Ten post edytował [Lethys] 18.10.2006, 20:29:13

[jarrod]

to jeszcze mi powiedz skąd bierzesz dane $email oraz $pass.

Ewidentnie ich nie ma - nie są zainicjowane. Dlatego mysql_error() nie pokazuje błędów (bo ich nie ma) a pytanie nie wykonuje się prawidłowo (nie zwraca rekordu).

Zatem sprawdź gdzie są inicjowane te zmienne i dlaczego one się nie inicjują.
Jak się z tym uporasz to wówczas powinno zadziałać.

[Lethys]

Wydaje mi sie ze to jest poprawnie

[PHP] pobierz, plaintext 
<?php
$stat = mysql_fetch_array(mysql_query("select * from players where email='".$email."' and pass='".$pass."'"));
?>
[PHP] pobierz, plaintext 

Tak mam zrobione w innym pliku php i wszystko dziala , a teraz jak to wpisalem wyskakuje blad

Leere Abfrage.
?>" title="Zobacz w manualu php" target="_manual

[jarrod]

To Ci przypomnę co sam napisałeś:

Teraz wyskakuje
select * from players where email='' and pass=''

W tych zmiennych nic nie ma. Tam gdzie się to udaje musi być includowany jakiś plik w którym jest to robione. Poszukaj.

Ten post edytował jarrod 18.10.2006, 20:52:06

[Lethys]

No wlasnie o to chodzi z tylko config ..

[PHP] pobierz, plaintext 
<?php include("config.php"); session_start(); ?>
 
<?php
if (!session_is_registered("email") || !session_is_registered("pass")) {
	print "Sesja sie skaczyla! .";
	exit;
}
$stat = mysql_fetch_array(mysql_query("select * from players where email='$email' and pass='$pass'"));
if (empty ($stat[id])) {
	print "Niepoprawny Login.";
	exit;
}
$numer = ********];
$numer1 =********;
$potworek = ********;
$potworek2 = ********);
$jajko = ********;
$potwor = ********;
$potwor2 = ********;
$potwor3 = ********;
$ctime = time();
mysql_query("update players set lpv=$ctime where id=$stat[id]");
$ip = "$HTTP_SERVER_VARS[REMOTE_ADDR]";
mysql_query("update players set ip='$ip' where id=$stat[id]");
?>
[PHP] pobierz, plaintext 

tyle ...

[jarrod]

To spróbuj może:

[PHP] pobierz, plaintext 
<?php
$stat = mysql_fetch_array(mysql_query("select * from players where email='".$_SESSION['email']."' and pass='".$_SESSION['pass']."'"));
?>
[PHP] pobierz, plaintext 

Sprawdź czy w tym zapytaniu masz zmienne.

Pozatym:

[PHP] pobierz, plaintext 
<?php
include("config.php"); session_start();
?>
[PHP] pobierz, plaintext 

czy:

[PHP] pobierz, plaintext 
<?php
session_start();  include("config.php");
?>
[PHP] pobierz, plaintext 

bo wykonujesz kawałek kodu bez inicjacji sesji.

[Lethys]

Teraz dziala ale nie do konca bo odejmuje juz wartosc glod i wyswietla ze nakarmiles chowanca ale nie dodaje pkt statystyk wiec cos zle ...

btw. powodem wczesniejszego nie dzialania bylo brak session_start();

Aktualny kod :

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

Prosze mi pomoc z naprawieniem karmienia

Ten post edytował [Lethys] 18.10.2006, 21:49:17

[1010]

te zapytanie daj tak

[PHP] pobierz, plaintext 
<?php
mysql_query("update players set karmienie=(karmienie+1) where id='".(int)$stat[id]."'");
?>
[PHP] pobierz, plaintext 

i powinno działać

[Lethys]

Nie dziala , poprostu nie dodaje pkt

[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 ". ."

[Lethys]

Ok dziekuje wam wszystkim za pomoc , nareszcie udalo sie .

Prosze o zamkniecie tamatu

[jarrod]

' date='19.10.2006, 16:31:08 ' post='302486']
Ok dziekuje wam wszystkim za pomoc , nareszcie udalo sie .

Prosze o zamkniecie tamatu

Zakładam że chodziło o to że w bazie nie było rekordu dla tego użytkownika i UPDATE się wykonywał i zwracał info: "#Zmodyfikowanych rekordów 0"

Przed wykonaniem update musisz być pewny że istnieje rekord na którym to robisz bo niczego nie z-update-ujesz.

Ten post edytował jarrod 20.10.2006, 01:35:03