cctt1014 / pe

0 stars 0 forks source link

Did not throw any exception or record the time duration appropriately #2

Open cctt1014 opened 4 years ago

cctt1014 commented 4 years ago

Did not handle this case, simply cut down the time duration instead of throwing an exception or handle those 30 minutes.

image.png

nus-pe-bot commented 4 years ago

Team's Response

fixedduration [task description] /for [duration] [unit]

  1. The user did not follow the format strictly, assumed that duration and unit can be repeated more than once.
  2. Everyone has their own expectations of checking whether it is eligible command or not. However, the program is capable enough to accept any inputs as long as it follows the format, and the extra/additional/unneccessary portion at the back will not be read by the program.

For example: "fixedduration eat lunch /for 1 hour 30 minutes". Returns the added fixedduration task of 1 hour. It is considered a valid command because the first portion of the input (fixedduration eat lunch /for 1 hour) is in correct format. It does not care about the remaining portion ( 30 minutes). This is to promote user-friendliness.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]