Skip to content
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.

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

Refactor the code #29

Merged
merged 5 commits into from
Nov 17, 2023
Merged

Refactor the code #29

merged 5 commits into from
Nov 17, 2023

Conversation

w0rm
Copy link
Owner

@w0rm w0rm commented Nov 16, 2023

This time it increases the code, but I made some changes that make things nicer:

  • moved entities into the Bodies module
  • moves sizes into the Bodies module
  • renamed the custom types
  • collapsed the ball in hand with the ball behind the head string — this has simplified the code quite a bit!
  • added comments to a few places in the code
  • removed cueBallPosition from the playingState in favour of reading it from model.world
  • renamed playingState to cue

@w0rm w0rm merged commit 7da4d67 into main Nov 17, 2023
1 check passed
@w0rm w0rm deleted the refactor4 branch November 17, 2023 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant