nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Lack of CTRL-C handling leads to data loss #1032

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Out of habit, users who are accustomed to using other command-line programs may use CTRL-C to exit the currently-running program instead of using the "exit" command. This leads to a NoSuchElementException stack trace being printed onto the screen, and more critically, causes the data saving routines to not be run, thus leading to the loss of whichever data was input during the session. It may be prudent to handle this case in a more graceful manner. This bug has been rated Medium due to the potential for data loss, while recognising that exiting the program in this manner is a deviation from the expected way to exit the program documented in the User Guide.


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

gaoyunfan commented 1 year ago

Team's Response

User guide mentions save only works when the user exits properly and dynamic saving is not part of feature due to poor efficiency.

Duplicate status (if any):

--