Servlet フィルターについて

Question

前回質問させた頂いた続きになります。

編集中のページに、別のユーザーで編集ページに入らせない処理を書いています。

編集ページを抜けた判定を、フィルターで実装しようと処理を書いてみましたが、フィルターを実行すると、NullPointerException で処理が止まってします状況です。

そもそもフィルターで実装は不可能なのでしょうか
知恵をお貸しいただけないでしょうか。

また、フィルターのdestroyはいつ実行されるかご教授いただけないでしょうか(Servletのdestroyとの違い等)

よろしくお願いします。

前回の質問

package fw;

import java.io.IOException;
import java.util.Map;

import javax.servlet.Filter;
import javax.servlet.FilterChain;
import javax.servlet.FilterConfig;
import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import javax.servlet.ServletRequest;
import javax.servlet.ServletResponse;
import javax.servlet.http.HttpServlet;

import wiki.WikiBean;

public class PostProcessing extends HttpServlet implements Filter {

	public void init(FilterConfig filterConfig)
			throws ServletException {
	}

	public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain)
			throws IOException, ServletException {
		System.out.println("後処理フィルター");

		/* 編集ページのアクセスの履歴があれば、mapからidを削除 */
		ServletContext sc = null;
		if (getServletContext() == null) {
		}else{
			sc = getServletContext();
			String name = request.getParameter("name");
			if (sc.getAttribute("tmpMap") != null) {
				int id = -1;
				if (request.getParameter("id") == null || request.getParameter("id").isEmpty()) {
					id = 0;
				} else {
					id = Integer.parseInt(request.getParameter("id"));
				}

				Map<String, Integer> map = WikiBean.autoCast(sc.getAttribute("tmpMap"));
				if (map.containsKey(name)) {
					if (map.get(name) == id) {
						map.remove(name);
						sc.setAttribute("tmpMap", map);
					}
				}
			}

		}
		chain.doFilter(request, response);
	}

	public void destroy() {
		ServletContext sc = null;
		if (getServletContext() != null) {
			sc = getServletContext();
		}
		if (sc.getAttribute("tmpMap") != null) {
			sc.removeAttribute("tmpName");
		}
	}
}

<?xml version="1.0" encoding="UTF-8"?>
<web-app xmlns="http://java.sun.com/xml/ns/javaee" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-app_2_5.xsd"
	version="2.5">

	<!-- セッションタイムアウト設定 -->
	<session-config>
    	<session-timeout>300</session-timeout>
	</session-config>

	<!-- フィルター -->
	<filter>
		<filter-name>EncodingFilter</filter-name>
		<filter-class>fw.EncodingFilter</filter-class>
		<init-param>
			<param-name>encode</param-name>
			<param-value>UTF-8</param-value>
		</init-param>
	</filter>

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

<!--
	 <filter>
		<filter-name>PostProcessing</filter-name>
		<filter-class>fw.PostProcessing</filter-class>
	</filter>

	<filter-mapping>
		<filter-name>PostProcessing</filter-name>
		<url-pattern>/*</url-pattern>
	</filter-mapping>
 -->
途中省略

Comments

[7tsuno]

NullPointerExceptionが発生している箇所を特定することは可能でしょうか？エラー内容から行数など特定できるはずです。

[n000n00]

回答ありがとうございます。恐らく29行目のgetServletContext() です。java.lang.NullPointerException javax.servlet.GenericServlet.getServletContext(GenericServlet.java:123) fw.PostProcessing.doFilter(PostProcessing.java:29) fw.EncodingFilter.doFilter(EncodingFilter.java:34)

Answer 1

Before

public class PostProcessing extends HttpServlet implements Filter {

    public void init(FilterConfig filterConfig)
            throws ServletException {
    }

    public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain)
            throws IOException, ServletException {

（中略）

        if (getServletContext() == null) {
        }else{

（中略）

        }
        chain.doFilter(request, response);
    }

    public void destroy() {
        ServletContext sc = null;
        if (getServletContext() != null) {
            sc = getServletContext();
        }
        if (sc.getAttribute("tmpMap") != null) {
            sc.removeAttribute("tmpName");
        }
    }

After

public class PostProcessing implements Filter {

    private FilterConfig filterConfig = null;

    public void init(FilterConfig filterConfig) throws ServletException {
        this.filterConfig = filterConfig;
    }

    public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain)
            throws IOException, ServletException {

（中略）

        /* 他の全ての箇所の getServletContext() の呼出しも、同様に修正。 */
        if (filterConfig.getServletContext() == null) {
        }else{

（中略）

        }
        chain.doFilter(request, response);
    }

    public void destroy() {

        /* 必要か否か不明。ご自分で検証・判断してください。 */
        // if (sc.getAttribute("tmpMap") != null) {
        //     sc.removeAttribute("tmpName");
        // }

        filterConfig = null;
    }

---

getServletContext() は HttpServletクラス（が継承している GenericServletクラス）のメソッドです。
http://mergedoc.osdn.jp/tomcat-servletapi-5-ja/javax/servlet/GenericServlet.html#getServletContext()

しかし、PostProcessingクラスは Filter インターフェースの実装クラスとして使用しているため、
HttpServletクラスとしての初期化処理は実行されません。
そのため、getServletContext() メソッドの実行で NullPointerException が発生します。

代わりに、上述のように init() メソッドの引数に渡された FilterConfig インスタンスをメンバーフィールドに保持し、
その FilterConfig インスタンスの getServletContext() メソッドを使用すれば、正常に ServletContext インスタンスにアクセスできるはずです。
http://mergedoc.osdn.jp/tomcat-servletapi-5-ja/javax/servlet/FilterConfig.html#getServletContext()

Comments

[n000n00]

回答ありがとうございます。
本日実装して、無事動きました。遅くなり、申し訳ございません。

大変勉強になりました。ありがとうございました。

[KiyoshiMotoki]

b

Answer 2

Servlet の init メソッド内で super.init(config) を呼びだしていないとconfigが設定されず、 getServletContext() メソッドで NullPointerException が発生することがあるらしいです。
参考

    public void init(FilterConfig filterConfig)
            throws ServletException {
        super.init(filterConfig);
    }

としてみてもらえますか

Comments

[n000n00]

回答ありがとうございます。

実装してみましたが、引数にfilterConfigを指定できませんでした。、
引数なしで、コンストラクタ(?)を呼び出しましたが、同じくExceptionが発生します。

Answer 3

destroyは基本的にはサーバ停止時やタイムアウト時に発生します。