New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactoring #112

Open

5 tasks

daniel-wer opened this issue Apr 14, 2014 · 0 comments

Labels

enhancement must have

Collaborator

daniel-wer commented Apr 14, 2014

game event messages (duplicate code, ...)
bikeview, bikecontroller, motionstate, bikemodel, bikeinputstate (long methods, unclear dependencies, hard to extend, ...)
maybe an event based approach for model/view communication would be an improvement
no methods in controller that directly call model XOR view methods, public getter for model/view
can the multiplayer integration be reworked - get rid of all the if multiplayer statements (traits?!)

daniel-wer added enhancement labels

philippotto added a commit that referenced this issue


          #112 major refactoring of updateState in BikeModel

81e1909

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment