Eine Variable in mehreren "switch"

[Der Neuling]

Hallo Leute,

ich habe jetzt nach ein paar Wochen Abwesenheit vom Programmieren wieder Zeit gefunden weiterzumachen. Ziel war dabei ein Programm, dass je nach Höhe des Einkommens und Anzahl der Kinder, die Höhe des Kindergeldes ausgibt. Nun habe ich mit "zahl1" eine Variable für die Einkommenshöhe, die ich dann jeweils auf > 5000 und < 5000 prüfen und so in die entsprechenden Kinderzahlabfragen und Ausgaben kommen möchte.
Jetzt weiß ich nur nicht, wie ich es schaffen kann, dass ich nach der Eingabe der Einkommenshöhe direkt in die jeweilige Funktion springen kann? Derzeit werden beide Abfragen durchlaufen. Ist irgendwie ein bisschen schwierig zu beschreiben, deshalb wußte ich auch nicht nach was ich suchen soll.

import java.io.*;

public class KinderG

     {
        public static void main (String argv []) throws IOException

        {

        String str;

        int zahl1;

        int grenzWert = 5000;
        int Kinder1;
        int Kinder2;
       

        BufferedReader input = new BufferedReader(

        new InputStreamReader(

        System.in));

       System.out.print("Bitte geben Sie Ihr jährliches Bruttoeinkommen ein: ");
       str = input.readLine();
       zahl1 = Integer.parseInt(str);
  
       if (grenzWert < zahl1);
           System.out.println ("Bitte geben Sie die Zahl Ihrer Kinder ein: ");
            str = input.readLine();
            Kinder1 = Integer.parseInt(str);
      
       
            switch (Kinder1)
            {
                case 1: System.out.println ("Sie erhalten 2,50 € Kindergeld"); break;
                case 2: System.out.println ("Sie erhalten 4,00 € Kindergeld"); break;
                case 3: System.out.println ("Sie erhalten 8,00 € Kindergeld "); break;
                default: System.out.println ("Ungültig "); break;
            } 
       
       
       if (grenzWert > zahl1);
       System.out.println ("Bitte geben Sie die Zahl Ihres Kinder ein: ");
       str = input.readLine();
       Kinder2 = Integer.parseInt(str);
       switch (Kinder2)
       {
       case 1: System.out.println ("Sie erhalten 100 € Kindergeld"); break;
       case 2: System.out.println ("Sie erhalten 200 € Kindergeld"); break;
       case 3: System.out.println ("Sie erhalten 400 € Kindergeld "); break;
        
                }
        }
}

Wäre nett, wenn ihr mir einen Tipp geben könntet. Der Text an sich stimmt ja, es fehlt mir nur eine "Trennung" oder?

Viele Grüße und danke für eure Hilfe

Der Neuling

 

[Ark]

if (grenzWert < zahl1);

So was würde Eclipse bei mir dick und fett mit einer Warnung hervorheben: Hier passiert nämlich genau nichts, und diese Zeile hätte man komplett weglassen können, da sie genau gar keinen Einfluss hat.

Grund: die einzige Anweisung, die ausgeführt wird, wenn grenzWert < zahl1 gilt, ist die leere Anweisung: ;

Soll heißen: Ein paar mehr geschweifte Klammern { und } täten deinem Code eventuell ganz gut.

Ark

 

[Der Neuling]

Oh Mist, so schnell kanns gehen, ";" hinter den if (grenzWert....) weg und dafür {} rein und schon klappt es.

NetBeans unterkringelt mir da komischerweise nichts...

Vielen Dank für die schnelle Hilfe und fürs Platz lassen zum selbst nachdenken

 

[hüteüberhüte]

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class KinderG {

    public static void main(String argv[]) throws IOException {
        String str;
        int zahl1;
        int grenzWert = 5000;
        int kinder1;
        int kinder2;
        BufferedReader input = new BufferedReader(new InputStreamReader(System.in));

        System.out.print("Bitte geben Sie Ihr jährliches Bruttoeinkommen ein: ");
        str = input.readLine();
        zahl1 = Integer.parseInt(str);

        if (zahl1 > grenzWert) {
            System.out.print("Bitte geben Sie die Zahl Ihrer Kinder ein: ");
            str = input.readLine();
            kinder1 = Integer.parseInt(str);
            switch (kinder1) {
                case 1:
                    System.out.println("Sie erhalten 2,50 € Kindergeld");
                    break;
                case 2:
                    System.out.println("Sie erhalten 4,00 € Kindergeld");
                    break;
                case 3:
                    System.out.println("Sie erhalten 8,00 € Kindergeld");
                    break;
                default:
                    System.out.println("Ungültig");
                    break;
            }
        } else {
            System.out.print("Bitte geben Sie die Zahl Ihres Kinder ein: ");
            str = input.readLine();
            kinder2 = Integer.parseInt(str);
            switch (kinder2) {
                case 1:
                    System.out.println("Sie erhalten 100 € Kindergeld");
                    break;
                case 2:
                    System.out.println("Sie erhalten 200 € Kindergeld");
                    break;
                case 3:
                    System.out.println("Sie erhalten 400 € Kindergeld");
                    break;
                default:
                    System.out.println("Ungültig");
                    break;
            } // switch
        } // else
    } // main
} // KinderG

Funktioniert bei mir. Variablen und Methoden immer lowerCamelCase, Klassen UpperCamelCase

 

[Landei]

Warum fragst du nicht vor dem if einmal nach der Kinderanzahl statt in jedem Zweig?

        System.out.print("Bitte geben Sie die Zahl Ihrer Kinder ein: ");
        str = input.readLine();
        int kinder = Integer.parseInt(str);
        if (zahl1 > grenzWert) {
            switch (kinder) {
                case 1:
                    System.out.println("Sie erhalten 2,50 € Kindergeld");
                    break;
                case 2:
                    System.out.println("Sie erhalten 4,00 € Kindergeld");
                    break;
                case 3:
                    System.out.println("Sie erhalten 8,00 € Kindergeld");
                    break;
                default:
                    System.out.println("Ungültig");
                    break;
            }
        } else {
            switch (kinder) {
                case 1:
                    System.out.println("Sie erhalten 100 € Kindergeld");
                    break;
                case 2:
                    System.out.println("Sie erhalten 200 € Kindergeld");
                    break;
                case 3:
                    System.out.println("Sie erhalten 400 € Kindergeld");
                    break;
                default:
                    System.out.println("Ungültig");
                    break;
            } 
        }

 

[hüteüberhüte]

NetBeans unterkringelt mir da komischerweise nichts...

Alt+Shift+F ist die Format-Funktion. Wenn er nicht einrückt, stimmt etwas nicht

 

[Der Neuling]

@ Landei deine Frage ist berechtigt...das hatte ich irgendwie nicht auf dem Schirm, guter Gedanke
@ Hütte was heißt LowerCamelCase und UpperCamelCase? Das versteh ich nicht :-/

Alt+Shift+F ist die Format-Funktion. Wenn er nicht einrückt, stimmt etwas nicht

Wenn Netbeans den Text nicht automatisch einrückt stimmt etwas nicht oder wie darf ich das verstehen?

 

[hüteüberhüte]

Ich hab Vorarbeit geleistet und ein anderer sahnt das Danke dafür ab Zurück zum Thema:

@ Hütte was heißt LowerCamelCase und UpperCamelCase? Das versteh ich nicht :-/

Wenn Netbeans den Text nicht automatisch einrückt stimmt etwas nicht oder wie darf ich das verstehen?

Google. 1. Treffer. Oder auch java code conventions. Und naja, was innerhalb eines Blocks steht, wird eingerückt, und wenn nicht, dann stimmt etwas nicht

 

[hüteüberhüte]

Hab den Ratschlag von Landei beherzigt und es sieht jetzt so aus:

    public static void main(String argv[]) throws IOException {
        String str;
        int zahl;
        int grenzWert = 5000;
        int kinder;
        BufferedReader input = new BufferedReader(new InputStreamReader(System.in));

        System.out.print("Bitte geben Sie Ihr jährliches Bruttoeinkommen ein: ");
        str = input.readLine();
        zahl = Integer.parseInt(str);

        System.out.print("Bitte geben Sie die Zahl Ihrer Kinder ein: ");
        str = input.readLine();
        kinder = Integer.parseInt(str);

        if (zahl > grenzWert) { // >= 5001 ?
            switch (kinder) {
                case 1:
                    System.out.println("Sie erhalten 2,50 € Kindergeld");
                    break;
                case 2:
                    System.out.println("Sie erhalten 4,00 € Kindergeld");
                    break;
                case 3:
                    System.out.println("Sie erhalten 8,00 € Kindergeld");
                    break;
                default:
                    System.out.println("Ungültig");
                    break;
            }
        } else { // 0-5000 ?
            switch (kinder) {
                case 1:
                    System.out.println("Sie erhalten 100 € Kindergeld");
                    break;
                case 2:
                    System.out.println("Sie erhalten 200 € Kindergeld");
                    break;
                case 3:
                    System.out.println("Sie erhalten 400 € Kindergeld");
                    break;
                default:
                    System.out.println("Ungültig");
                    break;
            } // switch
        } // else
    } // main

Stimmt das mit Zeile 16 und 31 ? Und hat das Beispiel einen Realitätsbezug ? (Kenne mich damit nicht aus)

 

[Jango]

Google. 1. Treffer. Oder auch java code conventions. Und naja, was innerhalb eines Blocks steht, wird eingerückt, und wenn nicht, dann stimmt etwas nicht

Und was genau hat das mit dem Markieren von Fehlern/Warnungen zu tun, darum ging es doch?

 

[hüteüberhüte]

Da du weder von dem einen noch von dem anderen etwas weißt, würde ich vorschlagen, du beschränkst deine Aktivitäten ausschließlich auf die Plauderecke. Ich hab jedenfalls keine Lust, auf jede Troll-Frage eine Antwort zu geben

 

[Jango]

Da du weder von dem einen noch von dem anderen etwas weißt, würde ich vorschlagen, du beschränkst deine Aktivitäten ausschließlich auf die Plauderecke. Ich hab jedenfalls keine Lust, auf jede Troll-Frage eine Antwort zu geben

Bitte hier um Klärung, da meine Frage berechtigt war...
Danke.

 

[hüteüberhüte]

Wenn man sieht, ob der nachfolgende Code eingerückt wird oder nicht, kann man ja daraus schließen, ob er zum If gehört oder nicht. Das wollte ich damit ausdrücken.

for(...); und while(...);

ist nicht unbedingt ein Fehler.

Deine Fragen beantwortet?

 

[Bile Demon]

So was würde Eclipse bei mir dick und fett mit einer Warnung hervorheben

Hab das eben mal aus Neugier ausprobiert, weil ich es nicht glauben konnte. Bei mir unterkringelt Eclipse da definitiv nichts. Wüsste auch nicht wieso. Könnte ja schließlich sein, dass das so ein Unkonstrukt ist, wo alle Arbeit schon in der Bedingung erledigt wird.

 

[Ark]

Window → Preferences → Java → Compiler → Error/Warnings

Da gibt es sehr viele nützliche Schalterchen, im konkreten Fall auch einen für "Undocumented Empty Block".

Ark

 

[faetzminator]

Da gibt es sehr viele nützliche Schalterchen [...]

Wir haben im Projekt gerade zig Warnings des Javadoc Compilers eingeschalten - nun gibts wieder genügend Warnings
Manchmal nerv ich mich schon über gewisse Settings... Warum sollte ein switch fall-through eine Warning auslösen ?

 

[Ark]

Wir haben im Projekt gerade zig Warnings des Javadoc Compilers eingeschalten - nun gibts wieder genügend Warnings

Na, dann ist ja alles so wie immer.

Manchmal nerv ich mich schon über gewisse Settings... Warum sollte
ein switch fall-through eine Warning auslösen ?

Beispiel: Du ersetzt für ein case ein return durch ein paar andere Anweisungen (ohne return). Dann ist es durchaus praktisch, wenn mich eine Warnung an das wahrscheinlich fehlende break erinnert. Und für den Fall, dass das mit dem fall-through seine Richtigkeit hat, kann man ja von Eclipse einen Kommentar einfügen lassen, sodass die Warnung nicht mehr kommt.

[OT]Und jetzt große Diskussion: Heißt es "eingeschalten" oder "eingeschaltet"? [/OT]
Ark

 

[faetzminator]

Ark, klar, aber irgendwie haben meine switchs - wenn überhaupt mal verwendet - immer irgendwelche fall-throughs. Bin ich ein pöser Programmierer ?

 

[Der Neuling]

Hallo Leute,

da hier noch ein bisschen was los ist dachte ich ich schreibe meine zweite Frage auch noch hier rein, da sie sich auch um eine Variable im "switch" dreht.

Ich schreibe im Moment am nächsten Programm nach meinem Lehrbuch (leider keine Musterlösung vorhanden). Das Programm soll abfragen, ob das Volumen eines Würfels oder eines Zylinders berechnet werden soll, dafür soll "w" oder "z" eingegeben werden und je nach Eingabe sollen dann die entsprechenden Werte abgefragt werden. Nun meckert Netbeans, dass "w" nicht initialisiert ("variable w might not have been initialized") wäre, dabei habe ich ganz am Anfang (wie bisher immer) "int w;" geschrieben. Was mache ich falsch? Ich werd langsam bekloppt :-/

import java.io.*;

public class Koerper {

    public static void main(String argv[]) throws IOException {
        String str;
        int a;
        int h;
        int r;
        int w;
        int x;
        int z;
       

        BufferedReader input = new BufferedReader(
                new InputStreamReader(
                System.in));
        System.out.println("Bitte wählen Sie, ob sie das Volumen "
                + "eines Würfels (w) "
                + "oder eines Zylinders (z) berechnen möchten");
        str = input.readLine();
        x = Integer.parseInt(str);

        if (x == w) {
            
            System.out.println("Bitte geben Sie die Seitenlänge "
                    + "des Würfels ein: ");

            str = input.readLine();
            a = Integer.parseInt(str);
            w = a * a * a;
            System.out.println("Das Volumen des Würfels beträgt: " + w);
        }
    }
}

Bin für eure Tipps wie immer sehr dankbar.

Viele Grüße

Der Neuling

 

[MrClave]

Du hast sie nicht initialisiert, sondern deklariert.

initialisiert wäre es so

int w = 0;

(Immer wenn du der Variable einen Wert zuweist)
Und deklarieren tut man sie, wenn man einfach nur

int w;

, ohne Wert, schreibt.

 

[Der Neuling]

Danke ich habe jetzt w= a*a*a initialisiert, jetzt beschwert er sich, dass a nicht initialisiert ist. Da ich dafür aber einen eingelesenen Wert verwenden möchte kann ich da ja nicht einfach irgendeine Zahl initialisieren oder?

Viele Grüße

Der Neuling

 

[MrClave]

Das Problem jetzt ist, dass du die Variablen erst nach dem initialisieren einliest, was nicht funktionieren kann.
Der Compiler geht den Code Schritt für Schritt ab. D.h., dass du den Wert erst einlesen musst, bevor du sie zuweist.

Bis zu dem Zeitpunkt deiner Deklarationen, kennt der Compiler die Werte ja noch nicht.
Also einfach Variablen und Reader tauschen ^^

 

[Der Neuling]

Ja das klingt logisch, ich habe jetzt nur das Problem, dass ich ja definiert habe w=a*a*a, also müsste ich "a" vorher einlesen, aber ich möchte ja "a" nur einlesen, wenn die Eingabe "w" (Würfel) heißt. "w" muss ich aber ja vor dem "if" einlesen, weil sonst wieder der Fehler mit der fehlenden Initialisierung kommt. Weißt du was ich meine?

 

[Landei]

Einfach erst dann deklarieren, wenn man die Variable auch braucht, das macht das Leben viel einfacher...

import java.io.*;
 
public class Koerper {
 
    public static void main(String argv[]) throws IOException {
        BufferedReader input = new BufferedReader(new InputStreamReader(System.in));
        System.out.println("Bitte wählen Sie, ob sie das Volumen "
                + "eines Würfels (w) "
                + "oder eines Zylinders (z) berechnen möchten");
        String str = input.readLine();
 
        if (str.equals("w")) {  //Strings immer mit equals vergleichen!
            
            System.out.println("Bitte geben Sie die Seitenlänge "
                    + "des Würfels ein: ");
 
            str = input.readLine();
            double a = Double.parseDouble(str); 
            double w = a * a * a;
            System.out.println("Das Volumen des Würfels beträgt: " + w);
        } 

        if (str.equals("z")) {  
            ///... und hier das gleiche Spiel für Zylinder           
        } 
    }
}

Die Variablen habe ich auf [c]double[/c] geändert. Beim Zylinder käme sowieso nichts ganzzahliges raus...

 

[Der Neuling]

Hallo Landei,

vielen Dank für deine Hilfe. Da wäre ich in 100 Jahren nicht selbst darauf gekommen. Dass ich einfach mit if (str.equals...) { vergleichen kann wusste ich garnicht...
Mit double hast du natürlich auch völlig Recht, das hatte ich so im ersten Moment noch garnicht überrissen

Nochmals vielen Dank für die Hilfe, das gilt auch den anderen Helfern in diesem Thread, ohne euch wäre ich aus dieser Sackgasse wohl nicht rausgekommen.

Viele Grüße

Der Neuling