Diskuze: Spojenie dvoch update
V předchozím kvízu, Online test znalostí PHP, jsme si ověřili nabyté zkušenosti z kurzu.
Ďakujem za kód aj za upozornenia.Zobrazeno 20 zpráv z 20.
V předchozím kvízu, Online test znalostí PHP, jsme si ověřili nabyté zkušenosti z kurzu.
zavana ti to rozbitim databazy, ak obe UPDATE nezlucis do jednej transakcie a na transakcie potrebujes InnoDB
$sql = "UPDATE members SET money=money+if(body<10, 2, if(body>10, -2,0)) WHERE id=$id";Bacha, může ti to kmitat stejně jako původní řešení.
Ovladač MySQL v PHP AFAIK neumí dva SQL dotazy v jednom příkazu.
Ještě dodatek: Čísla v SQL nepatří do apostrofů. MySQL to sice překousne, ale jiné databáze to tvrdě odmítají.
len som ho upozornil na nebezpecenstvo, co vznikne pri vzajomnom posobeni
tychto prikazov v databaze... a SQL prkaz UPDATE je destrukcny - meni
udaje v databazi, lahko sa moze stat, ze pri vzajomnom pristupe viacerych
navstevnikov k databaze sa cela tabulka stane nekonzistentnou...
tu treba potom zacat pouzivat transakcie, resp. zamky na citanie/zapis pri
zachovani konzistencie
Podle názvu proměnných to spíš vypadá, že to riziko není vysoké. Proč však dělat dva SQL dotazy a spojovat je do jedné transakce, když to jde jedním SQL dotazem, u kterého je transakčnost zajištěna?
5 proměnných není moc. Napiš si to na víc řádek, ať v tom neztratíš přehled
$sql = <<<EOT
UPDATE members
SET
money=money+if(body<10, 2, if(body>10, -2, 0)),
wood=wood+5,
steel=steel+2*body
WHERE
id=$id
EOT;To jsem si jen trochu vymýšlel, je to na tobě.
Předpokládal jsem, že pokud body==10, tak to nemá dělat nic. Alespoň to vyplývá z původního dotazu.
Obvykle je v takovém případě výhodnější udělat dva SQL dotazy a uzavřít do transakce. Napřiklad pokud dva hráči spolu obchodují a mění si zboží za peníze. Pokud však máš správně navrženou databázi, jde to i jedním SQL dotazem, jen je to možná o něco komplikovanější.
$sql = <<<EOT
UPDATE members
SET
money=money+if(id=$id1, $cena, if(id=$id2, -$cena,0)),
wood=wood+if(id=$id1, -$pocet, if(id=$id2, $pocet,0))
WHERE
id=$id1 OR id=$id2
EOT;V tom případě jsi udělal nějakou chybu. Stringy nemůžeš sčítat.
Na text ho musíš trochu upravit. Přece jen jsou operace se stringy odlišné od operací s čísly. A asi nebudeš chtít stringy sčítat, ale například vyměňovat.
Zobrazeno 20 zpráv z 20.