[REVIEW] Veranstaltungen

Dao:
Ich glaube, da hat jemand zu viel Speicher zu Verfügung: strings unnötig konkatenieren, und das auf die Art und Weise mit der höchsten Rate an unguten Memory-Instruktionen? Keine schöne Idee. C-Programmierer bekommen für so etwas einen Eisenbahner.

Die Suche wäre ebenso etwas effektiver, wenn wir zumindest Separatoren auch noch durch etwas flexibleres ersetzen würden. (Bier Fest != Bierfest != Bier-Fest aus den Systemtests) Da könnte man sich vlt. noch eine Vorbehandlung des Suchbegriffs überlegen.


Service:
Dein Parameter-Handling in findVeranstaltungen() gehört mmn. in das Dao.


UI:
"//Rebuld Table" schreit nach Auslagerung in eine Methode, da die Funktionalität nur eine Notwendigkeit und keine Aufgabe der Suche ist.


Dieses Review liest sich btw nicht ohne Grund ca. so wie das für die Aufführungen.