потому что любое малейшее изменение бизнес логики заставляет искать во многих местах, и править код

тогда как при правильной архитектуре, такого бы не было

практически всегда в

в этом классе на 4к строк

именно в этом методе на 500 строк?

метрика, кстати, зовётся code churn

именно в этом классе из 37 методов

и нескольких по 500 строк

это основная сущность - заявка. много статусов и состояний

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

Нарушены границы контекста и единственной обязанности.

> Yii > принцип единственной ответственности))))

а тут и с тестами беда начинается

и много мест где Order::find... дублируется

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

в yii вроде это activeQuery

переместить все запросы в модели, очень сильно сократится дублирование. Уже профит. Это позволит реффакторить дальше

> Yii > принцип единственной ответственности))))

в чем проблема? я не говорю о каком то 100% соблюдении этого. Здравый смысл

в чем проблема? я не говорю о каком то 100% соблюдении этого. Здравый смысл

Да я шучу

переместить все запросы в модели, очень сильно сократится дублирование. Уже профит. Это позволит реффакторить дальше

Модели это папка?

Всё правильно говоришь

в смысле папка?

я просто про то, что начинать надо с подсчёта churn

в смысле папка?

Ну как в байсик приложении, и там Order extends Model

и вычищать самое больное в первую очередь

Order (ar модель. анемичная) -> OrderService (божественный класс где все запросы find и остальная бизнес логика) -> OrderController (тонкий)

я не уверен, что репозитории — это оно

предпочитаю не гадать, когда есть метрики чёткие

Без тестов вычищать это стрельба в голову

не всегда

все покрыто тестами

1000 тестов

в этом плане плюс

А можно какой то пример, какая задача тяжела в выполнении на yii2, и для каких "легких" целей symphony лучше не использовать?

Элементарно. Если принесли замысловатую вёрстку не на Bootstrap, то ActiveForm и LinkPager не подходит. Если нужны замысловатые таблицы с colspan и rowspan, то выбрасываем GridView. Если проектируем сложные сущности со вложенными объектами-значениями, то сильно перепиливаем ActiveRecord или меняем его на Doctrine. Если подключаем сторонние компоненты, работающие по PSR, то пишем адаптеры PSR. Если хотим делать фреймворконезависимые PSR-7 контроллеры, то нужно самому программировать трансформеры для запроса и ответа и адаптеры для контроллеров. Если хотим делать маршрутизацию по именам маршрутов, а не по именам контроллеров и экшенов, то нужно переписать фреймворковский UrlManager или заменить на другой. Но у него нет интерфейса, поэтому придётся наследоваться. Потом хотим добавить мультиязычность в адресе. Можно было бы просто сдекорировать UrlManager, но он опять без интерфейса. И т.п.

@samdark loc phpщный поможет посчитать?

о, ну тогда офигенчик вообще

Я б начал с линтера

далее когда уберу все дублирование query. Внедрить машину состояний. Где на каждый статус будет собственный хендлер

@devium нет, он churn не умеет

а кто умеет?

но тулз полно, по "git churn" ищи

выбирай на вкус

в любом случае если не прибираться, то энтропия кода растет

принято

@symbios_zi что-то реальным говнищем пахнет в твоем текущем проекте... ))) Глаза видели, что брали?

проект с 2011 года

> уберу все дублирование query не всё то дубль, что дублем кажется

> уберу все дублирование query не всё то дубль, что дублем кажется

буду включать здравый смысл

> Внедрить машину состояний. Где на каждый статус будет собственный хендлер Чую, CRM-ка :)

в сфере ЖКХ

да

ну такая система ведения заявок

она интересная, просто нужно прибираться

Элементарно. Если принесли замысловатую вёрстку не на Bootstrap, то ActiveForm и LinkPager не подходит. Если нужны замысловатые таблицы с colspan и rowspan, то выбрасываем GridView. Если проектируем сложные сущности со вложенными объектами-значениями, то сильно перепиливаем ActiveRecord или меняем его на Doctrine. Если подключаем сторонние компоненты, работающие по PSR, то пишем адаптеры PSR. Если хотим делать фреймворконезависимые PSR-7 контроллеры, то нужно самому программировать трансформеры для запроса и ответа и адаптеры для контроллеров. Если хотим делать маршрутизацию по именам маршрутов, а не по именам контроллеров и экшенов, то нужно переписать фреймворковский UrlManager или заменить на другой. Но у него нет интерфейса, поэтому придётся наследоваться. Потом хотим добавить мультиязычность в адресе. Можно было бы просто сдекорировать UrlManager, но он опять без интерфейса. И т.п.

Дима, Вы закончили мастер класс, или заканчиваете по Ларавел. Там Eloquent был или доктрина?

жду вот технический спринт

https://habrahabr.ru/post/351686/

вот

ага читал

это про реальность, а не про идеальные проекты в вакууме

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

Дима, Вы закончили мастер класс, или заканчиваете по Ларавел. Там Eloquent был или доктрина?

Eloquent, так как это мастер-класс по Laravel. Doctrine будет в мастер-классе по Symfony.

Надо мастер-класс по Doctrine :)

Элементарно. Если принесли замысловатую вёрстку не на Bootstrap, то ActiveForm и LinkPager не подходит. Если нужны замысловатые таблицы с colspan и rowspan, то выбрасываем GridView. Если проектируем сложные сущности со вложенными объектами-значениями, то сильно перепиливаем ActiveRecord или меняем его на Doctrine. Если подключаем сторонние компоненты, работающие по PSR, то пишем адаптеры PSR. Если хотим делать фреймворконезависимые PSR-7 контроллеры, то нужно самому программировать трансформеры для запроса и ответа и адаптеры для контроллеров. Если хотим делать маршрутизацию по именам маршрутов, а не по именам контроллеров и экшенов, то нужно переписать фреймворковский UrlManager или заменить на другой. Но у него нет интерфейса, поэтому придётся наследоваться. Потом хотим добавить мультиязычность в адресе. Можно было бы просто сдекорировать UrlManager, но он опять без интерфейса. И т.п.

Мультиязычность на стандартном менеджере делается легко

отдельный

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

я не ищу виноватых, ищу пути решения появившихся проблем

большинство проектов такие

вот в Skyeng сейчас есть проект на Yii 1.1 с доктриной, например

Если хочется свой роутер, то можно его оформить как URL rule

Странные предъявы короч)

Ларавель калл, расходимся ?

я не ищу виноватых, ищу пути решения появившихся проблем

И это правильно. Только не надо гадать никогда и принимать решения на интуиции без перепроверки.

отдельный

Когда?)

Их чертовы фасады на все, хелперы в виде функций где под капотом контейнер в одну строчку (не считая модулей аля сессия - передали что либо, значит гет, не передали - просто возвращаем объект) и их чертова магия ?

Когда?)

Это к Диме вопрос.

Это к Диме вопрос.

Он только только закончил

вот в Skyeng сейчас есть проект на Yii 1.1 с доктриной, например

Сегодня Харитон из скайэнг рассказывал, что они гугл докс как cms используют

Их чертовы фасады на все, хелперы в виде функций где под капотом контейнер в одну строчку (не считая модулей аля сессия - передали что либо, значит гет, не передали - просто возвращаем объект) и их чертова магия ?

Да, магии в нём вагоны...

Их чертовы фасады на все, хелперы в виде функций где под капотом контейнер в одну строчку (не считая модулей аля сессия - передали что либо, значит гет, не передали - просто возвращаем объект) и их чертова магия ?

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

Он только только закончил

Что закончил? Мастер-класс по доктрине?

Что закончил? Мастер-класс по доктрине?

По Laravel

Сегодня Харитон из скайэнг рассказывал, что они гугл докс как cms используют

Там какой только жести не было.

Зато рост в 3 раза каждый год

Что закончил? Мастер-класс по доктрине?

Устал, думаю

Это к тому что бизнес первичен, а технологии и инструменты вторичны

А что там со скайенгом? Что за жуть?

По Laravel

А, это да. Я новости почитываю периодически, видел.

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

А еще у них шрифт в доке ущербный Слава богу я читал ее года 2 назад когда там не было его ?

А что там со скайенгом? Что за жуть?

19-е место в рейтинге Forbes, вот что...

http://www.forbes.ru/tehnologii-photogallery/357559-20-samyh-dorogih-kompaniy-runeta-2018-reyting-forbes?photo=19

растёт как на дрожжах

в слаке 1500 человек чатит

в гите 200 репок, из них около 70 — немаленькие проекты

и эти проекты коммуницируют друг с другом и с внешним миром :)

в общем, адок тот ещё :)

И вы там симфонистами командуете

программистами, дизайнерами, аналитиками, да

там нет симфонистов, Yii-стов или кого-то такого

там почти все могут всё

или готовы разбираться