s0574094 Marco Hamiti /PullRequest/Unvollständig

[Hamiti97]

Ich stelle schonmal einen PR, obwohl mir der letzte BugFix fehlt.
Irgendwie bekomme ich diesen Fehler nicht raus denn er addiert immer nur die letzten zwei DigitKeys und ignoriert den DotKey sowie den ersten Digitkey bei drei Eingaben.
Würde mich über nen kleinen Tipp freuen damit ich den letzten BugFix schnell abwickeln kann bis Freitag.
MfG

[ProfWider]

Gut, dass Sie schon mal den PR gestellt haben. Die Commit-Struktur ist super und Ihnen fehlt in der Tat nur noch der letzte Bugfix, den Sie dann auch einfach zu dem bestehenden Commit hinzufügen können.
Zu Ihrem letzten Test: schauen Sie sich doch mal die Zeilen 21-23 in der Calculator Klasse an, also der else-Zweig in den man hineinkommt, wenn zuvor eine Operation (in Ihrem Fall +) eingegeben wurde. Dort wird vorige Stand des Screens in die Variable latestValue gesichert und dann wird einfach der Screen-Inhalt mit der zuletzt eingegebenen Zahl ersetzt. Das muss an Hilfe reichen ;-)
Viel Erfolg!

[Hamiti97]

Also ich hab es geschafft den letzten Test grün zu bekommen jedoch habe ich jetzt Probleme mit dem ersten da er nicht mehr 2+2 normal rechnet, sondern die Strings "2" + "2" = 22. Ich habe es versucht mit verschiedenen If-Bedingungen zu lösen aber irgendwie klappt nichts. Ich Benötige die Screen-Ausgabe (Letzter Test)3,4 und (Erster Test)2 jedoch bekomme ich nur 3.4 und 22. Wenn man den Test so aufstellt das man zuerst eine Kommazahl + eine normale Zahl rechnet, bekommt man direkt einen Grünen Test zudem verwirren mich die ganzen Methoden ich sehe immer nur die einzelnen Fehler auf Ubuntu da mein IntelliJ nicht richtig funktioniert.

[ProfWider]

Ja, wenn Ihr Bugfix andere Tests wieder kaputt macht deutet das leider darauf hin, dass es kein echter Fix ist, da er nicht allgemeingülitg ist. Sie brauchen zum Bestehen aber einen Zustand, bei alle Tests gleichzeitig grün werden. Versuchen Sie es weiter. Ich denke Sie können das noch schaffen! Meiner Ansicht nach, haben Sie bei dem neuen Fix zu kompliziert gedacht, das lässt sich einfacher lösen und auch ohne negative Effekte auf andere Tests. Dass Sie nicht von IntelliJ ausführen können, ist kein Problem. Es ist sogar gut, dadurch sind Sie gezwungen immer alle Tests mit gradle test auszuführen und sehen sofort wenn woanders wieder etwas kaputt geht.

[Hamiti97]

So wahrscheinlich nicht die beste und eleganteste Lösung aber alle Test sind Grün

[ProfWider]

Ja, es gewinnt jetzt keinen Schönheitswettbewerb (da sind jetzt noch zwei Variablen, die gar nicht genutzt werden), aber die Tests sind grün, insofern bestanden 👍
Wie auch für die anderen: Wenn Sie Lust haben können Sie jetzt, wo Sie die richtige Lösung ja haben, einfach mal neu forken, clonen, dann Stück für Stück alles rüber ziehen, dabei immer schöne saubere Commits erzeugen, und dann einen neuen PR stellen.