Почему не работает исключение?

Question

[Евгений Мусонов]

Исключение

namespace App\Exceptions;

use Exception;
use Illuminate\Support\Facades\Log;

class ResizeDirNotFoundException extends Exception
{
    public function report(Exception $exception)
    {
        Log::error('Resize dir is empty');
        parent::report($exception);
    }
}

Вызываю

private function validateResizeParams($params)
    {
        if (empty($params['dirForSave'])) {
            throw new ResizeDirNotFoundException('Resize dir is empty');
        }
    }

Обрабатываю

public function resize($width, $height, $dirForSave, $quality = 100)
    {
        $params = [
            'width'      => $width,
            'height'     => $height,
            'dirForSave' => $dirForSave
        ];

        try {
            if (!is_null($this->file) && $this->validateResizeParams($params)) {
                //resize
            }
        } catch (Exception $exception) {
            return redirect('/');
        }
    }

Лог не создается, исключение не обрабатывается, форма успешно обрабатывается, подскажите, пожалуйста, где косяк?

Comments

[Дмитрий]

Проверяйте не бул, а инт

[Евгений Мусонов]

Дмитрий, вынес вызов функции из if - никаких изменений

[Дмитрий]

Евгений Мусонов,

try {
$this->validateResizeParams($params);

}

[Евгений Мусонов]

Дмитрий, да, именно это я и сделал

[Дмитрий]

Евгений Мусонов, и ни чего?

Answer 1

[Евгений Мусонов]

Само исключение работает, только надо убрать метод report, т.к. класс Extension его не содержит.
Чтобы заработал редирект, надо заменить
return redirect('/');
На
redirect('/')->send();

Answer 2

[Alex Wells]

А что должно было случится? } catch (Exception $exception) {
Вы же ловите ексепшены.

Comments

[Евгений Мусонов]

А что я должен ловить, когда бросаю exception? Должен быть редирект, после обработки

[Alex Wells]

Евгений Мусонов, ну обрабатывайте там, где ловите, либо не ловите в контроллере.

public function report(Exception $exception)
    {
        Log::error('Resize dir is empty');
        parent::report($exception);
    }

Этот метод вызывается exception handler'ом ларки для необработаных эксепшенов. Вы его словили. Он никогда не вызвется и не должен.

[Евгений Мусонов]

Alex Wells,

ну обрабатывайте там, где ловите, либо не ловите в контроллере.

Не совсем понял, я обрабатываю не в контроллере. Насчет метода report понял. Почему не срабатывает редирект в catch?

[Alex Wells]

Евгений Мусонов, вы обрабатываете вот тут:

} catch (Exception $exception) {
            return redirect('/');
        }

Я имел в виду что либо вы не ловите вообще, и эксепшен хэндлит report
, либо вы все таки ловите где-то и он не хэндлится report'ом.

Почему не срабатывает редирект в catch?
Срабатывает, дебажьте дальше и показывайте весь код.

[Евгений Мусонов]

Alex Wells, вызываю этот метод, параметр dirForSave делаю пустой строкой

public function resize($width, $height, $dirForSave, $quality = 100)
    {
        $params = [
            'width'      => $width,
            'height'     => $height,
            'dirForSave' => $dirForSave
        ];

        try {
            $this->validateResizeParams($params);
        } catch (ResizeDirNotFoundException $exception) {
            return redirect('/');
        }
    }

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

private function validateResizeParams($params)
    {
        if ($params['width'] === false && $params['height'] === false) {
            //Throw Exception
        }
        if (empty($params['dirForSave'])) {
            throw new ResizeDirNotFoundException('Resize dir is empty');
        }
    }

Собственно это всё

[Alex Wells]

Евгений Мусонов, ну продебажьте тогда, не экстрасенс.