search

Arif-Khalid / pe

0 stars 0 forks source link

Parser class UML class diagram errors #20

Open Arif-Khalid opened 1 year ago

Arif-Khalid commented 1 year ago

Parser class is labelled abstract but is not abstract. In addition, static methods are not underlined. image.png

nus-pe-bot commented 1 year ago

Team's Response

Hi thanks for pointing that out. But we disagree with the severity of this bug. If developers were to ever try to replicate our product based on our DG, and they use an abstract class for Paser, the IDE will give an error stating the instantiation of abstract class, which he / she can then change. It will only cause minor inconvenience. Not underlining static methods for this case does not affect the implementation, because static or non-static methods works equally well. The developer just has to instantiate the Parser class before using the methods.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]