Java Webアプリケーション - リファクタリング私は現在、以下は私のJSPファイルのすべての先頭に含まれてい

Question

ベストプラクティスのアプローチにスクリプトレットが含まれています

<%@ include file="inc/inc_cookie_login.jsp"%> 
<%@ include file="inc/inc_protect_page.jsp"%> 
<%@ include file="inc/inc_log_access.jsp"%> 

JSPはスクリプトレットcheck for cookie and set a user object in the session if cookie exists, prevents access to the jsp unless a session has been set, write to a text file the User IP, name, page accessed, etc.,それぞれのことを持っています。

上記のスクリプトレットのアプローチはうまくいきましたが、私はより良いサーバーをセットアップしてweb.xmlファイルを利用できるようになりましたので、ベストプラクティスにリファクタリングしています。上記はFIXMEを叫んでいます！リスナー、フィルタ、？を調査する必要がありますか、現在のアプローチは適切ですか？

=== inc_cookie_login.jspは====

<%@ page import="model.STKUser"%> 
<%@ page import="model.STKUserCookie"%> 
<%@ page import="data.STKUserDAO"%> 

<% 
if (request.getSession().getAttribute("STKUserSession") == null) { 
    STKUserCookie userCookie = new STKUserCookie(request); 
    String userBadge = userCookie.getUserID(); 
    STKUserDAO userDAO = new STKUserDAO(); 
    STKUser user = userDAO.getUser(userBadge); 
    if (user != null) { 
     user.setIpAddress(request.getRemoteAddr()); 
     userDAO.updateLoginCount(user); 
     request.getSession().setMaxInactiveInterval(36000); //set to 10 hours 
     request.getSession().setAttribute("STKUserSession", user); 
    } 
} 
%> 

Answer 1

これは、フィルタによって置き換えられるために良いもののように見えます。フィルタクラスを作成し、web.xmlにパターンで参照します。他のすべてのオプションがかなり使い果たされていない限り、スクリプトレットは使用しないでください。