search

nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

CommandParser inaccuracy in DG #370

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

Screenshot 2022-11-11 at 5.21.41 PM.png

Screenshot 2022-11-11 at 5.23.08 PM.png

As per observation, the createCommand method was called through static class CommandParser, similar to the Ui implementation. But in the sequence diagram, it's shown as an unnamed object. There is discrepancy between the diagram and the actual code. This is found in section 3.1 or DG.

[original: nus-cs2113-AY2223S1/pe-interim#388] [original labels: severity.Low type.DocumentationBug]

winston-lim commented 1 year ago

Team's Response

image.png Look at the full image and notice that createCommand is there.

Duplicate status (if any):

--