HA2

[Aurielle-t]

No description provided.

[ProfWider]

Sie sind spät dran! Zwei Dinge: die Commits sollten aufeinander aufbauen, d.h. im letzten Commit sollten 4 Tests und zwei Bugfixes gleichzeitig im Code sein und alle zusammen miteinander funktionieren. Soweit ich das überblicken kann widersprechen sich Ihre Bugfixes nicht, insofern ok. Das andere: eigentlich müssten Sie noch mindestens eine der funktionen pressClearKey, pressDotKey, oder pressNegative testen (habe ich zumindest auf Slack so formuliert). Aber auch hier ok, auf Moodle steht das nicht ganz explizit so. Also: gerade so bestanden! Aber stellen Sie bitte doch ruhig nochmal einen neuen PR (also nochmal neu forken, klonen, etc.), wo die Commits aufeinander aufbauen. Ist aber jetzt nicht schlimm, wenn das nicht mehr vor Mitternacht ist.

[ProfWider]

Achja, Ihr letzter Bugfix ist auch wirklich abenteuerlich - es ist zwar super, dass Sie die Division durch 0 abgedeckt haben, aber mit dem aktuellen Bugfix würde der Calculator-Screen bei 5 minus 4 auch Error anzeigen, da hätten Sie lieber so etwas wie NaN (not a number) nehmen können, um diesen Fall zu kommunizieren. Aber gut, auch hier, die Tests, die da sind, sind grün, also reicht das aus. Und ich freu mich, dass Sie es (gerade so) noch geschafft haben.