search

owenashurst / agar.io-clone

Agar.io clone written with Socket.IO and HTML5 canvas
MIT License
2.88k stars 1.12k forks source link

Modularization #424

Open aadilzbhatti opened 8 years ago

aadilzbhatti commented 8 years ago

I think it would be a good idea to break this project up into separate modules to ease the process of development/testing and also make it easier for incoming developers to be able to understand the project. I have already begun doing this in #421, #422, and #423 but I think there is more to be done. I foresee the project having one "main" module where the overall game is ran, including all other modules, and the set of modules which control the different components of the game (game logic, chat client, canvas, socket handlers, etc).

abalabahaha commented 8 years ago

Thanks for contributing! I do agree that separating functionality into different modules is easier to understand/work with than a large mixed JS file.

Also, I would suggest that If you're going to continue work on modularization, I think it would be better to make one larger pull request with a big batch of commits rather than a bunch of small PRs.

aadilzbhatti commented 8 years ago

No problem! And will do. Thanks for the feedback.