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

change shuffle to include enum instead of integer encoding #767

Open
bjornregnell opened this issue Oct 30, 2023 · 1 comment
Open

change shuffle to include enum instead of integer encoding #767

bjornregnell opened this issue Oct 30, 2023 · 1 comment

Comments

@bjornregnell
Copy link
Member

Many had problems with enums at the diagnostic test. One way to improve knowledge about enums is to make shuffle use them from start instead of starting off with integer encoding and then have an optional task on enums.

@jonmellerby
Copy link

This seems like a good idea to me. It's probably more educational to write good code and then reflect on why it's good, than to purposely write suboptimal code and reflect on why it's bad.

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