Mam klasy nadrzędnej z @ExceptionHandler i podklasy z jego własną wersją. Dostaję niejednoznaczne error metoda

Question

dominującej:

@ExceptionHandler(NoUserException.class) 
protected ModelAndView handleException(NoUserException e) { 
    Map<String, Object> model = new HashMap<String, Object>(); 

    model.put(ModelKeys.HOST_KEY, "message"); 

    return new ModelAndView("noAccess",model); 
} 

Dziecko:

@ExceptionHandler(NoUserException.class) 
protected void handleException(NoUserException e, HttpServletRequest request, HttpServletResponse response) throws IOException { 
    logger.error("Invalid user."); 
    respond(CLIENT_USER_ERROR,response); 
} 

Tak, trzeba je mieć różne parametry i wyjść.

Answer 1

Nie można mieć dwóch metod z tą samą obsługą wyjątku, przepraszam, to po prostu nie jest obsługiwane. Kod, który je rozwiązuje, nie rozróżnia między nadrzędnymi podklasami i uważa podklasę za "bardziej szczegółową". Kod jest w org.springframework.web.servlet.mvc.annotation.AnnotationMethodHandlerExceptionResolver, jeśli jesteś zainteresowany. Nie byłoby trudno stworzyć własną implementację AbstractHandlerExceptionResolver opartą na tej, która uwzględnia metodę bezpośrednio na kontrolerze, bardziej konkretny wynik niż dziedziczoną metodę.

edytuj: komentarz osobisty, Znalazłem z biegiem czasu, że lepiej jest powstrzymać chęć użycia "dziedziczenia jako narzędzia do szablonów" podczas tworzenia kontrolerów z kontrolkami Spring MVC. Kod widoku jest z natury nietrwały i proceduralny, dlatego najpierw tworzymy osobną "warstwę widoku". Zbytnie podejście do DRY i "ponownego użycia" nie przynosi ci korzyści. Jeśli to narusza zasadę Liskov Substitution, nie robię tego. YMMV oczywiście.

Answer 2

Dlaczego po prostu nie delegować wdrożenia handleException() na inną metodę?

// superclass 
protected ModelAndView handleExceptionImpl(
     NoUserException e, 
     HttpServletResponse response) { 
    Map<String, Object> model = new HashMap<String, Object>(); 
    model.put(ModelKeys.HOST_KEY, "message"); 
    return new ModelAndView("noAccess",model); 
} 

@ExceptionHandler(NoUserException.class) 
protected ModelAndView handleException(
     NoUserException e, 
     HttpServletResponse response) { 
    return handleExceptionImpl(e, response); 
} 

// subclass 
@Override 
protected ModelAndView handleExceptionImpl(
     NoUserException e, 
     HttpServletResponse response) { 
    logger.error("Invalid user."); 
    respond(CLIENT_USER_ERROR,response); 
}