Struts2 Griff-session-timeout mit Interceptor

Ich habe viele Kommentare zu diesem code/Konfiguration, einige trivial, andere nicht.

1. Es keine Grund für erstellen Sie Ihre eigene Sitzung; nicht.

2. Wenn Sie erklären, einen interceptor in einem action-Konfiguration müssen Sie angeben alle Abfangjäger. Wie konfiguriert, nur der Sitzung interceptor läuft menuAction.

3. Das bedeutet, dass keine Parameter gefüllt werden, da sonst kein Abfangjäger ausgeführt werden.

4. Im Allgemeinen, verwenden Sie nur SessionAware Zugriff auf die Sitzung. Es ist sehr selten zugreifen zu können, müssen den Antrag direkt.

5. Offensichtlich ist der Flieger nicht", setzt die Anfrage auf null", ist das gar nicht sinnvoll.

6. Ich habe keine Ahnung, was Ihre LoginAction soll, zu tun. Was ist der Absicht von Zeilen wie setServletRequest(request); oder setSession(session, user);? Es gibt nichts über diese beiden Linien, die Aussehen korrigieren.

7. Nennen Sie Ihre Erfolge und Fehler die Ergebnisse nur, dass "success" und "error" (tiefer Fall), wenn du gehst zu verwenden, die ActionSupport.SUCCESS und ActionSupport.ERROR Konstanten. Wenn Sie nicht Verwendung dieser Konstanten in diesem code, ich würde empfehlen, mit anderen Namen für Sie, weil sonst jeder, der tatsächlich verwendet Struts 2 vor.

8. Beim posten von code-Beispielen bitte entfernen Sie Sachen, die nicht relevant. Besonders, wenn Sie nicht explizit festlegen, syntax-highlighting, es macht die Dinge viel schwieriger zu Lesen.

9. Nicht verwenden, code wie if (isValidUser == true) verwenden if (isValidUser).

10. Daran Ihre Bedingungen: Sachen wie if (user == null) ... else if (user != null) ... else ... macht null Sinn. Der user gleich null ist, oder es ist es nicht: es gibt keine Dritte option.

11. Vermeiden Sie unnötige, verwirrende Logik, wie User currentUser = (User) (mapSession.get("User") == null ? null : mapSession.get("User")); die im Grunde sagt "Wenn es null, null, andernfalls den Rückgabewert muss man einfach haben, aber es wieder zu bekommen." WARUM???

12. Kommentare wie User user = null; //To store current user sind völlig wertlos. Ist es nicht offensichtlich was User user ist? Einen Benutzer. Nicht offensichtlich genug? Wie wäre User currentUser???

13. Namen Pluralformen (z.B. Sammlungen) als etwas im plural. Eine Karte der Kategorie die Namen zu Listen, die nicht einer einzelnen Kategorie.

14. Nicht Variablen deklarieren, weit entfernt, von wo Sie verwendet werden; es ist sehr verwirrend.

15. Setzen Sie Ihre JSP-Seiten unter WEB-INF irgendwo zu verbieten, direkten Clientzugriff.

16. Vermeiden Sie unnötige Konstrukte, wie wenn ein if Zweig zurück, ein else ist nicht unbedingt notwendig, und IMO, es fügt Rauschen. Ebenso betrachten die Rückkehr so bald wie Sie wissen, die Sie zurückgeben. Letzteres ist ein bisschen mehr umstritten, aber ich denke, dass die Leute kommen, um zu realisieren, es ist okay, mehrere return-Punkte in kurzen Methoden, und IMO ist es einfacher zu denken.

17. Weniger Programmcode Dinge zu sagen. Erstellen Sie kleine utility-Methoden zu wickeln triviale Funktionalität, so dass es nicht verschmutzen die mainline-code.

18. Use action-chaining fast nie.

Es gibt noch mehr, aber das ist genug für jetzt. Hier die aktuellen relevanten code, bereinigt. Einiges davon ist nicht wirklich relevant, aber ich ließ es trotzdem.

<filter-mapping>
  <filter-name>struts2</filter-name>
  <url-pattern>/*</url-pattern>
</filter-mapping>

<session-config>
  <session-timeout>1</session-timeout>
</session-config>

<package name="default" extends="struts-default">
  <interceptors>
    <interceptor name="sessionInterceptor" class="com.platform.web.security.SessionInterceptor" />
  </interceptors>

  <action name="doLogin" class="com.platform.web.action.LoginAction">
    <result name="input">/WEB-INF/jsp/login/login.jsp</result>
    <result name="error">/WEB-INF/jsp/login/login.jsp</result>
    <result type="redirectAction">menuAction</result>
  </action>

  <action name="menuAction" class="com.platform.web.action.MenuAction">
    <interceptor-ref name="sessionInterceptor"/>
    <result name="success">/WEB-INF/jsp/main.jsp</result>
    <result name="input">/WEB-INF/jsp/myFavourite.jsp</result>
  </action>

 public class SessionInterceptor extends AbstractInterceptor implements StrutsStatics {
    @Override
    public String intercept(ActionInvocation invocation) throws Exception {
        ActionContext context = invocation.getInvocationContext();
        HttpServletRequest request = (HttpServletRequest) context.get(HTTP_REQUEST);
        HttpSession session = request.getSession(false);

        //session will almost *never* be null. Check for a valid user object.
        if (session == null) {
            return "timeout";
        }

        return invocation.invoke();
    }
}

public class LoginAction extends MesActionSupport implements ServletRequestAware {
    @Override
    public String execute() throws Exception {
        if (!isValidUser) {
            return ERROR;
        }

        user = getUser();

        String strSessionId = request.getSession(true).getId();
        setServletRequest(request);
        session.put("SessionId", strSessionId);

        setSession(session, user);

        ServletActionContext.getRequest().getSession().setAttribute("User", user);

        return SUCCESS;
    }
}

public class MenuAction extends MesActionSupport implements SessionAware, ParameterAware, RequestAware {
    @Override
    public String execute() throws Exception {
        User currentUser = (User) mapSession.get("User");
        if (currentUser == null) {
            return ERROR;
        }

        Map<String, List<String>> categories; //Left in for naming.

        StringBuffer menu = menuView.getMenu(user);
        mapSession.put("Menu", menu.toString());

        StringBuffer dashboardMenu = new StringBuffer("");
        mapSession.put("dbMenu", dashboardMenu.toString());

        return SUCCESS;
    }
}

InformationsquelleAutor Dave Newton