GithubHelp home page GithubHelp logo

prgm8's People

Contributors

goef avatar oduivestein avatar

Watchers

 avatar

Forkers

oduivestein

prgm8's Issues

Peer Review Shane Severins

Klassendiagram
logisch dat het nog mist, mijn tip is wel om deze bij te houden zodat je niet alles op het einde hoef te doen.

Singleton:
De game heeft op dit moment geen Singleton. Mijn tip is deze toe te passen om levens toe te voegen aan je game en als deze om nul komen een "gameOver()" functie aan te roepen die in je game object zit en deze dan vanuit een ander class aanroepen. (misschien kunnen er meer manieren zijn om een game over te krijgen waardoor de singleton nog relevanter wordt )

Polymorfisme

  1. Je hebt gebruik gemaakt van een gameobject die verschillende classes kan aanspreken.
  2. Je kunt gebruik maken van het keyboardobject om zo meerde type children aan te spreken, ga voor de zekerheid even naar een leraar om te vragen of dit telt als polymorfisme. Omdat je het keyboardObject niet direct extend maar gebruik maakt van een "has a" in plaats van "is a".

Strategy
Nog niet op de juiste manier toegepast, wel in de goede richting. je kan er bijvoorbeeld voor kiezen om de class "character" een behavior element te geven die de functie update aanroept, en dit element laten veranderen wanneer er aan bepaalde criteria wordt voldaan. Zo kun je de moveObject een behavior maken en een crouch behavior maken die allebei gekoppeld zijn aan het behavior element in character.

Observer
nog niet toegepast. Wat je eventueel kan toevoegen om het wel te halen zijn wapens of spells die alle vijanden op het scherm tegelijkertijd aanvallen of bevriezen. Je moet er dan voor zorgen dat alle vijanden op het scherm een obeserver zijn en je character/wapen een subject. Wanneer je dan een wapen oppakt of gebruikt alle observers damage krijgen.

Peer Review

LEVEL ONE

Ontvankelijkheids eisen

✅ Broncode staat op GitHub
❌ Geen link om het online te spelen
❌ Geen installatieinstructies in ReadMe

LEVEL TWO

Klassendiagram

❌ Er is nog geen klassendiagram

Peer review

✅ Er staat een peer review in de ReadMe

Singleton

❌ Er is nog geen singleton toegepast
Hint: Dit is goed te doen voor Game

Polymorfisme

❌ Je hebt op 1 plek polymorfisme, namelijk het gameObject. Als je het KeyboardObject omschrijft kan je het makkelijk zo maken dat het ook onder dit criteria valt. Succes!

Strategy

❌ Je hebt nog geen strategy. Je kan dit goed toepassen op het item wat je kan oppakken. Misschien een power up?

Observer

❌Je hebt nog geen observer.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.