Правильно ли я использую исключения?

Question

[Сергей Беловенцев]

вот код

public function actionOrder($bonus, $id){

		$transaction = Yii::$app->db->beginTransaction();
		$order= Order::findOne($id);
	    try {
	        $paymentStatus = $this->getPaymentStatus($order, $bonus);
	        Yii::$app->db->createCommand()->update('"order"', ['payment_status' => $paymentStatus, 'points_bonus' => $paysum, 'bonus' => 
	        $transaction->commit();
	    } catch (\Exception $ex) {
	        $transaction->rollBack();
	        Yii::$app->session->setFlash('error', $ex->getMessage());
	    }
	}
    


    private function getPaymentStatus($order,$bonus){
        $result=Order::PAYMENT_STATUS_PARTED_PAYED ;
        if($bonus['bonus']-$order->sum<0){
            throw new \Exception('У вас не хватает бонусов для оплаты бонусами');
        }
        return $result;
    }

Собственно вопрос в чем правильно ли я кидаю исключение в приватном методе getPaymentStatus и выкинет ли меня в catch в публичном методе actionOrder в случае если сработает исключение в приватном методе. И если не правильно то как правильно ?

Comments

[Модератор]

Сергей Беловенцев дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац).
В противном случае, ваши вопросы будут удаляться по причине тег-спама.

Это последняя просьба.

Answer 1

[voronkovich]

Нужно сделать отдельное исключение NotEnoughBonusesException и бросать его, остальные логировать.

try {
}
catch (App\Exception\NotEnoughBonusesException $e) {
      $transaction->rollBack();
      Yii::$app->session->setFlash('error',  'Недостаточно бонусов для оплаты.');
}
catch (\Throwable $e) {
     Yii::$app->session->setFlash('error',  'Произошла ошибка.');
     $transaction->rollBack();

    // Нужно бросить исключение дальше, чтобы обработчик по уполчанию его занес в лог
    // Надеюсь в Yii есть обработчик по умполчанию?
    throw $e;
}

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

Если вы хотите передать сообщение для пользователя вместе с исключением, создайте интерфейс UserFriendlyExceptionInterface с методами setUserMessage/getUserMessage и используйте их для отправки сообщения об ошибке пользователю:

try {
} catch (App\Exception\UserFriendlyExceptionInterface $e) {
    Yii::$app->session->setFlash('error',  $e->getUserMessage());

    throw $e; // Передаем дальше для логирования
}

Comments

[Сергей Беловенцев]

То есть если я использую Ваш совет насчет интерфейсов мне надо будет метод getPaymentStatus организовать?

[voronkovich]

Сергей Беловенцев, Что вы имеете ввиду под "организовывать"?
Я имел ввиду это:

if ($bonus['bonus']-$order->sum<0) {
    $e = new Application\Exception\OrderException();
    $e->setUserMessage('У вас не хватает бонусов для оплаты бонусами');

    throw $e;
 }

Лично я бы все-таки сделал отдельное исключение на каждую ситуацию (это более правильный подход):

if ($bonus['bonus']-$order->sum<0) {
   throw new Application\Exception\NotEnoughBonusesException();
 }

Answer 2

[Антон Иванов]

Как правило, исключения должны срабатывать в тех ситуациях, когда не возможно вернуть user friendly ответ пользователю. Все исключения должны писаться в логи, и не должны выводиться на экран (в production-режиме). Вы же как я понимаю, хотите вывести пользователю статусное сообщение. Так что нет - в вашем случае исключения не к месту.

Comments

[Антон]

часто исключения используются для вывода штатных ошибок. в том же yii, например, есть HttpException(404)

[Антон Иванов]

Антон, Смотря что вы подразумеваете под понятием "штатный". Если страница не найдена (404) - это (на мой взгляд) внештатная ситуация, когда администратор сайта потерял страницу - да это исключение. Штатная ситуация, которую нужно обрабатывать другими методами - это сообщить пользователю о недостатке средств на его счете.

Answer 3

[Lander]

Да. Выкинет.