RPN Taschenrechner - keine Lösung!!
- Themenstarter javabeginners18
- Beginndatum
Hi,
ich habe das so gemacht und bei beiden versucht, ein i=0; drüber zu setzen, habe es auch bei beiden und auch einzeln probiert, trotzdem gibt er mir das aus:
U P N-TASCHENRECHNER
********************
Eingabe: 3 3 + fwehjfwoijef
Ungültige Eingabe.
Bitte erneut versuchen!
Ergebnis: 6.0
********************
Eingabe:
EDIT: Das passiert nur wenn ich die 3 3 + am Anfang eingebe,
wenn ich zum Beispiel: fheuwih pj 3 3 + eingebe kommt nicht das unerwünschte "Ergebnis: 6" raus
Zuletzt bearbeitet:
JStein52
Top Contributor
Mhmm bei mir funktioniert es:
Code:
import java.util.Scanner;
import java.util.Stack;
public class UPNRechner {
public static void main(String[] args) {
Scanner scan = new Scanner(System.in); //Benutzereingabe
boolean verlassen = false; //boolean zum Verlassen der while-Schleife bei true
Stack<Double> ergebnis = new Stack(); //Stack erstellt
int laengeVomStack = ergebnis.size();
int i = 0;
willkommen(); //Willkommenstext
Scanner scanner = new Scanner(System.in);
while (true) {
eingabe(); //
String nextLine = scanner.nextLine();
Scanner tokenizer = new Scanner(nextLine);
i = 0;
ergebnis.clear();
for (int ii = 0; tokenizer.hasNext(); ii++) {
i++;
String nutzereingabe;
nutzereingabe = tokenizer.next();
double zahl = 0.0;
try {
if (nutzereingabe.startsWith("0x") || nutzereingabe.startsWith("0X")) {
zahl = Integer.parseUnsignedInt(nutzereingabe.substring(2), 16);
} else if (nutzereingabe.startsWith("-0x") || nutzereingabe.startsWith("-0X")) {
zahl = Integer.parseInt(nutzereingabe.substring(3), 16);
} else if (nutzereingabe.startsWith("0b") || nutzereingabe.startsWith("0B")) {
zahl = Integer.parseInt(nutzereingabe.substring(2), 2);
} else if (nutzereingabe.startsWith("-0b") || nutzereingabe.startsWith("-0B")) {
zahl = Integer.parseInt(nutzereingabe.substring(3), 2);
} else {
zahl = Double.parseDouble(nutzereingabe);
}
} catch (NumberFormatException e) {
if (nutzereingabe.equals("+")
|| nutzereingabe.equals("-")
|| nutzereingabe.equals("*")
|| nutzereingabe.equals("x")
|| nutzereingabe.equals("/")
|| nutzereingabe.equals(":")
|| nutzereingabe.equals("pow")
|| nutzereingabe.equals("cos")
|| nutzereingabe.equals("sqrt")
|| nutzereingabe.equals("sin")
|| nutzereingabe.equals("cos")
|| nutzereingabe.equals("ln")
|| nutzereingabe.equals("~")
|| nutzereingabe.equals("e")
|| nutzereingabe.equals("tan")
|| nutzereingabe.equals("exp")
|| nutzereingabe.equals("pi")
|| nutzereingabe.equals("exit")) {
} else {
System.out.println(" Ungültige Eingabe! \n"
+ " Bitte erneut \n versuchen:");
i = 0;
break;
}
}
try {
// Rechenoperatoren
if (nutzereingabe.equals("+")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() + ergebnisTmp);
} else if (nutzereingabe.equals("-")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() - ergebnisTmp);
} else if (nutzereingabe.equals("x")
|| nutzereingabe.equals("*")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() * ergebnisTmp);
} else if (nutzereingabe.equals("/")
|| (nutzereingabe.equals(":"))) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() / ergebnisTmp);
} else if (nutzereingabe.equals("sqrt")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.sqrt(ergebnisTmp));
} else if (nutzereingabe.equals("sin")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.sin(ergebnisTmp));
} else if (nutzereingabe.equals("cos")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.cos(ergebnisTmp));
} else if (nutzereingabe.equals("tan")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.tan(ergebnisTmp));
} else if (nutzereingabe.equals("~")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push((ergebnisTmp * (-1)));
} else if (nutzereingabe.equals("ln")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.log(ergebnisTmp));
} else if (nutzereingabe.equals("exp")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.pow(Math.E, ergebnisTmp));
} else if (nutzereingabe.equals("pi")) {
ergebnis.push(Math.PI);
} else if (nutzereingabe.equals("pow")) {
Double ergebnisTmp = ergebnis.pop();
System.out.println(" Geben Sie einen "
+ "\n Exponenten ein:");
System.out.print(" ");
ergebnis.push(Math.pow(ergebnisTmp, scan.nextDouble()));
} else if (nutzereingabe.equals("e")) {
ergebnis.push(Math.E);
} else if (nutzereingabe.equalsIgnoreCase("exit")) {
System.exit(0);
} else {
ergebnis.push(zahl);
}
} catch (Exception e) {
System.out.println(" Ungültige Eingabe! \n"
+ " Bitte erneut versuchen:");
i=0;
break;
}
}
if (i >= 2 && ergebnis.size() == 1) {
double finErgebnis = ergebnis.peek();
System.out.println(" Ergebnis: ");
System.out.println(" " + finErgebnis);
}
}
}
public static void eingabe() {
System.out.println("\n Eingabe: ");
System.out.print(" ");
}
public static void willkommen() {
System.out.println(" U P N-TASCHENRECHNER");
System.out.println(" ********************");
}
}Meniskusschaden
Top Contributor
Warum holt ihr euch die Operanden beim Potenzieren eigentlich nicht ganz normal vom Stack? Das ist so ja eine ziemlich merkwürdige Bedienung.
Ausserdem würde ich den Stack auch nicht löschen, damit man damit weiter rechnen kann.
Ausserdem würde ich den Stack auch nicht löschen, damit man damit weiter rechnen kann.
Stimmt zwar, aber ich finde es mit switch/case trotzdem etwas besser lesbar:
Java:
switch (nutzereingabe) {
case "+":
ergebnis.push(ergebnis.pop() + ergebnis.pop());
break;
case "-":
ergebnis.push(-ergebnis.pop() + ergebnis.pop());
break;
case "x":
ergebnis.push(ergebnis.pop() * ergebnis.pop());
break;
...
Switch würd ich in dem Fall auch gegenüber if bevorzugen, da fällt der ganzen Bloat weg...
Mag vllt für Programmieranfänger etwas komplizierter sein, aber am besten finde ich die Variante mit Map (oder passenderem Interface). Vermeidet dann auch die Doppelungen beim prüfen auf valide Eingaben.
Wird dann sowas:
Operation anlegen:
Java:
operationen.put("+", stack -> stack.push(stack.pop() + stack.pop()));
operationen.put("-", stack -> stack.push(stack.pop() - stack.pop()));Anwenden:
Java:
operationen.get(nutzereingabe).consume(ergebnis);@JStein52 Ich habe sehr vieles noch kommentiert, kann ich vielleicht meinen Code einsenden? Bei "Ihrem" Code funktioniert es, aber an sich habe ich eigentlich nichts umgeändert.
@Meniskusschaden & @mrBrown
ich versuche die switch-case-Anweisung zu machen, jedoch gibt er mir das Ergebnis dann nicht mehr aus.. sehr merkwürdig.. mein Prof bevorzugt glaube ich aber auch switch-case..
Der Code wegen @JStein52
@Meniskusschaden & @mrBrown
ich versuche die switch-case-Anweisung zu machen, jedoch gibt er mir das Ergebnis dann nicht mehr aus.. sehr merkwürdig.. mein Prof bevorzugt glaube ich aber auch switch-case..
Der Code wegen @JStein52
Java:
import java.util.Scanner;
import java.util.Stack;
public class UPNRechner {
public static void main(String[] args) {
Scanner scan = new Scanner(System.in); // Benutzereingabe
Scanner scanner = new Scanner(System.in); // zwei Scanner - dieser hier für den "Tokenizer"
Stack<Double> ergebnis = new Stack(); // Stack mit dem Namen "ergebnis" erstellt
willkommen(); // Überschrift/Begrüßungstext
/*
while-Schleife, welche den Großteil des Taschenrechners beinhaltet
*/
int i = 0;
while (true) {
eingabefeld(); // Eingabefeld (Text)
System.out.print(" ");
String nextLine = scanner.nextLine();
Scanner tokenizer = new Scanner(nextLine); // der "Tokenizer" dient zum Zerlegen der Benutzereingabe
i = 0;
ergebnis.clear();
for (int j = 0; tokenizer.hasNext(); j++) {
i++;
String nutzereingabe;
nutzereingabe = tokenizer.next();
double zahl = 0.0;
/*
In dieser try-catch-Anweisung wird die
"Umwandlung" von binären und hexadezimalen
Codes in Integer ausgeführt. Ebenfalls werden
Fehlermeldungen generiert, die eine ungültige
Eingabe bestätigen, und interne Fehlermeldungen
seitens IntelliJ, die das Programm ungewollt
beenden, werden beseitigt.
*/
try {
if (nutzereingabe.startsWith("0x") || nutzereingabe.startsWith("0X"))
zahl = Integer.parseInt(nutzereingabe.substring(2), 16);
else if (nutzereingabe.startsWith("-0x") || nutzereingabe.startsWith("-0X"))
zahl = Integer.parseInt(nutzereingabe.substring(3), 16);
else if (nutzereingabe.startsWith("0b") || nutzereingabe.startsWith("0B"))
zahl = Integer.parseInt(nutzereingabe.substring(2), 2);
else if (nutzereingabe.startsWith("-0b") || nutzereingabe.startsWith("-0B"))
zahl = Integer.parseInt(nutzereingabe.substring(3), 2);
else zahl = Double.parseDouble(nutzereingabe);
}catch (NumberFormatException e) {
if (nutzereingabe.equals("+")
|| nutzereingabe.equals("-")
|| nutzereingabe.equals("*")
|| nutzereingabe.equals("x")
|| nutzereingabe.equals("/")
|| nutzereingabe.equals(":")
|| nutzereingabe.equals("÷")
|| nutzereingabe.equals("pow")
|| nutzereingabe.equals("cos")
|| nutzereingabe.equals("sqrt")
|| nutzereingabe.equals("√")
|| nutzereingabe.equals("sin")
|| nutzereingabe.equals("cos")
|| nutzereingabe.equals("ln")
|| nutzereingabe.equals("~")
|| nutzereingabe.equals("e")
|| nutzereingabe.equals("tan")
|| nutzereingabe.equals("exp")
|| nutzereingabe.equals("pi")
|| nutzereingabe.equals("exit")) {
}else{
ungueltigeEingabe();
i=0;
break;
}
}
/*
In der folgenden try-catch-Anweisung sind die Funk-
tionen der Rechenoperatoren definiert - ebenso wird
hier geprüft, ob die Eingabe des Benutzers richtig
ist und/oder der Reihenfolge entspricht, in der diese
sein soll. Falls sich die Eingabe nicht um den Typ
Double oder um einen definierten Rechenoperatoren,
etc. handelt, soll für den Benutzer eine Fehlermeldung
erscheinen, die den Nutzer dazu auffordert, die
Handlung in korrekter Weise zu wiederholen.
*
*/
/*Hier wurde ein Double "ergebnisTmp" (temporäres
Ergebnis) erstellt. Dieser "speichert" den letzten
Wert im bzw. "auf" dem Stack. Durch "ergebnis.pop()"
wird eine Zahl vom Stapel genommen, mit "ergebnis.push()"
hinzugefügt.
*/
try {
if (nutzereingabe.equals("+")) { // Addition im Stack
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() + ergebnisTmp);
} else if (nutzereingabe.equals("-")) { // Subtraktion im Stack
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() - ergebnisTmp);
} else if (nutzereingabe.equals("x") || // Multiplikation im Stack
nutzereingabe.equals("*")) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() * ergebnisTmp);
} else if (nutzereingabe.equals("/") || // Division im Stack
(nutzereingabe.equals(":"))) {
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(ergebnis.pop() / ergebnisTmp);
} else if (nutzereingabe.equals("sqrt") // Wurzelziehen
|| nutzereingabe.equals("√")){
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.sqrt(ergebnisTmp));
} else if (nutzereingabe.equals("sin")) { // Sinus-Funktion
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.sin(ergebnisTmp));
} else if (nutzereingabe.equals("cos")) { // Cosinus-Funktion
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.cos(ergebnisTmp));
} else if (nutzereingabe.equals("tan")) { // Tangens-Funktion
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.tan(ergebnisTmp));
} else if (nutzereingabe.equals("~")) { // Vorzeichen
Double ergebnisTmp = ergebnis.pop();
ergebnis.push((ergebnisTmp * (-1)));
} else if (nutzereingabe.equals("ln")) { // ln-Logarithmus
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.log(ergebnisTmp));
} else if (nutzereingabe.equals("exp")) { // E-Funktion
Double ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.pow(Math.E, ergebnisTmp));
} else if (nutzereingabe.equals("pi") || // pi-Funktion
nutzereingabe.equals("π")) {
ergebnis.push(Math.PI);
} else if (nutzereingabe.equals("pow")) { // Exponent einer Zahl
Double ergebnisTmp = ergebnis.pop();
exponenteingeben();
ergebnis.push(Math.pow(ergebnisTmp, scan.nextDouble()));
} else if (nutzereingabe.equals("e")) { // Eulersche Zahl
ergebnis.push(Math.E);
} else if (nutzereingabe.equalsIgnoreCase("exit")) { // Verlassen des Programms
System.exit(0);
} else {
ergebnis.push(zahl);
}
}catch (Exception e) {
ungueltigeEingabe(); // ungültige Eingabe
i=0;
break;
}
}
if (ergebnis.size() == 1) { // Wenn die Größe vom Stack "ergebnis" 1 beträgt und der Stack am Ende angelangt ist,..
double finErgebnis = ergebnis.peek(); // ..soll mit .peek() das Ergebnis angezeigt werden.
System.out.print(" Ergebnis: ");
System.out.println(finErgebnis);
System.out.println(" ");
}
}
}
public static void eingabefeld() {
System.out.println(" ********************");
System.out.print(" Eingabe:");
} // "Eingabe:" vor der Benutzereingabe
public static void willkommen() {
System.out.println(" ");
System.out.println(" U P N-TASCHENRECHNER");
} // Begrüßung
public static void ungueltigeEingabe() {
System.out.println(" Ungültige Eingabe. \n" +
" Bitte erneut versuchen!");
} // ungültige Benutzereingabe
public static void exponenteingeben() {
System.out.print(" Geben Sie einen " +"\n" +
" Exponenten ein: ");
} // Exponent eingeben
}Ich habe mal versucht switch-case einzubauen, wahrscheinlich wieder etwas vergessen oder nicht beachtet.. jetzt gibt er die Rechnung nicht aus
Java:
import java.util.Scanner;
import java.util.Stack;
public class UPNRechner {
public static void main(String[] args) {
Scanner scan = new Scanner(System.in); // Benutzereingabe
Scanner scanner = new Scanner(System.in); // zwei Scanner - dieser hier für den "Tokenizer"
Stack<Double> ergebnis = new Stack(); // Stack mit dem Namen "ergebnis" erstellt
willkommen(); // Überschrift/Begrüßungstext
/*
while-Schleife, welche den Großteil des Taschenrechners beinhaltet
*/
int i = 0;
while (true) {
eingabefeld(); // Eingabefeld (Text)
System.out.print(" ");
String nextLine = scanner.nextLine();
Scanner tokenizer = new Scanner(nextLine); // der "Tokenizer" dient zum Zerlegen der Benutzereingabe
i = 0;
ergebnis.clear();
for (int j = 0; tokenizer.hasNext(); j++) {
i++;
String nutzereingabe;
nutzereingabe = tokenizer.next();
double zahl = 0.0;
/*
In dieser try-catch-Anweisung wird die
"Umwandlung" von binären und hexadezimalen
Codes in Integer ausgeführt. Ebenfalls werden
Fehlermeldungen generiert, die eine ungültige
Eingabe bestätigen, und interne Fehlermeldungen
seitens IntelliJ, die das Programm ungewollt
beenden, werden beseitigt.
*/
try {
if (nutzereingabe.startsWith("0x") || nutzereingabe.startsWith("0X"))
zahl = Integer.parseInt(nutzereingabe.substring(2), 16);
else if (nutzereingabe.startsWith("-0x") || nutzereingabe.startsWith("-0X"))
zahl = Integer.parseInt(nutzereingabe.substring(3), 16);
else if (nutzereingabe.startsWith("0b") || nutzereingabe.startsWith("0B"))
zahl = Integer.parseInt(nutzereingabe.substring(2), 2);
else if (nutzereingabe.startsWith("-0b") || nutzereingabe.startsWith("-0B"))
zahl = Integer.parseInt(nutzereingabe.substring(3), 2);
else zahl = Double.parseDouble(nutzereingabe);
}catch (NumberFormatException e) {
if (nutzereingabe.equals("+")
|| nutzereingabe.equals("-")
|| nutzereingabe.equals("*")
|| nutzereingabe.equals("x")
|| nutzereingabe.equals("/")
|| nutzereingabe.equals(":")
|| nutzereingabe.equals("÷")
|| nutzereingabe.equals("pow")
|| nutzereingabe.equals("cos")
|| nutzereingabe.equals("sqrt")
|| nutzereingabe.equals("√")
|| nutzereingabe.equals("sin")
|| nutzereingabe.equals("cos")
|| nutzereingabe.equals("ln")
|| nutzereingabe.equals("~")
|| nutzereingabe.equals("e")
|| nutzereingabe.equals("tan")
|| nutzereingabe.equals("exp")
|| nutzereingabe.equals("pi")
|| nutzereingabe.equals("exit")) {
}else{
ungueltigeEingabe();
i=0;
break;
}
}
/*
In der folgenden try-catch-Anweisung sind die Funk-
tionen der Rechenoperatoren definiert - ebenso wird
hier geprüft, ob die Eingabe des Benutzers richtig
ist und/oder der Reihenfolge entspricht, in der diese
sein soll. Falls sich die Eingabe nicht um den Typ
Double oder um einen definierten Rechenoperatoren,
etc. handelt, soll für den Benutzer eine Fehlermeldung
erscheinen, die den Nutzer dazu auffordert, die
Handlung in korrekter Weise zu wiederholen.
*
*/
/*Hier wurde ein Double "ergebnisTmp" (temporäres
Ergebnis) erstellt. Dieser "speichert" den letzten
Wert im bzw. "auf" dem Stack. Durch "ergebnis.pop()"
wird eine Zahl vom Stapel genommen, mit "ergebnis.push()"
hinzugefügt.
*/
try {
Double ergebnisTmp = ergebnis.pop();
switch (nutzereingabe) {
case "+":
ergebnis.push(ergebnis.pop() + ergebnis.pop());
break;
case "-":
ergebnis.push(ergebnis.pop() - ergebnis.pop());
break;
case "*":
ergebnis.push(ergebnis.pop() * ergebnis.pop());
break;
case "x":
ergebnis.push(ergebnis.pop() * ergebnis.pop());
break;
case "/":
ergebnis.push(ergebnis.pop() * ergebnis.pop());
break;
case ":":
ergebnis.push(ergebnis.pop() * ergebnis.pop());
break;
case "÷":
ergebnis.push(ergebnis.pop() * ergebnis.pop());
break;
case "sqrt":
ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.sqrt(ergebnisTmp));
break;
case "√":
ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.sqrt(ergebnisTmp));
break;
case "sin":
ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.sin(ergebnisTmp));
break;
case "cos":
ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.cos(ergebnisTmp));
break;
case "tan":
ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.tan(ergebnisTmp));
break;
case "~":
ergebnisTmp = ergebnis.pop();
ergebnis.push((ergebnisTmp * (-1)));
break;
case "exp":
ergebnisTmp = ergebnis.pop();
ergebnis.push(Math.pow(Math.E, ergebnisTmp));
break;
case "pi":
ergebnis.push(Math.PI);
break;
case "π":
ergebnis.push(Math.PI);
break;
case "e":
ergebnis.push(Math.E);
break;
case "pow":
ergebnisTmp = ergebnis.pop();
exponenteingeben();
ergebnis.push(Math.pow(ergebnisTmp, scan.nextDouble()));
break;
case "exit":
System.exit(0);
break;
default:
ergebnis.push(zahl);
}
}catch (Exception e) {
ungueltigeEingabe(); // ungültige Eingabe
i=0;
break;
}
}
if (ergebnis.size() == 1) { // Wenn die Größe vom Stack "ergebnis" 1 beträgt und der Stack am Ende angelangt ist,..
double finErgebnis = ergebnis.peek(); // ..soll mit .peek() das Ergebnis angezeigt werden.
System.out.print(" Ergebnis: ");
System.out.println(finErgebnis);
System.out.println(" ");
}
}
}
public static void eingabefeld() {
System.out.println(" ********************");
System.out.print(" Eingabe:");
} // "Eingabe:" vor der Benutzereingabe
public static void willkommen() {
System.out.println(" ");
System.out.println(" U P N-TASCHENRECHNER");
} // Begrüßung
public static void ungueltigeEingabe() {
System.out.println(" Ungültige Eingabe. \n" +
" Bitte erneut versuchen!");
} // ungültige Benutzereingabe
public static void exponenteingeben() {
System.out.print(" Geben Sie einen " +"\n" +
" Exponenten ein: ");
} // Exponent eingeben
}
Zuletzt bearbeitet:
Meniskusschaden
Top Contributor
Einfach so? Oder gibt es auch eine Fehlermeldung?
Also mit der integrierten Fehlermeldung "Ungültige Eingabe........."
Meniskusschaden
Top Contributor
Dann würde ich - zumindest vorübergehend - in den catch-Blöcken
e.printStackTrace() aufrufen. Im StackTrace müsste man die Ursache dann sehen.ich habe das mit dem e.printStackTrace() gemacht, da kommt dann folgende Fehlermeldung:
U P N-TASCHENRECHNER
********************
Eingabe: 3 3 +
java.util.EmptyStackException
at java.base/java.util.Stack.peek(Stack.java:102)
at java.base/java.util.Stack.pop(Stack.java:84)
at UPNRechner.main(UPNRechner.java:114)
Ungültige Eingabe.
Bitte erneut versuchen!
********************
Eingabe:
U P N-TASCHENRECHNER
********************
Eingabe: 3 3 +
java.util.EmptyStackException
at java.base/java.util.Stack.peek(Stack.java:102)
at java.base/java.util.Stack.pop(Stack.java:84)
at UPNRechner.main(UPNRechner.java:114)
Ungültige Eingabe.
Bitte erneut versuchen!
********************
Eingabe:
Meniskusschaden
Top Contributor
Ja, aber jetzt hast du Informationen über die Ursache. Ein StackTrace kann sehr lang werden. Deshalb würde ich mir die eigentliche Meldung ansehen (java.util.EmptyStackException) und die erste Zeile, die zu einer selbst programmierten Klasse gehört (Zeile 114). Da liegt das Problem.
Hmm ich versteh das nicht so genau.. bin eben ein Anfänger
....
try {
Double ergebnisTmp = ergebnis.pop();
switch (nutzereingabe) {
case "+":
ergebnis.push(ergebnis.pop() + ergebnis.pop());
break;
....
das rote soll der Fehler sein... hmm
....
try {
Double ergebnisTmp = ergebnis.pop();
switch (nutzereingabe) {
case "+":
ergebnis.push(ergebnis.pop() + ergebnis.pop());
break;
....
das rote soll der Fehler sein... hmm
Meniskusschaden
Top Contributor
In der Zeile willst du dir offenbar etwas vom Stack holen, was aber nicht funktioniert, weil er gerade leer ist. Warum möchtest du an der Stelle denn etwas vom Stack holen?
Ok, das hat sich soweit geklärt 
Beim switch-case braucht man kein temporäres Ergebnis (ergebnisTmp), ich weiß nicht ob es anders auch so gegangen wäre, aber hier geht das jetzt auch nur mit dem ergebnis.pop().
Beim switch-case braucht man kein temporäres Ergebnis (ergebnisTmp), ich weiß nicht ob es anders auch so gegangen wäre, aber hier geht das jetzt auch nur mit dem ergebnis.pop().
Jetzt wäre der Rechner eigentlich fertig von den Funktionen,
das andere Problem, in dem das Ergebnis trotzdem gekommen ist
habe ich gelöst, in dem ich den alten Code genommen habe,
wahrscheinlich ist mir da etwas Falsches reingerutscht.
Jetzt hätte ich noch einige Wissensfragen, da ich auch kurze Kommentare
über die Funktion schreiben muss:
Was hat dieser Block genau für eine Funktion? Danke schonmal.
das andere Problem, in dem das Ergebnis trotzdem gekommen ist
habe ich gelöst, in dem ich den alten Code genommen habe,
wahrscheinlich ist mir da etwas Falsches reingerutscht.
Jetzt hätte ich noch einige Wissensfragen, da ich auch kurze Kommentare
über die Funktion schreiben muss:
Was hat dieser Block genau für eine Funktion?
Danke schonmal.
Java:
...
for (int j = 0; tokenizer.hasNext(); j++) {
i++;
String nutzereingabe;
nutzereingabe = tokenizer.next();
double zahl = 0.0;
...Nur noch kurz eine Frage zu dem Rechner:
Wenn ich zum Beispiel nur eine Zahl wie zum Beispiel 5 eingebe
und Enter drücke, kommt einfach nochmal "Eingabe:" ohne dass
es eine Fehlermeldung "Ungültige Eingabe" ausgibt. Wie kann ich
das auch hinbekommen?
Wenn ich zum Beispiel nur eine Zahl wie zum Beispiel 5 eingebe
und Enter drücke, kommt einfach nochmal "Eingabe:" ohne dass
es eine Fehlermeldung "Ungültige Eingabe" ausgibt. Wie kann ich
das auch hinbekommen?
Meniskusschaden
Top Contributor
Für mich wäre das eine gültige Eingabe. Wenn man 5 eingibt kommt eben 5 heraus. Was ist daran falsch?
Problematisch finde ich eher, dass kein Ergebnis angezeigt wird, wenn der Stack noch mehrere Elemente enthält. Ein echter UPN-Rechner lässt die einfach liegen. Vielleicht will man ja später etwas damit machen. Ich würde beispielsweise folgendes erwarten:
Code:
>3 3 x 4 4 x
Stack: 9, 16
Anzeige: 16
>+
Stack: 25, Anzeige: 25Also der Prof. hat gemeint, dass unser Projekt nur das Endergebnis anzeigen soll. Wäre auch fürs erste Semester etwas weit hergeholt, finde ich.
Kommt eben nicht raus, sondern es fängt einfach eine neue Rechnung an.
Meniskusschaden
Top Contributor
Ist dadurch nicht schwieriger. Man löscht den Stack eben nicht und gibt immer das oberste Element aus. Wäre meines Erachtens sogar ein wenig einfacher.
Kann ich nicht nachvollziehen. Bei mir sieht es so aus, wenn ich 5 eingebe:
Code:
U P N-TASCHENRECHNER
********************
Eingabe: 5
Ergebnis: 5.0
********************
Eingabe:Meniskusschaden
Top Contributor
Das widerspricht meinem Beispiel ja nicht. Den Stack habe ich zum Verständnis angegeben. Ausgegeben würde nur jeweils der Anzeigewert.
Meniskusschaden
Top Contributor
Übrigens rechnet dein Programm bei nicht kommutativen Operationen nicht mehr richtig:
Code:
U P N-TASCHENRECHNER
********************
Eingabe: 5 2 -
Ergebnis: -3.0
********************
Eingabe:Habe den Fehler gefunden, war in dem Fall Minus und Plus vertauscht...
Da stimm ich @Meniskusschaden zu, das wäre eher leichter geworden
Das Problem ist eher was anderes, nur in diesem Spezialfall ist’s +/- vertauscht...
Probier mal dividieren (wenn es denn jetzt implementiert ist), da sollte das auch auftreten.
Oh ja, hast vollkommen recht.. ich habe in der switch-Anweisung einfach beim Minus, wie hier schon empfohlen wurde, ein Minus vor das Ergebnis.pop()... wie löse ich das am Besten?
Oh man, kannst du mir nicht vielleicht helfen? Die Abgabe ist in wenigen StundenHabe es rausgefunden, danke.
Ok, das wird dann wohl meine allerletzte Frage zu diesem Thema sein:
Kann man zu dieser Art von Code bzw. ohne Methoden ein oder zwei
JUnit-Tests einbauen? Zum Beispiel dass wenn die Nutzereingabe 3 3 +
ist, dass 6 rauskommt, oder wenn man 3 3 * 4 4 * + sqrt (sqrt steht für Wurzel) eingibt,
das Ergebnis bzw. Stack 5 ist? Das wäre wirklich das letzte, dann wäre ich fertig ich versuche das schon seit Stunden
Kann man zu dieser Art von Code bzw. ohne Methoden ein oder zwei
JUnit-Tests einbauen? Zum Beispiel dass wenn die Nutzereingabe 3 3 +
ist, dass 6 rauskommt, oder wenn man 3 3 * 4 4 * + sqrt (sqrt steht für Wurzel) eingibt,
das Ergebnis bzw. Stack 5 ist? Das wäre wirklich das letzte, dann wäre ich fertig
ich versuche das schon seit Stunden
X
Xyz1
Gast
Kannst du nochmal von vorne anfangen? Wofür steht RPN?
X
Xyz1
Gast
(R)PN konnte ich schon erraten, aber das (R) -(copyright ist nicht damit gemeint )- sagte mir noch nichts...... also was normales Posfix-notation? POSTFIXNOTATION
Bearbeitung: Also - "die schlechten ins Kröpfchen, die guten ins Töpfchen" und dann nochmal wiederholen pull en und push en.
)- sagte mir noch nichts...... also was normales Bearbeitung: Also - "die schlechten ins Kröpfchen, die guten ins Töpfchen" und dann nochmal wiederholen pull en und push en.
Zuletzt bearbeitet von einem Moderator:
Ähnliche Java Themen
Ähnliche Java Themen
-
KErste Schritte "Taschenrechner" zeigt keine Komma Zahlen an.
- Gestartet von Kleechen
- Antworten: 8
-
-
Frage zu einem "Taschenrechner" code
- Gestartet von Jeremy....
- Antworten: 9
-
Operatoren Rechenoperation verwenden für Taschenrechner.
- Gestartet von Rimsemessine
- Antworten: 32
-
-
Taschenrechner mit mehr als 2 Zahlen.
- Gestartet von Johannes73
- Antworten: 18
-
-
-
Prüfen von Komma stelle beim Taschenrechner- Gestartet von Fiedelbambu
- Antworten: 5
-
-
-
GTaschenrechner ergibt Fehler in if-Abfrage
- Gestartet von Gelöschtes Mitglied 69333
- Antworten: 6
-
Potenzberechnung über switch case. Taschenrechner mit Eingabe über einen grafischen Dialog
- Gestartet von cocosch
- Antworten: 22
-
JavaFX Taschenrechner Eingabe im Textfield beschränken
- Gestartet von eiligerwind
- Antworten: 2
-
Taschenrechner GUI Problem mit Fehlerhandling
- Gestartet von eiligerwind
- Antworten: 6
-
Taschenrechner (switch) in Taschenrechner mit Methoden umwandeln
- Gestartet von Codix
- Antworten: 115
-
-
-
-
Taschenrechner mit GUI Problem bei der Berechnung
- Gestartet von 27Flo09
- Antworten: 8
-
Taschenrechner mit unendlich vielen Eingabemöglichkeiten
- Gestartet von Panda9296
- Antworten: 1
-
Erste Schritte Taschenrechner mit beliebig vielen Zahlen
- Gestartet von vulturas
- Antworten: 5
-
-
-
Compiler-Fehler Basic- Taschenrechner/ Fehler
- Gestartet von shimsho
- Antworten: 8
-
-
-
Erste Schritte Frage zu simplem Taschenrechner(switch)
- Gestartet von FloristenFloris
- Antworten: 16
-
Mein Taschenrechner lässt sich plötzlich nicht mehr öffnen
- Gestartet von leclerc16
- Antworten: 5
-
-
-
Variablen Taschenrechner mit Komfortfunktionen
- Gestartet von Redbeard
- Antworten: 2
-
-
Taschenrechner mit switch und while funktioniert noch nicht richtig
- Gestartet von L.E.O.C.Y.
- Antworten: 22
-
-
Taschenrechner NullPointerException
- Gestartet von RegenSonne
- Antworten: 1
-
Variablen Weitergehenderr Taschenrechner
- Gestartet von RegenSonne
- Antworten: 4
-
Taschenrechner "Error loading class..."
- Gestartet von Fubix
- Antworten: 5
-
-
Erste Schritte Taschenrechner mit if-else Anweisung
- Gestartet von philthad
- Antworten: 6
-
-
-
-
Kleiner Taschenrechner! Anfänger braucht Hilfe :(
- Gestartet von Jython
- Antworten: 2
-
-
Erste Schritte Java CMD Taschenrechner mit Parametern!
- Gestartet von javabeginner123
- Antworten: 16
-
-
Taschenrechner problem! Ich komme nicht voran
- Gestartet von ExceptionError
- Antworten: 18
-
Erste Schritte Taschenrechner mit Schleife
- Gestartet von rp1
- Antworten: 24
-
(Taschenrechner Netbeans) Komma nur einmal schreiben & 0 ersetzen- Gestartet von Lord.Djerun
- Antworten: 1
-
Taschenrechner - Additions/Substraktions/Multiplikations/Divionsketten!
- Gestartet von Zitrus
- Antworten: 4
-
-
Sehr simpler Taschenrechner - Hilfe!
- Gestartet von Zitrus
- Antworten: 10
-
Taschenrechner - Wie addiere,multipliziere und subtrahiere ich?
- Gestartet von Zitrus
- Antworten: 6
-
kleiner Taschenrechner mit switch fehlerhaft
- Gestartet von diehigh
- Antworten: 1
-
-
-
-
-
Erste Schritte Taschenrechner - Punkt-vor-Strich Rechnung
- Gestartet von Ralf.mei
- Antworten: 1
-
Operatoren Taschenrechner Ergebnis ausgeben
- Gestartet von Semiih58
- Antworten: 7
-
-
-
-
-
z = input.nextChar(); PROBLEM Taschenrechner
- Gestartet von fiabilite03
- Antworten: 7
-
-
Variablen Rechenzeichen und Zahlen verknüpfen für einen Taschenrechner
- Gestartet von Jänö
- Antworten: 17
-
Klassen Java Taschenrechner mit Klassen und Methoden
- Gestartet von Blablablub100
- Antworten: 12
-
Erste Schritte Brauche Hilfe bei einem Java-Taschenrechner
- Gestartet von Blablablub100
- Antworten: 11
-
-
-
Taschenrechner - Methoden implementieren
- Gestartet von Isnogud
- Antworten: 5
-
-
-
-
taschenrechner - Fehler beim Kürzen eines Bruches finden
- Gestartet von kirkikor
- Antworten: 20
-
JTaschenrechner Punkt vor Strich
- Gestartet von JavaLehrling
- Antworten: 9
-
A
-
Ausklappbares Fenster bei grafischem Taschenrechner
- Gestartet von marvin.mai
- Antworten: 2
-
Dividieren bei Grafischem Taschenrechner klappt nicht
- Gestartet von marvin.mai
- Antworten: 2
-
-
-
-
-
-
-
-
Taschenrechner-GUI mit BorderLayout?
- Gestartet von Astaroth
- Antworten: 6
-
[PROJEKT] Simpler grafischer Taschenrechner
- Gestartet von Woom
- Antworten: 8
-
-
Grafischen Taschenrechner programmieren - Frage dazu
- Gestartet von Jack159
- Antworten: 12
-
-
-
-
-
-
-
FDatentypen Ganz simpler Taschenrechner mit switch
- Gestartet von fragesteller
- Antworten: 10
Aktuelle Jobs
-
Wafios Softwareentwickler Cloud (JAVA/Spring) m/w/d - Reutlingen- Gestartet von Java_Jobs
- Antworten: 0
-
Inek Senior Java Softwareentwickler (m/w/d) - Siegburg
- Gestartet von Java_Jobs
- Antworten: 0
-
Atos Senior Java Architekt (m/w/d) - Fürth
- Gestartet von Java_Jobs
- Antworten: 0
-
-
FCA Bank Java-Entwickler (m/w/d) (Heilbronn)
- Gestartet von Java_Jobs
- Antworten: 0
Neue Themen
-
-
Breite und Höhe aller Monitore ausgeben.
- Gestartet von JavaRentner
- Antworten: 10
-
-
Signierte Exe Datei funktioniert nicht mehr
- Gestartet von AltesEisen
- Antworten: 1
-