khcbaude
commented
3 years ago Zoals besproken met Koen is de MainController verantwoordelijk voor de scherm navigatie en de LevelController voor de dingen in het level. Verder heb ik nog gekeken naar tests, maar nu kan ik niet veel testen, doordat bijna alles private is en geen getters heeft. Ik zou die kunnen toevoegen/veranderen maar de daadwerkelijk game heeft die functies niet nodig. Wat denken jullie hierover?
Mooi 👍 qua tests is het mogelijk fps te testen of is dat niet echt te doen? Bijvoorbeeld testen of als je 10 keer per seconde iets tekend de fps counter ook 10 zegt.
Voor de schermnavigatie had ik wat op whatsapp gestuurd maar ik zal het hier ook nog even benoemen.
Het is denk ik netter als de level controller kan unsubscriben wanneer het niet meer wordt getoond. Zo krijgt de level controller ook geen onbedoelde updates dus dat zou eventuele fouten kunnen voorkomen. Ook is het unsubscriben handig voor bijvoorbeeld het main menu of de game over en game gewonnen schermen, zo zou je bijvoorbeeld kunnen herstarten met enter ofzo.
Zoals besproken met Koen is de MainController verantwoordelijk voor de scherm navigatie en de LevelController voor de dingen in het level. Verder heb ik nog gekeken naar tests, maar nu kan ik niet veel testen, doordat bijna alles private is en geen getters heeft. Ik zou die kunnen toevoegen/veranderen maar de daadwerkelijk game heeft die functies niet nodig. Wat denken jullie hierover?