Стоит ли в контроллере обрабатывать исключения?

Question

[darksladen]

Скажите, нужно ли в контроллере обрабатывать исключения или они должны обрабатываться глобальным хэндлером? Или он нужен только для крайних случаев?

Answer 1

[JhaoDa]

Если есть потребность, то стоит. А если нет, то не стоит.

Comments

[darksladen]

хм, идеальный ответ на любой вопрос ) То есть когда нужно просто ошибку выкинуть, то не надо ничего обрабатывать или как?

[JhaoDa]

darksladen, ну, например, почти всегда какое-то исключение можно отдать глобальному обработчику, но вот в одном месте хоть тресни, но надо особым образом обработать. Ну и обрабатывайте там, где надо.

[Кирилл Несмеянов]

darksladen, в программировании всегда так. Даже говнокод можно аргументированно оправдать.

Answer 2

[Dmitry Pavlov]

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

Обработчики уровня actions в контроллерах - за содержание ошибки. К примеру, если у вас есть action MoveFile, который где-то там лезет во внешний сервис и пытается скачать файл и он падает по какой-то сугубо технической проблеме - таймаут у stream-а или еще что, то в конроллере имеет смысл этот Exception перехватить и завернуть во что-то более внятное:

public async Task<IActionResult> MoveFile()
{
    try {
        ...
    } catch (Exception exception) {
        throw new Exception("Moving file failed.", exception);
    }
}

В общем обработка исключений может быть многоуровневой. Например, сервис (то что раньше называли бизнес логикой) может перехватывать исключения уровня базы данных и заворачивать их в более осмысленные и перебрасывать дальше.

Answer 3

[dmitriy]

больше да чем нет