The Daily WTF на русском
Подписаться
CodeSOD: Вопрос Понимания
Разработчик написал неэффективный и плохо написанный код, который назвали "Гугл-соусом", потому что он, по всей видимости, был скопирован и вставлен из результатов поиска в Google. В коде используется StringBuffer для построения простого SQL-запроса, что является излишним, и в нем присутствует конструкция WITH в неправильном месте. Затем запрос подготавливается как инструкция, но не содержит никаких параметров, делая этот шаг ненужным. Затем разработчик пытается установить несуществующий параметр, что вызовет исключение. Вместо использования инструкции SELECT COUNT код перебирает результаты в цикле, чтобы подсчитать количество строк. Код синтаксически некорректен и не будет работать так, как предполагалось. Самое удивительное, что этот код попал в кодовую базу, не будучи замеченным или исправленным. Разработчик, написавший этот код, по-видимому, не понимал, что делает, и все равно закоммитил нерабочий код. Это подчеркивает отсутствие защитных мер, предотвращающих коммит плохо написанного кода. Задача очистки этого кода теперь ложится на плечи более компетентных разработчиков. Наличие такого кода в кодовой базе вызывает опасения по поводу общего качества и поддерживаемости программного обеспечения.