switch - Problem

[Luk10]

Hallo

Ich wollte mal switch ausprobieren und habe aber folgendes problem .. wenn ich die methode 3 mal aufrufe wird mir immer 0 ausgegeben ... was ist das falsch?

int zahl = 0;
        switch (zahl) {
            case 0: 
              System.out.println ("0");
              erntezahl++;
              break;
            case 1:
              System.out.println ("1");
              erntezahl++;
              break;
            case 2:
              System.out.println ("2");
              erntezahl++;
              break;
            case 3:
              System.out.println ("3");
              break;
            }
        }
    }

 

[nickname]

hi, wenn du der Variablen

int zahl = 0;

nichts anderes zuweisst, wird immer 0 aufgerufen. switch läuft bei dir nicht
durch, da du eine break-Anweisung stehen hast. Damit bricht es bei 0 ab. Schreib
doch mal

int zahl = 2;

, dann wirst du sehen was passiert. Oder lass doch mal alle break´s weg, dann hat switch
durchfall...

Besser so

public void ausgabe(int i){
int zahl = i;
        switch (zahl) {
            case 0: 
              System.out.println ("0");
              erntezahl++;
              break;
            case 1:
              System.out.println ("1");
              erntezahl++;
              break;
            case 2:
              System.out.println ("2");
              erntezahl++;
              break;
            case 3:
              System.out.println ("3");
              break;
            }
        }

ist natürlich nur auf bestimmte Zahlen festgelegt. Ist halt einfach nur einfach

gruß nickname

 

[Leck0rkuchen]

Hoi Leute hab da mal ein Problem.

Ich hab eine Aufgabe bekommen einen einfachen "Ticketautomaten" zu schreiben. Bin recht lang dran gesessen weil ich nicht wusste ob ich switch oder if - else if - else "falsch verwendet habe".
Das Programm ist fertig und funktioniert auch, aber es sieht echt umständlich aus obwohl die Aufgabe einfach scheint. Nun will ich fragen ob ich das besser hätte machen können.

Der Code:

public class Ticket 
{
	public static void main(String[] args)
	{
		System.out.println("1 = Einzelfahrkarte, 2 = Wochenkarte, 3 = Monatskarte.");
		System.out.println("Welchen Kartentyp möchten Sie: ");
		int karte = Eingabe.IntZahl();
		System.out.println("Wählen Sie 1 oder 2 Zonen: ");
		int zone = Eingabe.IntZahl();
		String kartenTyp = "";
		String zoneTyp = "";
		double preis = 1.80;
		
		switch (karte)
		{
		case 1: kartenTyp = "Einzelfahrkarte"; break;
		case 2: kartenTyp = "Wochenkarte"; break;
		case 3: kartenTyp = "Monatskarte"; break;
		default: System.out.println("FEHLER");
		}
		switch (zone)
		{
		case 1: zoneTyp = "Kurzstrecke"; break;
		case 2: zoneTyp = "Normalstrecke"; break;
		default: System.out.println("FEHLER");
		}
		
		if ((karte == 1) && (zone == 1))
		{
			System.out.println(kartenTyp + " " + zoneTyp + ": " + preis+" €");
		}
		else if ((karte == 1) && (zone == 2))
		{	
			System.out.println(kartenTyp + " " + zoneTyp + ": " + preis*2+" €");
		}
		else if ((karte == 2) && (zone == 1))
		{
			System.out.println(kartenTyp + " " + zoneTyp + ": " + preis*7+" €");
		}
		else if ((karte == 2) && (zone == 2))
		{
			System.out.println(kartenTyp + " " + zoneTyp + ": " + (preis*2)*7+" €");
		}
		else if ((karte == 3) && (zone == 1))
		{
			System.out.println(kartenTyp + " " + zoneTyp + ": " + preis*30+" €");
		}
		else System.out.println(kartenTyp + " " + zoneTyp + ": " + (preis*2)*30+" €");
	}
}

Nur so zur belustigung, die Auflösung vom Buch :lol:

class Ticket
{
  public static void main(String[] args)
  {
    int region = 1;
    int art = 2;
    double price = 0.0;

    if ((region == 1) || (region == 2))
    {
      if (region == 1)
      {
        price = 2.00;
      }
      else
      {
        price = 2.80;
      }
      switch (art)
      {
        case 1:  System.out.println(price);
                 break;
        case 2:  System.out.println(price * 7);
                 break;
        case 3:  System.out.println(price * 30);
                 break;
        default: System.out.println("falsche Fahrkartenart" );
      }
    }
    else
    {
      System.out.println("falsche Tarifzone");
    }
  }
}

PS: Mir wurde erst später klar das ich keine Eingaben gebraucht hätte LOL

 

[SlaterB]

was ist an der Buch-Lösung belustigend, die ist doch teils deutlich besser

durch den Block

     if (region == 1)
      {
        price = 2.00;
      }
      else
      {
        price = 2.80;
      }

wird z.B. für alle drei arten der Grundpreis bestimmt während du dafür mehr einzelne ifs brauchst

 

[Sonecc]

Finde auch nichts belustigend an der Lösung.
Nur mal als Beispiel, dort werden gerade mal 5 Vergleiche im Worst Case durchgeführt.
Bei dir sind es im Worst Case aber 9.

Zu viele Switch-Case bzw. zu viele If-Abfragen, für zu wenige Daten

 

[Landei]

Finde ich auch. Man bekommt das natürlich auch kürzer:

  public static void main(String[] args)
  {
    int region = 1;
    int art = 2;
    if ((region == 1) || (region == 2)) {
      double price = (region == 1) ? 2.00 : 2.80;
      if (art < 1 || art > 3) {
        System.out.println("falsche Tarifzone");
      } else {
        System.out.println(price * new int[]{1,7,30}[art-1]);
      }  
    } else {
      System.out.println("falsche Tarifzone");
    }
  }
}

 

[Leck0rkuchen]

Lustig war hier als Selbstkritik zu verstehen.
Lustig finde ich das weil ich nicht wusste das ich keine Eingaben benötige, natürlich ist die Buchlösung total logisch, und ich so umständlich viel geschrieben habe weil ich das Programm "eingabe-fähig" machen wollte. Sprich wenn ich kompiliere und starte nicht nur "Fakten" rauskommen sondern ich gefragt werde welche karte ich will, und welche Zone.


BTW: Wie könnte ich nun MEIN code verbessern? Ich nehme an das zB:

case 1: zoneTyp = "Kurzstrecke"; break;
case 2: zoneTyp = "Normalstrecke"; break;
default: System.out.println("FEHLER");

nicht als Switch ausgeführt werden müsste, nur weiß ich nicht wie ich das mit if-else machn könnte.

 

[Sonecc]

if (karte == 1) {
    kartenTyp = "Einzelfahrkarte";
} else if (karte == 2) {
   kartenTyp = (...)

usw.

Alle Switch-Case Fälle können in äquivalente If-Else Abfragen umgeformt werden.
Ist aber nicht immer sinnvoll.

Dein Problem ist auch nicht, dass du eine Eingabe ermöglichen wolltest, denn die Musterlösung wäre auch mit Eingabe nicht anders.
Man braucht schlicht nicht mehr Abfragen, als in der Lösung

 

[SlaterB]

@Leck0rkuchen
nein, ein solches switch kann man mit einfachen Mitteln nicht besser machen,
nur könntest du in diesem switch gleich noch den Preis mit setzen

case 1: zoneTyp = "Kurzstrecke"; break;
case 2: zoneTyp = "Normalstrecke"; preis *= 2; break;

dann kannst du dir alle if/ else am Ende sparen,

 

[Leck0rkuchen]

Supa Idee danke dir! Bin ja total dämlich case = Anweisung.