Code Review Philipp Merkle

Durchgeführt am 24.03.2011

Teilnehmer

Allgemeine Fragen

Fragen zur Funktionsweise:

Wie wird Stack verwaltet? Ist an Event angehängt mit nächster Aktion die ausgeführt werden sollte.

Wie könnte reliability eingehängt werden -> verwende dekorator auf aktuelle strategien.

Allgemeine Anmerkungen

Architekturbild hätte hilfeich sein können, allgemeine Funktion von event-basierter Simulation fehlt

Liste der unterstützten Funktionen.

Packet struktur simulation evtl zu allgemein. Sequenz-Diagramm für besseres Verständniss. Verbesserungen des Javadoc für Strategien. Roadmap für die Weiterentwicklung

Wie werden unchecked Errors gelogged von der workflow engine.

Code Kommentare

EventSim model ist so etwas wie ein Blackboard. Sollte dieses modell irgendwo abgelegt werden? Kein Singleton für parallel sim

Braucht evtl. einen Cleanup, da sehr Funktionalität enthalten.

Abhängigkeiten zu SSJ und Desmoj. Vieles gesäubert noch Abhängigkeiten zum Scheduler und evtl. Reliability. Sollte später über Extensionpoint gebunden werden?!

EventSimEntity 37
- idGenerators geht auch ein long für alles

Request 46
- was macht das activationEvent? Bindemittel zw. scheduler und rest

66
- evtl wie prozesse erstellt werden rausziehen an strategie, threadpool oder an Nutzer binden etc. Konfigurierbar machen. Verfälscht momentan evtl. Ergebniss des Schedulers.

Sefftraversal 105:
- getStack methode schlecht benannt.

Kleinigkeiten

Packet name abstractenegine: Kleinbuchstaben vs CamlCase.

Registries konsistente Namensgebung verwenden. AllocationRegistry, SimulatedResEnv.

Coding conventions, Checkstyle sollten angewendet und angepasst werden.

Atomics könnten evtl entfernt werden?!

ICommand und IPCMCommand Mehrwert