ulidtko 16.12.2011 16:16

Я сейчас вам покажу фрагмент переписки code review моего кода. Программка в одном файле на 200 строк решает алгоритмическую задачку.

— [...] В мейн должна вызываться одна функция для чтения, одна для обработки, одна для вывода, можно в цикле.
— 11. Сделал что-то и по этому пункту, стало получше. Правда, вывод результата (одно число) я так и оставил в одну строку вместо создания однострочной функции. Ведь если гипотетический программист захочет расширить этот код (например, изменить направление вывода, пересылать результат по сети), у него есть предельно понятная функция parking.GetTotalProceeds() для получения нужных данных, а строку с printf можно смело удалить, заменив своим собственным кодом вывода.
— Вывод отдельной функцией. При необходимости ее тоже можно будет гипотетическому программисту изменить
— Но ведь изменить одну строчку легче, чем функцию! Я не понимаю, зачем нужно ещё и это усложнение. Keep It Simple Stupid!
— Настаиваю, т.к. в будущем стоит понимать, что возможно расширение, а программа понятнее когда она имеет модульный вид: Вывод отдельной функцией. При необходимости ее тоже можно будет гипотетическому программисту изменить

видали чудеса проектирования?..
модульность одной строки, блджад!
Дедфуду такая архитектура и не снилась.

Recommended by: @238328, @werdn, @DZhon
1. ruda 16.12.2011 17:35 4030196457132430258703774

это логично с той точки зрения, что гипотетическому программисту абсолютно поебать сколько у тебя там сейчас строчек кода занимаются выводом данных, и как именно они это делают. с другой стороны, этот метод оче хуёво выглядит, но этим, кажется, страдают все курсы: они пытаются научить тебя сложным вещам на простых примерах, и тебе щяс надо дофантазировать, что помимо твоего класса (или чего угодно) есть ещё стомильёнов классов, которые будут наследовать твой код и переопределять метод вывода данных. так что окрести этот метод с принтф подсистемой вывода ухаха

2. DZhon 17.12.2011 09:26

Какое-то толкание в ступе вместо ПРОДУКТИВНОСТИ.

3. analizer 17.12.2011 09:34 talkonaut-iphone_5.91_67b1c873

синдром вахтёра детектед

4. DZhonanalizer /3 17.12.2011 09:39

yep

5. werdn 17.12.2011 12:51 Work

code review для helloworld-а

Do you really want to delete ?