search

nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

ISBN Number needs to be limited #1093

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

No details provided. Screenshot 2020-11-13 at 1.03.01 PM.png

Long ISBN numbers that are unrealistic are accepted by the command

[original: nus-cs2103-AY2021S1/pe-interim#995]

Caiyi34777 commented 3 years ago

Team's Response

I beg to argue with your problem report. From what we perceive in reality, ISBN for books keeps changing over time. ISBNs were 10 digits in length up to the end of December 2006, but since 1 January 2007 they now always consist of 13 digits. You won't know how number of ISBN bits will increase in the future (since there will be more and more books in the world) and if we set a hard upper limit, it will cause problems for future usage since it might exceed the limit and cause error messages. To achieve a better compatibility for older and newer books, we did not set a maximum number of digits for ISBN.

Duplicate status (if any):

--