Daniel95
commented
8 years ago Heey, bedankt voor de feedback! De feedback staat bij de master geschreven, waar darren's tank game uit matchmaker 1 op staat. Is deze feedback bedoelt voor de tank game die darren in matchmaker 1 heeft gemaakt, of voor onze one button game die we hebben gemaakt voor matchmaker 2?
Je hebt feedback gekregen van fisser op:
URL: https://github.com/DarrenVs/NodeJavascriptFramework/blob/master/public/main.js
Feedback: Goed bezig! spel net gespeeld op kantoor met wat collega's en ze vonden het leuk. Code ziet er goed uit en is duidelijk. Wel paar opmerkingen: Main.js doet wel veel... Het is zeker mogelijk om dit op te splitten.
Kijk is naar Prototype in javascript Info links: (https://developer.mozilla.org/nl/docs/Web/JavaScript/Inheritance_and_the_prototype_chain, https://developer.mozilla.org/nl/docs/Web/JavaScript/Reference/Global_Objects/Object/prototype, http://sporto.github.io/blog/2013/02/22/a-plain-english-guide-to-javascript-prototypes/)
Ook in index.html staan veel script tags naar alle javascript bestandjes. Dit kan veranderd worden naar 1 file met behulp van bijvoorbeeld GruntJS(of GulpJS). link: http://gruntjs.com/ , http://gulpjs.com/
Zou ook aanraden om een package.json te maken in de root directory zodat npm install werkt(hoeft men niet apart (npm install express, socket.io) te doen) Link: https://docs.npmjs.com/getting-started/using-a-package.json
Over het algemeen zeer goed project en goed bezig