nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Infomation display functions are too separated #1345

Open nus-se-script opened 7 months ago

nus-se-script commented 7 months ago

image.png As shown in the picture above, is it better to have a single command to just display calories vs calories burnt, instead of having two separate commands? Also, I would expect the listEverything command literally lists everything including at least calories and calories burnt, in view of this app being a fitness tracker.


[original: nus-cs2113-AY2324S2/pe-interim#1204] [original labels: type.FeatureFlaw severity.Medium]

edwardhumi commented 7 months ago

Team's Response

listEverything is meant to display all the meals, drinks, and exercises tracked, as per the UG. It is not meant to list our calories intake, nutritional information, and etc. If this was the case, there would be a long list of nutritional info and make our display even more complicated.

image.png

Duplicate status (if any):

--