ファイルに書き込んだオブジェクトを全て読み込みたいが、先頭にある１つ目しか読み込むことができない

Question

シリアライズとデシリアライズをし、ファイルにオブジェクトの読み書きをしたいです。追加の書き込みはできるのですが、読み込みが上手くいきません。
何度読み込んでも、ファイルに一番初めに書き込んだオブジェクトしか得ることが出来ません。

ファイルに書き込んだすべてのオブジェクトを読み込みたいです。

main関数でループを回すという方法を考え、RandomAccessFileを使いファイルポインタの動かしてというやり方をしていたのですが上手くいきませんでした。
１行ずつ読み込むという方法もあったのですが、オブジェクトでは型が違いますとエラーになってしまいます。

もしうまい方法を知っている方がいればぜひ教えてください。
よろしくお願い致します。

Member.javaファイル

package hello;

import java.io.Serializable;

public class Member implements Serializable {

	public String name; // 氏名
	public int age;     // 年齢
}

SerializableSampleTest.javaファイル

package hello;
import java.io.FileInputStream;

import java.io.FileOutputStream;
import java.io.IOException;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.util.ArrayList;
import java.util.List;


public class SerializableSampleTest {
	public static void main(String[] args) {
		SerializableSampleTest sst1 = new SerializableSampleTest();
		SerializableSampleTest sst2 = new SerializableSampleTest();


		// 一人目
		Member mb1 = new Member();
		mb1.name = "sakura";
		mb1.age = 20;

		List Akumi = new ArrayList();
		Akumi.add(mb1.name);
		Akumi.add(mb1.age);

		sst1.write(Akumi);

		// 二人目
		Member mb2 = new Member();
		mb2.name = "yuri";
		mb2.age = 19;

		List Bkumi = new ArrayList();
		Bkumi.add(mb2.name);
		Bkumi.add(mb2.age);

		sst2.write(Bkumi);


		System.out.println(sst1.read());
		System.out.println(sst2.read());



	}

 public void write(java.util.List<Member> Mem) {
     // シリアライズ
     try (ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream("member", true))) {
         oos.writeObject(Mem);
         System.out.println("シリアライズ完了");
         oos.close();

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

 public java.util.List<Member> read(){
	 java.util.List<Member> Mem = null;

     // デシリアライズ
     try (ObjectInputStream ois = new ObjectInputStream(new FileInputStream("member"))) {
         Mem = (List<Member>) ois.readObject();
         System.out.println("デシリアライズ完了");
         ois.close();
     } catch (IOException | ClassNotFoundException e) {
         System.out.println("error");
     }
     return Mem;
  }

 }

Answer 1

A-pZさんの回答に同意です。自分は以下の点について若干コメントします。

ファイルに書き込んだすべてのオブジェクトを読み込みたいです。

もし現在の書き込み方式(複数のObjectOutputStreamを用いてファイルへ追記する方法)のままとするなら、書き込みと同じ順番・回数だけ個別のObjectInputStreamを用い、かつ直前のオブジェクト読み込み完了時のInputStreamのリード位置を調べ次のObjectInputStreamを用いたリードの際にInputStreamに対してシークしておかなければなりません。質問者さんもこの点を意識しておられたと思いますが、try-resource文を使う関係で以下のようにされたのではないでしょうか。

1. RandomAccessFileを生成
2. RandomAccessFileをwrapするInputStreamを生成
3. InputStreamからObjectInputStreamを生成
4. RandomAccessFileをシーク
5. readObject

こうすると5.で例外が発生すると思います。原因はObjectInputStreamのコンストラクター内でInputStreamが先読みされることにあるようです。とりあえず回避は4のシーク処理を3より前に行うことですが、ただObjectInputStreamが先読みを用いている事実があるので個人的に気持ち悪さが残ります(readObject完了時点で本来必要なデータの終端を超えて先読みすることがないのかという点で)。もし可能なら単一のObjectOutputStreamで全てのオブジェクトを一気に書き込む(読み込みも同様)方が安心な実装だと思いました。

Comments

[fgfnabwym]

ありがとうございます。教えたいただいた方法で書き直してみます

Answer 2

少し気になるコードがあります。

public void write(java.util.List<Member> Mem) {

}

と、writeメソッドは List<Member>を受け取るメソッドですが、このメソッドを使っているmainメソッドでは

List Akumi = new ArrayList();
Akumi.add(mb1.name);
Akumi.add(mb1.age);

sst1.write(Akumi);

と、バラバラに足しこんでいますが、Akumi.add(mb1) ではないでしょうか。

あと、例外をキャッチしている箇所でエラーメッセージを出していますが、できれば、

} catch (IOException | ClassNotFoundException e) {
  System.out.println("error");
  e.printStackTrace();
}

と、例外が発生した内容を出力するようにした方が良いでしょう。

Comments

[fgfnabwym]

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