質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.49%

  • Java

    16139questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • XML

    768questions

    XMLは仕様の1つで、マークアップ言語群を構築するために使われています。

ログにエラー発生箇所情報が出力できない。

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,875

Linkey

score 30

ログの出力テストを行っているのですが、エラーの発生箇所情報がうまく出力できません。

[テスト内容]
1.Registration.javaの定数フィールド「PROPERTY_FILE」に存在しないプロパティファイル名を設定。
2.Htmlページからサーブレットにリクエストを送信。
3.プロパティファイルが存在しないため、NullPointerExceptionが発生する。
4.出力されたログを確認する。

ログの出力自体は問題なくできたのですが、エラー発生箇所の情報が期待通りになりません。
ログの出力内容は以下の通りです。

2017-05-04 10:42:47.426 WARN  [プロパティファイル未存在],取得ファイル名=url2.properties
 registration.servlet.RegistrationServlet.init(RegistrationServlet.java:46) 

()の部分の情報がログを出力している箇所(logger.warn(XXX)の行番号)が出力されました。やりたいことは(Registration.java:「エラー発生箇所であるprop.load(inStream)の行番号」)を出力させたいです。
このエラー発生箇所の情報はどのように設定するのか調べてみたのですが有力な情報が見つかりませんでした。ログ出力設定にお詳しい方がいましたらご回答をいただけないでしょうか。

原因がありそうな箇所のソースコード、設定内容を以下に共有いたします。

package registration.servlet;

import java.io.IOException;
import java.io.InputStream;
import java.io.PrintWriter;
import java.util.Properties;
import java.util.logging.Logger;

import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

import org.apache.logging.log4j.LogManager;

import registration.controller.RegistrationController;
import util.exception.UnexpectedProcessingException;

public class RegistrationServlet extends HttpServlet {
    private static final long serialVersionUID = 1L;
    static org.apache.logging.log4j.Logger logger = LogManager.getLogger(Logger.class.getName());
    final static String COMMON_URL = "CommonUrl";
    final static String PROPERTY_FILE = "url2.properties";
    final Properties prop = new Properties();

    public void init() throws ServletException{

        InputStream inStream = null;
        try {
            inStream = RegistrationServlet.class.getClassLoader().getResourceAsStream(PROPERTY_FILE);
            prop.load(inStream);
        } catch (IOException e) {
            throw new UnexpectedProcessingException("プロパティ読み込み中にエラーが発生", e);
        } catch (NullPointerException e) {
            logger.warn("[プロパティファイル未存在],取得ファイル名={}", PROPERTY_FILE);
            throw new UnexpectedProcessingException("予期せぬエラーが発生しました。", e);
        } finally {
            try {
                if (inStream != null) {
                    inStream.close();
                }
            } catch (IOException e) {
                System.out.println("予期せぬエラーが発生しました。");
                e.printStackTrace();
            }
        }
    }

    protected void doGet(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {

        // パラメータを取得する
        String process_code = request.getParameter("process_name");

        // コントローラを呼び出す
        RegistrationController rc = new RegistrationController();
        rc.invoke(process_code);
    }

    protected void doPost(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {

    }
}
log4j2.xml

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE configuration>

<configuration status="OFF">

<Properties>
    <Property name="error_daily_log">logs/app_error_%d{yyyy-MM-dd HH:mm:ss.SSS}</Property>
</Properties>
    <Appenders>
        <Console name="Console" target="SYSTEM_OUT">
            <PatternLayout pattern="%d{DEFAULT} | [%t] | %-5level | %c{36} | %msg%n"/>
        </Console>
        <File name="File" fileName="logs/normal_app.log">
            <PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} | [%t] | %-5level | %c{36} | %msg%n"/>
        </File>
            <RollingFile name="RollingFile" fileName="logs/app.log" filePattern="logs/app_%date{yyyyMMddHHmmss}.log">
            <PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} %-5level %msg%n %C.%M(%F:%L)"/>
            <Policies>
                <TimeBasedTriggeringPolicy />
            </Policies>
        </RollingFile>
    </Appenders>
    <Loggers>
        <Root level="trace">
            <AppenderRef ref="File" />
            <AppenderRef ref="RollingFile" />
        </Root>
        <Logger name="registration.servlet" level="debug" additivity="false">
            <AppenderRef ref="Console"/>
            <AppenderRef ref="RollingFile" />
        </Logger>
    </Loggers>
</configuration>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+1

ログのWARNの行の下が例外発生の文字列ということでしょうか。
これは最終的に発した例外の情報を示しているようなので、NullPointerExceptionによるものではなく、
自分で投げたUnexpectedProcessingExceptionの例外情報となります。
なので、この行番号は実はthrow new UnexpectedProcessingException("予期せぬエラーが発生しました。", e);の行です。

catchの引数のようになっているeはException系クラス型のインスタンスであり、
その中にはスタックトレースという情報が含まれています。
このスタックトレース情報は、ThrowableクラスのgetStackTrace()メソッドで取得できます。
後は、この中から自クラス内の実行ポイント、もしくは実行したinitメソッドを含む実行ポイントを探し出せばいいのだと思います。
こんな風に…(未検証)

} catch (NullPointerException e) {
    StackTraceElement[] elements = e.getStackTrace();
    String name = this.getClass().getName();
    for (StackTraceElement ste : elements) {
        if (ste.getClassName().equals(name)){
            String log = ste.toString();  //この文字列を出力すればいい
        }
    }
    logger.warn("[プロパティファイル未存在],取得ファイル名={}", PROPERTY_FILE);
    throw new UnexpectedProcessingException("予期せぬエラーが発生しました。", e);
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/06 10:55

    ログ出力の参考にさせていただきます。ありがとうございます。

    キャンセル

checkベストアンサー

0

prop.load(inStream) で発生した例外はアプリケーション内で catch してしまっているので、ハンドルされていない例外はその下の throw new UnexpectedProcessingException() となり、その行がスタックトレースとして出力されているだけで、ログ出力とは直接関係ないと思います。

ロガーは、ログに何かを出力する以上のことは何もしないので、例外が発生した行番号をログに出したいなら、logger.warn("[プロパティファイル未存在],取得ファイル名={}", PROPERTY_FILE); の所で、自力でスタックトレースを見て、行番号を出力に追加してください。

スタックトレースに単純に出力されればいいだけなら、UnexpectedProcessingException でラップするのを止めて、catch した NullPointerException をそのまま throw してもいけるかもしれません(できないかも)。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • Java

    16139questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • XML

    768questions

    XMLは仕様の1つで、マークアップ言語群を構築するために使われています。