search

kahhe / pe

Repo for potential bugs I found for CS2113T Practical Exam
0 stars 0 forks source link

Hang Man: only accepting single letters #7

Open kahhe opened 2 years ago

kahhe commented 2 years ago

Seems like a oversight of not allowing the kids to guess the word and the satisfaction from pulling it off in a single command? Finding the game rewarding can help encourage learning. Inputs accepted might be too inflexible here.

Yay! You have found the letter
You can have 5 more wrong attempts
Your guess word is:
algorit--
Enter your guess letter:
h

Yay! You have found the letter
You can have 5 more wrong attempts
Your guess word is:
algorith-
Enter your guess letter:
algorithm

You have already guessed this letter. Try something else!
You can have 5 more wrong attempts
Your guess word is:
algorith-
Enter your guess letter:
m
nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Guessing letter game takes in multiple letters when it should ideally take in only one

image.png

Perhaps it would be better to indicate this as an invalid input because hangman technically only should accept one letter?

[original: nus-cs2113-AY2122S1/pe-interim#2445] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Hi, thank you for your advice. It is shown in the game message. Only accept the first letter.

pe-1.png

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: I understand the game only allows for one letter at a time. My concern was also not with the absence of an error message to indicate that one shouldn't type multiple letters.

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I was thinking of playing hang man in class when we were younger. Often you can shout the word,
once you have figured out the word and no longer needed to guess it letter by letter. The teacher
would accept the answer. Here I wish the app could also mimic that. Allow children to take a chance
of guessing the word, at the risk of getting it wrong and wasting a chance that could have been used to
guess a letter instead. This would have made the dynamics of the game more interesting.

So this was a matter of game design, rather than error messages and whatnot.

:question: Issue severity

Team chose [severity.Medium] Originally [severity.Low]

Reason for disagreement: Severity might have been changed for the 'original bug', but I have given my reason that the 'original bug' and my bug are different.