LWachtel1 / pe

0 stars 0 forks source link

Multi-character guess not recognised #2

Open LWachtel1 opened 2 months ago

LWachtel1 commented 2 months ago

image.png

image.png

Hangman game allows single character guesses and whole word guesses as specified in the in-game tutorial and in the UG.

However, hangman game does not allow multi-character guesses that are shorter than whole word length; please look at the photos where I guess "fr" and hangman does not recognise this as correct, but guessing 'f' and 'r' separately is accepted.

Adding multi-character guesses that are shorter than whole word length as an additional feature would be nice as whole word guesses are already allowed.

soc-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Seems like an invalid alphabet input is counted as a guess

When I key in "zz" or other duplicated alphabets, it is counted as a guess.

Perhaps can add in a validation such that only inputs that corresponds to the length of the word to guess will affect the hangman result. (Mouthful..)

e.g: Word to guess is 4letters, if i give in inputs such as 2 lettered inputs, then it wouldnt affect the hangman result.


[original: nus-cs2113-AY2324S2/pe-interim#2575] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

Feature mentioned was currently not implemented but can be included in future releases. Mentioned in user guide and game tutorial that whole word guesses and an alphabet are the accepted inputs.

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: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]