Code Review JavaFX Movie Application

[josroskam]

Hi,

Hoewel ik net zei geen specifieke review te willen hebben ben ik toch benieuwd wat je vind van de implementatie van de:

1. BaseController
2. SceneController
3. Context (model) - Deze gebruik ik om verschillende objecten mee te geven aan de scenes (bijvoorbeeld: sceneController.changeScene("Login", new Context(null, null, inMemoryDatabase));)

Ik heb weinig affiniteit met JavaFX, dus hoop ik dat deze classes van voldoende kwaliteit zijn.

Verder ben ik nog erg benieuwd wat je vind van de fxml files, en of het project in het algemeen voldoende is voor het tentamen maandag.

Alvast bedankt!

[burgeraw]

Ha Jos, Ziet er goed uit.

• Je kunt iets duidelijker maken welke menu opties ge-disabled / enabled zijn, bijvoorbeeld door ze uit te faden als ze niet beschikbaar zijn.
• Ook moet je je database met initiële data populaten.
• inMemoryDatabase wordt 2 keer geset in MovieTheaterApplication line 50&53
• is de instance reference ergens voor nodig? MovieTheaterApplication line 54
• BaseController line 24; je catch, maar doet er niets mee.
• Het is me niet helemaal duidelijk waarom je zowel een BaseController als een NavigationController nodig hebt.
• de changeScene in SceneController kan gewoon bij Base en/of NavigationController toch? Kijk even naar de opdrachten van week 6, de code die daar is gegeven, daarbij hebben ze alleen een MainController, en dan Controllers voor de specifieke views. Jij hebt 3 Controllers, waar denk ik 1tje genoeg zou kunnen zijn.
• ik denk dat je zonder Context zou kunnen, maar ik ben er niet boos over.

In het kort denk ik dat je het al prima hebt gedaan. Mark zal vast nog andere punten voor je hebben, maar ik denk dat hij het ook prima zal vinden. Succes met het tentamen!