search

GeNiaaz / pe

0 stars 0 forks source link

Long commands #16

Open GeNiaaz opened 3 years ago

GeNiaaz commented 3 years ago

It is supposed to be a program optimised for Command line interface. If so, the commands should be short. For example, addP instead of addPerson which is cumbersome and takes additional time to type.

nus-se-bot commented 3 years ago

Team's Response

Firstly, for user clarity, especially since the commands are new to the user, we would still prefer to keep addPerson. In addition, saving 5 letters would not amount for much since the user still has to type a long amount of details when using addPerson command. Hence, severity is marked as low.

Secondly, the logic that "It is supposed to be a program optimised for Command line interface. If so, the commands should be short." does not flow. The requirement of Command Line interface only implies that the commands should be typed.

Thirdly, this qualifies as a feature flaw since the issue is with the format of the command that the app has. Feature is functional and works as expected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As a person who only had 1 hour to test this program, I found myself scrambling to enter commands quickly. Now, I may just be someone trying to test this system in the span of an hour and I may be very pressed for time yes, however the target audience of this program are healthcare staff who have to comb through plenty of data to access a person's record. While yes, 5 letters is not much, this can add up to a lot of frustration and hassle over time when a person has to access tens of even hundreds of records a day.

When the system is new, sure the person would have time adjusting. But a person using this system would potentially have to use it for a long period, and getting used to and remembering the commands would be less of an issue in the long run. (or even the short term of a week)

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.DocumentationBug]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: Combined with the reasons I stated above, I also would like to add that this program would be used by (generally) overworked and tired healthcare workers. Any barrier to make their lives more comfortable should be a barrier that should be removed. It would indeed benefit a new user yes, but this program is intended to be used in the long term, and remembering commands in the long run becomes less of an issue over time, if at all.