[MySQL][PHP]Jednorazowa funkcja-przycisk Opcje

[edwarpan]

[PHP] pobierz, plaintext 
<?php
$state = "SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem";
if($state==0){
if ($_POST['submit']) {
$kwota = $_POST['kwota'];
$query = "UPDATE users SET kwota = kwota - ".$kwota." WHERE username='".safe($_COOKIE['username'])."'";
mysql_query($query);
$sql = "UPDATE users SET redeem = '1' WHERE username='".safe($_COOKIE['username'])."' ";
mysql_query($sql);
}
}
else
{
echo "Już jesteś zapisany";
}
?> 
[PHP] pobierz, plaintext 

I cuś nie działa.

Ten post edytował edwarpan 13.05.2010, 17:34:08

[wookieb]

Temat: Jak poprawnie zada pytanie

[edwarpan]

Poprawione, proszę o pomoc, manual już cały przeczytany, lecz nic nie działa.

[kapuch]

[PHP] pobierz, plaintext 
<?php
$state = "SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem";
if($state==0){
if ($_POST['submit']) {
$kwota = $_POST['kwota'];
$query = "UPDATE users SET kwota = kwota - ".$kwota." WHERE username='".safe($_COOKIE['username'])."'";
mysql_query($query);
$sql = "UPDATE users SET redeem = '1' WHERE username='".safe($_COOKIE['username'])."' ";
mysql_query($sql);
}
}
else
{
echo "Już jesteś zapisany";
}
?> 
[PHP] pobierz, plaintext 

I cuś nie działa.

Opisz dokladnie o co ci chodzi, bo nie widze zwiazku tytulu z kodem, ktory tu dajesz.
Poza tym ustawiasz $state = "SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem";
a pozniej praktycznie caly kod uzalezniasz od tego, czy $state==0. W tym kodzie $state nigdy nie bedzie == 0, wiec jedyne co bedzie wykonane to echo "Już jesteś zapisany";.
NIe mowie juz o reszcie kodu, ktoremy zarzucic mozna wiele, ale po co, najlepiej poczytaj jakis poradnik o php i sql.

Ten post edytował kapuch 13.05.2010, 18:46:32

[croc]

W ogóle jakiś pokićkany ten kod...

[edwarpan]

W cookie zapisuje się nazwa użytkownika.
Na stronie ma być button z zapisem do tabeli + odjęciem wartości 1 z innej tabeli.
I chodzi o to aby można tylko raz tam kliknąć.

[PHP] pobierz, plaintext 
<?php
 
if ($logged_in == 1) {  
 
	?>
	Logged in as: <?php echo $_COOKIE['username']; ?><br />
        <br /><b>Powiadamianie email o restocku! <strong>1Punkt</strong></b><br>
        <form action="zredeem.php" method="post">
	<input type="hidden" name="kwota" value="1" />
	<input type="submit" name="submit" value="modyfikuj" />
	</form>
 
<?php
$state = "SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem";
if($state==0){
if ($_POST['submit']) {
$kwota = $_POST['kwota'];
$query = "UPDATE users SET kwota = kwota - ".$kwota." WHERE username='".safe($_COOKIE['username'])."'";
$sql = "UPDATE users SET redeem = '1' WHERE username='".safe($_COOKIE['username'])."' ";
mysql_query($query);
}
}
else
{
echo "Już jesteś zapisany";
}
?> 
[PHP] pobierz, plaintext 

cały kod.

Ten post edytował edwarpan 13.05.2010, 18:47:55

[piotrooo89]

mysql_error zwraca Ci coś?

[croc]

Co za różnica co zwraca mysql_error skoro cały kod jest pochrzaniony?

[bolverk]

[PHP] pobierz, plaintext 
$state = "SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem";
if($state==0)
[PHP] pobierz, plaintext 

To pierwsza i druga linijka twojego kodu PHP zaraz po formularzu. Przecież wynika jasno z tego że $state zawsze będzie równe

Kod

"SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem"

[kapuch]

mysql_error zwraca Ci coś?

A co ma zwracac mysql_error, skoro nawet do operacji w sql nie dochodzi, skoro dal warunek zeby $state rownalo sie 0, a zaraz przed tym nadal wartosc $state rowna zapytaniu.
Jak ma ten kod sie wykonac?

[PHP] pobierz, plaintext 
$state = "SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem";
if($state==0)
[PHP] pobierz, plaintext 

To pierwsza i druga linijka twojego kodu PHP zaraz po formularzu. Przecież wynika jasno z tego że $state zawsze będzie równe

Kod

"SELECT * FROM users WHERE username='".safe($_COOKIE['username'])."' SET redeem"

No dokladnie o tym pisalem na samym poczatku, ale autorowi chyba nie chcialo sie tego przeczytac.
Fajnie ze ktos tez to zauwazyl

[croc]

A poza tym co to za zapytanie?

[SQL] pobierz, plaintext 
SELECT cośtam FROM cośtam SET cośtam
[SQL] pobierz, plaintext 

Jakiś nowy standard SQL?

[kapuch]

A poza tym co to za zapytanie?

[SQL] pobierz, plaintext 
SELECT cośtam FROM cośtam SET cośtam
[SQL] pobierz, plaintext 

Jakiś nowy standard SQL?

Hehe chyba tak, mi sie podoba to

Kod

UPDATE users SET kwota = kwota - ".$kwota." WHERE username='".safe($_COOKIE['username'])."'";

Do Autora:
tak powaznie, to musisz jeszcze soporo poczytac manuali, zarowno tych od PHP, jak i sql.
Bo ty nawet nie rozumiesz jakie warunki stawiasz skryptowi, ktory ma sie wykonac, nie mowic juz o blednie sformulowanych zapytaniach sql.
Skoro dajesz warunek, ze aby sie skrypt wykonal, zmienna $state musi byc rowna 0 (zero), a chwile wczesniej tej samej zmiennej nadajesz jakas wartosc (zapytanie sql), ktora napewno nie jest rowna zero, wiec jak ma ci sie cokolwiek wykonac?
Z drugiej strony, moze i lepiej ze sie nie wykonalo, bo przy tych zapytaniach, to moglby ci serwer siasc


Ten post edytował kapuch 13.05.2010, 21:03:27

[bolverk]

Kocham ten serwis właśnie za użytkowników zadających banalne pytania, i za użytkowników na nie odpowiadających z takim poczuciem humoru. Na takie teksty nie wpadliby żadni Łowcy.B czy nawet Kabaret Moralnego Niepokoju. Oto reklama: Nigdzie indziej nie pośmiejesz się i nie nauczysz jednocześnie, tylko na wortal.php.pl Ubawiłem się setnie czytając ostatnie posty

A tak na marginesie, gdzie są ci programiści którzy w poszukiwaniu zagubionego przecinka analizwali własny kod przez dwa dni? Pewnie zaglądają na ten serwis właśnie.

Ten post edytował bolverk 13.05.2010, 21:22:02

[kapuch]

Kocham ten serwis właśnie za użytkowników zadających banalne pytania, i za użytkowników na nie odpowiadających z takim poczuciem humoru. Na takie teksty nie wpadliby żadni Łowcy.B czy nawet Kabaret Moralnego Niepokoju. Oto reklama: Nigdzie indziej nie pośmiejesz się i nie nauczysz jednocześnie, tylko na wortal.php.pl Ubawiłem się setnie czytając ostatnie posty

A tak na marginesie, gdzie są ci programiści którzy w poszukiwaniu zagubionego przecinka analizwali własny kod przez dwa dni? Pewnie zaglądają na ten serwis właśnie.

Hehe nie ma to jak mozolne analizowanie kodu, w celu znalezienia zgubionego przecinka, gdy juz myslisz ze go znalazles, okazuje sie ze to mucha na monitor narobila

PS. No pozartowac nie zaszkodzi - smiech to zdrowie, kazdy jakos zaczynal i nie wysmiewamy tu nikogo, tylko smiejemy sie z zaistnialej sytuacji
Poza tym dalismy rozwiazanie problemu - pouczyc sie php i sql, no bo na poprawe bledow w tym kodzie szkoda czasu, bo autor i tak by pewnie nie zrozumial o co chodzi, nie dlatego ze jest inteligentny inaczej, ale dlatego ze brakuje mu podstawowej wiedzy na ten temat.

Ten post edytował kapuch 13.05.2010, 21:30:48

[bolverk]

Kod

Logged in as: <?php echo $_COOKIE['username']; ?><br />
<br /><b>Powiadamianie email o restocku! <strong>1Punkt</strong></b><br>
<form action="zredeem.php" method="post">
<input type="hidden" name="kwota" value="1" />
<input type="submit" name="submit" value="modyfikuj" />
</form>

Kod

UPDATE users SET kwota = kwota - ".$kwota." WHERE username='".safe($_COOKIE['username'])."'";

Już jako ostatni post w tym temacie napiszę że odwołanie do zmiennej $kwota świadczy że posługujesz się przestarzałym i do tego niebezpiecznym dla użytkownika kodem PHP. Do zmiennej w polu formularza name="kwota" odwołuj się przez pole method na początku formularza, czyli w tym przypadku:

Kod

$_POST['kwota']

[edwarpan]

Jakoś sobie bez Was poradziłem, poczytałem. Działa, może nie tak jak chciałem, bo jak ktoś ma 0 w tabeli kwota to i tak może się zapisać.