search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

"save meal" does not handle special characters in string #71

Open nus-pe-bot opened 3 days ago

nus-pe-bot commented 3 days ago

I suppose that the meal menu should only contain upper case or lower case alphabetical characters from "a" to "z". A wrong string format exception should be thrown And the string after "/" in the meal string is not handled in this case

save meal mat-cha /c800
      _____________________________________________________________________________
      Added to options: mat-cha with 800 calories
      _____________________________________________________________________________
save meal hot/dog /c1000
      _____________________________________________________________________________
      Added to options: hot with 1000 calories
      _____________________________________________________________________________

[original: nus-cs2113-AY2425S1/pe-interim#152] [original labels: severity.Medium type.FunctionalityBug]

kennethSty commented 1 day ago

Team's Response

Thank you for spotting the bug. We should have specified a constraint on meal names not having '/' inside of them. However, as this bug does not cause the programme to break, and is a rare occurrence, we have thus reclassified it to 'low'.

Duplicate status (if any):

--