Skip to content
Benjamin Reitzammer edited this page Jul 30, 2013 · 1 revision

Treffen #8 der Softwerkskammer Rhein-Main

Datum: 27.08.2012, 19 Uhr
Ort: Bei Namics

Agenda

  • Vorstellungsrunde
  • Lightning Talks
  • Code Kummerkasten
  • Software Archäologie

Protokoll

Lightning Talks

Code Kummerkasten

Gregor fragt wie man DI und non-DI Teile einer Klasse gut mischen kann. Einhellige Erkenntnis: Geht nicht gut.
WebCast von Greg Young dazu (DI) http://skillsmatter.com/podcast/home/simple-is-better.

Software Archäologie

Wir haben uns das MCB Projekt von Andreas Leidig angeschaut, nachdem er es uns mit den folgenden Worten auf der Mailingliste schmackhaft gemacht hat.

Ist ein kompletter Eclipse Workspace. Das Projekt, um das es geht ist MCB. Die obersten drei Projekte haben nix damit zu tun, sind aus Schlamperei mit drin. :) Die anderen braucht Ihr aber. Der zentrale Umbau, der stattgefunden hat, war, die Persitenzschicht von MySQL auf filebasiertes JSon umzustellen. Ob ich das sauber aufgeräumt habe, weiss ich nicht. Ansonsten gibts noch ein paar andere "ungewöhnliche" Ansätze im Code zu sehen. Ich hoffe, es gruselt Euch nicht zu sehr.

Was macht der Code?

nach etwa 45 Minuten habe ich den Entschluss gefasst, dass es sich bei der Software um eine Frühstücksverwaltung handelt muss, da sich der Großteil des Quellcodes mit der Berechnung / Erfassung von Frühstücksbuchungen beschäftigt.  
-- [@coderbyheart](http://twitter.com/coderbyheart)

Verbesserungsvorschläge?

Also, wenn ich es nicht neu schreiben könnte, würde ich beim mcb.model.Treffen anfangen und dort aufräumen:

  • E-Mail-Zeugs komplett auslagern

  • die ganzen get*String() auslagern

  • hart kodiertes wie in isGespann() konfigurierbar machen

  • generell das Bennenungsschema einheitlich englisch halten

  • setString(String val) in set(String val) refactoren

  • Dokumentieren!

    Wenn man damit anfängt und das auf die anderen Klassen überträgt würde das dem Verständnis schon deutlich helfen.
    -- @coderbyheart

Erkenntnisse?

Ein bisschen Quellcode-Dokumentation hätte soviel weiterhelfen können.
-- [@coderbyheart](http://twitter.com/coderbyheart)