тогда нельзя было так кодить вот и всё.. есть же прицип DRY

не мне Вам, вероятно, объяснять

вот мой драй. всё в одном месте всегда

Называется по одному - class Debug а вызывается статически в 346 местах.

где нужна гибкость - вертикальная иерархия, если нужны всякие мапперы или декораторы - отдельный разговор, но в моём проекте и без них нормально.

ну. вызвали в 346 местах один и тот же класс Debug

верно?

да да... мазать синяк вазилином можно и без мытья рук

Потом находишь в интернете более крутой класс SymfonyLogger и хочешь начать логировать им везде. Но для этого тебе понадобится в 346 местах переписать Debug на SymfonyLogger

НАФИГА

да вы че ну мужики ема

что вам религия в Debug нt позволяет написать SymfonyLogger ?

Ну а как ты заменишь?

может кто-то не выспался?

руками как

ппц

То есть ты хочешь внутри Debug начать вызывать SymfonyLogger ?

верно. спасибо

хворост для сожжения начинать собирать теперь?

Ну так это же нифига не DRY. Зачем тебе лишняя сущность в виде Debug если можно сразу передать инстанс SymfonyLogger ?

я больше скажу, можно ещё выполнить крутой неглупый поиск автоматизированный по коду и зафигачить везде где есть use path\path2\Debug нечто вроде use path\path2\Debug as SymfonyLogger

В такой ситуации получается что класс Debug это лишняя обертка которая ничего не делает а только дергает другие классы.

для тех кому плохо, если это не симфония

да

нечто, похожнее на декоратор

а может оно и есть я не помню точно наизусть паттерны

так и что плохого я так и не понял

наоборот только разрешил проблему секундным добавлением строки

нечто, похожнее на декоратор

Ну вот и зачем засирать код таким мусором если можно гораздо меньшими усилиями обойтись инстансами объекта.

каким мусором это низкий уровень

вот если управень управления приложением грязный - это оч плохо

Можно обойтись без этой обертки -> не нужно.

а внизу то что страшного, я и в yii2 уже недочёты видел

хорошо. а потом выясняется что в коде нельзя использовать инстансы

что там надо передть CALLBACK

и что Вы будете делать с этой теорией Инстансов тогда? прощу прощения за напор

Так yii2 далеко не по лучшим канонам написан.

не спорю

но он мой первый fw и как уже сказал не было времени

и что Вы будете делать с этой теорией Инстансов тогда? прощу прощения за напор

Ээ с какой теорией? Использовать, быстро и просто :)

@chebotarevp ещё раз

пришёл нач

скзаал чувааак ненадо инстансы

ведь тут логирование делается через callback-и

пиши callback-и, спасибо. удачи.

и что тогда?

Коллбек это уже не статика как минимум

И не надо тут это примешивать

мы не про статику

а про то, что надо менять одно на другие на уровне идентификаторов (имён функций, например)

и что плохого в обобщающей обёртке я реально не понимаю

когда она вообще закрывает глаза людей на то как и что там устроено

это обычная инкапсуляция по сути

что в ней плохого?

Это не инкапсуляция.

Навскидку, 1. ты нихрена не протестируешь, ибо мокнуть зависимость не сможешь. 2. у тебя в принципе нереюзабельный код, ты его слишком завязал на логику приложение, всякие ветвления, типа если один if - то OneLogger, если другой - то SecondLogger, опять же, 2.1 - сильная связанность, т.е.крест на реюзабельностию 3. Повышаетя общая сложность этого класса, всякие ветвления в больших количествах - уже признак, а с отсуствием тестирования - рассадник багов.

Ну да реюзабельность низкая.

но это все попытка объяснить аборигенам, чо руки мыть нужно... а они возражают, что мажут мазь прекрасно и так... и ведь, в принципе, не так уж и не правы ;) не будет толка от этого, если человек не учится, а фигачит код - то или будет вечным джуниором или набъет шишки и поумнеет

А в ооп она очень важна - там везде пытаются использовать уже имеющиеся абстракции и интерфейсы, чтобы достроить свое поведение.

но в общем оснавная проблема это нетестируемость

это функциональная инкапсуляция, если позволите

сокрытие части функционала

Давай называть это своими словами - это обертка, без которой можно обойтись.

если обойдётесь то когда гром грянет у вас громооттвод уже не влезет в вашу отладку

ERROR: S client not available

Она решает несуществующую проблему подсовывания различных реализаций логгера, которая реализуется обыкновенным внедрением объекта.

потому что она будет начальным и она же конечным звеном и всё что будет уметь это давать Вам в руку ток с неба напрягом в 10кв

про внедрение поподробнее, если можно

и чем оно лучше я так и не понял. я идиот.

Не я уже спать собирался

Потом тогда как-нибудь допишу..

второй пункт у @miksir принят к рассмотрению, не хватает оперативки ща понять, но чувствую запах сути где-то рядом

@chebotarevp я Вам тогда в личку можно напишу чтобы не забыли

@chebotarevp ..тема то интересная

)

да не.. не очень интересная если честно

но если можешь - завтра днем (по москве) помучай @adel этим вопросм ;) он у нас официальный ментор ;)

невопрос. мне нужны советы

Но с настроем "я лишь только еще больше убедился в своей правоте" конечно лучше не мучать никого.

ну дада, ну хреновая черта, знаю =)

спасибо за помощь всё-равно!

буду думать, перечитывать

Поддержите топик плиз https://habrahabr.ru/company/devconf/blog/330564/

Она решает несуществующую проблему подсовывания различных реализаций логгера, которая реализуется обыкновенным внедрением объекта.

кто-нибудь может это разъяснить?

что подразумевал человек под внедрением объекта?

доброе утро неспящим)

Спам

ясно. может быть Вы поможете с вопросом?

@confguru

Каким?

Она решает несуществующую проблему подсовывания различных реализаций логгера, которая реализуется обыкновенным внедрением объекта.

кто-нибудь может это разъяснить?

что подразумевал человек под внедрением объекта?

Не понял контекста

Наверно надо автора спросить