Skip to content

2. Hausaufgabe YG#64

Open
Goekalp wants to merge 6 commits intoProfWider:masterfrom
Goekalp:master
Open

2. Hausaufgabe YG#64
Goekalp wants to merge 6 commits intoProfWider:masterfrom
Goekalp:master

Conversation

@Goekalp
Copy link

@Goekalp Goekalp commented Oct 29, 2020

Bevor ich alles nochmal sauber machen muss, wollte ich prinzipiell wissen, ob das so korrekt ist?

Aus Angst evtl alles zu löschen ist der erste rote Test doppelt drinnen, aber bitte den zweiten roten test beachten.

Tut mir leid für die späte Abgabe.

@DanielWagner87 DanielWagner87 self-requested a review October 29, 2020 11:43
@DanielWagner87 DanielWagner87 self-assigned this Oct 29, 2020
Copy link
Collaborator

@DanielWagner87 DanielWagner87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht soweit nicht schlecht aus. Die Prozentfunktion deckt aber keinen Fehler in der Implementierung auf. sondern fügt eine neue Funktion hinzu.

Hier wäre es schön, wenn Sie noch einen dritten Test samt Bugfix hinzufügen können. Versuchen Sie doch mal zwei zweistellige Zahlen miteinander zu addieren.

@Goekalp
Copy link
Author

Goekalp commented Oct 29, 2020

Danke sehr Herr Wagner, in der Übungsstunde heute beim Herrn Wider meinte er, die Funktion Prozent ist okay, da dies in unserem Calculator noch nicht funktioniert hatte und man das ergänzt hat. Zudem meinte Herr Wider, da er es nicht genauer in der Beschreibung erklärt hat, wäre es okay.
Meine Frage: ist das aktuell dann "Bestanden" und ich soll trotzdem noch ein dritten Bugfix machen oder gilt es momentan als "nicht bestanden"?
@DanielWagner87 @ProfWider

@DanielWagner87
Copy link
Collaborator

Wenn das mit Herrn Wider abgesprochen ist, dann passt das auch für mich.

@Goekalp
Copy link
Author

Goekalp commented Oct 29, 2020

Ich hab mich vllt. falsch ausgedrückt, der Herr Wider hatte es mir in der Übung so erklärt, aber ich weiß nicht, ob ich es in der Umsetzung auch so richtig gemacht habe, dass er es als "bestanden" bewertet oder nicht. Das entscheidet ja er und/oder sie.

Mir ging es oben nur darum, ob ich ein dritten Bugfix zum bestehen brauche oder nicht. Auch wenn ich das bestanden habe, könnte ich ein dritten Bugfix machen.

@DanielWagner87
Copy link
Collaborator

Jetzt haben Sie das Bestanden Label erhalten. Wenn Sie mögen, steuern Sie gerne noch einen dritten Bugfix hinzu.

@Goekalp
Copy link
Author

Goekalp commented Oct 29, 2020

Sehr Gerne. Vielen Dank. Mach ich dann, nach meinen restlichen Vorlesungen heute.

Schönen Tag noch 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants