BenedictChannn / pe

0 stars 0 forks source link

Ctrl-C issue #10

Open BenedictChannn opened 8 months ago

BenedictChannn commented 8 months ago

image.png

If the user enters Ctrl + C, a whole bunch of "No input provided by the user" will appear before exiting the application. Also, since there is already an exit command, would it be better to not have Ctrl + C cause the application to close, but rather display an error message?

nus-se-bot commented 8 months ago

Team's Response

Thank you for bringing this up. But crtl C is taken as a “special character” and we have specified in user guide to refrain from using any special characters. For future iterations we will be sure to replace with an error message instead

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Weird output when I control-c to exit program

image.png

Says no input provided by user multiple times when exiting with control-c


[original: nus-cs2113-AY2324S1/pe-interim#1079] [original labels: type.FunctionalityBug severity.VeryLow]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** I think the handling of Ctrl+C is very important and should be a priority for your application instead of pushing it to future iterations.
## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [x] I disagree **Reason for disagreement:** It is a flaw that was not thought of.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** This affects the usage of the application so the severity is definitely not "very low"