nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Application does not accept floats #1835

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

add property n/Bishan t/Hdb a/Blk 150 Bishan Street 11 #02-101 p/570150 d/30-06-2021 ca/20,2

App gives an error message

Note the following conditions for specifying an asking price:

  1. The dollar sign is optional. (ok)
  2. There should not be any leading zeros in the number specified. (ok)
  3. The asking price should not be negative. (ok)
  4. Either do not use commas at all or be consistent in the usage of commas throughout, where each comma should separate every three digits from the back. (what if the price is not an integer) E.g. $1000000 or $1,000,000 are valid but $1000,000 or $100,00,00 are not valid.

[original: nus-cs2103-AY2021S2/pe-interim#1835] [original labels: severity.Low type.FeatureFlaw]

w2vgd commented 3 years ago

Team's Response

From the provided test input ca/20,2, this input is invalid because it was stated that each comma should separate every three digits from the back. Since the input 20,2 does not follow the required format, there is an error message produced.

Duplicate status (if any):

--