nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Doctors can have 2 appointments at the same time, which does not follow real world logic #3081

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png With reference to the image above, the doctor with ID 3 can have 2 appointments at the exact same time but with different patients. this can be due to user error when keying in the date and time. There should be error messages to inform the user that the doctor already has an appointment at that time slot instead of allowing the second appointment to be successfully created. this can lead to a lot of confusion when managing appointment data and can potentially cause great admin trouble for the clinic, greatly affecting user satisfaction. in the worse case scenario, 2 patient can turn up at the same time slot.


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

Nigeltzy commented 1 week ago

Team's Response

Hi! Thank you all for the bug report and the observation! We understand your concerns with regards to the supposedly clashing appointment timings. During the development process, our team did consider whether or not to restrict such appointment creation.

However, ultimately, considering the limits of this project, we arrived at the consensus that restricting such functionality would prove more detrimental to our target group (healthcare workers) within the context of the healthcare industry.

We believe that restricting "duplicate" appointments would fall under the scope of overzealous input validation.

The healthcare industry with regards to consultation, can have incredibly varied formats and purposes depending on the healthcare context. To provide maximum flexibility and functionality to our users, we have decided to allow such appointment creation.

Consider the following scenario, or as given in the above bug report:

Patient (ID: 2) along with Patient (ID: 4) has a consultation scheduled with Doctor (ID: 3) on the same day and the same time.

This is an entirely valid and plausible appointment that could happen in the real world. Not ALL doctors or healthcare staff may do them, but as long as SOME of them do, we believe that not restricting them is the right decision. The following are some examples:

Consider another scenario, where instead of multiple patients having an appointment with the same doctor, you have multiple doctors with an appointment with the same patient:

We understand your concerns with regards to ease of appointment scheduling, however, at most, we believe this could be addressed through perhaps a UI pop-up to verify / "double-confirm" with the user before adding or editing such "clashing" appointments. However, we believe that this would not be a priority for our development given the constraints and focus of this project, and that the appointment management system with regards to duplicate appointments would suffice and provide flexibility and functionality in its current "clash-able" state.

Duplicate status (if any):

--