Skip to content

Commit

Permalink
Merge pull request #544 from yixiann/final-dg
Browse files Browse the repository at this point in the history
Remove composition of items and tags in list
  • Loading branch information
tingkai-mai authored Nov 7, 2022
2 parents 30c988a + 522beed commit 479610a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/_dg/implementations/ItemFeatures.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ The related attributes of an item are:
Regarding the attributes, here are a few things to note:

* The Item value is the `ItemPrice` multiplied by the `ItemQuantity`. This is used for the [Statistics](#statistics-features) feature.
* Each attribute in `Item` is dependent on a respective `Validator` and `ItemComparator`. `Validator` is an interface that facilitates validation of fields when `Item` is created. `ItemComparator` is `Comparator` that facilitates sorting of items by its respective fields.
* Each attribute in `Item` is dependent on a respective `Validator` and `ItemComparator`. `Validator` is an interface that facilitates validation of fields when `Item` is created. `ItemComparator` is `Comparator` that facilitates sorting of items by its respective attributes.

![model_diagram](images/ItemFieldClassDiagram.png)

Expand Down
4 changes: 2 additions & 2 deletions docs/diagrams/ItemClassDiagram.puml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ skinparam arrowThickness 1.1
skinparam arrowColor MODEL_COLOR
skinparam classBackgroundColor MODEL_COLOR

UniqueTagList *-> "0...100" Tag
UniqueItemList *-> "0...10000" Item
UniqueTagList -> "0...100" Tag
UniqueItemList -> "0...10000" Item
note right : 10,000 items enforced by UniqueItemList

Item *-up-> "0...100" Tag
Expand Down
Binary file modified docs/images/ItemClassDiagram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 479610a

Please sign in to comment.