yingx9 / pe

0 stars 0 forks source link

Invalid move not specific enough #5

Open yingx9 opened 11 months ago

yingx9 commented 11 months ago

Steps to reproduce:

  1. Enter b to choose black for colour
  2. Enter 1 to choose difficulty level
  3. Enter move a1 a2

Expected: Move is not valid and why

Actual:

Oops, that move isn't valid!

I think the message can be more specific to explain why the move isn't valid, since this program aims to help chess players train and learn how to play chess better and make learning and training accessible for beginners

image.png

nus-pe-script commented 11 months ago

Team's Response

I think the message can be more specific to explain why the move isn't valid, since this program aims to help chess players train and learn how to play chess better and make learning and training accessible for beginners

Having a chess game which explains why a move is not valid would be of great help to beginner chess players. However, this is a feature which was not in scope for v2.1 of ChessMaster. The goal for these 3 iterations was to set up a basic chess game which adhered to all the formal chess rules, and provided some basic features to the user such as checking the history of moves in the game. Hints or specific explanations were not part of our accepted user stories in this project, though we agree that this would be a great feature for a future version release.

The 'Original' Bug

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

Unable to parse coordinate error message not specific enough

Steps to reproduce:

  1. Enter b for black* to choose colour
  2. Enter 1 to choose difficulty
  3. Enter move a1

Expected:

Error message telling me I am missing the second coordinate

Actual:

Unable to parse coordinate!

Error messages that are not specific enough can be a feature flaw

image.png


[original: nus-cs2113-AY2324S1/pe-interim#1058] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

No details provided by team.

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]