Докатился…

[dil]

Пишу скрипт для веба на PHP. Потому что исправлять тот ужас, который ранее написали быдлокодеры методом copy-paste, очень не хочется. Естественно, ни одного комментария там нет, о  замысле аффтара можно только догадываться.

В общем, ЭТО проще переписать заново, чем менять одно и то же в пяти местах, а потом внезапно обнаружить, что оно ещё и в шестом есть, а я его не заметил.

Чувствую себя ассенизатором.

Оригинал этой записи в личном блоге.
Любые материалы из этого блога запрещается использовать на сайте livejournal.ru в любой форме и любом объёме.

Comments

[mcjabberwock]

БГГ
Любой русский программист, после пары минут чтения кода, обязательно вскочит и произнесет, обращаясь к себе: переписать это все нафиг. Потом в нем шевельнется сомнение в том, сколько времени это займет, и остаток дня русский программист потратит на то, что будет доказывать самому себе, что это только кажется, что переписать это много работы. А если взяться и посидеть немного, то все получится. Зато код будет красивый и правильный.
...

[dil]

Код будет не сильно красивый, я всё же не профессиональный php-программист. Но всяко лучше, чем
select * from servers where type='2' and status='0' для целых полей в таблице,
или
echo "<div id=pageset style=\"width:1024px\">"; отдельно после _каждого_ case: в switch'е,
или использования конструкции cat $FOO | sed 's/XXXXXXXXXX/$line/' для поиска и замены подстроки в файле, который потом всё равно считывается в программу.

Про инъекции в самых разнообразных местах вследствие использования совершенно невалидированных значений непосредственно из POST'а я уж не говорю.

[dil]

Собственно, я его уже переписал ;)

[filonovd.myopenid.com]

Да-да-да. Я как раз через это проходил недавно. Сначала был корявый код. Потом я в него долго вносил изменеия "на коленке". Потом год кода не видел. А потом он мне понадобился. Посмотрел я на это безобразие и решил переписать. Переписал. Все заработало. И даже функциональность стала получше и багов поменьше и вообще неплохо. Но выглядит оно при отрешенном взгляде... прямо скажем, выглядит хреново. Вот не программист я. И через год, боюсь, придется опять переписывать с нуля :)

[dil]

Я-то как раз по образованию программист :) Но на PHP не специализировался.

[filonovd.myopenid.com]

Образование - это дело такое... Я по образованию тоже программировать учился. Но это было 20 лет назад и проектов больше, чем на пару сотен строк кода за время учебы писать не приходилось. Ну и подход соответствующий был: написал - сдал - забыл.

[dil]

Писать мне приходилось гораздо больше, но это не принципиально. Если основы усвоились, то писать более-менее прилично можно на любом языке. Может, не очень эффективно, если не знать конкретных тонкостей, но прилично. А если основ нет, то на любом языке быдлокод получится.

[filonovd.myopenid.com]

На счет любого языка я-бы поспорил.
Просто вспоминаю на чем нас учили писать - ассемблер, паскаль, пролог, вижуал бейсик, немного лиспа.
Вот честное слово, слабо себе представляю чего хорошего из этого набора можно перенести на PHP или, не дай Б-же, Perl. :)

[dil]

Ну лисп и пролог - это отдельная песня с принципиально отличными концепциями, а процедурные языки все плюс-минус одинаковые.
Элементарные вещи типа "говорящих имён" переменных, типизации, насколько она поддерживается языком, выноса инвариантов за пределы цикла, общих операций - в отдельные процедуры, - это всё одинаковое.

[a2is]

> выноса инвариантов за пределы цикла

Не обязательно. Современные компиляторы с этим вполне справляются сами.

[dil]

Подозреваю, что компилятор PHP с этим не справится. И в любом случае программисту не стОит полагаться на интеллект компилятора, лучше проявлять его самостоятельно.

[a2is]

Я в последние 5 лет лет как раз таким ассенизатором и работаю.

Стахановский метод. Это когда один очень одарённый сотрудник херачит, не приходя в сознание, а за ним, яростно матерясь, идут десять человек с лопатами.
Все награды и прочие ништяки, естественно, Стаханову.