Enum durch String auswählen

[sunnyandy]

Hallo!

habe folgende Enum-Klasse geschrieben:

public enum Geschlecht {
	Maennlich("Männlich"), Weiblich("Weiblich");
	
	private final String bezeichnung;
	
	private Geschlecht(String bezeichnung){
		this.bezeichnung = bezeichnung;
	}
	
	public String getBezeichnung(){
		return bezeichnung;
	}
	
	public char getAbkuerzung(){
		return (this == Maennlich)? 'm': 'w';
	}
		
}

Nun möchte ich aus einer ComboBox das ausgewählte Geschlecht eintragen (Es stehen als Einträge "Männlich" und "Weiblich" zur Verfügung).
Das habe ich so gemacht:

student.setGeschlecht(Geschlecht.valueOf(cmbGeschl.getSelectedItem()
				.toString()));

Ich hole mir also den String des ausgewählten Items und lasse mir mit valueOf das passende Geschlecht zurückgeben.

Der Compiler meldet aber: java.lang.IllegalArgumentException: No enum const class model.Geschlecht.Männlich
at java.lang.Enum.valueOf(Unknown Source)

Was mache ich falsch? Geschlecht.valueOf("Männlich") funktioniert auch nicht, muss ich für sowas eine extra Methode schreiben?

Viele Danke schonmal!!
Grüße, Andy

 

[dieta]

Das könnte daran liegen, dass du im enum Maennlich mit ae geschrieben hast, und er nach Männlich mit ä sucht, wobei er natürlich nichts findet.

 

[sunnyandy]

Ja wenn ich "Maennlich" eingebe, dann funktioniert es.

Aber in der Combobox steht ja gerade die Bezeichnung jedes Geschlechts. Also "Männlich" und "Weiblich".

Ich möchte das Enum-Objekt ja durch die Werte bekommen, die als String hinter dem Enum-Namen stehen, wie du unten siehst:

public enum Geschlecht {
   Maennlich("Männlich"), Weiblich("Weiblich");

 

[dieta]

Ach so... dann wirst du dir warscheinlich eine eigene Methode schreiben müssen, die nacheinander die Werte der einzelnen Einträge mit dem Suchwort vergleicht und bei einer Übereinstimmung das entsprechende Objekt ausgibt.

So nebenbei: Benenne doch einfach deine beiden Ojekte um, Java hat keine Probleme mit Umlauten!

 

[sunnyandy]

Hallo, danke für deine Antwort! Das gleich habe ich mir auch gedacht und habe es nun so gemacht:

public static Geschlecht getGeschlecht(String bezeichnung){
		if (bezeichnung == "Männlich") return Maennlich;
		else if (bezeichnung == "Weiblich") {
			return Weiblich;			
		} else
		{
			throw new IllegalArgumentException("Ungültiges Geschlecht");
		}
	}

Nun überprüfe ich selbst alle Strings und gebe den entsprechenden Typen zurück. Blöd finde ich nur, dass ich bereits in der Enum-Klasse die gleichen Strings benannt habe (in Klammern hinter den Objekten) und sie nun nochmal aufzählen muss...

 

[dieta]

Strings muss man mit String.equals(...) vergleichen:

public static Geschlecht getGeschlecht(String bezeichnung){
      if (bezeichnung.equals("Männlich")) return Maennlich;
      else
         return Weiblich;
   }

an sonsten gibt er dir nämlich immer Männlich zurück.

 

[Yzebär]

Das else if würde ich aber nicht weglassen, ansonsten wird ja jeder Schrott weiblich...

 

[FelixB]

Das else if würde ich aber nicht weglassen, ansonsten wird ja jeder Schrott weiblich...

wenn in der ComboBox nur die Möglichkeiten "m" und "w" sind, gibt es da keinen Schrott, der weiblich werden könnte...

 

[hupfdule]

Das else if würde ich aber nicht weglassen, ansonsten wird ja jeder Schrott weiblich...

wenn in der ComboBox nur die Möglichkeiten "m" und "w" sind, gibt es da keinen Schrott, der weiblich werden könnte...

Trotzdem würde ich in diesem Fall ein "else if" und ein zusätzliches "else" einbauen. Das else sollte nie eintreten (wäre dann der Schrott) und schmeißt daher eine Exception. Du kannst dir nie sicher sein, dass durch irgendwelche Programmierfehler nicht auf einmal Schrott in der ComboBox steht. Deswegen solltest du auch die Fälle, die normalerweise nie eintreten sollten, abdecken.