Meine Lösung der Aufgabe

[MK-NEUKO]

Hallo Jan,
hier meine Lösung zu dem Kurs.
Ich habe mir noch keine Lösungshinweise, oder gar deine Musterlösung angesehen.
Ich freue mich auf dein Feedback.

[MK-NEUKO]

@suchja
Klopf, Klopf ?
Ich will nochmal an mein Issue erinnern.
Mittlerweile ist es 8 Tage her, vielleicht hast du viel zu tun, aber wenn man garkeine Rückmeldung bekommt, hatt man den Eindruck, dass nichts angekommen ist.
Alles Gut, Grüße Michael

[suchja]

@MK-NEUKO Moin Michael,
entschuldige die Wartezeit. Bin gerade ziemlich ins Hintertreffen mit der Beantwortung der Fragen in den Kursen gekommen. Es gibt halt auch Nachteile, wenn plötzlich die Teilnehmerzahlen explodieren ;-)!

Zu deiner Lösung:

• Ich konnte sie so komplett von oben nach unten lesen und habe sofort verstanden was du machst. SUPER!
• Im Vergleich zu meiner "Musterlösung" ist deine Lösung wesentlich strukturierter, weil du mehr in Methoden gepackt hast. Auch wenn das nicht unbedingt Teil der Übung ist bzw. für mich nicht im Fokus stand, ist das natürlich super.
• Mir gefiel der Methodennamen CheckForLettersOnly nicht ganz so gut. Insbesondere wenn eine Methode ein bool zurück gibt, ist häufig ein Methodenname mit Is... also z.B. IsLetter sehr aussagekräftig. (wirklich nur eine Kleinigkeit!!!)
• Für den Algorithmus würde es noch ein paar Optimierungsmöglichkeiten geben. Beispielsweise brauchst du nicht unbedingt ein Array mit den Buchstaben. Das wäre jedoch maximal eine Performanceoptimierung und die ist bei dem Programm und mit den heutigen Rechnern nun weiß Gott nicht notwendig.
• Die Verwendung einer RegEx könntest du dir sparen, weil es beispielsweise Char.IsLetter gibt. Für mich persönlich habe ich immer das Ziel, dass ich Code so einfach wie möglich erstellen möchte. Das muss für dich nicht so sein und es kann eben so gut sein, dass dein Ziel war unbedingt RegEx mit dieser Übung auszuprobieren.

Wenn du dir meine Musterlösung jetzt mal anschaust, dann wirst du noch einige Unterschiede erkennen. Wichtig ist jedoch, dass du bei der Übung etwas gelernt hast und das du eine lauffähige und gut strukturierte Lösung gefunden hast.

[MK-NEUKO]

@suchja Hallo Jan
Vielen Dank für dein Feedback.
Das die Teilnehmerzahlen explodieren freut mich für dich, Corona ist daran bestimmt nicht ganz unschuldig. Vielleicht musst du dir Verstärkung holen.
Zwei Tage nachdem ich Aufgabe gemacht habe, habe ich alle übrigen Videos geguckt.

• IsLetter hat mich echt geärgert, die Methode hab ich übersehen, obwohl ich danach gesucht habe. Regex habe ich kurz vorher erst verwendet, damit war das dann schnell erledigt.
• Deine Musterlösung des Algorithmus habe ich in meinen Vorüberlegungen auch angestrebt. Beim Entwickeln habe ich überlegt, wie ich die erforderliche Rechnerrei um im Alphabet zu bleiben, umgehen kann und mir kam die Idee mit dem Array.
  Ich habe diese Lösung genommen, weil ich sie als besser lesbar empfunden habe.

Nachwievor macht mir das Entwickeln und Lernen jede Menge Spaß. Ich freue mich schon auf das intensive Lernen sobald ich meine Sachen erledigt habe.