Javaの質問です。テキストファイルのデータを上書き(部分的に削除)したいです。お力をお貸しください。

Question

初めて利用させていただきます。

現在私は、テキストファイルを用いるJavaの勉強をしております。
テキストファイルに、コンソールで入力した「name」「job」「sex」を出力する、また、テキストファイルに書き込まれたデータをコンソールに表示する機能はなんとか作成することが出来ました。

しかし、テキストファイルに書き込まれているデータを上書き(部分的に削除)する機能で詰まってしまいました。自分なりに調べたり、試してみたのですが思うように出来ませんでした。
上書き(一部削除)する時は、数字が入力されたら、それに対応した「Person」クラスの「name」「job」「sex」を一括でしたいです。

どなたかお力をお貸しください。お手数ですが、よろしく御願い致します。

---

//txtのデータ
List.1//1が入力されたら削除
田中
医者
男性
List.2//2が入力されたら削除
鈴木
看護師
女性

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.util.ArrayList;

public class Editing {
class Person { // 1人分のデータを格納するクラス
private String name;
private String job;
private String sex;

	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	public String getJob() {
		return job;
	}
	public void setJob(String job) {
		this.job = job;
	}
	public String getSex() {
		return sex;
	}
	public void setSex(String sex) {
		this.sex = sex;
	}
}

private boolean checkBeforeReadFile(File file) {
	if (file.exists()) {
		if (file.isFile() && file.canRead()) {
			return true;
		}
	}
	return false;
}

// ファイルを読み込む
public ArrayList<Person> readFile(File file) {
int i = 0;
ArrayList<Person> list = new ArrayList<Person>();
BufferedReader br = null;
try {
if (checkBeforeReadFile(file)) {
br = new BufferedReader(new FileReader(file));
String line = null;
Person person = null;
while ((line = br.readLine()) != null) {
switch (i) {
case 0: // 名前
person = new Person(); // Person クラスのインスタンスを生成
person.setName(line); // 名前を格納
i++; // 次は i = 1 → 職業
break;
case 1: // 職業
person.setJob(line); // 職業を格納
i++; // 次は i = 2 → 性別
break;
case 2: // 性別
person.setSex(line); // 性別を格納
list.add(person); // 1人分のデータの格納が完了したので、リストに追加
i=0; // 次は i = 0 → 名前
break;

				}
			}
		}
	}
	catch (IOException e) {
		e.printStackTrace();
	}
	finally {
		if (br != null) {
			try {
				br.close();
			}
			catch (IOException e) {
				e.printStackTrace();
			}
		}
	}
	return list;
}

// 全員分のデータを表示する
private void showPeople(ArrayList<Person> list) {
for (Person person : list) {
System.out.println("名前：" + person.getName());
System.out.println("職業：" + person.getJob());
System.out.println("性別：" + person.getSex());
}
}
public static void main(String[] args) {
try{
File file = new File("sample.txt");

	    	  PrintWriter pw = new PrintWriter(new BufferedWriter(new FileWriter(file,true)));
	    	  BufferedReader br = new BufferedReader(new FileReader(file));
	    	  Editing editing = new Editing();
	    	  ArrayList<Person> list = editing.readFile(file);
	    	  editing.showPeople(list);

	    	  System.out.println("0:新規登録 1:削除");
	    	  int input = new java.util.Scanner(System.in).nextInt();
	    	  if(input == 0){
	    		  System.out.println("名前を入力してください");
		  		  String name = new java.util.Scanner(System.in).nextLine();
		  		  System.out.println("仕事を入力してください");
		  		  String job = new java.util.Scanner(System.in).nextLine();
		  		  System.out.println("性別を入力してください");
		  		  String sex = new java.util.Scanner(System.in).nextLine();
		  		  pw.println(name);
		  		  pw.println(job);
		  		  pw.println(sex);
		 /* 		  pw.println("\n");*/
		  		  pw.close();
		  		  System.out.println("連絡先を追加しました");
	    	  }

	    	  if(input == 1){
  	  		//削除
	    	  }

	}
	catch(IOException e){
    	System.out.println(e);
}
}

}

Answer 1

与えられたファイルの形式は適切とはおもいませんが、ひとまずはそれに従うコードを書いてみました。
部分的削除は面倒なので、追加・削除にかかわらず、つねにフィアルは全体を書きなおしています。

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Editing {
    private static int MAX_ID_LENGTH = 100;
    private static int MAX_NAME_LENGTH = 100;
    private static int MAX_JOB_LENGTH = 100;
    private static int SEX_LENGTH = 2;

    class Person { // 1人分のデータを格納するクラス
        private final String id;
        private final String name;
        private final String job;
        private final String sex;

        public Person(String id, String name, String job, String sex) {
            if (id.length() == 0 || id.length() > MAX_ID_LENGTH) {
                throw new IllegalArgumentException("id is illegal." + id);
            }
            if (name.length() == 0 || name.length() > MAX_NAME_LENGTH) {
                throw new IllegalArgumentException("name is illegal." + name);
            }
            if (job.length() == 0 || job.length() > MAX_JOB_LENGTH) {
                throw new IllegalArgumentException("job is illegal." + job);
            }
            if (sex.length() != SEX_LENGTH) {
                throw new IllegalArgumentException("sex is illegal." + sex);
            }
            this.id = id;
            this.name = name;
            this.job = job;
            this.sex = sex;
        }

        public String getId() {
            return id;
        }

        public String getName() {
            return name;
        }

        public String getJob() {
            return job;
        }

        public String getSex() {
            return sex;
        }

        public String toString() {
            String ans = "";
            ans += (this.getId());
            ans += (",[" + this.getName() + "]");
            ans += (", " + this.getJob() + "]");
            ans += (", " + this.getSex() + "]");
            return ans;
        }
    }

    // path が存在する、ファイルである、読み込み可能である ことをチェックする。
    private boolean checkBeforeReadFile(File path) {
        if (path.exists() && path.isFile() && path.canRead()) {
            return true;
        } else {
            return false;
        }
    }

    // ファイルに出力する。
    public void writeFile(String file, List<Person> people) throws IOException {
        PrintWriter pw = null;
        try {
            pw = new PrintWriter(new BufferedWriter(new FileWriter(file)));
            for (Person person : people) {
                pw.println(person.id);
                pw.println(person.name);
                pw.println(person.job);
                pw.println(person.sex);
            }
        } finally {
            if (pw != null) {
                pw.close();
            }
        }
    }

    // ファイルを読み込む。
    public List<Person> readFile(String file_name) throws IOException {
        List<Person> list = new ArrayList<Person>();
        BufferedReader br = null;
        try {
            File file = new File("sample.txt");
            if (checkBeforeReadFile(file)) {
                br = new BufferedReader(new FileReader(file));
                String line = null;
                int i = 0;
                String str[] = { null, null, null, null }; // { id, name, job,
                                                           // sex }
                while ((line = br.readLine()) != null) {
                    str[i % 4] = line;
                    i++;
                    if (i % 4 == 0) {
                        list.add(new Person(str[0], str[1], str[2], str[3]));
                    }
                }
            }
        } finally {
            if (br != null) {
                try {
                    br.close();
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }
        return list;
    }

    public void addPerson(String file_name, List<Person> people, String name, String job, String sex)
            throws IOException {
        String id = "List:" + (people.size() + 1);
        Person person = new Person(id, name, job, sex);
        people.add(person);
        writeFile(file_name, people);
    }

    public boolean deletePerson(String file_name, List<Person> people, String id) throws IOException {
        for (Person person : people) {
            if (person.getId().equals(id)) {
                people.remove(person);
                writeFile(file_name, people);
                return true;
            }
        }
        return false;
    }

    // 全員分のデータを表示する
    private void showPeople(List<Person> list) {
        for (Person person : list) {
            System.out.println("ＩＤ：" + person.getId());
            System.out.println("名前：" + person.getName());
            System.out.println("職業：" + person.getJob());
            System.out.println("性別：" + person.getSex());
        }
    }

    public static void action() {
        String file_name = "sample.txt";
        try {
            Scanner scanner = new Scanner(System.in);
            Editing editing = new Editing();

            while (true) {
                System.out.println("0:新規登録 1:削除 2:表示 9: 終了");
                String line = scanner.nextLine();
                int input = Integer.parseInt(line);
                if (input == 0) {
                    System.out.println("名前を入力してください");
                    String name = scanner.nextLine();
                    System.out.println("仕事を入力してください");
                    String job = scanner.nextLine();
                    System.out.println("性別を入力してください");
                    String sex = scanner.nextLine();
                    List<Person> list = editing.readFile(file_name);
                    editing.addPerson(file_name, list, name, job, sex);
                    System.out.println("追加しました。");
                } else if (input == 1) {
                    System.out.println("ＩＤを入力してください。");
                    String num = scanner.nextLine();
                    List<Person> list = editing.readFile(file_name);
                    boolean ret = editing.deletePerson(file_name, list, "List:" + num);
                    if (ret) {
                        System.out.println("削除しました。");
                    } else {
                        System.out.println("指定されたidのデータはありませんでした。");
                    }
                } else if (input == 2) {
                    List<Person> list = editing.readFile(file_name);
                    editing.showPeople(list);
                } else if (input == 9) {
                    System.out.println("終了");
                    break;
                }
            }
        } catch (Exception e) {
            System.out.println(e);
        }
    }

    public static void main(String[] args) {
        action();
    }
}

Answer 2

ルール変えましょう。
テキストファイルはいかのようにします。

name,job,sex
木村,センター,男
中居,リーダー,男
草薙,平社員,女

ヘッダ行付きで、半角カンマ区切りとする。
読み込む時は、ヘッダを読み飛ばし、1行読み込み、半角カンマで分割、person クラスを初期化して、長さ3の配列要素が、name,job,sex 順で並んでるのでそのまま対応する変数のセッターで設定する。

登録処理は、名前、仕事、性別をよみこみ、半角かんまでれんけつした1行をリストに格納、ファイルに保存するときに、既存読み込み行の後ろにアペンドして、ファイル戦先頭に、name,job,sex のヘッダをかきこみ、そのあとリスト内容をかきこみ、。

削除処理は、メニューで削除を選択したら、予め読込、list<Person> 状態のリストを先頭から順に、
行番号:名前、仕事、性別
のヘッダをしゅつりょくし、その後リスト内容を、先頭から、行番号を1から出力する。リストの先頭は0だけど、一般的には、0行目ではなくて、1行目でしょ、だから1から表示するんです。

で、入力プロンプトに、「削除する行番号を入力してkyださい。」ってやって、1 以上 リスト長さ＋1いかの数値チェックを行ったあと、list.remove(lineNo-1); することで、行の削除は、完了でづね。

あとは想像力だけでづ。自分ほんい、データ本位ではなくて、自分が使うとしたらこういうとこが気が利いているといいなぁ、と思う部分を考えて、かいぜんすればもっとよくなりまづ。

日本語が少々変ですが、iPadの画面のキーボードを左手人差し指1本で叩いてるため、どうしても思う位置のキーが押せなくて、、、、

Comments

[tapipi]

横から失礼いたします。
読み込む時は、ヘッダを読み飛ばし、1行読み込み、半角カンマで分割…
の部分がわからないのですが、ご教授願えませんか？

Answer 3

java.io.RandomAccessFileを使えばいいんじゃないかな？

randomaccess.seek(1);//番号で何バイト目にアクセスするかを設定

確か、英語が1バイト、日本語が2バイトです。

Comments

[退会済みユーザー]

英語1バイト、日本語2バイトは、MS932(Windows-31j)の話です。
そうではない文字セットもあります。

[sun-solar-arrow]

そうでしたね。ありがとうございます。

Answer 4

果たしてプログラムからファイルを「部分的に消す」という動作ができたかどうか。
やるとしたら「該当データを削除した状態でファイルを書き込み、上書きする」という形にするしかないように思うのですが…
そのようにデータを操作したいのであれば、データベースを使うべき場面であると思います。

Comments

[ipadcaron]

データベースはファイルベースですよ。
ファイル操作で、挿入、削除、追加、変更を理解しておくことは重要だと思いますが。
メモリ1Mb しかなくても、Hddに10gbあれば、8gb程度のデータを格納可能なデータベースが作れます。

[heignamerican]

> データベースはファイルベース
それは間違ってはいないんですが、人間可読の形式と機械可読(というより機械最適化)の形式では性質が異なります。
各カラム固定長という制約をつければ java.io.RandomAccessFile でなんとかできますけど…

今回の質問者さんは初心者マーク付けてますし、単一 plain text (オレオレ行区切りフォーマットですが)でなんとかしたいのでしょう。まあ妥当な判断だと思います。
ファイル保存時は(性能は置いといて)まるっと上書きがシンプルで良いのでは

[tkturbo]

java.io.RandomAccessFileのことも忘れないであげて欲しい。。。

[sun-solar-arrow]

確かに