Java ファイル読み込みループ内のネスト数を減らしたい

###実現したいこと
ファイルの読み込みを一行づつ行って、最終的にはoutputLogListというリストに一行書き込むことを目標としたループ処理の中でネストが深くなってしまっているのですが、できるかぎりネスト数を減らしたいと思っています。どのような方法が考えられるでしょうか。

###現在のソースコード

Java
1// 一行づつ読み込みループ開始
2String logLine = reader.readLine();
3while (logLine != null) {
4        logLineCount++;
5        inputData.setLogLine(logLine);
6        // スタックトレース部ではない
7        if (!logLine.startsWith("\t")) {
8                // 日時のフォーマットが期待通りである
9                if (logLineCheck.isDateFormat(inputData)) {
10                        // 指定した期間内である
11                        if (logLineCheck.isDateScope(inputData)) {
12                                logTargetLineCount++;
13                                // タブ区切りオプションが指定されていたら実施
14                                if (selectedOption != null
15                                                && selectedOption.indexOf(Execute.TAB_OPTION) != -1) {
16                                        option.doTabDelimiter(inputData, inputFileName);
17                                }
18                                // 出力ログリストへ追加
19                                logDate.append(sdfStandard.format(inputData.getLogLineDate()));
20                                logDate.append("\t");
21                                logDate.append(logLine);
22                                outputLogList.add(logDate.toString());
23                                logDate.setLength(0);
24                        }
25                //日時フォーマットが期待通りでない        
26                } else {
27                        // ログメッセージ出力
28                        message.append("入力ファイルの日時フォーマットが定義と一致しないため、次の行を読み込みます。（");
29                        message.append(inputFileName);
30                        message.append("：");
31                        message.append(logLineCount);
32                        message.append("行目）");
33                        logger.warn(message.toString());
34                        message.setLength(0);
35                }
36        //スタックトレース部を読み込んだ        
37        } else {
38                // ログメッセージ出力
39                message.append("スタックトレース部分のため、次の行を読み込みます。（");
40                message.append(inputFileName);
41                message.append("：");
42                message.append(logLineCount);
43                message.append("行目）");
44                logger.debug(message.toString());
45                message.setLength(0);
46        }
47        // 次の行を読み込む
48        logLine = reader.readLine();
49}

行動規範の内容に同意します

回答2件

ベストアンサー

下記のように書けばネストを減らせると思います。

JAVA
1if ( logLine.startsWith("\t") ) {
2    /* エラー処理 */
3    continue;
4}
5if ( !logLineCheck.isDateFormat(inputData) ) {
6    /* エラー処理 */
7    continue;
8}
9if (!logLineCheck.isDateScope(inputData)) {
10    continue;
11}
12    /* 通常処理 */

投稿2016/05/29 05:15

kei344

総合スコア69407

Hyonta

2016/05/29 05:37

ありがとうございます！！！ continueをうまく使えばこんなにもスッキリするんですね！！素早い回答でとても助かりました。

行動規範の内容に同意します

kei344さんの回答をもとに修正しました。
これで同じ処理になっていますよね？
回答ありがとうございました！

###修正後のソースコード

Java
1// 一行づつ読み込みループ開始
2String logLine = reader.readLine();
3while (logLine != null) {
4	logLineCount++;
5	inputData.setLogLine(logLine);
6	
7	// スタックトレース部の場合、メッセージを出力し次の行へ
8	if (logLine.startsWith("\t")) {
9		message.append("スタックトレース部分のため、次の行を読み込みます。");
10		message.append(inputFileName);
11		message.append("：");
12		message.append(logLineCount);
13		message.append("行目）");
14		logger.debug(message.toString());
15		message.setLength(0);
16		continue;
17	}
18	// 日時のフォーマットが定義と一致しない場合、メッセージを出力し次の行へ
19	if (!logLineCheck.isDateFormat(inputData)) {
20		message.append("入力ファイルの日時フォーマットが定義と一致しないため、次の行を読み込みます。");
21		message.append(inputFileName);
22		message.append("：");
23		message.append(logLineCount);
24		message.append("行目）");
25		logger.warn(message.toString());
26		message.setLength(0);
27		continue;
28	}
29	// 指定期間内でない場合、次の行へ
30	if (!logLineCheck.isDateScope(inputData)) {
31		continue;
32	}
33	logTargetLineCount++;
34	// タブ区切りオプション処理
35	if (selectedOption != null && selectedOption.indexOf(Execute.TAB_OPTION) != -1) {
36		option.doTabDelimiter(inputData, inputFileName);
37	}
38	// 出力ログリストへ追加
39	logDate.append(sdfStandard.format(inputData.getLogLineDate()));
40	logDate.append("\t");
41	logDate.append(logLine);
42	outputLogList.add(logDate.toString());
43	logDate.setLength(0);
44
45	// 次の行を読み込む
46	logLine = reader.readLine();
47}