Когда это нормально бросать ServletException из сервлета?

Question

Я бросал ServletException в прошлом, когда что-то или что-то пошло не так в сервлете, в основном просто обертывая исключение в ServletException.

Теперь я думаю, что на самом деле лучше не бросать ServletException, а отвечать response.sendError(sc) и использовать правильные коды состояния HTTP.

Если я не могу отправить сообщение об ошибке с помощью reponse.sendError, (IOException), я обернуть IOException в ServletException.

Является ли выше лучший способ ответить? Когда все в порядке, просто нажмите ServletException?

Answer 1

Я только что пришел к противоположному выводу @alamar. Моя ситуация заключается в написании сервлета типа REST, который вызывается кодом в базе данных Oracle, а не людьми.

Я хочу вернуть код HTTP 400 Неверный запрос вызывающему абоненту, когда информация запроса недействительна или отсутствует. Бросок ServletException заставляет контейнер возвращаться 500 Внутренняя ошибка сервера, в которой говорится, что с сервером что-то не так, а не с запросом.

Вот решение, которое я принял.

1. Создать простой класс исключений RequestException, который расширяет Exception.
2. Проверьте правильность запроса с помощью методов (методов), которые выбрасывают new RequestException(message).

3. Поймайте RequestException в doPost метод сервлета и вызвать HttpServletResponse.sendError() так:

   try { 
       validateRequest(request); 
       // Do stuff with a valid request. 
   } catch (RequestException e) { 
       response.sendError(HttpServletResponse.SC_BAD_REQUEST, e.getMessage()); 
   } 
   

сообщение возвращается вызывающему абоненту, как это было бы с ServletException.

Answer 2

Лучше всего выбросить ServletException.

Вы можете на самом деле поймать это исключение с помощью директивы error-page в своем web.xml и предпринять соответствующие действия: отобразить страницу причудливых ошибок пользователю, отправить исключение вместе со стеком для разработчиков, записать его в журналы и так далее на.

Если вы просто используете sendError(), вы не можете этого сделать, в различной степени.