search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Wrong error message displayed for numbers that is not within the index size #984

Open nus-pe-bot opened 2 weeks ago

nus-pe-bot commented 2 weeks ago


video:https://raw.githubusercontent.com/nus-cs2103-AY2425S1/pe/master/files/9f5cb04f-c633-4759-aaef-366cd04bcdf3.mov When I delete a index that is out of range of the patient limit, for instant 10 when I only have 8 person, I will get an invalid index error message. However, when try to delete an index that is very large and obviously not within the range of the patient limit, instead of getting the same error message, the error messages changed to invalid format.

Workaround this error is due to the limit of int, it will be better to include it under constraints or mention it in the UG

[original: nus-cs2103-AY2425S1/pe-interim#1369] [original labels: severity.Low type.FunctionalityBug]

Xczheng0105 commented 2 weeks ago

Team's Response


video:https://raw.githubusercontent.com/nus-cs2103-AY2425S1/pe-dev-response/main/files/94b6d5bc-36ee-4e2e-90b8-5a206ad121a3.mp4

As can be seen from the video above, this issue is likely to be caused by the 32-bit integer limit. While valid, this is a rather trivial issue because in reality one is most likely not going to have more than 2147483647 gaming contacts.

Duplicate status (if any):

--