デシリアライズで複数のオブジェクトを読み取りたい。

Question

質問お願いします。
自分はjavaFXでスケジュール表を作っていて、追加した予定を保存して表示するためにSerializeを使っています。

前回の質問で複数のオブジェクトを保存できるようになったのですが、今度は一つのオブジェクトしかデシリアライズできない問題が起きました。そこで調べてみると、このサイトに,Whileで無限ループしてEOFExceptionが発生したら抜け出すという方法が書いてあったのでやってみたのですが、closeしているのにIOExceptionが起きてしまい失敗してしまいました。
正規表現を使ってなんとかできるかな、とも思ったのですが、バイナリファイルですのでやり方がわからずあきらめている状況です。知恵をお貸しください、お願いします。

fxmlは文字数を超えるため書けませんでした、ご容赦ください。

package application;
	
import java.io.EOFException;
import java.io.File;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.ObjectInputStream;
import java.util.ArrayList;
import java.util.List;

import javafx.application.Application;
import javafx.fxml.FXMLLoader;
import javafx.scene.Scene;
import javafx.scene.control.Label;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;


public class ScheduleMain extends Application {
	

	public static ScheduleTableController stController;
	
	@Override
	public void start(Stage primaryStage) {
		try {
			FXMLLoader fxmlLoader = new FXMLLoader(getClass().getResource("ScheduleIndividual.fxml"));
			Scene scene = new Scene((VBox)fxmlLoader.load(),150,600);			
			scene.getStylesheets().add(getClass().getResource("application.css").toExternalForm());
			stController = fxmlLoader.getController();
			
			List<ScheduleData> list = new ArrayList<ScheduleData>();
			try {
				if(!isEmpty()) {
					loading(list);
					for(ScheduleData sd : list) {
						initScheduleLabel(sd);
					}
				}
			} catch (IOException e) {
				e.printStackTrace();
			}
			
			primaryStage.setScene(scene);
			primaryStage.show();
			System.out.println(stController.toString());
		} catch(Exception e) {
			e.printStackTrace();
		}
	}
	
	public static void main(String[] args) {
		launch(args);
	}
	
	boolean isEmpty() throws IOException {
    	File file = new File("SaveDataFile.ser") ;
    	boolean empty = !file.exists() || file.length()==0;
    	return empty;
    }
	
	void loading(List<ScheduleData> datas) {
		try (var in = new ObjectInputStream(new FileInputStream("SaveDataFile.ser"))){
    		while(true) {
    			ScheduleData saveData = (ScheduleData)in.readObject();
        		System.out.println(saveData.getDetail());
        		datas.add(saveData);
        		in.close();
    		}
    	}catch 		(EOFException e) {
    	    System.out.println("デシリアライズ完了");
    	   }	catch (IOException e) {
    	    System.out.println("IOerror");
    	  }catch ( ClassNotFoundException e) {
      	    System.out.println("CNFerror");
      	  }
	}
	
	@SuppressWarnings("static-access")
	void initScheduleLabel(ScheduleData data) {
	    var sLabel = new Label();
	 	double stNum = (data.getsTime().getHour()+data.getsTime().getMinute()/60)*30+4;
	 	double ftNum = (data.getfTime().getHour()+data.getfTime().getMinute()/60)*30+4;
	 	double tNum = ftNum-stNum;
	 	String str = data.gettitle()+"\n"+data.gettime()+"\n"+data.getDetail();
	 	sLabel.setText(str);
	 	sLabel.setStyle("-fx-background-color:green;");
	 	stController.getaPane().getChildren().add(sLabel);
	 	stController.getaPane().setTopAnchor(sLabel, stNum);
	 	sLabel.setPrefHeight(tNum);
	 		
	}
}

/**
 * Sample Skeleton for 'ScheduleIndividual.fxml' Controller Class
 */


public class ScheduleTableController  {


    @FXML // ResourceBundle that was given to the FXMLLoader
    private ResourceBundle resources;

    @FXML // URL location of the FXML file that was given to the FXMLLoader
    private URL location;

    @FXML // fx:id="pane"
    private VBox pane; // Value injected by FXMLLoader
    
    @FXML // fx:id="aPane"
    private AnchorPane aPane; // Value injected by FXMLLoader
    
    @FXML // fx:id="aPane"
    private VBox scheduleIndividual; // Value injected by FXMLLoader
    
    
    //ここからScheduleSelectのid

   
    
    @FXML
    void sets(MouseEvent event) {
    	try {
			showSecondWindow();
		} catch (IOException e) {
			e.printStackTrace();
		}
    }

   
    
    void showSecondWindow() throws IOException {

		FXMLLoader fxmlLoader = new FXMLLoader(getClass().getResource("ScheduleSelect.fxml"));
		AnchorPane root = (AnchorPane) fxmlLoader.load();
		Scene scene = new Scene(root);
		Stage stage = new Stage();
		stage.setScene(scene);
		stage.showAndWait();

	}
    
    
	

	public AnchorPane getaPane() {
		return aPane;
	}



	@FXML // This method is called by the FXMLLoader when initialization is complete
    void initialize() {
   		
		
 		
        assert pane != null : "fx:id=\"pane\" was not injected: check your FXML file 'ScheduleIndividual.fxml'.";
       
    }
	
}

package application;

import java.io.FileOutputStream;
import java.io.IOException;
import java.io.ObjectOutputStream;
import java.time.LocalTime;

import javafx.fxml.FXML;
import javafx.scene.control.Button;
import javafx.scene.control.ComboBox;
import javafx.scene.control.Label;
import javafx.scene.control.TextArea;
import javafx.scene.control.TextField;
import javafx.scene.input.MouseEvent;
import javafx.scene.layout.AnchorPane;

public class ScheduleSelectController {

	 @FXML // fx:id="scheduleName"
	    private TextField scheduleName; // Value injected by FXMLLoader

	    @FXML // fx:id="sHour"
	    private ComboBox<String> sHour; // Value injected by FXMLLoader

	    @FXML // fx:id="sakusei"
	    private Button sakusei; // Value injected by FXMLLoader

	    @FXML // fx:id="packageSelect"
	    private ComboBox<?> packageSelect; // Value injected by FXMLLoader

	    @FXML // fx:id="month"
	    private ComboBox<?> month; // Value injected by FXMLLoader

	    @FXML // fx:id="scheduleSelct"
	    private AnchorPane scheduleSelect; // Value injected by FXMLLoader

	    @FXML // fx:id="year"
	    private ComboBox<String> year; // Value injected by FXMLLoader

	    @FXML // fx:id="fHour"
	    private ComboBox<String> fHour; // Value injected by FXMLLoader

	    @FXML // fx:id="fMinute"
	    private ComboBox<String> fMinute; // Value injected by FXMLLoader

	    @FXML // fx:id="memo"
	    private TextArea memo; // Value injected by FXMLLoader

	    @FXML // fx:id="sMinute"
	    private ComboBox<String> sMinute; // Value injected by FXMLLoader

	    @FXML // fx:id="day"
	    private ComboBox<String> day; // Value injected by FXMLLoader

	    @FXML
	    void clickMemo(MouseEvent event) {
	    	
	    }
	    
	    @FXML
	    void jikkou(MouseEvent event) {
	    	addScheduleLabel(addData());
	    	save();
	    	System.out.println("追加しました。");
	    }
	    
	    ScheduleData addData() {
	    	String name = scheduleName.getText();
	    	var sTime = LocalTime.of(Integer.parseInt(sHour.getValue()),Integer.parseInt(sMinute.getValue()));
	    	var fTime = LocalTime.of(Integer.parseInt(fHour.getValue()),Integer.parseInt(fMinute.getValue()));
	    	String detail = memo.getText();
	       	var sd = new ScheduleData(name, sTime, fTime, detail);
	    	return sd;
	    }
	    
	    
	    @SuppressWarnings("static-access")
		void addScheduleLabel(ScheduleData data) {
	    	var sLabel = new Label();
	 		double stNum = (data.getsTime().getHour()+data.getsTime().getMinute()/60)*30+4;
	 	    double ftNum = (data.getfTime().getHour()+data.getfTime().getMinute()/60)*30+4;
	 	    double tNum = ftNum-stNum;
	 		String str = data.gettitle()+"\n"+data.gettime()+"\n"+data.getDetail();
	 		sLabel.setText(str);
	 		
	 		ScheduleTableController controller = ScheduleMain.stController;	 		
	 		controller.getaPane().getChildren().add(sLabel);
	 		controller.getaPane().setTopAnchor(sLabel, stNum);
	 		sLabel.setPrefHeight(tNum);
	 		}
	    
	    @FXML private void initialize() throws IOException {
	    	
		}
	    
	    
	    void save() {   		    	
	    	try (var out = new ObjectOutputStream(new FileOutputStream( "SaveDataFile.ser",true))){
	    		out.writeObject(addData());
	    		
	    		
	    	}catch (IOException e) {
				e.printStackTrace();
				
			}
	    }
	    
}

ｬ・ sr application.ScheduleData       
 L detailt Ljava/lang/String;L 
finishTimet Ljava/time/LocalTime;L 	startTimeq ~ L timeq ~ 
L titleq ~ 
xpt hrhr4wsr 
java.time.Ser評┝"Hｲ  xpwxsq ~ w
xt 
01:02・・5:01t grsegｬ・ sr application.ScheduleData       
 L detailt Ljava/lang/String;L 
finishTimet Ljava/time/LocalTime;L 	startTimeq ~ L timeq ~ 
L titleq ~ 
xpt gvseaghsr 
java.time.Ser評┝"Hｲ  xpwxsq ~ w
砡t 
01:04・・4:01t fcwea

一回目

二回目

Answer 1

わざわざ調べていただきありがとうございました！新規で開くか追記で開くかで処理が変わってしまうんですね！とても参考になりました！

Comments

[shiketa]

オブジェクトをObjectOutputStreamでシリアライズする。その内容については、https://docs.oracle.com/javase/jp/6/platform/serialization/spec/protocol.html を読むといいでしょう。

[shiketa]

> スケジュール表を作っていて、追加した予定を保存して表示するためにSerializeを使っています。

以下、老婆心ながら。
オブジェクトをシリアライズしたとき、そのオブジェクトのクラスの定義が変わると、あなたの例でいえば ScheduleDataクラスの定義を変更するとデシリアライズできなくなる、可能性があります。ScheduleDataクラスの実装については、その点も十分に配慮する必要があります。
スケジュールのように、長期に渡ってデータを蓄積し運用するモノであれば、わたしならシリアライズの機能を使ってデータを保存しようとは思いません。100%思いません。どうしてもヒトが読める形のファイルで管理したいのであれば、JSON形式やxml形式を選びます。けれどスケジュール管理なのであれば、間違いなくデータベースを選択します。

[ryu-bamboo]

ありがとうございます！実はシリアライズが危険という話は聞いたことがあるのですが、それ以外の方法を知らなかったので、簡単なシリアライズを使おうと思っていたんです。
せっかくの機会ですのでデータベースを勉強してみたいと思います、本当にありがとうございます！

Answer 2

JavaFXは、まったく関係ない部分のおはなしです。

デシリアライズで複数のオブジェクトを読み取りたい。

1. 複数のオブジェクトを読み取りたいのであれば、まずは、複数のオブジェクトを書き出す必要がある。
2. あなたの実装では、FileOutputStreamを追記状態で作成して、ObjectOutputStreamに渡している。
3. https://docs.oracle.com/javase/jp/8/docs/api/java/io/ObjectOutputStream.html のサンプルでは、ファイルを追記状態ではなく新規で開いてObjectOutputStreamに渡している。

      FileOutputStream fos = new FileOutputStream("t.tmp");
      ObjectOutputStream oos = new ObjectOutputStream(fos);

      oos.writeInt(12345);
      oos.writeObject("Today");
      oos.writeObject(new Date());

      oos.close();

1. 複数のオブジェクトを読み取りたいのであれば、複数のオブジェクトをwriteXX()で書き出して、その順番通りにreadXX()すればいい。

import java.io.*;
import java.time.LocalTime;
import java.util.UUID;

public class HogeSerialize {
  public static void main(final String[] args) throws Exception {
    final File file = new File("/tmp/xxx");
    file.delete();

    try (final ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(file))) {
      for (int i = 0; i < 10; i++) {
        switch (i % 3) {
          case 0:
            oos.writeObject(Long.valueOf(System.nanoTime()));
            break;
          case 1:
            oos.writeObject(LocalTime.now());
            break;
          case 2:
            oos.writeObject(UUID.randomUUID());
            break;
        }
      }
    }

    try (final ObjectInputStream ois = new ObjectInputStream(new FileInputStream(file))) {
      while (true) {
        final Object obj = ois.readObject();
        System.out.println(String.format("debug: clz=%s, obj=%s", obj.getClass(), obj));
        if (obj instanceof LocalTime) {
          // ...
        } else if (obj instanceof Long) {
          // ...
        } else if (obj instanceof UUID) {
          // ...
        }
      }
    } catch (EOFException e) {
      ; // 無視する
    }
  }
}