Taschenrechner mit Unterprogra

[Nijuu]

Hallo Leute,

In der Schule nehmen wir gerade Unterprogramme in Java durch. Leider verstehe ich es noch nicht so ganz. ;(
Ich soll einen Taschenrechner erstellen. Wichtig dabei ist dass die Unterprogramme addieren(),subtrahieren(), mutliplizieren(),dividieren(), eingaben() und ausgeben() existieren sollen.
Das Programm dass ich selbst geschrieben haben erscheint mir ein wenig seltsam. Ich habe alle Unterprogramme aus das ausgeben(); verwendet. (Bei den Unterprogrammen addieren etc. habe ich die Ausgabe gemacht anstatt in einem anderen Unterprogramm)

//Taschenrechner soll aus 6 Unterprogrammen bestehen Eingabe(); Ausgabe(); addieren(); subtrahieren();multiplizieren(); dividieren();
Kann mir jemand helfen? :)
MFG Nijuu

import java.io.*;
public class taschenrechner
{
    public static int zahl1,zahl2;

  public static void main (String args[])   throws IOException
  {
       Eingabe();
     //Ausgabe();
  }
   public static void Eingabe()  throws IOException
   {
      int Auswahl;
      int erg;
      InputStreamReader Input=new InputStreamReader(System.in);
      BufferedReader Dataln=new BufferedReader(Input);
      System.out.println ("...Willkommen im Taschenrecher...");
      //System.out.println ("Drücken Sie die Zahl für ihre Auswahl");
      System.out.println (" [1]Addition \n [2]Subtrahtion \n [3]Multiplikation \n [4]Division");
      System.out.println ("Geben Sie ihre Auswahl ein.");
      Auswahl=Integer.parseInt(Dataln.readLine());
    switch (Auswahl)
    {
      case  1:

       addieren("Sie haben sich für Addition entschieden");

        break;
        
      case  2:
       subtrahieren("Sie haben sich für Subtrahtion entschieden");
        break;
      case 3:
       multi("Sie haben sich für Multiplikation entschieden");
        break;
      case 4:
      dividieren("Sie haben sich für Division entschieden");
         break;
      default:
        System.out.println ("###F A T A L  // E R R OR###");
    }
  }
  private static void addieren(String text) throws IOException
  {
    System.out.println(text);
    InputStreamReader Input=new InputStreamReader(System.in);
   BufferedReader Dataln=new BufferedReader(Input);
    System.out.println("Geben sie eine Zahl ein");
    zahl1=Integer.parseInt(Dataln.readLine());
         System.out.println("Geben sie eine Zahl ein");
    zahl2=Integer.parseInt(Dataln.readLine());
        int erg=zahl1+zahl2;
    System.out.println (zahl1+"+"+zahl2+"="+erg);

  }
  private static void subtrahieren(String text)  throws IOException
  {
     System.out.println(text);
    InputStreamReader Input=new InputStreamReader(System.in);
   BufferedReader Dataln=new BufferedReader(Input);
    System.out.println("Geben sie eine Zahl ein");
    zahl1=Integer.parseInt(Dataln.readLine());
         System.out.println("Geben sie eine Zahl ein");
    zahl2=Integer.parseInt(Dataln.readLine());
        int erg=zahl1-zahl2;
   System.out.println (zahl1+"-"+zahl2+"="+erg);
  }
  private static void multi(String text)  throws IOException
  {
   System.out.println(text);
    InputStreamReader Input=new InputStreamReader(System.in);
   BufferedReader Dataln=new BufferedReader(Input);
    System.out.println("Geben sie eine Zahl ein");
    zahl1=Integer.parseInt(Dataln.readLine());
         System.out.println("Geben sie eine Zahl ein");
    zahl2=Integer.parseInt(Dataln.readLine());
   int erg=zahl1*zahl2;
   System.out.println (zahl1+"*"+zahl2+"="+erg);
  }
 private  static void dividieren(String text)  throws IOException
  {
    System.out.println(text);
    InputStreamReader Input=new InputStreamReader(System.in);
   BufferedReader Dataln=new BufferedReader(Input);
    System.out.println("Geben sie eine Zahl ein");
    zahl1=Integer.parseInt(Dataln.readLine());
         System.out.println("Geben sie eine Zahl ein");
    zahl2=Integer.parseInt(Dataln.readLine());
    int erg=zahl1/zahl2;
    System.out.println (zahl1+"/"+zahl2+"="+erg);
  }
}

 

[Camill]

Und die Frage lautet..?

 

[BJ]

Ich nehme an es geht um die Separierung der Ausgabe Methoden von den Rechen Methoden indem eine Ausgabe Methode abstrahiert wird.

Oder?

public static void Eingabe(){

int Auswahl;
      int erg;
      InputStreamReader Input=new InputStreamReader(System.in);
      BufferedReader Dataln=new BufferedReader(Input);
      System.out.println ("...Willkommen im Taschenrecher...");
      //System.out.println ("Drücken Sie die Zahl für ihre Auswahl");
      System.out.println (" [1]Addition \n [2]Subtrahtion \n [3]Multiplikation \n [4]Division");
      System.out.println ("Geben Sie ihre Auswahl ein.");
      Auswahl=Integer.parseInt(Dataln.readLine());


InputStreamReader Input=new InputStreamReader(System.in);
   BufferedReader Dataln=new BufferedReader(Input);
    System.out.println("Geben sie eine Zahl ein");
    zahl1=Integer.parseInt(Dataln.readLine());
         System.out.println("Geben sie eine Zahl ein");
    zahl2=Integer.parseInt(Dataln.readLine());


 switch (Auswahl)
    {
      case  1:
 
       addieren("Sie haben sich für Addition entschieden");
 
        break;
        
      case  2:
       subtrahieren("Sie haben sich für Subtrahtion entschieden");
        break;
      case 3:
       multi("Sie haben sich für Multiplikation entschieden");
        break;
      case 4:
      dividieren("Sie haben sich für Division entschieden");
         break;
      default:
        System.out.println ("###F A T A L  // E R R OR###");
    }
}

Jetzt kannst du in allen Methoden (addieren(), subtrahieren()...) das Einlesen löschen...

 

[Nijuu]

Die Frage lautet, wie kann ich es so machen dass ich ein Unterprogramm habe, dass für die Ausgabe zuständig ist. Dass ich nicht in den Unterprogrammen wie addieren etc. die Ausgabe machen muss.
Danke für die Antworten

 

[tuttle64]

Oder?

Weshalb sind die Felder Input und Dataln doppelt deklariert und wiese fehlt die Typangabe bei zahl1 und zahl2?

Die Frage lautet, wie kann ich es so machen dass ich ein Unterprogramm habe, dass für die Ausgabe zuständig ist.

Innerhalb des switch-Blocks könnte der Aufruf so lauten: addieren(zahl1, zahl2);
Das Unterprogramm, welches für die Ausgabe zuständig ist, könnte lauten:

	private void addieren(double zahl1, double zahl2) {
				System.out.println(zahl1 + zahl2);

	}

 

[Nijuu]

Meine Frage bezieht sich eher darauf und zwar dass ich nach dem Eingabe Unterprogramm, die werte zurückgeben, die in das Ausgabe Unterprogramm gehen und dort dann Ergebnis ausgeben lasse.
Danke im Voraus und für die bisherigen Antworten
Mfg Nijuu

import java.io.*;
public class taschenrechner
{
    public static int zahl1,zahl2; //zahlen die berechnet werden sollen
 
  public static void main (String args[])   throws IOException
  {
       Eingabe();   // Unterprogramme addieren(); subtrahieren(); multi(); dividieren();
     //Ausgabe(); // Hier sollen die Werte zurück gegeben werden von den Unterprogrammen + Ausgabe stattfinden
  }

 

[pro2]

Dann veränderst du deine Funktionen so, dass sie dir Werte zurückgeben statt sie direkt auszugeben. Also statt

System.out.println (zahl1+"+"+zahl2+"="+erg);

meinetwegen

return erg

. Dann kannst du dir eine Funktion zum Ausgeben schreiben, wobei das irgendwie total überflüssig ist, aber egal. Dann kommt da sowas hin wie:

public void ausgeben(int ergebnis){
    System.out.println("Das Ergebnis ist: " + erg);
}

Die könntest du dann so aufrufen:

  public static void main (String args[]) 
  {
      ausgeben(addieren("..."));
  }

Oder sowas ähnliches. Aber der Aufbau des Programms und Co sind bei dir eh nicht sonderlich empfehlenswert. Kannst du mal die ganze Aufgabenstellung posten?