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

Rewrite checks? #10

Open
WouterVrielink opened this issue Nov 6, 2020 · 0 comments
Open

Rewrite checks? #10

WouterVrielink opened this issue Nov 6, 2020 · 0 comments

Comments

@WouterVrielink
Copy link
Member

WouterVrielink commented Nov 6, 2020

De checks zijn op het moment erg onoverzichtelijk. Gameloop en checks lopen door elkaar, waardoor het super lastig is om te begrijpen wat er precies gecheckt wordt, en wat er wel/niet gebeurt. Dit gaat in Adventure een stuk beter, daar heb je meer mogelijkheden voor input, maar is het alsnog bijzonder leesbaar.

Daarnaast is er nu (minstens) een verschil in aanpak tussen de gameloop die gegeven wordt in de opdracht en die in de checks: in de gegeven code wordt altijd if len(letter) != 1 or not game.is_valid_guess(letter): vóór game.guess() gedaan, terwijl game.is_valid_guess() helemaal niet gebruikt wordt in de checks. Dit veroorzaakt soms nare bugs die niet goed te vinden zijn als je de checks niet goed kent. Ik zou persoonlijk hier zeggen dat de aanpak van de checks logischer is, en dat game.is_valid_guess() een functie is die alleen intern in .guess() wordt aangeroepen.

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

1 participant