brandonyuen / prg01-8_game Goto Github PK

Strictly seen, you could improve your code by using getters and setters to access instances of libraries (like PIXI) defined in class Game. In your case, only a getter would be appropriate since other classes are not allowed to overwrite these libraries.

Singleton

The singleton pattern has been implemented in the Game class but you forgot to make the constructor private.

When implementing the singleton pattern the constructor should be private. If the constructor is not private, anyone can make an instance of Game and it would no longer be a singleton. For a singleton by definition there can exist only one instance.

Polymorphism

Currently you are using polymorphism in one place in your code. Inheritance is used since class Player extends from GameObject. For the final assessment you will need to have 2 implementations of polymorphism. Do remember that polymorphism used in the strategy or observer pattern does not count as an implementation of polymorphism.

Strategy

It looks like you made a start with the strategy pattern, though your implementation does not seem to be finished. You have an interface for different types of movement, but there is only one type of movement: walking. In the player class you define the type of movement at:

private movement:Movement = new Walking(this)

The strategy pattern is correctly implemented, you just need to add another type of movement that you can switch between.

Observer

Oberver pattern has not been implemented yet.

Your game is looking good so far! Keep up the good work.

Je hebt heel goed nagedacht over wat je in je game wilt hebben en maakt goed gebruik van tools die beschikbaar zijn. Met de combinatie van PIXI, Howler & Tiled heb je een hele toffe combinatie aan Game Aesthetics!

Je hebt de meeste patterns volledig verwerkt in je game:

• Je Game is een singleton waardoor er maar één game aan te kunnen maken. (Voor Player geldt hetzelfde!)
• De Entity implementeert een Subject en Gun implementeert de observer. Wanneer een Entity zich update doen de observers van die Entity dat ook.
• Doordat een Entity een Movement heeft (die bijvoorbeeld van het type Walking is maar ook een ander type zou kunnen hebben) en aangepast zou kunnen worden. Hiermee pas je de Strategy pattern correct toe. Zorg er in ieder geval voor dat je uiteindelijk in runtime van Movement kan wisselen.
• De basis van Polymorphism heb je: je Player is een Entity en een Pistol is een Gun.

Je mist nog twee dingen: Type Assertion en Type Guards. Op dit moment ben je nog een is stage van development dat dit niet optimaal is toe te passen. Maar denk bijvoorbeeld aan mogelijkheden voor
type guards wanneer je Enemies gaat toevoegen. Als je ook nog van plan bent om menuschermen te implementeren kan type Assertion bij clickEvents ook erg van pas komen.

Ik ben heel benieuwd naar het eind resultaat van je game, zeker met de kwaliteit van je animaties van nu! Success met het laatste stuk 😸