deserialize singleton

[ZebaX]

Hallo,

ich hab folgenden Code

ublic class Properties implements Serializable{

	private static final long serialVersionUID = 895L;
	private static Properties properties;
	private String path = "../fx_Prop/properties.ser";
	private String name;
	
	private Properties() {
		
	}
	
	public static Properties getInstance(){
		if(properties == null)
			properties = new Properties();
		
		return properties;
	}

	public void setName(String name){
		this.name = name;
	}

	public String getName(){
		return this.name;
	}

	public void LoadProperties(){
		try
		{
			FileInputStream fileIn = new FileInputStream(this.getClass().getResource(path).toURI().getPath().toString());
			if(fileIn.available() > 0){
			      ObjectInputStream in = new ObjectInputStream(fileIn);
			      properties = (Properties) in.readObject();
			      in.close();
			      fileIn.close();
			}
			
		}catch(IOException i){
			i.printStackTrace();
			return;
		}catch(ClassNotFoundException c){
			c.printStackTrace();
			return;
		} catch (URISyntaxException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}
	
	private Object readResolve() throws ObjectStreamException { 
		return getInstance(); 
	} 
	
	public void SaveProperties(){
		try
		{
			
			FileOutputStream fileOut = new FileOutputStream(this.getClass().getResource(path).toURI().getPath().toString());
			ObjectOutputStream out = new ObjectOutputStream(fileOut);
			out.writeObject(properties);
			out.close();
			fileOut.close();
		}catch(IOException | URISyntaxException i)
		{
			i.printStackTrace();
		}
	}

	public int hasCode() {
		return 810;
	}


}

Aber jedesmal wenn ich mir den Namen über getName() holen möchte ist dieser null. Kann jemand weiter helfen?

 

[ARadauer]

was soll er stonst sein? Wo setzt du ihn den?

 

[ZebaX]

ich ruf iwo außerhalb Properties.getInstance().setName("Test") auf und anschließend Properties.getInstance().SaveProperties();

und beim erneuten Start vom Programm halt erst Properties.getInstance().LoadProperties(); und anschließend System.out.println(Properties.getInstance().getName());

 

[tagedieb]

Ich wuerd mal die Logik zum lesen und schreiben und das Datenobjekt in 2 Klassen aufteilen und somit die Komplexitaet zu verringern. Am besten hat das Datenobjekt auch keine statischen variablen. Dann sollte es ohne Probleme funktionieren.

Ausserdem sollten methoden kleinGeschrieben werden und

hasCode()

sollte wohl eher

hashcode()

heisen??

 

[ZebaX]

Ich würde das aber nur ungern wollen.

jupp hashCode() sollte das heißen und das mit den Methoden kleinschreiben ist bekannt trägt aber grad nicht wirklich zur lösung des Problems bei

 

[anti-held]

Schmeiß die readResolve Methode aus deiner Klasse und es sollte funktionieren!

Warum hast du diese überhaupt implementiert?

 

[ZebaX]

hab iwo nen paar zeilen code gefunden und da stand die mit bei, wenn ich die rausnehme funktioniert es auch nur wenn ich den code wie folgt abänder:

public String getName(){
		return properties.name;
}

 

[anti-held]

Ach ja was ich dazu noch verbessern würde ist, dass die beiden Methoden load und save statisch sein sollten.
Auch die Variable path.

Dann sollte es auch mit dem this.name funktionieren.

 

[ZebaX]

habs jetzt die methoden und path statisch gemacht aber ändert nix

 

[anti-held]

Also bei mir funktioniert es mit folgendem Code.

public class Properties implements Serializable {
	private static final long serialVersionUID = 895L;
	private static Properties properties;
	private static String path = "properties.ser";
	private String name;

	public static void main(String[] args) {
		// Properties.getInstance().setName("Test2");
		// Properties.SaveProperties();

		Properties.LoadProperties();
		System.out.println(Properties.getInstance().getName());
	}

	private Properties() {
	}

	public static Properties getInstance() {
		if (properties == null)
			properties = new Properties();
		return properties;
	}

	public void setName(String name) {
		this.name = name;
	}

	public String getName() {
		return this.name;
	}

	public static void LoadProperties() {
		try {
			FileInputStream fileIn = new FileInputStream(path);
			if (fileIn.available() > 0) {
				ObjectInputStream in = new ObjectInputStream(fileIn);
				properties = (Properties) in.readObject();
				in.close();
				fileIn.close();
			}
		} catch (IOException i) {
			i.printStackTrace();
			return;
		} catch (ClassNotFoundException c) {
			c.printStackTrace();
			return;
		}
	}

	public static void SaveProperties() {
		try {
			FileOutputStream fileOut = new FileOutputStream(path);
			ObjectOutputStream out = new ObjectOutputStream(fileOut);
			out.writeObject(properties);
			out.close();
			fileOut.close();
		} catch (IOException i) {
			i.printStackTrace();
		}
	}

	public int hasCode() {
		return 810;
	}
}

also mit this.name