starblaster69
commented
2 years ago klassendiagram hebben we inderdaad niet, eentje aanmaken zou wel handig zijn nu dat het spel steed scomplexer wordt. de comments by Quest tracker kloppen, aangezien dit nog heel erg een work in progress is haha! dit wordt zeker gefixt als ik er verder mee ga vandaag. get & set zijn inderdaad nog niet echt bestaand, omdat ze nog niet echt nodig waren. inderdaad geen link nee! vergeten helaas, als je wilt zou npm start gewoon moeten werken, however
4 Ik zie meerdere classes aangemaakt met andere eigenschappen, zoals npc of player. Er is in GitHub geen Klassendiagram aanwezig.
Ik mis comments bij stukken code, als je als voorbeeld het bestand Quest tracker neemt, dan staat er alleen //variables en //behaviour. Wat de code verder doet, geen idee
5 Dit gaat gaat goed, ik zie geen objecten die nog in een apart bestand geplaatst moeten worden.
6 Encapsulation gaat ook goed, in elke class worden zowel public als private variables gebruikt. Alleen zie ik nog geen get & set gebruikt worden (kan zijn wel en niet goed gekeken, of niet nodig gevonden)
7 dit gaat ook goed, er wordt goed x extends PIXI.Sprite gedaan en ik zie nergens dat super() mist.
9 ik heb de game nog niet kunnen spelen, er stond in de readme geen link.