Ja tam vidim chyb hned nekolik. Mel jsem moznost v posledni dobe par podobnych skriptu videt a uz delsi dobu uvazuju o tom, ze bych sem o tom par veci napsal, ale zatim nebyl cas.
Predne bych rad predeslal, ze ten kod by mel mozna fungovat a ze v nem nevidim funkcni chybu a dokonce u me funguje spravne. Ovsem, styl jakym je ten kod napsanej...
V kostce...nevim od koho to mas, ale sepisu co me drasa:
1)
String id. Ja vsadim boty, ze ten programator neni takovej idiot, aby udelal id jako string, presto je takovej idiot, aby se na nej jako na string dotazoval. Tohle je trest bozi za to, ze je mysql (ale i php a taky html) malo striktni. Proste jazyk pro lamy, ktery skousne vse. Predpokladam, ze to nejak preprocesor rozlouskne a nakonec s tim jako se stringem nepracuje, ale...
2)
Pocitani ve floatu. Za 1/100000 bych na miste popravoval a dokonce verim, ze tohle je slabinou tohoto konkretniho kusu kodu. Proste tam bylo 10000, ty sis tam hodil 100000, ale v databazi mas precision o rad dolu. Pri pocitani penez pouziva float snad jedine naprostej tatar a programatorskej analfabet. Kazdej programator vi, jake problemy float zpusobuje.
Min kriticky veci:
3) V updatu
chybi limit...nejlepsi cesta, jak si pri blbym vstupu prepsat celou databazi.
4) Nazvat si
id jako
$id je fajn, ale lepsi je si alespon v kodu id oznacit nejak smysluplne...napriklad
$user_id. Kdo se v tom ma vyznat? Nakonec se ty id pomichaji a problem je na svete...
Omlouvam se, jestli jsem se rozcilil moc, ale opravdu to ve me peni dyl, tak pardon.
Tobe doporucuju nejaky lehky debug. Dej si tam
echo '$id', abys mel jistotu, ze tam dorazi spravne id, nebo jeste lip, dej si echo misto toho query, at vidis co vlastne volas. Do $m doporucuju pro zacatek vlozit celociselnou hodnotu, treba 1. Mozna je tam nejaka jina chyba, kterou ted pri rudem videni nevidim...