search

LernMoment / einstieg-csharp-taschenrechner

Quellcode für den Taschenrechner welcher im Udemy Kurs "Einstieg in C#" erstellt wird.
http://www.lernmoment.de/einstieg-csharp/
15 stars 84 forks source link

Überprüft Grenzwert für die fortlaufende Berechnung #103

Closed Ial0kin closed 2 years ago

Ial0kin commented 3 years ago

Hallo @suchja, Nachdem ich nun endlich ein pull request hinbekomme, die Ergänzung zu der Grenzwertüberprüfung.

suchja commented 2 years ago

@Ial0kin Auch wenn es einige Monate gedauert hat 🙄, habe ich deinen PR nun endlich akzeptiert.

Als Verbesserung würde ich noch vorschlagen, dass die Do-While-Schleife nicht in der Methode HoleEingabenFuerErsteBerechnungVomBenutzer, sondern in HoleZahlVomBenutzer umgesetzt wird. Ähnlich habe ich es für den Operator auch gemacht. Das hat den Vorteil, dass du einiges von dem doppelten Code in HoleEingabenFuerErsteBerechnungVomBenutzer eliminieren kannst. Da du allerdings wahrscheinlich schon weitergezogen bist und hier nicht unbedingt noch etwas machen möchtest, lasse ich es einfach mal so und wir schauen ob jemand anderes sich der Sache annimmt.

suchja commented 2 years ago

Vielen Dank für deine Mühe. Ich hoffe du hast auch etwas gelernt dabei.