Taschenrechner

[wiesner95]

Moin Moin,

Hab in der Schule n Taschenrechner gemacht der mir das Ergebnis nicht ausgibt!

import java.util.*;
public class Taschenrechner{
  public static void main(String[] args){

   Scanner sc=new Scanner(System.in);
   System.out.print("Erste Zahl: ");
   String z1=sc.next();
   System.out.print("Rechenzeichen: ");
   String r=sc.next();
   System.out.print("Zweite Zahl: ");
   String z2=sc.next();
    Integer zahl1=new Integer (z1);
    int a=zahl1.intValue();
    Integer zahl2=new Integer (z2);
    int b=zahl2.intValue();
    int x = a+b;
    int y = a-b;
    int z = a*b;
    int v = a/b;
      if(r==("+")){
        System.out.println(a+"+"+b+"="+x);
      }
      if(r==("-")){
        System.out.println(a+"-"+b+"="+y);
      }
      if(r==("*")){
        System.out.println(a+"*"+b+"="+z);
      }
      if(r==("/")){
        System.out.println(a+"/"+b+"="+v);
      }
  }
}

Er kommt nicht in die "if"-entscheidung rein!!

Wieso ?

mfg und Hoffnung auf baldige Antwort
wiesner95

 

[Haave]

Strings vergleicht man mit equals(), nicht mit ==.
(Wie oft wurde dieser Satz in diesem Forum wohl schon geschrieben? )

Übrigens ist es meines Erachtens nicht so klug, schon alle Ergebnisse vorzuberechnen, aber am Ende nur das auszugeben, das zum vom Benutzer eingegebenen Rechenzeichen passt. Warum nicht so:

if(r.equals("+")){
System.out.println(a+"+"+b+"="+(a+b));
}
if(r.equals("-")){
System.out.println(a+"-"+b+"="+(a-b));
}
if(r.equals("*")){
System.out.println(a+"*"+b+"="+(a*b));
}
if(r.equals("/")){
System.out.println(a+"/"+b+"="+(a/b));
}

Wenn das alles ist, was du für den Taschenrechner willst, könnte man das so machen; wenn der Taschenrechner noch mehr können soll (z.B. sich eine Eingabe merken und damit weiterrechnen), ist diese Lösung natürlich auch Quatsch. Dennoch, alles vorzuberechnen, obwohl es gar nicht gebraucht wurde, ist nicht so toll.

 

[chalkbag]

Guten Morgen,

ich hab es noch nicht ausprobiert,

aber grundlegend

Du darfst nur Chars (und numerische Werte) mit == vergleichen
also z.b. a == 'c'

Wenn du Strings vergleichen willst musst du das mit equals machen
also a.equals("a")

Zusätzlich könnt sein, das Leerzeichen im r drinnen sind bzw. der Linebreak \n

Lass dir am besten r mal ausgeben, bzw. debugge und schau was genau dort drinnen steht.

Leerzeichen kannst du mit String.trim beseitigen.

Grüße