Bitte um Hilfe bei nem Taschenrechner

[v2elite]

hat sich erledigt danke

 

[Marco13]

Hm. Nach einer vollendeten Rechnung wie
1+2=
ist er

            if (anfang)
            {
                ...
            }
            else
            {
                if(operation.equals("C"))
                {
                    ....
                }
                else
                {
                    // ============================================ HIER
                    berechne(Double.parseDouble(anzeige.getText()));
                    letzteOperation = operation;
                    anfang = true;
                }

angekommen - dort setzt er "Anfang" auf "true", d.h. wenn man direkt danach "C" drückt, landet er garnicht mehr bei der Abfrage nach "operation.equals("C")".

Hilft das?

 

[v2elite]

Hi

ja danke jetzt weiss ich wenigstens wo der Fehler liegt ...

Das Problem ist aber nun wenn ich es weglasse dann mach er zwar einen reset dafür kann er aber nicht mehr rechnen ^^

wenn ich dann 9 + 9 eingebe gibt er 9.09 aus also er hängt an die erste 9.0 einfach die zweite 9 an

 

[Marco13]

Das Denken wollte ich eigentlich dir überlassen aber ... ich habe ja nichts von "weglassen" gesagt. Ganz pragmatisch könnte man einfach die Abfrage
if(operation.equals("C"))
als erstes machen (d.h. egal, ob "anfang" nun true oder false ist).

Du solltest dir aber genau überlegen und genau klarmachen, wie du die verschiedenen Zustände so eines Rechners beschreiben und "konsistent halten" kannst. Wenn der Rechner (oder ein anderes Programm) mal ein bißchen komplizierter werden, wird das immer wichtiger...

 

[v2elite]

So habe es dank deiner hilfe nun halbwegs zum laufen gebracht

Es hat aber noch einen Wurm drin ...

und zwar wenn ich die rechnung so eingebe: 1+2*2-3=..... funktioniert es und es kommt -3 raus wenn ich dann = drücke

wenn ich aber 1 + 2= -3 = eingebe müsste ja 0 rauskommen, es kommt aber -3 raus ...

Hier der aktuelle Code

 

[v2elite]

weiss niemand rat ?

 

[Marco13]

Du solltest dir aber genau überlegen und genau klarmachen, wie du die verschiedenen Zustände so eines Rechners beschreiben und "konsistent halten" kannst. Wenn der Rechner (oder ein anderes Programm) mal ein bißchen komplizierter werden, wird das immer wichtiger...

Genau deswegen Überleg' dir, welche Zuständes es geben kann, und wodurch der Rechner von einem Zustand in den anderen übergehen kann. Spätestens bei
1+2*2-3=....
müßtest du ja auch Punkt-Vor-Strich-Rechnung berücksichtigen - das kann man dann nur mit einer Art "Puffer" machen, der einige der letzten Zahlen/Ergebnisse und die Operation dazwischen speichert.

Ein Beispiel:

1
+
2
+ Sobald das gedrückt wurde, kann 1+2 ausgerechnet und als 3 gespeichert werden

D.h. man hat dann im Puffer:
3
+
...

Im anderen Fall:

1
+
2
* Hier kann noch NICHTS ausrechnet werden - man muss die nächste Eingabe abwarten
3
+ Jetzt kann 1+2*3 ausgerechnet werden

Im Puffer steht dann
7
+
...


Noch ein anderes Beispiel:

1
+
2
* Hier kann noch NICHTS ausrechnet werden - man muss die nächste Eingabe abwarten
3
* Hier kann jetzt NUR 2*3 ausgerechnet werden.

Im Puffer steht dann
1
+
6
*
...


Ideal wäre, eine Klasse "Rechner" zu erstellen, der OHNE das GUI funktioniert - das kann dann noch ein bißchen schwieriger werden, aber wäre sehr gut in bezug auf eine saubere Trennung von GUI und eigentlicher Rechner-Funktionalität...

 

[v2elite]

hi

Der Taschenrechner soll ganz einfach sein ohne punkt vor strich... wenn man 1+2 dann * eingibt dann soll er 1+2 berechnen und dann einfach mal die nächste zahl die kommt berechnen

und zur zeit ist irgendwo ein wurm drin --> jedes zeichen ist eine eingabe:

1 + 2 = - 3 = nun soll er also 3-3 rechnen als ergebnis kommt aber -3 raus das verstehe ich nicht

Das mit der Klasse Rechner ist eine gute Idee wie setze ich das nun am besten um ? Also dass die Rachnung ohne GUI funktioniert

Ich bin noch ein ziemlicher noob

 

[Marco13]

Hab' gerade nicht viel Zeit, nur der Allgemeine Tipp: Mal in den ActionPerformed-Methoden solche Ausgaben machen wie

System.out.println("Command war "+command+" letzteOperation war "+letzteOperation .... usw)

dann findet man die Ursache leichter.

Die Rechner-Klasse - die Grundidee wäre eine Klasse, die das Rechnen übernimmt, und nur durch Methodenaufrufe von der GUI aus gesteuert wird - bei einem so einfachen Rechner "lohnt" sich das kaum, aber bei komplexeren Sachen "müßte" man das so machen, d.h. es wäre eine Gute Übung, sich zu überlegen, wie man das aufbauen könnte... Bei Fragen kann ich vielleicht am Wochenende noch was dazu sagen