Serializeで複数のオブジェクトを保存したい。

Question

自分は、今、javaFXでスケジュール表を作っていて、一度実行して追加した予定を、実行を終了してからまた実行したときにも、前に追加した予定が表示されるように、Serializeを使ってファイルに保存できるようにしました。

ここで問題があって、保存されるオブジェクトは、直前に実行された、一番最後のオブジェクトだけで、他のオブジェクトが表示されないのです。そこで、オブジェクトを保存するファイルを見てみると、（文字化けしていてはっきりとはわからないのですが）一つのオブジェクトしか保存されていないみたいでした。

どうすれば複数のオブジェクトを保存できるのでしょうか。教えてください！

下のcodeは
1.ScheduleMainがMainクラスで、実行時にファイルを読み込む。
2.ScheduleTableControllerでスケジュール表の本体を表示
3.ScheduleSelectControllerで予定を追加し、追加するごとにSaveDataFile.serにオブジェクトを書き込む

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

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();
			
			try {
				if(!isEmpty()) {
					initScheduleLabel(loading());
				}
			} 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;
    }
	
	ScheduleData loading() {
    	try (var in = new ObjectInputStream(new FileInputStream("SaveDataFile.ser"))){
    		var saveData = (ScheduleData)in.readObject();
    		return saveData;
    	}catch (Exception e) {
			return null;
		}
	}
	
	@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'.";
       
    }
	
}

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"))){
	    		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 vgswgsr 
java.time.Ser評┝"Hｲ  xpwsq ~ w
ﾈxt 
01:55・・3:07t wservgwseag

一回目

二回目

Answer 1

FileInputStreamのコンストラクタにはappendという第二引数があって、そこをtrueにすると、writeしたときに前の書き込みを上書きせずに続きから書き込んでくれるみたいでした。

ObjectOutputStreamは、追記されることを想定してはいないとおもうので、追記できたようにみえても、読み出すことはきっと単純ではありませんよ。

https://docs.oracle.com/javase/jp/8/docs/api/java/io/ObjectOutputStream.html をじっくりと読んでみてはいかがかと。

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

public class HogeSerialize {
  public static void main(final String[] args) throws Exception {
    final File file = new File("/tmp/xxx");
    file.delete();
    for (int i = 0; i < 10; i++) {
      final ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(file, true));
      oos.writeObject(LocalTime.now().toString());
      oos.close();
    }

    final ObjectInputStream ois = new ObjectInputStream(new FileInputStream(file));
    for (int i = 0; i < 10; i++) {
      System.out.println(ois.readObject());
// java.io.StreamCorruptedException: invalid type code: AC 
    }
  }
}

Comments

[ryu-bamboo]

そうなんですか！わかりました、じっくり読んでみます。

Answer 2

一つのオブジェクトしか保存されていないみたいでした。

保存しているのはここ。

        void save() {
            try (var out = new ObjectOutputStream(new FileOutputStream( "SaveDataFile.ser"))){
                out.writeObject(addData());
            }catch (IOException e) {
                e.printStackTrace();
            }
        }

addData()が返すのは、ScheduleDataのインスタンスひとつだけ。

　　　　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;
        }

オブジェクトをひとつだけしか保存していないのですから、「一つのオブジェクトしか保存されていないみたい」なのです。あなたが書いたとおりの動作です。

複数のScheduleDataを保存したいのであれば、配列なりListなりMapなりの形で保存すればよいのではないでしょうか。

Comments

[ryu-bamboo]

回答ありがとうございました！！ですがすみません、質問後に調べてると、FileInputStreamのコンストラクタにはappendという第二引数があって、そこをtrueにすると、writeしたときに前の書き込みを上書きせずに続きから書き込んでくれるみたいでした。もっと調べてから質問するべきでした…考えていただいてありがとうございました！

それと、保存することはできたのですが、今度は読み込むときにエラーが起きてしまってどうしようもないので質問させてください…