Klassen-konstanten durchlaufen

[Painii]

Hi,
ich hab in einer Klasse 7 Konstanten:

public class Stein{
        public static final int STEIN_L = 1;
	public static final int STEIN_L_REVERT = 2;
	public static final int STEIN_N = 3;
	public static final int STEIN_N_REVERT = 4;
	public static final int STEIN_LINE = 5;
	public static final int STEIN_BLOCK = 6;
	public static final int STEIN_TRIANGLE = 7;

Ich würde jetzt gerne im Konstruktor ein Argument überprüfen ob es gleich einer Konstanten ist -> evtl. möchte ich aber später noch andere Konstanten hinzufügen, also wär ich mit einem Konstruktor der Art wie unten nicht so zufrieden...

 public Stein(int art){
  if(art==STEIN_L||art==STEIN_L_REVERT||...){ //wenn art== einer konstante
  }else{
   throw new IllegalArgumentException(); //art != jeder konstante
  }
 }
}

Gibt es da eine Möglichkeit von java aus direkt durch die Feldkonstanten zu gehen?
also z.b. so:

for(Konstante k: Stein)

ohne dass ich die konstanten in einem eigenen array hab?

 

[Der Müde Joe]

mach ne enum....

 

[The_S]

du könntest über Reflection gehen. Oder du machst es sauber und verwendest Enumerations.

 

[tfa]

Klar, mit Reflection ginge das.
Aber dieses Problem (wenn ich es richtig interpretoiert habe) würde ich eher mit Enums lösen. Das sind sozusagen typsichere Konstanten. Da musst du dann nichts mehr prüfen.

 

[Painii]

Hmm ok, Enums sehen ja schon ganz gut aus...
Hab mir jetzt die Enum so geschrieben:

public enum Stein_art {
	STEIN_L,STEIN_L_REVERT,STEIN_N,	STEIN_N_REVERT,STEIN_LINE,STEIN_BLOCK,STEIN_TRIANGLE ;
}

und den Konstruktor dan so gemacht:

public Stein(int art){
		boolean found=false;
		for(Stein_art a:Stein_art.values())if(art==a.ordinal())found=true;
		if(found){
			this.art=art;
		}
		else{
			throw new IllegalArgumentException("Nur Konstanten aus Stein");
		}

Probleme macht mir jetzt aber woanders mein code -> hier hab ich ein switch-case konstrukt mit art...

switch(art){
		case Stein_art.STEIN_L.ordinal(): // arbeite
		case Stein_art.STEIN_L_REVERT.ordinal(): //arbeite anders
}

Aber da krieg ich den Fehler:
"case expressions must be constant expressions"

 

[The_S]

Du kannst einfach über Enums switchen. Brauchst nicht über die Zahl gehen.

Java Blog Buch : 06.05 Enumerations

 

[Der Müde Joe]

zudem:

public Stein(SteinArt art){

 

[Gast2]

Moin,

erstmal kanst Du es - der Übersichtlichkeit wegen - untereinander schreiben

public enum Stein_art {
    STEIN_L,
    STEIN_L_REVERT,
    STEIN_N,
    ....
    STEIN_TRIANGLE ;
}

und dann kannst Du noch das STEIN weglassen ... Dein Enum sagt das ja schon aus

public enum Stein_art {
    L,
    L_REVERT,
    N,
    ....
    TRIANGLE ;
}

Stein_art.L

hand, mogel

 

[Painii]

Ah ok...

Ich hatte art noch als normales int, das hab ich jetzt in die Enum geändert -> es geht

Wieder ein neues Kapitel java gerlernt, danke für eure Aufmerksamkeit

 

[Der Müde Joe]

>erstmal kanst...

und wenn wir schon dabei sind... In Java benutzt man bevorzugterweise CamelCase
also

Klasse, Enum, IF:
SteinArt
StringBuffer
InputStream

Methoden:
getSteinArt()
queenOfTheStoneage()

Variablen:
int bestRandom =
SteinArt weakSofties