Read a file and Output the files

Question

前提・実現したいこと

お知恵をお貸しいただけると幸いです。
＜実現＞
・ファイルに書き込みとコンソールに出力は任意のメソッドを作成して処理を行いたい。
・一行ずつ読み込んでファイルに書き込みとコンソールに出力をさせたい
・extractメソッドにあるclose()が必要なところにfinally句で必ず実行させるようにしたい。　
<前提>
1:「app.log」というファイルを読み込んで、「ERROR」「FATAL」行の先頭に「★」印を印字したファイル「app_error.log」を作成
2:「app.log」というファイルを読み込んで、「2019年1月11日」の記録を抽出した「app_20190111.log」を作成
3: [ERROR」「FATAL 」行の先頭に「★」印を印字して、引数で指定されたファイルに出力するメソッド「markErrorLine」を作成
4: 引数で渡された日付（Calendar-Class）に該当する行をファイルに出力する　メソッド「extractDateLog」を作成
5: 結果として、以下のように記載の内容を表示したい。

ExampleOfOutput:
1.実行処理
ログファイルの解析処理を開始します
ログファイルの解析処理を終了しました
2. app_error.log
★2019/01/12 13:30:23.530 ERROR (DBLogic.java getConnection) - DBコネクション取得
★2019/01/12 13:30:23.530 FATAL (AbstractLogic.java execute) - システム例外 発生
2019/01/13 13:30:10.861 DEBUG (DBLogic.java getConnection) - DBコネクション取得
2019/01/13 13:30:10.923 DEBUG (DBLogic.java closeConnection) - DBコネクションクローズ
3.app_20190111.log
2019/01/11 13:30:10.861 DEBUG (DBLogic.java getConnection) - DBコネクション取得
2019/01/11 13:30:10.923 DEBUG (DBLogic.java closeConnection) - DBコネクションクローズ

該当のソースコード

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.Calendar;
import java.util.Date;
import java.util.regex.Matcher;
import java.util.regex.Pattern;

/**
 *ログファイル「app.log」を読み込んで、下記ファイルを作成するプログラム
 *「FATAL」「ERROR」行の先頭に「★」印を印字したファイル「app_error.log」
 *「2019年1月11日」の記録を抜き出したファイル「app_20190111.log」
 */
public class Exercise10 {
	/**
	 * main
	 * execute method を呼び出す
	 * @param 実行時引数
	 *
	 *
	 * */
	public static void main(String[] args) {

		Exercise10 ex10 = new Exercise10();
		try {
			ex10.execute();
		} catch (IOException e) {
			System.out.println("入出力例外が発生しました");
			System.exit(1);

		}
	}

	/**
	 *Execute method
	*markErrorLine method and extractDateLog method を呼び出し、
	 * @throws IOException
	 */
	public  void execute() throws IOException {

		System.out.println("1. 実行処理");

		System.out.println("ログファイルの解析処理を開始します");

		ArrayList<String> textLines = new ArrayList<String>();


		File file = new File("..\app.log");
		try( FileReader fr = new FileReader(file);
				BufferedReader br = new BufferedReader(fr);){





			String line;
			while ((line = br.readLine()) != null) {
				
				textLines.add(line);
			}


			System.out.println("ログファイルの解析処理を終了しました");
		}catch(FileNotFoundException f) {
			System.out.println("ファイルが見つかりません。[リンク内容](https://eng-entrance.com/java-finally)");
			System.exit(1);
		} catch (IOException e) {
			System.out.println("入出力例外が発生しました");
			System.exit(1);
		}

		//「app_error.log」ファイルを作成
		File newErrorfile = new File("..\app_error.log");
		newErrorfile.createNewFile();


		System.out.println("2. app_error.log");
		// Exercise10 ex10 = new Exercise10();

		markErrorLine(newErrorfile, textLines);

		Calendar cal = Calendar.getInstance();
		cal.set(2019, 00, 11);

		System.out.println("3. app_20190111.log");
		extractDateLog(cal, textLines);

	}

	/**
	 *markErrorLine method
	 * @param textLines
	 * @param newErrorfile
	 * @throws IOException
	 */

	public void markErrorLine(File newErrorfile, ArrayList<String> textLines)
			throws IOException {


		//ファイル・コンソールに出力
		try(PrintWriter pw = new PrintWriter(new BufferedWriter(new FileWriter(newErrorfile)));){
			for (String l : textLines) {
				pw.println(l.replaceFirst("^(.*)(FATAL|ERROR)(.*)$", "★$0"));
				System.out.println(l.replaceFirst("^(.*)(FATAL|ERROR)(.*)$", "★$0"));
			}
			//pw.close();
		}
	}



	/**
	 *extractDateLog method
	 *
	 * @param cal
	 * @param textLines
	 * @throws IOException
	 */
	public static void extractDateLog(Calendar cal, ArrayList<String> textLines) throws IOException {
		Date date = cal.getTime();
		SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy/MM/dd");
		String strDate = dateFormat.format(date);

		SimpleDateFormat df = new SimpleDateFormat("yyyyMMdd");

		//「app_20190111.log」ファイルを作成
		String s = "..\app_"+ df.format(date) + ".log";


		File newDatefile = new File(s);
		newDatefile.createNewFile();

		//  ファイル出力
		PrintWriter pw = new PrintWriter(new BufferedWriter(new FileWriter(newDatefile)));
		for (String l : textLines) {
			Pattern p = Pattern.compile(strDate);
			Matcher m = p.matcher(l);
			if (m.find()) {
				pw.println(l);
				System.out.println(l);
			}
		}
		pw.close();


	}
}

試したこと

何故かERROR or FATAL 以外のやつはコンソールに出力されるが、ファイルには一行しか書き込まれていない。
ERRORやFATALに合致したログを★を付け加えて編集させて出力とファイルに書き込みしたいのに、これは出力も書き込みもされていませんでした。

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.text.SimpleDateFormat;
import java.util.Calendar;
import java.util.Date;
import java.util.regex.Matcher;
import java.util.regex.Pattern;

/**
 *ログファイル「app.log」を読み込んで、下記ファイルを作成するプログラム
 *「FATAL」「ERROR」行の先頭に「★」印を印字したファイル「app_error.log」
 *「2019年1月11日」の記録を抜き出したファイル「app_20190111.log」
 */
public class Test10 {
	/**
	 * main
	 * execute method を呼び出す
	 * @param 実行時引数
	 *
	 *
	 * */
	public static void main(String[] args) {

		Test10 ex10 = new Test10();
		try {
			ex10.execute();
		} catch (IOException e) {
			System.out.println("入出力例外が発生しました");
			System.exit(1);

		}
	}

	/**
	 *Execute method
	*markErrorLine method and extractDateLog method を呼び出し、
	 * @throws IOException
	 */
	public  void execute() throws IOException {

		System.out.println("1. 実行処理");

		System.out.println("ログファイルの解析処理を開始します");
		System.out.println("ログファイルの解析処理を終了しました");
		//ArrayList<String> textLines = new ArrayList<String>();


		File file = new File("..\app.log");
		try( FileReader fr = new FileReader(file);
				BufferedReader br = new BufferedReader(fr);){
			//「app_error.log」ファイルを作成
			File newErrorfile = new File("..\app_error.log");
			newErrorfile.createNewFile();

			Calendar cal = Calendar.getInstance();
			cal.set(2019, 00, 11);



			String line;
			while ((line = br.readLine()) != null) {
				markErrorLine(newErrorfile, line);
				extractDateLog(cal, line);

				//textLines.add(line);
			}



		}catch(FileNotFoundException f) {
			System.out.println("There is no files to be read");
			System.exit(1);
		} catch (IOException e) {
			System.out.println("入出力例外が発生しました");
			System.exit(1);
		}




		System.out.println("2. app_error.log");
		// Exercise10 ex10 = new Exercise10();

		//markErrorLine(newErrorfile, textLines);


		System.out.println("3. app_20190111.log");


	}

	/**
	 *markErrorLine method
	 * @param line
	 * @param newErrorfile
	 * @throws IOException
	 */

	public void markErrorLine(File newErrorfile,  String line)
			throws IOException {
		if(line.isEmpty()){
			String str2 = line.replaceFirst("^(.*)(FATAL|ERROR)(.*)$", "★$0");
			Fileoutput(newErrorfile, str2);

		//ファイル・コンソールに出力
		//try(PrintWriter pw = new PrintWriter(new BufferedWriter(new FileWriter(newErrorfile)));){
			//for (String l : textLines) {
				//pw.println(l.replaceFirst("^(.*)(FATAL|ERROR)(.*)$", "★$0"));
				//System.out.println(l.replaceFirst("^(.*)(FATAL|ERROR)(.*)$", "★$0"));
			//}
			//pw.close();
		}
	}

	//ファイル・コンソールに出力用にメソッドを作成
	public void Fileoutput(File  s ,  String str2 ){
	PrintWriter pw = null;
	try {
		pw = new PrintWriter(new BufferedWriter(new FileWriter(s)));
		pw.println(str2);
		System.out.println(str2);
	} catch (IOException e) {

		e.printStackTrace();
	}finally {pw.close();

	}

}

	/**
	 *extractDateLog method
	 *
	 * @param cal
	 * @param line
	 * @throws IOException
	 */
	public void extractDateLog(Calendar cal, String line) throws IOException {
		Date date = cal.getTime();
		SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy/MM/dd");
		String strDate = dateFormat.format(date);
		SimpleDateFormat df = new SimpleDateFormat("yyyyMMdd");

		//「app_20190111.log」ファイルを作成
		String s = "..\app_"+ df.format(date) + ".log";

		File newDatefile = new File(s);
		newDatefile.createNewFile();
		Pattern p = Pattern.compile(strDate);
		Matcher m = p.matcher(line);
		if (m.find()) {
			Fileoutput( newDatefile , line);
		}


	}
}

Finally句

補足情報（FW/ツールのバージョンなど）

java_1.8
eclipse_version_14
###追記1
変数に結果を代入してファイル・コンソールに出力させた。

	try(PrintWriter pw = new PrintWriter(new BufferedWriter(new FileWriter(newErrorfile)));){
			for (String l : textLines) {
				String s1 = l.replaceFirst("^(.*)(FATAL|ERROR)(.*)$", "★$0");
				pw.println(s1);
				System.out.println(s1);
			}
			//pw.close();
		}
	}

###追記2
extractDateLog メソッドを実行した後に"3.app_201911.log"を出力しているため、コンソール出力・ファイル書き込み後に出力されていることに気がつきました。
ただ、while文のなかでextractDateLogメソッドを実行する直前に記述しても一行ずつ"3.app_201911.log"が出力される気がします。
どう修正すればよいかどなたかご教示願います。

String line;
			while ((line = br.readLine()) != null) {
				markErrorLine(newErrorfile, line);
				extractDateLog(cal, line);


			}
System.out.println("3. app_20190111.log");

出力結果;

2019/01/16 13:30:23.467 DEBUG (WriteOperationLogLogic.java insert) - ログ証跡出力 終了
2019/01/16 13:30:23.530 DEBUG (DBLogic.java commit) - トランザクションコミット
3. app_20190111.log

###追記3
mark method を用いたら解決しました。

			br.mark((int)file.length());
			String line;
			while ((line = br.readLine()) != null) {
				markErrorLine(newErrorfile, line);


			}
			br.reset();
			System.out.println("3. app_20190111.log");

			String line1;
			while ((line1 = br.readLine()) != null) {

				extractDateLog(cal, line1);


			}

Answer 1

全部Listに読むclass Exercise10から、1行ずつ処理をするclass Test10に書き換えているのですね。

とりあえず下記2点で意図通りの動作でしょうか。

public void markErrorLine(File newErrorfile, String line) {
    if (!line.isEmpty()) {

条件が逆です。

public void Fileoutput(File s, String str2) {
    try (PrintWriter pw = new PrintWriter(new FileWriter(s, true))) {

trueがないと常に新しい1行で上書きされてしまいます。

Comments

[退会済みユーザー]

exercise10 class では、おっしゃる通り全て読み込んでからリストに格納しておりましたが、読み込むlogは実務では膨大なため Out of memoryが発生すると考え,Test10 classで一行ずつ読み込んで処理を行う方法に切り替えました。
ご教示頂いたコードに変更しましたらコンソールに出力・ファイル書き込み出来ました。
ありがとうございました。
上書きされていることに気がつきませんでした。

Answer 2

追記2に関して
markメソッドを用いたらできました。

			br.mark((int)file.length());
			String line;
			while ((line = br.readLine()) != null) {
				markErrorLine(newErrorfile, line);


			}
			br.reset();
			System.out.println("3. app_20190111.log");

			String line1;
			while ((line1 = br.readLine()) != null) {

				extractDateLog(cal, line1);


			}