search

pzl111 / pe

0 stars 0 forks source link

Transaction holds a reference but not written #38

Open pzl111 opened 9 months ago

pzl111 commented 9 months ago

image.png

soc-pe-bot commented 9 months ago

Team's Response

Unclear what this is pertaining to as there is a lack of description and only one picture and the title message does not seem to be addressing anything.

Items for the Tester to Verify

:question: Issue response

Team chose [response.IssueUnclear]

Reason for disagreement: The following is a screenshot of the code:

image.png

It shows that the Transaction class holds references to each of the Name, Amount, DateTime, Location, Type, UniqueCategoryList. This means the Transaction class can navigate to these classes, and should use arrowheads to indicate the navigability of the associations. However, this is missing from the UML diagram of the DG, which would lead to me thinking that there is no such navigability of the associations although it exists, which can cause misunderstanding of the association. This is only a low severity since it is not merely cosmetic as it affects the reader's understanding of the association.

Below is a screenshot from the textbook that shows this:

image.png