search

JWTP / PRG4GAME

0 stars 0 forks source link

Feedback #1

Open perrydrums opened 7 years ago

perrydrums commented 7 years ago

Code is netjes en to-the-point. Ik kan meteen zien wat er gebeurd. Het spel werkt als behoren.

Let wel wat meer op wanneer je wel en niet een puntkomma ( ; ) gebruikt. Dit doe je de ene keer wel, andere keer niet. Blijf consequent!

Ik snap niet waarom je een Clicker en Clickers class heb. De Clickers erven over van Clicker maar daar zie ik het nut niet van in. Je maakt de Clickers objecten in je Game en vult daar alle parameters in. Ik zie de functie van de Clicker class niet.

De alert die je krijgt als je iets niet kan betalen is best irritant. Dit is een clicker game en die moet snel kunnen lopen zonder onderbrekingen van een popup.

Voor de rest super nette code en heel leesbaar. 👍

perrydrums commented 7 years ago

Het spel mist een eindscherm maar dat is begrijpelijk kijkend naar het soort spelletje. Een eindscherm is niet echt van toepassing.

perrydrums commented 7 years ago

Ik zie nu (20-06-17) nog geen Readme bestand. Deze moet je nog even toevoegen!

Ook een link naar het spelletje is handig. (Ik had hem maar gekopieerd)