nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Delete command takes long to finish #2155

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

When using deleting command, it runs for a while before finishing the command. There may be too much search in the code.


[original: nus-cs2103-AY1920S2/pe-interim#2113]

HubertHalim commented 4 years ago

Team's Response

What is your definition of 'runs for a while'? Since transaction list is stored as an array, deletion will always take more time than insertion as deletion is an O(n) process. The delay from delete is barely noticeable even when the list is very long ( < 1 second) so it does not really affect the productivity of the users.

Additionally, in the non functional requirements section of the DG, Appendix D point 3 indicates that "The software should be able to respond within 5 seconds."

Duplicate status (if any):

--