Falsche Ausgabe

[ProSonixs]

Sehr geehrte Community ich habe zur Übung einen Java Code geschrieben, in dem es um folgendes ging:

AUFGABE:
Es sollen die Volumina und Oberflächen von bestimmten Körpern berechnet werden.

Erstellen sie ein Programm, das nach Auswahl von Oberfläche oder Volumen eine Auswahl zu den gegebenen Körpern bittet
und diese dann gegebenenfalls nach Eingabe der fehlenden Werte berechnet.


FRAGE:
Wenn ich das Programm Ausführe und die Taste "1" drücke wird der Bereich Oberfläche ausgewählt.
Dannach wähle ich die "2" um die Oberfläche des Würfels berechnen zu können.
Ich tippe dann die Fehlenden Angaben ein, die von mir verlangt werden.
Es wird natürlich berechnet, aber dann startet er nach Ausgabe der Lösung aufeinmal
die zweite if Schleife die für die Berechung der Voluminas zur verfügung steht.

/*
 * @author ProSonixs
 */

import java.util.*;
public class Koerper
{
  public static void main(String argv[])
  {
    //Eingabetyp
    Scanner input = new Scanner(System.in);
    
    //Deklaration der Variablen
    byte eingabe;
    double o,r,h,a,b,c,s,v;
    
    //Auswahl zwischen Oberfläche- /Volumenberechnung
    System.out.println("Zum Berechnen der Oberfläche geben Sie eine \"1\" ein / Zum Berechnen des Volumens geben Sie eine \"2\" ein");
    eingabe = input.nextByte();
    
    //Berechnung der Oberfläche div. Körper
    if(eingabe==1)
    {
      System.out.println("Sie berechnen jetzt die Oberfläche bestimmter Körper!\n");
      System.out.print("Sie haben die Auswahl zwischen folgenden Körpern: ");
      
      System.out.println("1=Zylinder / 2=Würfel / 3=Quader / 4=Kreiskegel / 5=Kugel\n");
      
      System.out.print("Geben Sie für den gewünschten Körper, die richtige Zahl ein: ");
      eingabe = input.nextByte();
      
      //Berechnung der Oberfläche eines Zylinders
      if(eingabe==1)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Zylinders");
        
        System.out.print("Geben Sie den \"Radius\" des Zylinders in cm an: ");
        r= input.nextDouble();
        
        System.out.print("\nGeben Sie die \"Höhe\" des Zylinders in cm an: ");
        h= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o=2*Math.PI*r*(r+h);
        System.out.println("\nDas Ergebniss lautet: "+o+"cm²" );
      }
      
      //Berechnung der Oberfläche eines Würfels
      else if(eingabe==2)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Würfels");
        
        System.out.print("\nGeben Sie den Wert a in cm an: ");
        a= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o=6*(a*a);
        System.out.println("\nDas Ergebniss lautet: "+o+"cm²");
      }
      
      //Berechnung der Oberfläche eines Quaders 
      else if(eingabe==3)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Quaders");
        
        System.out.print("\nGeben Sie den Wert a in cm an: ");
        a= input.nextDouble();
        
        System.out.print("\nGeben Sie den Wert b in cm an: ");
        b= input.nextDouble();
        
        System.out.print("\nGeben Sie den Wert c in cm an: ");
        c= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o=2*((a*b)+(a*c)+(b*c));
        System.out.println("\nDas Ergebniss lautet: "+o+"cm²");
      }
      
      //Berechnung der Oberfläche eines Kreiskegels
      else if(eingabe==4)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Kreiskegels");
        
        System.out.print("\nGeben Sie den \"Radius\" in cm an: ");
        r= input.nextDouble();
        
        System.out.print("\nGeben Sie den Wert \"s\" in cm an: ");
        s= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o=Math.PI*r*(r+s);
        System.out.println("\nDas Ergebniss lautet: "+o+"cm²" );
      }
      
      //Berechnung der Oberfläche einer Kugel
      else if(eingabe==5)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche einer Kugel");
        
        System.out.print("\nGeben Sie den \"Radius\" in cm an: ");
        r= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o=4*Math.PI*(r*r);
        System.out.println("\nDas Ergebniss lautet: "+o+"cm²" );
      }
    }
    
    //Berechnung des Volumens div. Körper
    if(eingabe==2)
    {
      System.out.println("Sie berechnen jetzt das Volumen bestimmter Körper!\n");
      System.out.print("Sie haben die Auswahl zwischen folgenden Körpern: ");
      
      System.out.println("1=Zylinder / 2=Würfel / 3=Quader / 4=Kreiskegel / 5=Kugel\n");
      
      System.out.print("Geben Sie für den gewünschten Körper, die richtige Zahl ein: ");
      eingabe = input.nextByte();
      
      //Berechnung des Volumens eines Zylinders
      if(eingabe==1)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Zylinders");
        
        System.out.print("Geben Sie den \"Radius\" des Zylinders in cm an: ");
        r = input.nextDouble();
        
        System.out.print("\nGeben Sie die \"Höhe\" des Zylinders in cm an: ");
        h = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v=Math.PI*(r*r)*h;
        System.out.println("\nDas Ergebniss lautet: "+v+"cm³");
      }
      //Berechnung des Volumens eines Würfels
      else if(eingabe==2)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Würfels");
        
        System.out.print("Geben Sie den Wert \"a\" an: ");
        a = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v = a*a*a;
        System.out.println("\nDas Ergebniss lautet: "+v+"cm³" );
      }
      //Berechnung des Volumens eines Quaders
      else if(eingabe==3)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Quaders");
        
        System.out.print("Geben Sie den Wert \"a\" an: ");
        a = input.nextDouble();
        
        System.out.print("Geben Sie den Wert \"b\" an: ");
        b = input.nextDouble();
        
        System.out.print("Geben Sie den Wert \"c\" an: ");
        c = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v = a*b*c;
        System.out.println("\nDas Ergebniss lautet: "+v+"cm³" );
      }
      //Berechnung des Volumens eines Kreiskegels
      else if(eingabe==4)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Kreiskegels");
        
        System.out.print("Geben Sie den \"Radius\" des Kreiskegels in cm an: ");
        r = input.nextDouble();
        
        System.out.print("\nGeben Sie die \"Höhe\" des Kreiskegels in cm an: ");
        h = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v = (1/3)*Math.PI*(r*r)*h;
        System.out.println("\nDas Ergebniss lautet: "+v+"cm³" );
      }
      //Berechnung des Volumens einer Kugel
      else if(eingabe==5)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen einer Kugel");
        
        System.out.print("Geben Sie den \"Radius\" des Kreiskegels in cm an: ");
        r = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v = (4/3)*Math.PI*(r*r*r);
        System.out.println("\nDas Ergebniss lautet: "+v+"cm³" );
      }
    }
  }
}

 

[SlaterB]

verwende verschiedene Variablen, wenn du eingabe im inneren neu abfragst und auf 2 änderst, dann kommt beim späteren Code eben 2 dran,

mit Untermethoden käme es auch nicht zu so viel Durcheinander,
Scanner input dann nur einmal statisch ablegen, nicht mehrfach erzeugen,

sinnvoll ist auch

if (..) {

} else if (..) {

}

wie du zum Teil schon verwendest, das würde den Fehler hier auch ausschließen,
noch schöner mit switch, breaks nicht vergessen

------

if-schleife.de

 

[FaxXer]

Habs mir jetzt nicht so genau angeschaut aber du hast ja als letztes eine 2 eingegeben und prüfst dann eben genau dort wo er hinspringt ob die Eingabe eine 2 ist. Also musst du irgendwie überprüfen ob man noch in die zweite Schleife will oder nicht.

    //Berechnung des Volumens div. Körper
    if(eingabe==2) // Zeile 116

 

[ProSonixs]

Ich habe jetzt einfach jedem verschiedene Werte zugewissen, so das Sie sich nicht mehr überschneiden.
Es ist zwar nicht die 100% Lösung aber für einen Javaanfänger wie mir, sollte da fürs erste reichen ;-)

Mein Problem ist nur das ich recht neu bin
und nicht gewusst hätte wie ich das mit "switch" zum Beispiel hätte machen können.

Oder ich weiß zum Beispiel nicht was mit Untermethoden gemeint ist ^^


Mein Code sieht nach dem Abändern, so aus:

/*
 * @author ProSonixs
 */

import java.util.*;
public class Koerper
{
  public static void main(String argv[])
  {
    //Eingabetyp
    Scanner input = new Scanner(System.in);
    
    //Variable für den Eingabetype
    byte eingabe;
     
    //Deklaration der Variablen für die Oberfläche
    double o1,r1,h1;
    double o2,a2;
    double o3,a3,b3,c3;
    double o4,r4,s4;
    double o5,r5;
    
    //Deklaration der Variablen für das Volumen
    double v6,r6,h6;
    double v7,a7;
    double v8,a8,b8,c8;
    double v9,r9,h9;
    double v10,r10;
    
    //Auswahl zwischen Oberfläche- /Volumenberechnung
    System.out.println("Zum Berechnen der Oberfläche geben Sie eine \"1\" ein / Zum Berechnen des Volumens geben Sie eine \"7\" ein");
    eingabe = input.nextByte();
    
    //Berechnung der Oberfläche div. Körper
    if(eingabe==1)
    {
      System.out.println("Sie berechnen jetzt die Oberfläche bestimmter Körper!\n");
      System.out.print("Sie haben die Auswahl zwischen folgenden Körpern: ");
      
      System.out.println("2=Zylinder / 3=Würfel / 4=Quader / 5=Kreiskegel / 6=Kugel\n");
      
      System.out.print("Geben Sie für den gewünschten Körper, die richtige Zahl ein: ");
      eingabe = input.nextByte();
      
      //Berechnung der Oberfläche eines Zylinders
      if(eingabe==2)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Zylinders");
        
        System.out.print("Geben Sie den \"Radius\" des Zylinders in cm an: ");
        r1= input.nextDouble();
        
        System.out.print("\nGeben Sie die \"Höhe\" des Zylinders in cm an: ");
        h1= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o1=2*Math.PI*r1*(r1+h1);
        System.out.println("\nDas Ergebniss lautet: "+o1+"cm²" );
      }
      
      //Berechnung der Oberfläche eines Würfels
      else if(eingabe==3)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Würfels");
        
        System.out.print("\nGeben Sie den Wert a in cm an: ");
        a2= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o2=6*(a2*a2);
        System.out.println("\nDas Ergebniss lautet: "+o2+"cm²");
      }
      
      //Berechnung der Oberfläche eines Quaders 
      else if(eingabe==4)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Quaders");
        
        System.out.print("\nGeben Sie den Wert a in cm an: ");
        a3= input.nextDouble();
        
        System.out.print("\nGeben Sie den Wert b in cm an: ");
        b3= input.nextDouble();
        
        System.out.print("\nGeben Sie den Wert c in cm an: ");
        c3= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o3=2*((a3*b3)+(a3*c3)+(b3*c3));
        System.out.println("\nDas Ergebniss lautet: "+o3+"cm²");
      }
      
      //Berechnung der Oberfläche eines Kreiskegels
      else if(eingabe==5)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche eines Kreiskegels");
        
        System.out.print("\nGeben Sie den \"Radius\" in cm an: ");
        r4= input.nextDouble();
        
        System.out.print("\nGeben Sie den Wert \"s\" in cm an: ");
        s4= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o4=Math.PI*r4*(r4+s4);
        System.out.println("\nDas Ergebniss lautet: "+o4+"cm²" );
      }
      
      //Berechnung der Oberfläche einer Kugel
      else if(eingabe==6)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt die Oberfläche einer Kugel");
        
        System.out.print("\nGeben Sie den \"Radius\" in cm an: ");
        r5= input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        o5=4*Math.PI*(r5*r5);
        System.out.println("\nDas Ergebniss lautet: "+o5+"cm²" );
      }
    }
    
    //Berechnung des Volumens div. Körper
    if(eingabe==7)
    {
      System.out.println("Sie berechnen jetzt das Volumen bestimmter Körper!\n");
      System.out.print("Sie haben die Auswahl zwischen folgenden Körpern: ");
      
      System.out.println("8=Zylinder / 9=Würfel / 10=Quader / 11=Kreiskegel / 12=Kugel\n");
      
      System.out.print("Geben Sie für den gewünschten Körper, die richtige Zahl ein: ");
      eingabe = input.nextByte();
      
      //Berechnung des Volumens eines Zylinders
      if(eingabe==8)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Zylinders");
        
        System.out.print("Geben Sie den \"Radius\" des Zylinders in cm an: ");
        r6 = input.nextDouble();
        
        System.out.print("\nGeben Sie die \"Höhe\" des Zylinders in cm an: ");
        h6 = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v6=Math.PI*(r6*r6)*h6;
        System.out.println("\nDas Ergebniss lautet: "+v6+"cm³");
      }
      //Berechnung des Volumens eines Würfels
      else if(eingabe==9)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Würfels");
        
        System.out.print("Geben Sie den Wert \"a\" an: ");
        a7 = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v7 = a7*a7*a7;
        System.out.println("\nDas Ergebniss lautet: "+v7+"cm³" );
      }
      //Berechnung des Volumens eines Quaders
      else if(eingabe==10)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Quaders");
        
        System.out.print("Geben Sie den Wert \"a\" an: ");
        a8 = input.nextDouble();
        
        System.out.print("Geben Sie den Wert \"b\" an: ");
        b8 = input.nextDouble();
        
        System.out.print("Geben Sie den Wert \"c\" an: ");
        c8 = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v8 = a8*b8*c8;
        System.out.println("\nDas Ergebniss lautet: "+v8+"cm³" );
      }
      //Berechnung des Volumens eines Kreiskegels
      else if(eingabe==11)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen eines Kreiskegels");
        
        System.out.print("Geben Sie den \"Radius\" des Kreiskegels in cm an: ");
        r9 = input.nextDouble();
        
        System.out.print("\nGeben Sie die \"Höhe\" des Kreiskegels in cm an: ");
        h9 = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v9 = (1/3)*Math.PI*(r9*r9)*h9;
        System.out.println("\nDas Ergebniss lautet: "+v9+"cm³" );
      }
      //Berechnung des Volumens einer Kugel
      else if(eingabe==12)
      {
        //Text und Eingabe der notwendigen Daten
        System.out.println("\n\nSie berechnen jetzt das Volumen einer Kugel");
        
        System.out.print("Geben Sie den \"Radius\" des Kreiskegels in cm an: ");
        r10 = input.nextDouble();
        
        //Berechnung und Ausgabe des Ergebnisses
        v10 = (4/3)*Math.PI*(r10*r10*r10);
        System.out.println("\nDas Ergebniss lautet: "+v10+"cm³" );
      }
    }
  }
}

 

[Andi_CH]

die zweite if Schleife

if Schleife

(Das hat aber lange gedauert, bis der wieder mal kam.)

Aber dein Code sieht recht schön aus - da gibt es wohl nicht mehr viel zu bemängeln.

 

[ProSonixs]

Es freut mich, mir hilft das nur.

Jetzt bin ich etwas schlauer:

Es gibt keine if-Schleifen nur if-Abfragen

(Ich wusste es ja eigentlich, aber warum ich Schleife sage weiß ich nicht, muss ne schlechte Angewohnheit sein)

if Schleife

(Das hat aber lange gedauert, bis der wieder mal kam.)