- 1
- 2
- 3
$result = doquery("SELECT * FROM {{table}} WHERE username='".$username."';",users,true);
$username = $result['username'];
unset($result);-
PHP / Говнокод #104
−57.3
В одной браузерной стратегии...
Запостил:
guest,
11 Декабря 2008
Follow us!
Это он про COUNT(*) не знал??
Могу конечно ошибиться, но COUNT(*) здесь совсем ни причём.. Лично я вижу в коде 2 жуткие ошибки:
1) $username в запросене обрабатывается через mysql_real_escape_string (=== SQL Injection)
2) SELECT * ... - эт конечно ппц, тем более из-за того, что нужно только поле username.
2Автор: А там далее по коду не было точно такого же селекта, де он брал ещё одно поле, а потом снова удалял результат? ^__^
Все в норме))) нагадил убери за сабой xDDD
2 ^_^:
А жуткая ошибка то что автор кода выбирает поле username по полю username которое известно до выборки - она не так заметна? :)
Правильно навреное вот так:
а еще вернее:
Зачем лишние переменные?
LoneCat, +1
LoneCat, +1 как от опубликовавшего этот код.
^_^, не факт, что username пришёл из пользовательского ввода, или маджик-квоты выключены.
LoneCat, Ну а в чём же тут ошибка? Если пользователя нет в базе, то $username на выходе будет пустым.
О нормализации запросов и не слышали
[quote=dioteos]О нормализации запросов и не слышали [/quote]
о нормализации запросов я тоже не слышал :)
а я знаю, да, ogame клон называется (ugamela в гугле ;) )))
кстати я ей занимался )
но через год забил из-за кривизны кода ))