Taschenrechner

[Knizz]

So wiedermal ein Frage wegen Java
Ich wollte heute einen Taschenrechner Programmieren, den man über cmd öffnet und wo man auswählen kann, welche Rechenart man haben möchte. Das hat leider nicht Funktioniert. Hab auch mal die foren durchsucht aber nichts gefunden.
Ich hoffe mir kann einer helfen ???:L
Falls fragen aufkommen kann ich die leider erst morgen beantworten, weil ich jetzt seeeehr müde bin
Vielen Dank im Vorraus

Quelltext:

import java.util.Scanner;
import java.io.*;

public class TR
{
public static void main(String[] args);

float a, b, c;

{
Scanner scanner = new Scanner(System.in);

System.out.println("Welche Rechenart moechten Sie benutzen?");
System.out.println("1=+");
System.out.println("2=-");
System.out.println("3=*");
System.out.println("4=/");




{
if (a==1)
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b+c) );
}
{
if (a==2)
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b-c) );
}
{
if (a==3)
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b*c) );
}
{
if (a==4)
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b/c) );
}
}
}

 

[cosmic]

Du liest a ja auch nie ein, sagt dir doch auch der Compiler. Das 4fache Einlesen von b und c könntest du dir übrigens sparen.

 

[Spin]

import java.util.Scanner;
import java.io.*;

public class Taschenrechner
{

public static void main(String[] args){


// wenn du in einer if bedingung Variablen nutzen möchtest:
//  INITIALISIEREN

float a=0;
float b=0;
float c=0;


Scanner scanner = new Scanner(System.in);

System.out.println("Welche Rechenart moechten Sie benutzen?");
System.out.println("1=+");
System.out.println("2=-");
System.out.println("3=*");
System.out.println("4=/");
int auswahl = scanner.nextInt();


if (auswahl==1){
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b+c) );
}

if (auswahl==2){
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b-c) );
}

if (auswahl==3){
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b*c) );
}

if (auswahl==4){
System.out.println("Geben Sie Zahl 1 ein");
b = scanner.nextFloat();

System.out.println("Geben Sie Zahl 2 ein");
c = scanner.nextFloat();

System.out.println("Ihr Ergebnis lautet: " + (b/c) );
}
}
}

So funktioniert das einwandfrei ^^..


Habe ja auch mal so angefangen.
Tip:

Deine Klasse heißt jetzt : Taschenrechner.

Weiterhin ....habe ich deine auswahl einlesen lassen.
und in die variable auswahl gespeichert.

Und du musst variablen initialisieren, also mit werten füllen ^...

grüße

 

[Knizz]

Vielen Dank für die Antworten, Funktioniert alles einwandfrei

-closed-

 

[Landei]

Du kannst deine Threads als "erledigt" markieren (Knopf unter dem letzten Post).

 

[homiie8897]

Du liest a ja auch nie ein, sagt dir doch auch der Compiler. Das 4fache Einlesen von b und c könntest du dir übrigens sparen.

Kannst du mir vielleicht kurz deine vereinfachte Version Posten ?
Ich kriegs selbst nicht hin.
Danke für die Hilfe

 

[0din]

ohne mich zuweit aus dem fenster lenen zu wollen, grade was die aufgabenstellung angeht... aber ich würde sagen der richtige code is mies...
1. es springt jedes if an, auch wenns nich nötig is
2. es hat das einlesen der zahlen doppelt un dreifach implementiert
3. wenn ich mich net verschaut hab, wird float a niemals gebraucht

ich habs mal fix umgeschrieben, so sollte es meiner meinung nach um einiges besser sein

import java.util.Scanner;
import java.io.*;

public class Taschenrechner
{

	public static void main(String[] args)
	{

		// wenn du in einer if bedingung Variablen nutzen möchtest:
		// INITIALISIEREN

		int rechenzeichen = -1;
		int a = 0;
		int b = 0;
		int c = 0;

		Scanner scanner = new Scanner(System.in);

		System.out.println("Welche Rechenart moechten Sie benutzen?");
		System.out.println("1=+");
		System.out.println("2=-");
		System.out.println("3=*");
		System.out.println("4=/");
		rechenzeichen = scanner.nextInt();
		System.out.println("Geben Sie Zahl 1 ein");
		a = scanner.nextInt();
		System.out.println("Geben Sie Zahl 2 ein");
		b = scanner.nextInt();
		
		switch(rechenzeichen)
		{
		case 1:
		{
			c = a+b;
			System.out.println(a + " + " + b + " = " + c);
			break;
		}
		case 2:
		{
			c = a-b;
			System.out.println(a + " - " + b + " = " + c);
			break;
		}
		case 3:
		{
			c = a*b;
			System.out.println(a + " * " + b + " = " + c);
			break;
		}
		case 4:
		{
			c = a/b;
			System.out.println(a + " / " + b + " = " + c);
			break;
		}
		default:
		{
			System.out.println("Irgendwer hat da pfusch gebaut!");
			break;
		}
		}

	}
}

 

[homiie8897]

Perfekt !

Vielen Dank

 

[Landei]

Ich hätte es so geschrieben:

import java.util.Scanner;
import java.util.Map;
import java.util.HashMap;

public class Taschenrechner
{
    private static interface Op{ double eval(double p, double q); }

    private static Map<String, Op> operatorMap() {
      Map<String, Op> map = new HashMap<String,Op>();
      map.put("+", new Op(){ public double eval(double p, double q){ return p+q;}});
      map.put("-", new Op(){ public double eval(double p, double q){ return p-q;}});
      map.put("*", new Op(){ public double eval(double p, double q){ return p*q;}});
      map.put("/", new Op(){ public double eval(double p, double q){ return p/q;}});
      return map;
    }

   public static void main(String[] args)
    {
        Scanner scanner = new Scanner(System.in);
        System.out.println("Welche Rechenart moechten Sie benutzen?");
        System.out.println("+\n-\n*\n/");
        String sign = scanner.next();
        Op op = operatorMap().get(sign);
        if(op == null) {
          System.out.println("Irgendwer hat da Pfusch gebaut!");
        } else {
          System.out.println("Geben Sie Zahl 1 ein");
          double a = scanner.nextDouble();
          System.out.println("Geben Sie Zahl 2 ein");
          double b = scanner.nextDouble();
          System.out.printf("%f %s %f = %f%n",a,sign,b,op.eval(a,b));
        }        
    }
}