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

consider dataclass for LifePath/Character #61

Open
miron opened this issue Feb 13, 2023 · 0 comments
Open

consider dataclass for LifePath/Character #61

miron opened this issue Feb 13, 2023 · 0 comments

Comments

@miron
Copy link
Owner

miron commented Feb 13, 2023

Given the heterogenous data, a dataclass could make it easier
Another possibility is to put Lifepath into json file and put random logic into Character class
Decision Flowchart:

                   ___ Homogenous ___ Homogenous or Heterogenous? ___ Heterogenous ___
                  |                                                                   |
      ___ Scalar or Collection? ___                                    ___ Do you have Invariants? ___
     |                             |                                  |                               |
     | Scalar                      | Collection                       | No                            | Yes
     |                             |                                  |                               |
Enumeration                    Dictionary                         Data class                        Class
@miron miron changed the title consider dataclass for LifePath consider dataclass for LifePath/Character Mar 11, 2023
@miron miron added this to NeonCore Mar 11, 2023
@miron miron moved this to 📋 Backlog in NeonCore Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant