nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Storage feature only works upon exit #67

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

The storage feature only works on exiting. This is a dangerous feature flaw as sudden crashes or termination of the program would cause data to be lost. This wastes time as the user will then have to re-input the data again.

This is a feasible fix as this was a requirement in the Duke IP project to update the storage during the program run.

image.png

image.png

image.png


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

jadenlimjc commented 1 week ago

Team's Response

The storage feature is designed to save data only upon exiting the program, ensuring that all updates made during the session are consolidated into the storage file. This approach was chosen to simplify the implementation and avoid potential performance overhead caused by frequent writes to the storage file during program execution.

The program currently saves data only upon exiting to consolidate all changes made during a session. This approach was implemented without prioritizing real-time or periodic saving, as the focus was on ensuring a functional and user-friendly experience for managing data.

We understand that this design choice introduces a risk of data loss if the program crashes or is terminated unexpectedly. We acknowledge that this could cause frustration, as users would need to re-enter their data.

While real-time saving was not a priority in the initial development phase, we recognize its importance in providing a more robust and reliable experience. For now, we recommend users exit the program after making significant updates to ensure their changes are saved.

Duplicate status (if any):

--