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

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

ただいまの
回答率

88.35%

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

解決済

回答 2

投稿

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

Hyonta

score 32

実現したいこと

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

現在のソースコード

// 一行づつ読み込みループ開始
String logLine = reader.readLine();
while (logLine != null) {
        logLineCount++;
        inputData.setLogLine(logLine);
        // スタックトレース部ではない
        if (!logLine.startsWith("\t")) {
                // 日時のフォーマットが期待通りである
                if (logLineCheck.isDateFormat(inputData)) {
                        // 指定した期間内である
                        if (logLineCheck.isDateScope(inputData)) {
                                logTargetLineCount++;
                                // タブ区切りオプションが指定されていたら実施
                                if (selectedOption != null
                                                && selectedOption.indexOf(Execute.TAB_OPTION) != -1) {
                                        option.doTabDelimiter(inputData, inputFileName);
                                }
                                // 出力ログリストへ追加
                                logDate.append(sdfStandard.format(inputData.getLogLineDate()));
                                logDate.append("\t");
                                logDate.append(logLine);
                                outputLogList.add(logDate.toString());
                                logDate.setLength(0);
                        }
                //日時フォーマットが期待通りでない        
                } else {
                        // ログメッセージ出力
                        message.append("入力ファイルの日時フォーマットが定義と一致しないため、次の行を読み込みます。(");
                        message.append(inputFileName);
                        message.append(":");
                        message.append(logLineCount);
                        message.append("行目)");
                        logger.warn(message.toString());
                        message.setLength(0);
                }
        //スタックトレース部を読み込んだ        
        } else {
                // ログメッセージ出力
                message.append("スタックトレース部分のため、次の行を読み込みます。(");
                message.append(inputFileName);
                message.append(":");
                message.append(logLineCount);
                message.append("行目)");
                logger.debug(message.toString());
                message.setLength(0);
        }
        // 次の行を読み込む
        logLine = reader.readLine();
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+2

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

if ( logLine.startsWith("\t") ) {
    /* エラー処理 */
    continue;
}
if ( !logLineCheck.isDateFormat(inputData) ) {
    /* エラー処理 */
    continue;
}
if (!logLineCheck.isDateScope(inputData)) {
    continue;
}
    /* 通常処理 */

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/05/29 14:37

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

    キャンセル

0

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

修正後のソースコード

// 一行づつ読み込みループ開始
String logLine = reader.readLine();
while (logLine != null) {
    logLineCount++;
    inputData.setLogLine(logLine);

    // スタックトレース部の場合、メッセージを出力し次の行へ
    if (logLine.startsWith("\t")) {
        message.append("スタックトレース部分のため、次の行を読み込みます。");
        message.append(inputFileName);
        message.append(":");
        message.append(logLineCount);
        message.append("行目)");
        logger.debug(message.toString());
        message.setLength(0);
        continue;
    }
    // 日時のフォーマットが定義と一致しない場合、メッセージを出力し次の行へ
    if (!logLineCheck.isDateFormat(inputData)) {
        message.append("入力ファイルの日時フォーマットが定義と一致しないため、次の行を読み込みます。");
        message.append(inputFileName);
        message.append(":");
        message.append(logLineCount);
        message.append("行目)");
        logger.warn(message.toString());
        message.setLength(0);
        continue;
    }
    // 指定期間内でない場合、次の行へ
    if (!logLineCheck.isDateScope(inputData)) {
        continue;
    }
    logTargetLineCount++;
    // タブ区切りオプション処理
    if (selectedOption != null && selectedOption.indexOf(Execute.TAB_OPTION) != -1) {
        option.doTabDelimiter(inputData, inputFileName);
    }
    // 出力ログリストへ追加
    logDate.append(sdfStandard.format(inputData.getLogLineDate()));
    logDate.append("\t");
    logDate.append(logLine);
    outputLogList.add(logDate.toString());
    logDate.setLength(0);

    // 次の行を読み込む
    logLine = reader.readLine();
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/05/29 14:44

    > // 指定期間内でない場合、次の行へ
    ここは if (!logLineCheck.isDateScope(inputData)) になりませんか?

    あと、回答欄もコードブロック使えますよ。

    キャンセル

  • 2016/05/29 16:00

    ご指摘ありがとうございます。
    修正いたしました。

    キャンセル

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

  • ただいまの回答率 88.35%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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