Я сейчас вам покажу фрагмент переписки code review моего кода. Программка в одном файле на 200 строк решает алгоритмическую задачку.
— [...] В мейн должна вызываться одна функция для чтения, одна для обработки, одна для вывода, можно в цикле.
— 11. Сделал что-то и по этому пункту, стало получше. Правда, вывод результата (одно число) я так и оставил в одну строку вместо создания однострочной функции. Ведь если гипотетический программист захочет расширить этот код (например, изменить направление вывода, пересылать результат по сети), у него есть предельно понятная функция parking.GetTotalProceeds() для получения нужных данных, а строку с printf можно смело удалить, заменив своим собственным кодом вывода.
— Вывод отдельной функцией. При необходимости ее тоже можно будет гипотетическому программисту изменить
— Но ведь изменить одну строчку легче, чем функцию! Я не понимаю, зачем нужно ещё и это усложнение. Keep It Simple Stupid!
— Настаиваю, т.к. в будущем стоит понимать, что возможно расширение, а программа понятнее когда она имеет модульный вид: Вывод отдельной функцией. При необходимости ее тоже можно будет гипотетическому программисту изменить
видали чудеса проектирования?..
модульность одной строки, блджад!
Дедфуду такая архитектура и не снилась.
ulidtko
16.12.2011 16:16
Do you really want to delete ?
это логично с той точки зрения, что гипотетическому программисту абсолютно поебать сколько у тебя там сейчас строчек кода занимаются выводом данных, и как именно они это делают. с другой стороны, этот метод оче хуёво выглядит, но этим, кажется, страдают все курсы: они пытаются научить тебя сложным вещам на простых примерах, и тебе щяс надо дофантазировать, что помимо твоего класса (или чего угодно) есть ещё стомильёнов классов, которые будут наследовать твой код и переопределять метод вывода данных. так что окрести этот метод с принтф подсистемой вывода ухаха
Какое-то толкание в ступе вместо ПРОДУКТИВНОСТИ.
синдром вахтёра детектед
yep
code review для helloworld-а