重複している要素を抽出し、ファイルに出力ができません。

Question

前提・実現したいこと

Javaの課題を解いています。
List1の中にある要素を、HashSetを利用して抽出し、ファイルに「b,c,d」と出力したいのですが、
92行目にあるSystem.out.println(str);のstrには、「b,b,c,d」が格納されてしまい、しかもファイルには「ｄ」のみしか出力されています。

該当のソースコード

import java.io.BufferedReader;
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.util.ArrayList;
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Scanner;
import java.util.Set;

public class Kadai{

	public static void main(String[] args) throws IOException {

		/**
		 * ・3つのファイルの中身を行単位で読み込みます。 ・3つのファイルの内容で一致するものがあれば、別ファイルにその一致した内容を書き出します
		 * ・出力ファイル名、入力ファイル名は問いません ・入力ディレクトリ、出力ディレクトリも問いません
		 */

		ques1();
		System.out.println("3つのファイルの中身を行単位で読み込みました");
	

	}

	private static void ques1() throws IOException {

		try {

			File input1 = new File("/Users/Owner/OneDrive/会社　資料/20_開発課題/01_Java基礎課題L0-L10/参照ファイル/input1.txt");
			File input2 = new File("/Users/Owner/OneDrive/会社　資料/20_開発課題/01_Java基礎課題L0-L10/参照ファイル/input2.txt");
			File input3 = new File("/Users/Owner/OneDrive/会社　資料/20_開発課題/01_Java基礎課題L0-L10/参照ファイル/input3.txt");

			if (input1.exists() && input2.exists() && input3.exists()) { // ファイルが存在しているか確認

				List<String> list1 = new ArrayList<>();

				FileReader filereader1 = new FileReader(input1);
				FileReader filereader2 = new FileReader(input2);
				FileReader filereader3 = new FileReader(input3);

				BufferedReader br1 = new BufferedReader(filereader1);
				BufferedReader br2 = new BufferedReader(filereader2);
				BufferedReader br3 = new BufferedReader(filereader3);

				String data1;
				while ((data1 = br1.readLine()) != null) {
					list1.add(data1);
				}
				String data2;
				while ((data2 = br2.readLine()) != null) {
					list1.add(data2);
				}
				String data3;
				while ((data3 = br3.readLine()) != null) {
					list1.add(data3);
				}

				// setを設定し、する。
				Set set = new HashSet();

				FileWriter fw = null;
				PrintWriter out = null;

				// リストに入っている要素分繰り返す
				for (String str : list1) {

					// setに追加してみて、追加できなかった(重複していた)ら、strに格納

					if (!set.add(str)) {

						String path = "/Users/Owner/OneDrive/java.txt/input4.txt";

						//ここで「b,c,d」と、ファイルに書き込まれる

						fw = new FileWriter(path);
						out = new PrintWriter(fw);

						System.out.println(str);//「b,b,c,d」

						out.write(str);//「d」のみ

					}

				}
				fw.close();
				out.close();

			}

		} catch (FileNotFoundException e) {
			System.out.println("ファイルが存在しません。");
		} catch (IOException e) {
			e.printStackTrace();

		}

	}
}

試したこと

ほかにも、Setの代わりにMapを使って重複を抽出しましたが、「ｂ、ｃ、ｄ」は格納できたのですが、ファイルにはやはり同じように[d]のみ出力されます。

Answer 1

                        fw = new FileWriter(path);
                        out = new PrintWriter(fw);

の定義位置、もしくは使い方ががおかしい。

あなたが使っているコンストラクタはこれ。https://docs.oracle.com/javase/jp/8/docs/api/java/io/FileWriter.html#FileWriter-java.io.File-。ファイルに追記したいのであれば、 https://docs.oracle.com/javase/jp/8/docs/api/java/io/FileWriter.html#FileWriter-java.io.File-boolean- をつかい、fw = new FileWriter(path, true)とする。

もしくは、上記二行の定義位置を見直してみるといい。fw.close()とfw = new FileWriter(path)が対になるように。

Comments

[Sharifa]

ご回答ありがとうございます！なるほどですね。
とりあえずは、
・ファイルに追記モードで書く
・上記2つの定義位置を変える
の2点で修正してみます！

[Sharifa]

あの後、Mapを使って、close()；の位置を変えたら、上手く
「ｂ、ｃ、ｄ」と出力されました！ありがとうございます！