ist das wirklich ein fehler?

[darkeye2]

Hallo, hab ne einfache swich case anweisung und eclipse zeigt es mir als fehler an, dass nach jeder case anweisung ein break steht :

protected String genreToString(byte zahl){
		int z = zahl;
		switch (z){
			case 0:
				return "Blues";
				break;
			case 1:
				return "Classic Rock";
				break;
			case 2:
				return "Country";
				break;
			case 3:
				return "Dance";
				break;
			case 4 :
				return "Disco";
				break;
			case 7:
				return "Hip-Hop";
				break;
			case 8:
				return "Jazz";
				break;
			case 9:
			case 22:
				return "Metal";
				break;
			case 12:
				return "Other";
				break;
			case 13:
				return "Pop";
				break;
			case 15:
				return "Rap";
				break;
			case 16:
				return "Reggae";
				break;
			case 17:
				return "Rock";
				break;
			case 18:
				return "Techno";
				break;
			case 20: 
				return "Alternative";
				break;
			case 24:
				return "Soundtrack";
				break;
			case 31:
				return "Trance";
				break;
			case 32:
				return "Klasik";
				break;
			case 33:
				return "Instrumental";
				break;
			case 43:
				return "Punk";
				break;
			case 78:
				return "Rock´n Roll";
				break;
			case 79:
				return "Hard Rock";
				break;
			default:
				return "Other";
				break;
		}
	}

 

[eRaaaa]

naja, eine Zeile vorher steht ja schon jedes mal return ;D Dadurch wird die Methode direkt beendet...macht ja also keinen Sinn danach ein break; einzufügen ^^

 

[darkeye2]

achso, habs ganz vergessen, das return auch immer am ende steht ... hab mich schon so dran gewöhnt immer break; zu schreiben^^
Danke für die schnelle hilfe... gibts dazu eigentlich ne einfach lösung,also ne switch case anweisung ? Ich kireg immer eine zahl überliefert und brauche den dazugehörigen string.

MfG
darkeye

 

[SlaterB]

und wichtig:
in kleinen Schritten üben,
programmiere erst

protected String genreToString(byte zahl){
        int z = zahl;
        switch (z){
            case 0:
                return "Blues";
                break;
            case 1:
                return "Classic Rock";
                break;
            default:
                return "Other";
                break;
        }
    }

lasse das kompilieren, testen usw.,
da wäre dir dieser Fehler oder jedes vergessene Semikolon auch aufgefallen

ich hoffe du hast die ca 5000 anderen return-break-Kombinationen weitgehend nur kopiert statt alles einzeln einzutippen,
dennoch darfst du nun jedes einzelne case korrigieren..


> gibts dazu eigentlich ne einfach lösung,also ne switch case anweisung ?

kürzer geschrieben wäre evtl. ein Array, wenn du nicht so große Lücken wie 43 zu 78 hättest,
ne Map geht auch noch,
aber switch macht sich dafür eigentlich gut

 

[eRaaaa]

achso, habs ganz vergessen, das return auch immer am ende steht ... hab mich schon so dran gewöhnt immer break; zu schreiben^^
Danke für die schnelle hilfe... gibts dazu eigentlich ne einfach lösung,also ne switch case anweisung ? Ich kireg immer eine zahl überliefert und brauche den dazugehörigen string.

MfG
darkeye

Verstehe ich jetzt nicht? Eig. genau so wie du das da auch schon hast, nur halt ohne break; ^^
Besser wäre vllt. eine Enumeration zu benutzen o.ä...aber funktionieren sollte das doch auch?!

 

[Landei]

Ich würde in solchen Fällen zu einer Map raten. Wenn es tatsächlich mal zuviele Einträge werden, kann man eine Map problemlos aus einer Textdatei oder Datenbank füllen, mit deinem switch ist dann Ende im Gelände. Außerdem lässt sich der Inhalt einer Map zur Laufzeit verändern, aber bei einem switch ist alles "hartcodiert".

(ungetestet)

class X {
   private static Map<Integer, String> map = new HashMap<Integer, String>();
   static {
      //das hier könnte man auch aus einer Text-Datei oder so lesen
      map.put(0, "Blues");  
      map.put(1, "Classic Rock");  
      map.put(2, "Polka-Punk");  
   }

   protected String genreToString(int zahl){
      return map.containsKey(zahl) ? map.get(zahl) : "Other"; 
   }

}

 

[darkeye2]

Danke für die tipps, werde mal schauen, ob ich das mit der Map ausprobiere, swich case funktioniert zwar soweit, aber irgendwie sieht das mit der map besser aus^^.
@SlaterB ja, öffter testen wäre net schlecht, aber ein paar break; zu löschen geht relative schnell.