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

Organize classes into new packages #4

Open
jwflory opened this issue Mar 2, 2015 · 4 comments
Open

Organize classes into new packages #4

jwflory opened this issue Mar 2, 2015 · 4 comments
Assignees

Comments

@jwflory
Copy link
Member

jwflory commented Mar 2, 2015

Seeing as how IceBall has a large number of classes, it would make more sense to organize the classes into multiple packages based on the type of function the class performs, e.g. "commands", "mobs", "gameplay", etc.

This is not connected to a milestone as it has no effect on the actual plugin itself, but it is a general convention that should be followed when possible to prevent messy code.

@wood73
Copy link
Contributor

wood73 commented Mar 2, 2015

Love the idea!

@jwflory
Copy link
Member Author

jwflory commented Mar 2, 2015

@woodrow73
For a good example of this, take a look in CrystalSpace and see how his classes are laid out in multiple packages.

@wood73
Copy link
Contributor

wood73 commented Mar 2, 2015

I see.. even better organization. Packages to hold the class's to hold the methods to hold the fields lol. I plan on doing so in the future for sure; it's part of the game programming tutorial I'm taking as well.

@jwflory
Copy link
Member Author

jwflory commented Mar 2, 2015

@woodrow73
Precisely, hahah! Anyways, it's something that might be worthwhile since there's a good number of packages in this plugin. Hopefully it would make maintaining this easier!

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

No branches or pull requests

2 participants