@ControllerAdvice 异常处理与 @ResponseStatus

Question

我有@ControllerAdvice 类，它处理一组异常。比我们有一些其他的例外，用@ResponseStatus 注释进行注释。为了结合这两种方法，我们使用博客文章中描述的技术：http://spring.io/blog/2013/11/01/exception-handling-in-spring-mvc，即在ControllerAdvice 中，我们按以下方式处理通用Exception：

    @ExceptionHandler(value = Exception.class)
    public ModelAndView defaultErrorHandler(HttpServletRequest req, Exception e) throws Exception {
        // If the exception is annotated with @ResponseStatus rethrow it and let
        // the framework handle it - like the OrderNotFoundException example
        // at the start of this post.
        // AnnotationUtils is a Spring Framework utility class.
        if (AnnotationUtils.findAnnotation(e.getClass(), ResponseStatus.class) != null)
            throw e;

        // Otherwise setup and send the user to a default error-view.
        ModelAndView mav = new ModelAndView();
        mav.addObject("exception", e);
        mav.addObject("url", req.getRequestURL());
        mav.setViewName(DEFAULT_ERROR_VIEW);
        return mav;
    }

它就像一个魅力，但是，使用这种技术会导致应用程序日志中出现以下文本的错误：

2014-06-11 15:51:32.907 错误 o.s.w.s.m.m.a.ExceptionHandlerExceptionResolver - 无法调用 @ExceptionHandler 方法：...

这是ExceptionHandlerExceptionResolver中的这段代码造成的：

try {
            if (logger.isDebugEnabled()) {
                logger.debug("Invoking @ExceptionHandler method: " + exceptionHandlerMethod);
            }
            exceptionHandlerMethod.invokeAndHandle(webRequest, mavContainer, exception);
        }
        catch (Exception invocationEx) {
            logger.error("Failed to invoke @ExceptionHandler method: " + exceptionHandlerMethod, invocationEx);
            return null;
        }

有没有人知道如何结合这两种方法来正确处理异常以避免日志中的错误？

谢谢， 一月

Answer 1

这是一个老问题，但我今天才遇到这个问题，并找到了比禁用 ExceptionHandlerExceptionResolver 的日志记录更好的解决方案。事实证明，可以通过升级到最新版本的 spring 框架来解决这个问题（4.3.8 对我有用）。 ExceptionHandlerExceptionResolver 已修复以检测原始异常是否从 @ExceptionHandler 重新抛出。在这种情况下，不再记录异常。

Answer 2

我以稍微不同的方式处理它，我认为这可以解决您的问题。

由于我知道基本上我想以不同的方式处理 404 到 500 的任何色调，我会查找 NOT_FOUND 状态并相应地发送它，这似乎有效，然后您不会重新抛出异常。

这意味着

@ControllerAdvice
public class MVCExceptionHandler {
    private static final Logger log = LogManager.getLogger();

    @ExceptionHandler(Exception.class)
    public ModelAndView defaultErrorHandler(HttpServletRequest req, HttpServletResponse res, Exception ex) throws Exception {

        // If the exception is annotated with @ResponseStatus check if it's a 404 in which case deal with it, otherwise 500 it.
        if (AnnotationUtils.findAnnotation(ex.getClass(), ResponseStatus.class) != null) {
            ResponseStatus rs = AnnotationUtils.findAnnotation(ex.getClass(), ResponseStatus.class);
            if (HttpStatus.NOT_FOUND.equals(rs.value())) {
                res.setStatus(HttpStatus.NOT_FOUND.value());
                return new ModelAndView("error/404");
            }
        }

        log.error("Error while loading", ex);

        return new ModelAndView("error/500");

    }
}

Answer 3

我错过了import时遇到了同样的问题：

import org.springframework.http.HttpStatus;

Eclipse 没有让我在 Quick Fix 中添加它，我手动添加了该行并且它有所帮助。

Answer 4

问题是您抛出异常而不是返回ModelAndView。本质上，您正在尝试在您的 ControllerAdvice 类中重新实现 ResponseStatusExceptionResolver。您可以这样做（例如，通过从ResponseStatusExceptionResolver 复制/粘贴代码），但除了重复之外，这种方法也是不完整的。 Spring 还为一些内置异常提供了DefaultHandlerExceptionResolver，这种方法可能无法正确处理。

相反，我所做的并不是在我的 ControllerAdvice 类中为 Exception.class 提供包罗万象的处理程序。我不知道这是否是最好的解决方案，但这是我所知道的最好的解决方案。