JSF 2: Ist dies ein guter Ansatz, um Business Exceptions zu behandeln?

Question

Nach meiner vorherigen Frage in Creating FacesMessage in action method outside JSF conversion/validation mechanism?, versuche ich die Ausnahmen aus der Business-Schicht außerhalb meiner verwalteten Beans zu behandeln.

Die Strategie besteht darin, Business Exceptions nach Faces-Nachrichten in einem PhaseListener zu suchen und zu konvertieren.

Es funktioniert wie ich erwartet habe, aber ich frage mich nur, ob ich gerade das Rad neu erfinden, oder es richtig mache mit dem falschen Weg?

Hier ist mein Beispielcode-Schnipsel:

public class BusinessExceptionHandler implements PhaseListener { 

    @Override 
    public void afterPhase(PhaseEvent phaseEvent) { 
     ExceptionHandler exceptionHandler = phaseEvent.getFacesContext().getExceptionHandler(); 

     // just debugging the handled exception, nothing here 
     /* 
     for (ExceptionQueuedEvent event : exceptionHandler.getHandledExceptionQueuedEvents()) { 
      ExceptionQueuedEventContext context = (ExceptionQueuedEventContext) event.getSource(); 
      System.out.println("handled exception : " + context.getException()); 
     }*/ 

     for (Iterator<ExceptionQueuedEvent> it = exceptionHandler.getUnhandledExceptionQueuedEvents().iterator(); 
       it.hasNext();) { 

      ExceptionQueuedEvent event = it.next(); 
      ExceptionQueuedEventContext eventContext = (ExceptionQueuedEventContext) event.getSource(); 
      Throwable e = eventContext.getException(); 
      System.out.println("unhandled exception : " + e); 

      // get the root cause exception 
      while (e.getCause() != null) { 
       e = e.getCause(); 
      } 

      System.out.println("cause exception : " + e + 
       ", cause exception is BE : " + (e instanceof BusinessException)); 

      // handle BE 
      if (e instanceof BusinessException) { 
       BusinessException be = (BusinessException) e; 
       System.out.println("processing BE " + be); 
       FacesMessage message = Messages.getMessage(
        "com.corejsf.errors", 
        be.getMessage(), 
        be.getParamValues() 
       ); 
       FacesContext context = FacesContext.getCurrentInstance(); 
       context.addMessage(null, message); 
       it.remove(); // remove the exception 

       // works fine without this block, if BE is thrown, always return to the original page 
       /* 
       NavigationHandler navigationHandler = context.getApplication().getNavigationHandler(); 
       System.out.println("navigating to " + context.getViewRoot().getViewId()); 
       navigationHandler.handleNavigation(context, context.getViewRoot().getViewId(), null); 
       */ 
      } 
     } 
    } 

    @Override 
    public void beforePhase(PhaseEvent phaseEvent) { 
    } 

    @Override 
    public PhaseId getPhaseId() { 
     return PhaseId.INVOKE_APPLICATION; 
    } 

} 

Thank you!

Grüße, Albert Kam

Answer 1

Ihr Ansatz ist eine Mischung aus JSF 1.x und JSF 2.x Ausnahmebehandlung Ansätze. Da Sie JSF 2.x verwenden, würde ich einen reinen JSF 2.x-Ansatz ohne PhaseListener und mit Hilfe des neuen ExceptionHandler API vorschlagen.

Sie können ein Beispiel finden, die die ViewExpiredException in Seite 282 des Buches behandelt "JSF 2.0: The Complete Reference" die online verfügbar here ist (klicken Sie auf das erste Ergebnis Link).

Answer 2

nicht sicher, warum Sie diesen Weg zu gehen, erstellen und Ihre eigene Exception-Handler Registrierung ist wirklich einfach. Obwohl einfacher wird Seam Catch (http://seamframework.org/Seam3/CatchModule und http://docs.jboss.org/seam/3/catch/3.0.0.Alpha1/reference/en-US/html_single/). Ich habe noch keine JSF-Brücke, aber es wird sehr einfach sein. Dann müssen Sie nur eine Methode schreiben, die eine bestimmte Ausnahme behandelt, und Sie werden fertig sein!