search

wwweert123 / pe

0 stars 0 forks source link

loading up after changing pantry_stock.txt by adding extra line #6

Open wwweert123 opened 10 months ago

wwweert123 commented 10 months ago

image.png I edited the pantry_stock.txt by adding an extra line

image.png However, the application seems to think that there was some input in that line and stated that that stock will be removed

A very minor bug, but perhaps it is better to skip a line if the line will be empty when scanning the file

soc-se-bot commented 10 months ago

Team's Response

Thanks for your suggestion! It is indeed a good suggestion and we will implement it in the future! However, as this is just a suggestion and not a bug, we decided to reject it as per the guidelines below:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I understand that my "bug" seems to have been phrased as a suggestion. However, I feel that the error message printed is slightly misleading as it seems to imply that a stock has been removed when in fact no stock was removed. To prevent such a misleading error message, the implementation in the code could have skipped an empty line. Thus, I believe that there could be a cosmetic bug here which might be misleading to the user