Как правильно отловить Exception?

Question

[Алексей Воропаев]

Как обработать Exception, чтоб появилась стандартная yii2 страница 404?
Без создания своего класса и чтоб обработка выполнялась для всех исключений

try
            {
                $activeCategory = NewsCategory::find()
                        ->where(['name_en' => Yii::$app->request->get('category')])
                        ->one();
                $model = News::find()
                        ->where(['newsCategory_id' => $activeCategory->id]);
            } catch (Exception $ex) {
                /* */
            }

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

Comments

[landergate]

А это не стандартная страница 404 у Yii2?

[Алексей Воропаев]

landergate: Стандартная. Я выложил скриншот, чтоб поняли, что я имел ввиду под "стандартная"

[landergate]

С точки зрения SEO, возвращать 404 вообще на любые ошибки - неверно.
Поисковики понимают под возвратом 404 сигнал о том, что этой страницы не существует, и поисковик больше не будет пытаться индексировать этот url в будущем, удалив его существующий индекс.

404 используют там, где URL гарантированно не существует и его точно не появится.
Для серверных ошибок или временного внепланового поведения лучше возвращать 503. Поисковик попытается проиндексировать эту страницу вновь через какое-то время.

Answer 1

[polar-bear]

Добавьте в ваш код слеш перед Exception. Тогда будет работать как вам нужно.

} catch (\Exception $ex) {
    throw new NotFoundHttpException('Not found');
}

Comments

[Алексей Воропаев]

Спасибо! Как я и говорил проблема легко решается, аж 1 символ :)

Answer 2

[Алексей Воропаев]

Сделал как вы сказали, ничего не изменилось

Comments

[imdeveloper]

У вас же ошибка не в самом формате исключения , а в коде.

[Алексей Воропаев]

imdeveloper: в коде я загружаю модель категории из GET, затем загружаю модель новостей, соответствующих этой категории. Если ввести правильное название категории(к примеру "updates"), то всё хорошо работает, а если ввести крагозябры(к примеру "asdasdgsdfg"), то вылетает ошибка(скриншот выше). Как мне обработать эту ошибку, чтоб вылетела страница 404 ? Извините, если сразу не правильно объяснил. Я только учусь

[Алексей Воропаев]

imdeveloper: пока писал вам сообщение, понял как сделать по-другому. Как делать через Exception я так и не понял :(

Answer 3

[Никита]

Уберите try,

$activeCategory = NewsCategory::find()
                        ->where(['name_en' => Yii::$app->request->get('category')])
                        ->one();
if(!$activeCategory) { // Или ошибка
  throw new NotFoundException(); 
}
$id = $activeCategory->id; // Иначе всё ок

Comments

[Алексей Воропаев]

Сейчас я так и решаю проблему, но всё же хочется чтоб ошибка выскакивала для всех исключений.
polar-bear показал решение ниже, всё оказалось очень просто

[Никита]

Алексей Воропаев: это неправильно. Мной написанное решение взято из шаблонов yii (FindModel метод).
С другой стороны перехватывать синтаксические ошибки вместе с бизнес-логикой это как минимум не логично, не говоря уже об общих хороших правилах тона.