Yii Framework

Давно интересует вопрос: как справляться без findOrFail? Я имею ввиду то что разработчики Yii упорно отказываются добавлять в ORM методы которые кидают исключение при не найденной модели (якобы это нарушает какие-то паттерны, хотя AR сама по себе является антипаттерном). Писать повсюду сотни проверок на null и вручную кидать 404 - нереально (будет просто охренительное захламление кода, превращая его в нечитабельное гавно). Так что у меня например это вообще игнорируется и падает с NullReferenceException в местах использования модели. Знаю что это ненормально, но не знаю как быть по другому...

сделайте свой промежуточный SuperActiveRecord и в нем добавьте этот метод, и все все свои классы наследуйте от вашего нового.

andku83 писал(а): ↑2018.05.31, 14:20 сделайте свой промежуточный SuperActiveRecord и в нем добавьте этот метод, и все все свои классы наследуйте от вашего нового.

Сложно. Там ведь не только статичный findOneOrFail у AR нужен, но и у Query/ActiveQuery oneOrFail.

Brainfuck писал(а): ↑2018.05.31, 14:35

andku83 писал(а): ↑2018.05.31, 14:20 сделайте свой промежуточный SuperActiveRecord и в нем добавьте этот метод, и все все свои классы наследуйте от вашего нового.

Сложно. Там ведь не только статичный findOneOrFail у AR нужен, но и у Query/ActiveQuery oneOrFail.

1) Создаёте класс ActiveQueryExtended, отнаследованный от ActiveQuery;
2) переопределяете в нём все поисковые методы (->one(), ->all() и т.п., чтобы результат выполнения проверялся и кидалось исключение);
3) создаёте класс ActiveRecordExtended отнаследованный от ActiveRecord;
4) добавляете в нём метод findOrException() (можно конечно и родной find() переопределить, но чревато, ибо не во всех 100% случаев нужны исключения). В этом методе создаёте и возвращаете экземпляр ActiveQueryExtended на основе класса модели;
5) наследуете свои модели от ActiveRecordExtended.

Alexum писал(а): ↑2018.05.31, 15:29

Brainfuck писал(а): ↑2018.05.31, 14:35

andku83 писал(а): ↑2018.05.31, 14:20 сделайте свой промежуточный SuperActiveRecord и в нем добавьте этот метод, и все все свои классы наследуйте от вашего нового.

Сложно. Там ведь не только статичный findOneOrFail у AR нужен, но и у Query/ActiveQuery oneOrFail.

1) Создаёте класс ActiveQueryExtended, отнаследованный от ActiveQuery;
2) переопределяете в нём все поисковые методы (->one(), ->all() и т.п., чтобы результат выполнения проверялся и кидалось исключение);
3) создаёте класс ActiveRecordExtended отнаследованный от ActiveRecord;
4) добавляете в нём метод findOrException() (можно конечно и родной find() переопределить, но чревато, ибо не во всех 100% случаев нужны исключения). В этом методе создаёте и возвращаете экземпляр ActiveQueryExtended на основе класса модели;
5) наследуете свои модели от ActiveRecordExtended.

Черт. Вот почему я должен делать столько лишних действий если в других фреймворках это есть из коробки? Кстати может есть уже какие-нибудь готовые врапперы над Yii такого типа? Чтоб композером подцепить и все.

Другие фреймворки - это Laravel, по-видимому.

mkramer писал(а): ↑2018.05.31, 15:51 Другие фреймворки - это Laravel, по-видимому.

И в Django тоже есть. А в ASP.NET этого и не требуется, т.к. там Linq предоставляет аналогичную возможность, если конечно говорить об EF.

Если только findOrFail необходим, то можно даже трейтом обойтись И примешивайте к своим моделям.

mkramer писал(а): ↑2018.05.31, 16:20 Если только findOrFail необходим, то можно даже трейтом обойтись

Код: Выделить всё

trait ExceptionOnFindFail {
     public static function findOrFail($param) {
         if (($res = static::findOne($param)) === null) {
                throw new NotFoundHttpException();
         }
         
         return $res;
     }
}

И примешивайте к своим моделям.

Не только. ActiveQuery->oneOrFail тоже бы не помешал. По сути только эти два и нужны - т.к. для массивов и так нормально когда пустой массив возвращается. Т.е. итерация по нему не вызовет ошибки, а просто ничего не произойдет.

Ну сделайте ещё свой ActiveQuery или примесь/поведение для него. Всё равно ActiveQuery почти всегда переопределяется даже в простых проектах

mkramer писал(а): ↑2018.05.31, 16:28 Ну сделайте ещё свой ActiveQuery или примесь/поведение для него. Всё равно ActiveQuery почти всегда переопределяется даже в простых проектах

Как показывали выше - это довольно не простое занятие переопределять ActiveQuery, т.к. тогда придется все методы AR переопределять чтобы они мою реализацию использовали. И я ни разу не видел чтобы кто-то его переопределял. С чего вы такое взяли я без понятия.

Хорошо бы чтобы тут отписался кто-нибудь из разработчиков Yii. Так сказать хочется посмотреть в глаза человеку заставляющему так страдать своих пользователей.

Мне так и не ответили по сути: как вы справляетесь без такого метода? Если не говорить о переопределении всего этого барахла (т.к. я слабо верю что кто-то реально этим занимается). Ставите овер 100500 проверок и вручную кидаете исключения? Игнорите это как я? Или как-то еще?

Ну я руками или как gii генерит отдельный метод. Мне три лишние строчки не трудно написать.

Переопределить ActiveQuery, чтоб добавить туда oneOrFail() - очень просто. И я его всегда переопределяю, поскольку scopes вынесены в Yii туда, и я предпочитаю писать Model::find()->newest(10)->all() вместо Model::find()->orderBy(["dt" => SORT_DESC])->limit(10)->all()

т.к. тогда придется все методы AR переопределять чтобы они мою реализацию использовали

Вот ничего подобного. Вcё работает через ActiveRecord::find(), поэтому переопределить достаточно его, и об этом написано в документации, которую я прочёл первым делом, когда перешёл на Yii2 с Kohana.
https://www.yiiframework.com/doc/guide/ ... ry-classes

зачем Alexum посоветовал кидать исключение из all() - не знаю, честно говоря

Нет высокой активности в сторону этой фичи: https://github.com/yiisoft/yii2/issues/15937
Пообсуждали и забили.

mkramer оказался прав - переопределить не так уж сложно:
Ладно сойдет. Придется правда это таскать за собой в другие проекты...

Таскать лучше трейтами. Вдруг не от ActiveRecord надо будет унаследоваться

mkramer писал(а): ↑2018.05.31, 18:08 Таскать лучше трейтами. Вдруг не от ActiveRecord надо будет унаследоваться

Как ты подмешаешь трейт в ActiveQuery?

Brainfuck писал(а): ↑2018.06.01, 09:32 Как ты подмешаешь трейт в ActiveQuery?

Запилишь свой ActiveQuery для класса и к нему подмешаешь. Ну, правда, это я почти всегда перекрываю ActiveQuery, чтоб добавить функции для нужных мне выборок