JirkaDellOro / Softwaredesign

Modul "Softwaredesign": Lektionen und Übungsaufgaben
12 stars 12 forks source link

Durchsicht Jirka Lektion 04 #37

Open JirkaDellOro opened 4 years ago

JirkaDellOro commented 4 years ago

Das wir das Kapitel zweimal da haben ist sehr ungünstig, da man jetzt zwei mal das gleiche Pflegen muss. Ich habe gerade in der C#-Section den detail-Tag getested, der den Kollaps erlaubt. Bitte also das Kapitel wieder auf eine Seite zusammenführen und die Code-Beispiele in solchen Sectionen unterbringen. Siehe unten

Test collapsible section

TypeScript (click to expand) ``` typescript this.isTypeScript("Code"); ```
C#(click to expand) ``` c# this.isCSharp("Code"); ```

-->fixed!

JirkaDellOro commented 4 years ago

Ebenso verfahren mit dem letzten Abschnitt "Konventionen" und für TypeScript meine einsetzen, wenn sie auch für Christoph (Fleig) passen.

-->Habe den Code überprüft und er scheint den Konventionen zu entsprechen (englisch, bezeichnende Namen für Variablen und Funktionen, lowerCamelCase). Hab ich was vergessen?

JirkaDellOro commented 4 years ago

Im Kapitel, gleich zu Anfang, darauf aufmerksam machen, dass Code viel häufiger gelesen als geschrieben wird.

-->fixed

JirkaDellOro commented 4 years ago

https://github.com/JirkaDellOro/Softwaredesign/tree/master/L04_CleanCode/TypeScript#magic-numbers

800 und 799 widersprechen sich

-->fixed

JirkaDellOro commented 4 years ago

die TypeScript-Beispiele müssen den Konventionen folgen also camelCase. Außerdem müssten die Funktionen mit dem Schlüsselworf function deklariert werden. Es sei denn, es sind Methoden, solche wurden aber bislang gar nicht besprochen und sind für diese Beispiele auch nicht zwingend erforderlich.

-->fixed!

Dabei fällt auf, dass Funktionen im Kapitel Programmieren1 gar nicht behandelt wurden. Das sollte geändert werden!

--> Habe nach bestem Gewissen einen Abschnitt bei L03 hinzugefuegt. Nach den foreach - Schleifen

JirkaDellOro commented 4 years ago

https://github.com/JirkaDellOro/Softwaredesign/tree/master/L04_CleanCode/TypeScript#single-responsibility-principle

Das eine Änderung in dieser Klasse soll keine andere Klasse betreffen.

Satzbau ergibt keinen Sinn, "Das" muss wohl weg.

-->fixed!

JirkaDellOro commented 4 years ago

https://github.com/JirkaDellOro/Softwaredesign/tree/master/L04_CleanCode/TypeScript#open-closed-principle

Ertweiterungen

-->fixed!

JirkaDellOro commented 4 years ago

https://github.com/JirkaDellOro/Softwaredesign/tree/master/L04_CleanCode/TypeScript#interface-segregation-principle

Darum sollten alle Funktionen der implementierenden Klasse nutzen.

Unverständlich

-->fixed!