Erste Schritte Falscher case wir ausgeben

[jofri]

Hallo zusammen,

ich beschäftige mich gerade mit einem Java Buch und bin nun bei den Bedingungen angekommen. Doch irgendwie wird mir immer der letzte case ausgeben obwohl ich eigentlich ein anderen wähle. Hier einmal der Code:

package paket1;
import javax.swing.*;
public class Klasse1 {

    public static void main(String[] args)
    {
        String Note = JOptionPane.showInputDialog
                ("Gib eine Schulnote ein");
       
        int Wert = Integer.parseInt (Note);
        switch (Wert)
        {
        case 1: Note = "Sehr gut";
        case 2: Note = "Gut";       
        case 3: Note = "Befriediegend";
        case 4: Note = "Ausreichend";
        case 5: Note = "Mangelhaft";
        case 6: Note = "Ungenügend";       
        }
        JOptionPane.showMessageDialog(null, Note);
       
        }

    }

Was mach ich falsch?

Danke schonmal im Vorraus,

jofri

 

[Blender3D]

{
         case 1: Note = "Sehr gut";
                    break;
         case 2: ...

Du hast das break nach den cases vergessen!

 

[Meniskusschaden]

Du mußt die einzelnen case-Zweige noch mit break abschliessen, denn sonst werden alle auf die erste zutreffende case-Bedingung folgenden Zweige ausgeführt. Angenommen Wert enthält die Zahl 4. Dann wird Note erst auf Ausreichend gesetzt, dann mit Mangelhaft überschrieben und schließlich nochmals mit Ungenügend überschrieben.

 

[Xyz1]

Machen wir das mal zu 100 % korrekt:

    private static String noteToString(int note) {
        String rueck = "";
        switch (note) {
            case 1:
                rueck = "Sehr gut";
                break;
            case 2:
                rueck = "Gut";
                break;
            case 3:
                rueck = "Befriediegend";
                break;
            case 4:
                rueck = "Ausreichend";
                break;
            case 5:
                rueck = "Mangelhaft";
                break;
            case 6:
                rueck = "Ungenügend";
                break;
            default:
                throw new AssertionError();
        }
        return rueck;
    }

    public static void main(String[] args) {
        System.out.println(noteToString(5));
    }

Ausgabe:

Mangelhaft

Entspricht aber nicht meiner Note, also Realität falsch Abgebildet.

 

[Blender3D]

Da Du nur konstante Strings zurückgibts, wäre hier die bessere Lösung ein Array zu verwenden.

public static String noteToString(int note) {

         if( note <=0 || note >6  ) return null; // oder werfe exeption
         String [] noten = {"Sehr gut","Gut","Befriediegend","Ausreichend", "Mangelhaft","Ungenügend"};
        return noten[note-1];
     }

 

[jofri]

Vielen Dank jetzt funktioniert es mit den Breaks funktioniert es @DerWissende wozu dieses rueck? Ist das ein Leerzeichen? @Blender3D Okay, das mit dem case war nur ne Übung aber ich probiere deine Lösung trotzdem mal aus

 

[Xyz1]

Jau, das ist die amerik. Abkürzung für ein whitespace.

 

[Xyz1]

Edit-time abgelaufen. Das ist natürlich nicht die Abkürzung für ein Leerzeichen, sondern bedeutet einfach nur Rueckgabe, das ist einfach nur ein Spaß.

 

[Blender3D]

Entspricht aber nicht meiner Note, also Realität falsch Abgebildet.

Wäre dann das richtig ?

public static void main(String[] args) {
         System.out.println(noteToString(6));
     }

 

[Xyz1]

Jow, das ist fast die Differenz zwischen Deiner und meiner Note. Blende hier mal nicht so rum.

 

[Jardcore]

Statt der ganzen breaks kann man auch einfach gleich den String zurückgeben.
Bei der Array Lösung jedesmal ein neues Array anzulegen ist auch nicht das gelbe vom Ei

Würde als optimale Lösung die von derWissende nehmen und die rueck variable rauskicken. Dafür

switch(note) {
    case 1: return "sehr gut";
    case 2: return "gut";
    //...
    default: throw IllegalArgumentException("Für die Note " + note + " ist kein Text hinterlegt"); // oder was auch immer man will
}

 

[Flown]

Diskussion wurde hierhin verschoben: http://www.java-forum.org/thema/performance-diskussion-array-enum-hashmap-etc.172186/