There should be a check for the correct number of fields.

[geraldkoh4]

An incorrect number of fields can be added perpetually.

[soc-se-bot]

Team's Response

No details provided by team.

The 'Original' Bug

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

There is no checks for loading up of save file

Since a correct input was accepted and it has exited successfully. But the input has crashed the application due to the loading up of the save file. This means that as a user, I can no longer use that safe file unless i go and find out what is wrong in the safe file.

---

_{[original: nus-cs2113-AY2223S2/pe-interim#2098] [original labels: type.FunctionalityBug severity.Medium]}

Their Response to the 'Original' Bug

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

Because the user is inputting overzealous input validation, it is considered a FeaturFlaw.

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)

• [x] I disagree

Reason for disagreement: One refers to the use of the application in CLI, which should be parsed correctly, while the 2nd issue refers to the save file not being parsed at all. They are not the same.

---

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:**   "A legitimate user behavior is not handled e.g. incorrect commands, extra parameters" There are extra parameters. It is a functionality bug as stated.

---

## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:**   This is due to product design. In this case, what I think is happening is that the string is separated by " " (whitespace). But consider this. /cadd fried rice with eggs, sausage, carrots and chicken 500 The string is a perfectly normal user input to enter, with spaces and commas. But according to how they parse it, there are actually 10 fields (9 white spaces) when there should only be 3 accepted. As seen in my original issue, you can put in as many spaces as you want. This means that there is no check for the number of spaces. Functionality bug : "A legitimate user behavior is not handled e.g. incorrect commands, extra parameters" Severity: A very common user input is not correctly handled, but it (the additional fields) does not crash the program. Hence, the severity should still be medium. (The command crashes the program due to the presence of "," which could have stopped most user inputs from failing due to limiting the amount of " ").

---