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

Lab2 Scrabble doesn't have a test for the first and last letter of the alphabet #56

Open
TumblingCoconut opened this issue Sep 1, 2022 · 0 comments

Comments

@TumblingCoconut
Copy link

TumblingCoconut commented Sep 1, 2022

I had submitted the lab yesterday because it initially passed all of the tests, but today while referencing back to it I realized a mistake that ignores z in calculating the score. The if statement that I used to check for letters looks at the lowercase ascii values was written as such letter > 96 && letter < 122 . I had fixed the issue for the letter a, but forgotten for the letter z.

There should be a test case for the first and last letter because those are the letters that would be most likely left out in the declared range by accident.

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