aaronvark / PeerReview1819

Repo for peer review assignments for year 2 development class of 18/19
0 stars 0 forks source link

Chantal van der Voort - Opdracht Blok 1 #20

Closed Chan92 closed 4 years ago

Chan92 commented 5 years ago

Mijn concept is een trefbal breakout.

Het is een breakout game waar stenen naar beneden vallen die je moet ontwijken. Doe je dit niet, zal je plankje breken en ben je game over.

Chan92 commented 5 years ago

There are still a lot bugs that have to be fixed, i tried using an interface and a more controllable version of object pooling as i usually used.

UML

aaronvark commented 5 years ago

Qua class verdeling heb je al een best aardige opzet waarin classes duidelijk verdeelde responsibilities hebben. Ik denk dat de meeste leeropbrengst te halen is uit het experimenteren met alternatieve methoden om je references te verkrijgen. Nu hanteer je eigenlijk overal het "public / serializable field waar je iets op sleept" om instances te delen tussen classes. Dit veroorzaakt wel een hoge mate van afhankelijkheid van de "scene setup", en het is in die zin vrij makkelijk om issues te veroorzaken zonder dat je dit in de code kunt waarborgen.

Vooral mezelf probeer ik altijd de regel aan te houden dat je wel referenties "binnen een hierarchy" kunt instellen op deze manier (bijv. een parent heeft referenties naar child objecten, of andersom), maar liever niet daarbuiten (maar ik verbied het verder niet). In code zou je de in-hierarchy references kunnen vergelijken met "GetComponentInChildren" of "GetComponentInParent". Referenties naar andere scene objects (buiten de eigen hierarchy) probeer ik zo veel mogelijk te vermijden, of op een andere manier te verkrijgen (bijv. een Singleton, of het zo ontwerpen dat de grotere dingen kleinere zaken "spawnen", en dan zichzelf als variabele (meestal een interface) meegeven).

Je code is verder goed leesbaar, en je comments zijn to-the-point en verhelderend.

Chan92 commented 4 years ago

Link to my Github: https://github.com/Chan92/Y2K1