boonjuey / pe

0 stars 0 forks source link

Data is stored as a .json file, not a text file #11

Open boonjuey opened 3 years ago

boonjuey commented 3 years ago

image.png

Based on the TP constraints, the data is to be stored in a human editable text file. However, the program stores it in a .json file. I do agree though, that parsing of a JSON file would be more reliable.

nus-pe-bot commented 3 years ago

Team's Response

JSON is the commonly used standard for both parsing and understanding by humans. As compared to usage of delimiters, the better structure makes each parameter much easier to understand. (e.g. compare [add, 1, A, C, cs1010, modulename] with { name: modulename, grade: A, type: C, code: cs1010 }). The use of JSON in fact, makes understanding and editing the storage file easier with the clearly marked parameters names.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]