Wątek: php czasem zawodzi...

niech mi ktoś to wyjaśni

Kod php:


$query = mysql_query("select * from users where user = ".$id);
$dane = mysql_fetch_array($query);

if(isset($_GET['x']) && $dane['kasa'] > 0) {
$query = mysql_query("update users set kasa = kasa - 1 where user = ".$id);
} 


jakim cudem w bazie kasa może wejść na wartość poniżej 0?

a jakiego tupu jest pole w bazie.

int..

ogólnie napisali mi na innym forum, że tak może się właśnie zdarzyć i nawet takie rzeczy trzeba zabezpieczać, np poprzez transakcje...

yehh tego się nie spodziewałem ;]

mi się nigdy nie zdarzyło przy typu pola int ale kto wie.

no nie chodzi o typ pola, tylko o ilość przeładowań strony i w pewnym momencie może być tak, że dasz 2 jednoczesne zadania dla danego usera

jedno się nie skończy a już drugie zacznie lecieć

Nie bardzo rozumiem to pytanie. Przecież to chyba jasne, że gdyby nie ten warunek, że kasa ma być >0 to można by zjechać z kasą grubo ponad kreskę. A w ten sposób UPDATE wykona się gdy kasa==1 ale już nie gdy kasa==0.

no właśnie nie zrozumiałeś

bo jak się okazało, to sam ten warunek if(isset($_GET['x']) && $dane['kasa'] > 0) { nie wystarcza

w sytuacjach dużej ilości przekliknięć/odwiedzin strony to okaże się, że się wywali w momencie gdy np 2x klikniesz w link, który potem miałby wykonać tą akcję

Co znaczy 'się wywali'? Mniemam, że chodzi o to, że zapytanie się wykona i sprowadzi wartość kasy poniżej 0. A wykona się nie z winy PHP, tylko z winy MySQL.

Użycie transakcji w takim wypadku powinno zagwarantować prawidłowe działanie.

to ze transakcja to zalatwi to jest to pewne i sam to napisalem juz dawno poczytaj wczesniejsze posty

problem pojawia sie gdy ten prosty kod, bardziej skomplikujemy

i nie chodzi o to ze ja nadal nie wiem jak co zrobic, tylko pokazuje userom, ze taka sytuacja moze sie zdazyc choc niby IF mial przed tym chronic

(sorry za brak pl, pisane z komorki)

Kod php:

if(isset($_GET['x'])) { 
$query = mysql_query('UPDATE users SET kasa = kasa - 1 WHERE user = ".$id.' AND kasa>0'); 
} 


Taki, prostszy kod, zadziała bez transakcji.