и нихуя понять не могу

читай вслух и с выражением

посмотрите как фактори прекрасно работают в ларке

жтио тёмная магия мне она не нравится, но это наглядно показывает

Да забейте вы на эти формальные объяснения что такое фабрика, а что такое хуй. Фабрика - тупо сокрытие логики при создании объектов.

она нигде не мешает

не, я к тому, что надо избегать ситуации когда "фабрика на фабрике, командой подгоняет" (непомню кто из чатика сказал)

да там какая-то хуйня написана

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

тикет устно обсуждался до того

и его и сома, ога

но это было 2 месяца назад

вброшу кусок кода, чтоб подстегнуть обсуждения =)

вот так я пилю это дело

ещё важная деталь втосоздания - условие по которому создаётся тот или иной дочерний клссс. это как раз и надо делать в фабрике

а не ифом в контроллере/моделе

но пхп простительно всё. вообще всё :)

? ну мы же стремимся к самосовершенствованию

вот так я пилю это дело

ну ифы у тебя один хрен останутся с фабрикой

<тонкий юмор>говнокод какой-то</тонкий юмор>

ну ифы у тебя один хрен останутся с фабрикой

вот с этим я не спорю =)

<тонкий юмор>говнокод какой-то</тонкий юмор>

не, Еж, я не отрицаю, что говенный код =)

вот с этим я не спорю =)

а так, в принципе можешь и через фабрику.

<тонкий юмор>говнокод какой-то</тонкий юмор>

тут больше половины сделано "бля, давай вот эту еще хуйню пилим... и вот эту..."

Только вот эти ифы бы изменить

sploshnoi ^^

Только вот эти ифы бы изменить

например

да не, код как код

sploshnoi ^^

не я базу создавал =) мне сразу модель выдали, мол "вот, мы долго думали и нам надо что-то такое"

например

1) вынеси это в отдельный метод

не я базу создавал =) мне сразу модель выдали, мол "вот, мы долго думали и нам надо что-то такое"

AS в запросах отменили? ?

sploshnoi ^^

и оно даже было плюсом в начале, потому что особо не надо было задумываться чо там в таблицы ебенить

У тебя там все ифы отрабатывают

AS в запросах отменили? ?

м?

У тебя там все ифы отрабатывают

и выдадут тип для фабрики?

у нас тут поначалу хуячили поля в таблицах на родной мове

но потому поняли, что это хуета какая-то

и выдадут тип для фабрики?

не, просто хреново. У тебя нету выхода от туда.

Просто вынеси эту хрень в отдельный метод для начала

м?

ну в код данные можно под нормальными именами подать через AS

и выдадут тип для фабрики?

У тебя там каждый иф проверяется

и похуй чё там в базе

ну в код данные можно под нормальными именами подать через AS

а... дык я там eloquent

мы по началу так спасались

У тебя там каждый иф проверяется

а, да, есть такое... там низзя взять и захуячить =)

теперь пишем всё на инглише

а, да, есть такое... там низзя взять и захуячить =)

Просто если он уже нашел совпадение, зачем следующие ифы проверять?

Причем там 3 разных свойства проверяется

Что за пиздец вообще

ну кстати про все ифы да

можно элсиф заюзать ж

или свитч тру и погнали ?

Что за пиздец вообще

вот не спрашивай =) там пиздец

вот не спрашивай =) там пиздец

Там есть какой-то приоритет у разных свойств?

это чудовище стремится к стандартизации, но как-то... хуевенько

Типа если client_name_code > order_name_mc

там тип выбирается во всех ифах

а значит выигрывает только один иф

а значит можно заюзать элсиф

мы вчера ждали вебки (

а значит можно заюзать элсиф

да вот непонятно есть ли там приоритет

есть

если есть, то надо перестроить все это дело

сейчас выигрывает последний правильный иф

даже если их было несколько

ERROR: S client not available

=)

почему else не используешь?

сейчас выигрывает последний правильный иф

не, по логике, а не по факту

по факту тоже

да там обычно не бывает ситуации, когда оно пересечется

пока что

=)

2 значения в $type не записать

по факту тоже

по факту да, а вот по логике он не сказал

так что да, можно разобрать на красивые кусочки

да там обычно не бывает ситуации, когда оно пересечется

даже если бывает

выигрывает все равно последний

выигрывает все равно последний

это да, но тогда возможно нарушится логика =)

не нарушится :/

Только я заметил что у него вся эта хрень в контроллере?)

переворачиваем проверки

и пишем элс перед ифами

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

но без лишних проверок

но без лишних проверок

Я про это и написал с самого начла

да понятно, что про это -.-

все мы про это

всё про это да про это

давайте лучше стейки обсуждать

Только я заметил что у него вся эта хрень в контроллере?)

а надо не в контролере? =)

а надо не в контролере? =)

Ну как бы ))

Лан, я тоже в контроллере такое пишу

ну а где, поведай?

Когда нахуй не нужно выносить

вот у меня есть ссылка "[export]"

я в нее тыкаю и попадаю в роут /order/id/export