search

JodyLorah / pe

0 stars 0 forks source link

Blist INDEX should not be a valid command #15

Open JodyLorah opened 3 years ago

JodyLorah commented 3 years ago

Command listed here (blist INDEX) is not a valid command, hence it can be quite confusing for the reader reading the DG

image.png

nus-pe-bot commented 3 years ago

Team's Response

As stated above, "The blist INDEX command will create a new Person with the same information as the person at INDEX in AddressBook". It is clear that INDEX refers to the position of the person in AddressBook.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: You cannot assume that everything is clear and represents something that you know but the reader may not. No where is it stated in the DG what INDEX is when it is used, and as such since the reader might think that the string "INDEX" is a possible input.