Collections (De)-Serialisierung meines HashSets

[Tsunami]

Hallo,

ich stehe momentan vor einem Problem, dass mich wirklich zur Verzweiflung bringt und ich habe leider überhaupt keinen Ansatz mehr.

Ich möchte ein kleines, selbst lernendes Schachspiel entwickeln (mehr oder weniger zu Übungszwecken) die eigentliche Spielsteuerung funktioniert soweit auch. Momentan haperts am "Dazulernen". Das Verfahren beruht darauf, dass eine Spielsituation gespeichert wird, die zum Verlust geführt hat um diese in Zukunft zu vermeiden.

Das Grundverfahren soll so laufen:

Ich habe ein Objekt vom Typ Situation (mehrdimensionales Array mit int-Werten), und ein Objekt von Memory-Einträgen (int-Wert für den Spieler schwarz/weiß, Objekt der Spielsituation). Diese Memory-Einträge serialisiere und deserialisiere ich, damit die gelernten Situationen auch nach dem Programmstart noch verfügbar sind.

Das Problem dabei ist: alle Einträge, die durch das Deserialisieren geladen werden, werden nur teilweise korrekt verarbeitet. Der Wert für den Spieler wird korrekt eingelesen, die Spielsituation allerdings nicht, dort wird überall nur eine 0 wiedereingelesen.

Leider kann ich nicht herausfinden, ob das Problem bei der Serialisierung oder der Deserialisierung liegt, weil ja eins vom anderen abhängt.

So, nun genug geschrieben, hier kommt der Code. Entscheidend sollten storeMemory() und restoreMemory() sein, dennoch der komplette Zusammenhang:

import java.util.*;
import java.io.*;

public class Memory
{	
	static HashSet<MemoryEntry> BannedSituations;
		
	static void resetMemory()
	{
		BannedSituations = new HashSet<MemoryEntry>();
	}
	
	public static void addSituation(Situation s, int player)
	{
		if(BannedSituations.add(new MemoryEntry(s, player)))
			System.out.println("Banned Situation added!");
	}

	static void storeMemory()
	throws IOException
	{
		FileOutputStream fos = new FileOutputStream("memory.ser");
		ObjectOutputStream oos = new ObjectOutputStream(fos);
		oos.writeObject(BannedSituations);
		oos.close();		
	}
		
	@SuppressWarnings("unchecked")
	static void restoreMemory()
	throws IOException, ClassNotFoundException
	{
		FileInputStream fis = new FileInputStream("memory.ser");
		ObjectInputStream ois = new ObjectInputStream(fis);
		BannedSituations = ((HashSet<MemoryEntry>) ois.readObject());
		ois.close();
	}
	
	public static boolean isBanned(Situation s, int player)
	{
		if(BannedSituations.contains(new MemoryEntry(s, player)))
			return true;
		return false;
	}
}

class MemoryEntry implements Serializable
{	
	private static final long serialVersionUID = 1L;
	
	Situation BannedSituation;
	int player;
	
	MemoryEntry(Situation s, int player)
	{
		this.BannedSituation = s;
		this.player = player;
		//this.print();
	}
	
	@Override
	public boolean equals(Object obj){
		MemoryEntry entry;
		if(obj instanceof MemoryEntry)
			entry = (MemoryEntry) obj;
		else
			return false;

		if(this.player != entry.player)
			return false;
		
		if(this.BannedSituation.equals(entry.BannedSituation) == false)
			return false;
		
		return true;
	}
	
	@Override
	public int hashCode() {
		int result = 17;
		result = 31 * result + this.BannedSituation.hashCode();
		result = 31 * result + this.player;
		return result;
	}
}

import java.io.Serializable;

import NM.*;

public class Situation extends SituationNM implements Serializable
{
	private static final long serialVersionUID = 1L;
	
	public Situation()   // Konstruktor zur Initialisierung 
	{
		feld = new int[][] {{1,1,1,1},{0,0,0,0},{0,0,0,0},{2,2,2,2}};
			   // Ausgangssituation f¸r das Spiel
	}
	
	@Override
	public boolean equals(Object obj){
		Situation sit;
		if(obj instanceof Situation)
			sit = (Situation) obj;
		else
			return false;

		for(int i = 0; i < 4; i++){
			for(int y = 0; y < 4; y++)
				if(this.feld[i][y] != sit.feld[i][y])
					return false;
		}
		
		return true;
	}
	
	@Override
	public int hashCode() {
		int result = 17;
		result = 31 * result + this.feld.hashCode();
		return result;
	}
}

Ich hoffe mir kann jemand auf die Sprünge helfen, inzwischen würden mich sogar Lösungen erfreuen.

Vielen Dank im Voraus und schöne Pfingsttage

Tsunami

 

[Volvagia]

Was mir auf Anhab auffällt ist, dass du flush vom ObjectOutputStrem nicht aufrufst. Versuch erstmal das.

 

[eRaaaa]

Was mir auf Anhab auffällt ist, dass du flush vom ObjectOutputStrem nicht aufrufst. Versuch erstmal das.

Er ruft close() auf, da wird automatisch geflusht, das sollte also nicht das Problem sein! Der Rest ist etwas undurchsichtig. Wie hast du feld in der Oberklasse deklariert usw. Wie testest du das Ganze?

	public static void main(String[] args) throws Exception {
		Situation s = new Situation();
		s.feld = new int[][]{{1,2,3},{4,5,6},{7,8,9},{10,11,12}}; //testweise feld = public
		Memory.resetMemory();
		Memory.addSituation(s, 3);
		Memory.addSituation(new Situation(), 1); //fuellen mit def werten
		Memory.addSituation(new Situation(), 2); //fuellen mit def werten
		Memory.storeMemory();
		Memory.resetMemory();
		Memory.restoreMemory();
		for (MemoryEntry e : Memory.BannedSituations) {
			int[][] situation = e.BannedSituation.feld;
			for (int[] is : situation) {
				System.out.println("Player "+e.player+" "+Arrays.toString(is));
			}
			System.out.println("-----");
		}
	}

liefert mir

Banned Situation added!
Banned Situation added!
Banned Situation added!
Player 1 [1, 1, 1, 1]
Player 1 [0, 0, 0, 0]
Player 1 [0, 0, 0, 0]
Player 1 [2, 2, 2, 2]
-----
Player 2 [1, 1, 1, 1]
Player 2 [0, 0, 0, 0]
Player 2 [0, 0, 0, 0]
Player 2 [2, 2, 2, 2]
-----
Player 3 [1, 2, 3]
Player 3 [4, 5, 6]
Player 3 [7, 8, 9]
Player 3 [10, 11, 12]
-----

Also wird die Situation, sowie der int für den Player korrekt deserialisiert. Also, wie erstellst du die Situations-Objekte etc.?! Vlt. wird es dann etwas klarer

 

[Tsunami]

Vielen Dank ersteinmal für die Antworten. Mit dem Codeschnipsel, den Du geschrieben hast, eRaaaa funktioniert es bei mir auch. Das Problem schien daran zu liegen, dass ich mit nur einer Situation gearbeitet habe, die ich dann durch eine Methode inhaltlich verändert habe.

		System.out.println("Memory reset!"); Memory.resetMemory();
		System.out.println("Memory restore!"); Memory.restoreMemory();

		int[][] f0 = new int[][] {{1,1,1,1},{0,0,0,0},{0,0,0,0},{2,2,2,2}};
		int[][] f1 = new int[][] {{1,1,1,0},{0,0,1,0},{0,0,0,0},{2,2,2,2}};
		int player = 1;
		Situation S = new Situation();
		
		S.change(f0);
		Memory.addSituation(S, player);
		S.change(f0);
		Memory.addSituation(S, player);

		System.out.println("Memory store!"); Memory.storeMemory();
		System.out.println("Memory reset!"); Memory.resetMemory();
		System.out.println("Memory restore!"); Memory.restoreMemory();

        for (MemoryEntry e : Memory.BannedSituations) {
            int[][] situation = e.BannedSituation.feld;
            for (int[] is : situation) {
                System.out.println("Player "+e.player+" "+Arrays.toString(is));
            }
            System.out.println("-----");
        }

Momentan siehts erstmal so aus als würde es mit neuen Objekten funktionieren - Danke!

Hat evtl. auch noch jemand die überschreibende hashCode()-Methode für Arrays, die, sofern jeder Wert an der gleichen Position gleich ist, auch einen gleichen hashCode zurück liefert? Sonst wird 1. das hashSet mit gleichen Situationen gefüllt und 2. die gebannte Situation nicht erkannt.

Leider habe ich da keinen weiteren Ansatz als den im ersten Post verfügbaren in der Situation-Klasse, der im Nachhinein sogar für mich sinnlos erscheint.

Ich gehe mal davon aus, dass das ein Problem ist, für das es inzwischen eine feste Lösung gibt. Sollte es komplexer sein eröffne ich dafür natürlich auch gerne einen neuen Thread.

Gruß Tsunami