Drazhar / lost_spc

Library for statistical process control during the lecture of the OST in Buchs
GNU General Public License v3.0
0 stars 4 forks source link

Alles was i han. #6

Closed HansPeter21 closed 2 days ago

HansPeter21 commented 5 days ago

Hilfsklassen:

Drazhar commented 4 days ago

Oha, das ist ziemlich vieles auf einmal 😅

Dennis und ich haben auch ein paar PRs erstellt. Ich hatte geplant diese morgen zu Beginn der Vorlesung durch zu sprechen. Dann gehen wir auch deinen durch, wenn es für dich okay ist.

Generell sind es zwei Sachen die direkt auffallen:

  1. Einen StorageManager innerhalb von Funktionen zu verwenden ist mega gefährlich. Generell sollte probiert werden Funktionen in sich abgeschlossen zu lassen ohne globalen State (Variablen) zu verändern. Das führt sonst zu Problemen beim Debuggen und zu sogenannten Spaghetti Code.
  2. Die Ermittlung der Kontrollgrenzen beinhaltet viel Copy&Past. Evtl könnte man das in schlankere Funktionen verpacken.

Das können wir dann gerne besprechen. Wenn es dir lieber ist auch vor der Vorlesung.

Drazhar commented 2 days ago

Zu viel und nicht aufgeräumt. Probiere bitte die Funktionen etwas sauberer in die Bibliothek einzubauen, ohne den Storage_manager.