Skip to content

Organize classes into new packages #4

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

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

Organize classes into new packages #4

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

Comments

@justjwheelin
Copy link
Member

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!

@justjwheelin
Copy link
Member Author

@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.

@justjwheelin
Copy link
Member Author

@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