search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Error with interactions with findmedcon #3212

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

image.png

when I use findmedcon Stage 4 cancer, all patients with different stages of cancer show up, rather than the ones with only stage 4 cancer. This limits the ability to store different types of illness.

[original: nus-cs2103-AY2425S1/pe-interim#1986] [original labels: severity.Medium type.FeatureFlaw]

yong-zhuo commented 1 week ago

Team's Response

Thank you for your feedback! As mentioned in the User Guide, the current implementation of the findMedCon command will only find patients with conditions containing the given keyword. Patients with medical conditions matching at least one keyword will be returned (i.e. OR search).

image.png

As such, based on your example, patients with conditions containing the keywords Stage , 4 and cancer will be found and this is intended based on how we designed this command.

While our implementation of the findMedCon command may result in some patients being displayed in the Patient List with conditions the doctor might not be looking for, we feel that this causes only minor inconvenience for our users as they will still be able to pinpoint and identify the patients with the specific conditions they are looking for, since those patients will also be displayed in the Patient List. Therefore, we have decided to lower the severity of this issue to Low.

However, we do acknowledge that it would be better to modify findMedCon to allow users to find patients with the given conditions instead of the given keywords as it allows doctors to easily locate the specific patients they might be looking for. This can be improved in future iterations of our application.

Duplicate status (if any):

--