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

Group Model Class #8

Closed
jconnuck opened this issue Apr 27, 2011 · 3 comments
Closed

Group Model Class #8

jconnuck opened this issue Apr 27, 2011 · 3 comments
Assignees

Comments

@jconnuck
Copy link
Owner

We need some way to model a group, which is different from a club.

@ghost ghost assigned wzimrin Apr 27, 2011
@wzimrin
Copy link

wzimrin commented Apr 27, 2011

The problem is that we also want to be able to specify a group in the source code. I can add a group class that is similar to the club class, but then we have to keep track of the id of the group, which is annoying, at least. It is much prettier to have groups be enums or strings, with a DataStore.getPeopleForGroup() method. Currently, I am using strings, though I can swap over to enums very easily if necessary.

@wzimrin
Copy link

wzimrin commented Apr 27, 2011

Unless I am missing something, the only operations that groups need to support is getMembers and getName. getMembers is implemented just fine as DataStore.getPeopleForGroup() and getName is the string for a string and .toString for an enum.

@jconnuck
Copy link
Owner Author

Okay, no problem. Sorry I should have looked deeper into your code.

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

No branches or pull requests

2 participants