alternativen zu if .. else ..

[hubertk]

Hallo zusammen,

ich lerne gerade Java und habe folgende Frage:

Ich würde gerne zu einem code in Form eines Strings die dazugehörige Nachricht zurückgeben ...
da ich aber vor habe viele Nachrichten zur Auswahl zu haben, ist mir das mit else if ... etwas zu unübersichtlich und ich würde das gerne anders lösen.
Was für Möglichkeiten habe ich ?
evtl. mit einer enum oder einer HashMap ?

private String getErrorMessage( String code )
{
	if (code.equalsIgnoreCase("100")) {
	   return "Die entsprechende Fehlermeldung zu 100";
	else if (code.equalsIgnoreCase("200")) {
	   return "Die entsprechende Fehlermeldung zu 200";
	} else if ......
        usw.
}

vielen dank schon mal

Hubert

 

[Aldimann]

Weiß ja nicht was du vor hast, aber kannst du deinen String nicht gegen einen integer ersetzen?
Dann könntest du eine Switch abfrage machen:

		int abc = 100;
		String fehlermeldung;

		switch (abc) {
		case 100:
			fehlermeldung = "Fehlermeldung zu 100";
			break;
		case 200:
			fehlermeldung = "Fehlermeldung zu 200";
		default:
			break;
		}

Außerdem ist .equalsIgnoreCase nur dafür da, dass er die Groß- und Kleinschreibung ignoriert, macht also bei Zahlen keinen Sinn (ich seh ihn jedenfalls nicht, bitte verbessert mich ).
Ein normales .equals sollte da auch reichen...

 

[Gastredner]

Du könntest z. B. eine Map verwenden:

private Map<String,String> codes;

/*
 * Diese Methode einmalig - z. B. im Konstruktor deiner Klasse - aufrufen.
 */
private void initCodes() {
	codes = new HashMap<String,String>();
	codes.put("100", "Die Fehlermeldung zu 100");
	codes.put("200", "Die Fehlermeldung zu 200");
	codes.put("300", "Die Fehlermeldung zu 300");
	// Und so weiter...
}

/*
 * Sollte der String code in deiner Map nicht als Key vorhanden sein,
 * so gibt diese Methode null zurück.
 */
private String getErrorMessage(String code) {
	return codes.get(code);
}

 

[hubertk]

ok, vielen Dank.
Also : equalsIgnoreCase("100") macht ja tatsächlich dann keinen Sinn !

und die Lösung mit der Map finde ich super. das werde ich versuchen!
Vielen Dank!

 

[Landei]

Anderer Vorschlag:

enum ErrorCode {
E100("100","Fehler 100"),E200("200","Fehler 200",E4711("4711","Zuviel Kölnisch Wasser");
private code;
private description;
private ErrorCode(String code, String description) {
   this.code = code;
   this.description = description;
}
public String getCode() { return code; }
public String getDescription() { return description; }
public static ErrorCode fromCode(String code) {
   for(ErrorCode ec : values()) {
       if(ec.code.equals(code)) return ec; 
   }
   return null;
} 
}

 

[Noctarius]

Was bin ich froh wenn mit Java 7 endlich Strings im Switch/Case funktionieren

 

[maki]

Bloch schlägt in seinem Buch eine Map(statisch initialisiert) & statische Methode vor, um von den Werten auf die Instanz zu kommen.

 

[hubertk]

die ich dann wie folgt anlege :

	public static HashMap<String,String> ErrMessages = new HashMap<String,String>();

kann ich die denn bei der deklaration schon mit werten füllen?
also z.B. mit einem 2 dim. array mit key und value werten ?

 

[0x7F800000]

"direkt" bei der initialisierung eher nicht, aber du kannst eine zeile weiter in einem

static{
   ...
}

-block alles machen was du willst, unter anderem auch die map füllen.

 

[hubertk]

maki ,

Bloch schlägt in seinem Buch eine Map(statisch initialisiert) & statische Methode vor, um von den Werten auf die Instanz zu kommen.

hmm, warum eigentlich statisch ?
macht doch nur dann einen sinn, wenn ich von anderen klassen, bzw. objecten auf diese map zugreifen will, oder ?

 

[Noctarius]

Nein, so kannt du auch sicherstellen, dass eventuelle (möglicherweise ausversehens) erstellte Instanzen alle die gleichen Daten sehen.

 

[hubertk]

ahh, jetzt hat es klick gemacht.

vielen dank !