Пойду параметризованные SQL пилить... А что, это так важно прям? Я просто питонист вообще-то, RAW SQL обычно не пишу, но не думаю что из-за этого там что-нибудь сломается
Это важно тем, что используя PDO, где есть подготовленные запросы, а значит какой никакой контроль типов, скрипту скармливается RAW SQL... К тому же если у вас 100к продуктов, то это архиважно, так как при использовании подготовленных запросов возрастает быстродействие. Ну и второе - по стилистике - это не код, это лапша. Человек котрый это писал, через пару месяцев сам не поймет что у него где, а уж сторонний человек и подавно не вникнет сходу.
В общем совет такой: Разносите классы по файлам, весь sql с html в константы, конфиг вынести отдельно. Начал сейчас в файл вникать и у меня вопросов все больше. Сначала идет HTML. Говно конечно, ну ок, автор так видит. А потом идет отправка заголовков. Это норм вообще? Ниче что заголовки должны отправляться до какого либо вывода, а не после? Скрипт тупо варнинги кидать будет. Опять же, к конструкции while ($value = $query->fetch(PDO::FETCH_ASSOC)) Все места использования в коде можно безболезненно заменить на fetchAll. В общем рефакторить и рефакторить, так это не взлетит
Обсуждают сегодня