OOP technieken

[Eviltoastey]

Misschien een beetje laat maar ik moet nog een peer review doen voor mijn herkansing. Het meeste is al gefixt de code zag er namelijk goed gestructureerd en duidelijk uit. Toch zal ik wat feedback achter laten ookal is deze grotendeels positief.

Strategy pattern
Ook dit ziet er gewoon netjes uit. Je wisselt van shooting behaviours als er een upgrade wordt opgepakt. De default shooting behaviour is de single shot waar de speler altijd mee begint.

Polymorfisme
De classes asteroid.ts, bullet.ts, bomb.ts, asteroidParts.ts en ship.ts extenden allemaal van gameobject.ts wat ze allemaal een GameObject maakt. Door het gebruik van instanceof is het mogelijk om door alle GameObjects heen te lopen, zo kan er makkelijk op collision gecontroleerd worden.

Alle onderdelen die getekend moeten worden zijn gameobjects. Hierdoor is het makkelijk om polymorphisme te gebruiken bij het checken van wat voor type het object is. Ik heb polymorphisme zelf ook op deze manier toegevoegd in de collision checker.

Omdat deze structuur zo duidelijk was heb ik zelf een fastshoot behaviour gemaakt. Ik maak mijn strategy pattern ook altijd zoals jij dit doet dus ik heb hier verder geen commentaar op.

Singleton
De meeste mensen hebben een singleton op de game manager omdat hier altijd maar 1 van gemaakt wordt. Het werkt en het klopt maar je zou bijvoorbeeld ook andere managers kunnen maken.

Observer pattern
netjes de asteroids laten registeren en unregisteren als ze kapot geschoten zijn.

conclusie
Zoals ik boven al heb geschreven, de code ziet er netjes en duidelijk uit. Het was voor mij erg makkelijk om er veranderingen in te maken. Voor zover mijn kennis gaat kloppen alle patterns ook en zijn ze goed geïmplementeerd. Je zou de game wel iets uitgebreider kunnen maken met extra upgrades en verschillende soorten asteroids die ook hun eigen behaviour hebben.

Een leuke toevoegen zou een random asteroid factory kunnen zijn.