final Instanz

[usrr]

Hallo,

eine Klasse, die verschiedene Daten speichert, wurde als static final Klassenkonstante deklariert. Jetzt ist aber nur die Instanzvariable final und die gespeicherten Werte lassen sich munter ändern. Wie bekommt man es hin, dass alle enthaltenen Daten auch final sind?

public class Test {

  public static final CONST = new Test(1,2);

  int a;
  int b;

  public Test(pa,pb) {
    a=pa;
    b=pb;
  }

  public change() {
    Test test = CONST;

    test.a = 999;
    test.b = 999;
  }
}

 

[AngryDeveloper]

Ein final vor die Instanzvariablen schreiben?
final int a

 

[usrr]

Dann wären aber alle Instanzen der Klasse unveränderbar.

 

[Gonzo17]

Generell indem man jede Variable auch

final

deklariert.

Was ich aber sehr verwirrend finde ist das Konstrukt mit

CONST

und der Methode

change()

. Die Methode

change()

ist NICHT statisch, deswegen erwartet man, dass sich auch etwas ändert, wenn man diese Methode aufruft.

Test instance = new Test(1,2);
instance.change(); // Die Werte für a und b bleiben für instance auf 1 und 2!

Was du *eventuell* machen möchtest ist ein Singleton, schau dir dieses Pattern vielleicht mal an.

 

[usrr]

Das war jetzt nur im Beispiel ungünstig.

Die Klasse hält Punktkoordinaten und ich dachte es wäre schön eine Konstante für die Punkte (0,0),(1,0) und (0,1) zu haben. Ich mach es jetzt mit static methoden .... giveZeroPoint(), giveXPoint() ...

Dann wird halt immer eine neue Instanz erzeugt.

 

[Gonzo17]

Dann würde sich eventuell ein Enum anbieten.

public enum Coordinates {

	ORIGIN(0, 0), POINT1(1, 0);

	double positionX, positionY;

	private Coordinates(double x, double y) {
		positionX = x;
		positionY = y;
	}
}

Wobei ich jetzt generell nicht verstehe wieso man Koordinaten als Konstanten haben möchte? Wenn man eine Klasse

Coordinates

hat und mit

new Coordinates(0,0)

den Nullpunkt bekommt, ist das für mich jetzt nicht der riesige Aufwand.

 

[usrr]

Dann wird aber jedesmal eine neue Instanz erstellt.

 

[Gonzo17]

Ja und wo ist das Problem? Das ist vielleicht der springende Punkt.

 

[usrr]

Die Instanzerzeugung soll kostenintensiv sein.


----

Mit dem Enum geht es auch nicht:

Coordinates test1 = Coordinates.ORIGIN;
		
test1.positionX = 100;

Coordinates test2 = Coordinates.ORIGIN;

// test2.positionX == 100

 

[AngryDeveloper]

Die Instanzerzeugung soll kostenintensiv sein.

Dann lies mal hier: Java theory and practice: Urban performance legends, revisited

 

[bygones]

ganz generell zur Information.

wenn ein Objekt final ist, so ist seine Referenz nicht aenderbar, sein Inhalt schon

final List<String> list = new ArrayList();
list.add("foo"); // geht
list.remove("foo"); // geht
list = new ArrayList(); // geht nicht

Die Instanzerzeugung soll kostenintensiv sein.

das ist im Normallfall unsinn

Mit dem Enum geht es auch nicht:

Coordinates test1 = Coordinates.ORIGIN;
		
test1.positionX = 100;

Coordinates test2 = Coordinates.ORIGIN;

// test2.positionX == 100

wenn du oeffentliche variablen hast brauchst du die nicht auch nicht wundern....

 

[Gonzo17]

Mit dem Enum geht es auch nicht:

Natürlich geht das so nicht. Du müsstest zum Beispiel die Variablen x und y als private deklarieren, hab ich jetzt eben nicht getan. Wenn du aber die Koordinaten ändern willst, dann ist das auch keine Konstante mehr!

Die Instanzerzeugung soll kostenintensiv sein.

Und du willst die Last auf den Anfang schieben? Je nachdem wieviele Konstanten du eben willst wird das nicht so schön, generell find ich aber AngryDevelopers Link nicht schlecht, je nachdem was du machst kannst du auch einfach die Erzeugung von Instanzen reduzieren?