nus-cs2113-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Invalid move not specific enough #1066

Open nus-pe-bot opened 12 months ago

nus-pe-bot commented 12 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


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

antrikshdhand 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.

Duplicate status (if any):

Duplicate of #1067