fsgmhoward / CS2113T-pe

0 stars 0 forks source link

Inappropriate behaviour in handling the search of nearest facilities #3

Open fsgmhoward opened 3 years ago

fsgmhoward commented 3 years ago

The facility itself should not be included into the search result, when the target facility type is the same as the original one.

For example, not like this:

What else would you like to do?
findFacility<library1><library><1>
library1@N1-02-02

nor this:

findFacility<canteen1><canteen><3>
canteen1@N4-01-01  <---------------------------------------
canteen4@N5-04-02
canteen3@N5-04-01

The rationale for this is, when we try to search a facility of the same type, it is usually because the original one is not accessible (e.g. library being full or canteen temporary closed) and we want to find a nearest one. Printing out the same building does not solve the user's needs at all.

nus-pe-bot commented 3 years ago

Your response not required for this bug as the team has accepted the bug as it is.