использовать функцию или повторять код?

whirlwind

TDD infected, paranoid
Понимаешь, вот это пере- для меня сейчас так же выглядит как всем понятная преждевременная оптимизация. Ну как-то так.
 

cDLEON

Онанист РНРСlub
А помоему проблема ТСа не в количестве кода и не в не достаточной "детальности"(хотя, скорее всего, с этим тоже есть проблемы). А в том, что он сунул финал туда, куда не нужно и придумывает костыли.
 

Духовность™

Продвинутый новичок
ой, ну причем тут финал, что вы до финала докопались?

ладно, закроем тему, все равно все уже изменяю..
 

whirlwind

TDD infected, paranoid
Не, кстати, тема финала в пыхе очень даже интересная. Вот этот бы топ в теорию перевести само то имхо. Можно обсудить - нафига в пыхе финал?
 

fixxxer

К.О.
Партнер клуба
Автор оригинала: whirlwind
Понимаешь, вот это пере- для меня сейчас так же выглядит как всем понятная преждевременная оптимизация. Ну как-то так.
Не совсем. :)

Вот у меня лежит фреймворк на серверах в /usr/share/php/<FrameworkName>, его юзают все сайты.

Обнаруживается такое "недо"-, и мучительно думаешь, что делать: переделать по уму (и ломать работающее, править код всех сайтов? нафиг"!), добавить "маленький хак" типа setFooMode() - говно, переделать старый вариант фасадом к новому (здравствуй KISS)?...

Хотя что все нужно в меру - это бесспорно. :)

Тут помогает базовый критерий "тестируемость" как условно небходимый и достаточный, кстати.
 

whirlwind

TDD infected, paranoid
Вот! Если код тестируется, скорее всего, он расширяется без хаков. Плюхи TDD сходу невидны ;) На такой код легко навешать любой паттерн. А это - бесконечная гибкость (в рамках практического применения по кр.мере).
 

fixxxer

К.О.
Партнер клуба
triumvirat

Не знаю чем плох, но по опыту получилось так - везде, где я по каким либо причинам ставил final я его в итоге убрал. :D
 

whirlwind

TDD infected, paranoid
final class = immutable object. То есть = композиция. Immutable один из способов избежать проблем в мультипоточном приложении. Т к в пыхе нет мультитрединга, для него это практически не актуально.
 

whirlwind

TDD infected, paranoid
Да какая разница? Более важнывх вещей в пыхе нету. А менее важные есть. И мне это не нравится :(
 
Сверху