Overzealous input validation

[jereeemyyyy]

According to the website:

• Currently, the nurses/doctors are quite restricted when it comes to adding NOTES and DIAGNOSIS.
• It is totally valid for them to have NOTES or DIAGNOSIS more than 80 characters (if the patient is problematic) if they want to.
• Hence, the input validation might be too restrictive.

[nus-pe-script]

Team's Response

Thank you for your input. The consideration behind these fields is to convey that these are meant to store quick and convenient information, and was not designed to store large amounts of information regarding a patient. Having long inputs could potentially disrupt the CLI-usability as they might have to use their mouse to scroll through the data, which defeats the purpose of a CLI Application. In addition, we also have the patient notes field to add additional information regarding a patient, which could be used for a more general purpose (i.e. to store more diagnosis data if the 80 characters is insufficient.). Hence, this was an intended implementation.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

• [x] I disagree

Reason for disagreement: Thank you for your response!

I understand the restriction is indeed an intended implementation which is nothing wrong. However, I am suggesting that this particular restriction is overzealous in the sense that it restricts the user's input for the NOTE and DIAGNOSIS fields to only 80 characters when in fact, it is reasonable for a Nurse/Doctor to have a DIAGNOSIS or NOTE longer than 80 characters. This was the reason why I think that this could be an issue of type.FeatureFlaw and severity.Low.

The following points are to address the response: `"In addition, we also have the patient notes field to add additional information regarding a patient, which could be used for a more general purpose (i.e. to store more diagnosis data if the 80 characters is insufficient.)":

• If a Doctor/Nurse needs to add more diagnosis data into the NOTES field when he/she realizes that the DIAGNOSIS is actually longer than 80 characters, it could potentially lead to inefficiency as the user would need to split the diagnosis into two separated portions to fit into the character restrictions of the DIAGNOSIS and NOTE fields.

• Additionally, it doesn't make any sense for the users to "count" the number of characters that they input to meet the character restrictions, especially in a fast-paced work environment in the hospital where Nurses and Doctors have more urgent things to attend to.

  • For e.g. If user inputs, Patient requires intensive care............MORE_CHARACTERS........ (more than 80 characters) into DIAGNOSIS, it doesn't make sense / is not efficient for the user to manually count the number of characters that went off the limits, and then input the additional diagnosis input into NOTES.

• This could also potentially cause some confusion as it is intuitive to have the diagnosis of a patient to only be under DIAGNOSIS, not both DIAGNOSIS and NOTE.

Therefore, I believe that this issue is still of type.FeatureFlaw and severity.Low which are the original types that I have inputted during the PE.

---