Разбираясь в коде проекта, над которым сейчас работаю, обнаружил довольно забваный фрагмент:
$sql = "DELETE FROM cross_presenters_sessions
WHERE session_id = ".(int)$_REQUEST["session_id"]."
AND user_id = ".(int)$_REQUEST["user_id"];
queryExecute($sql);
$sql = "INSERT into cross_presenters_sessions (session_id, user_id)
VALUES (".(int)$_REQUEST["session_id"]." , ".(int)$_REQUEST["user_id"].")";
queryExecute($sql);
Долго пытался понять, есть ли в этом какой-нибудь скрытый смысл, а потом пришёл к выводу, что разработчик не знает (или забыл) SQL оператор “REPLACE”. Хотя, с другой стороны, какие претензии можно предъявлять к коду с комментарием:
// Direct access to DB // YES, i know, its VERY bad // sorry // we have a deadline now:(( // refact it if you can/want

Комментарии