Skip to content

Commit

Permalink
docs: update component on UniqueTagList
Browse files Browse the repository at this point in the history
AddressBook previously had a UniqueTagList linked to it as the
master tag list, which was reflected in the architecture diagrams
in the DeveloperGuide and LearningOutcomes.

Since AddressBook does not use a master tag list anymore, and
UniqueTagList has been removed, these architecture diagrams are no
longer consistent with the program.

Let's update these architecture diagrams in the DeveloperGuide
and LearningOutcomes.
  • Loading branch information
yamidark committed Aug 8, 2018
1 parent 2aeb07b commit 756dccc
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 4 deletions.
Binary file modified docs/Diagrams.pptx
Binary file not shown.
7 changes: 3 additions & 4 deletions docs/LearningOutcomes.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ e.g. `Main.VERSION`, `Name.EXAMPLE`, `Utils.isAnyNull(...)`

* Convert the `Parser::parseCommand(...)` method (i.e. the `parseCommand()` method of the `Parser` class) to a
class-level method. Note how this method can be either class-level or instance-level.
* Note how the `setTags` method of the `Person` class cannot be converted to a class-level method.
* Note how some instance-level methods, such as the `setTags` method of the `Person` class, cannot be converted to a class-level method.
* Add an instance-level member `int sequenceNumber` and a class-level variable `int nextSequenceNumber`
to the `Person` class. Using these two variables, ensure that each `Person` object has a unique sequence number
that indicates the order in which `Person` objects were created. e.g.
Expand All @@ -168,8 +168,8 @@ Note the following examples of _composition_ (filled diamond):
[cols="<,<",options="header",]
|=================================================
|Whole |Parts
|`AddressBook` |`UniquePersonList` `UniqueTagList`
|`Person` |`Name` `Phone` `Email` `Address`
|`AddressBook` |`UniquePersonList`
|`Person` |`Name` `Phone` `Email` `Address` `Tag`
|=================================================

Contrast with these examples of _aggregration_ (empty diamond):
Expand All @@ -178,7 +178,6 @@ Contrast with these examples of _aggregration_ (empty diamond):
|============================
|Container |Contained
|`UniquePersonList` |`Person`
|`UuniqueTagList` |`Tag`
|============================

=== References
Expand Down
Binary file modified docs/images/TaggingClass.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified docs/images/TaggingsInTagging.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified docs/images/mainClassDiagram.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 756dccc

Please sign in to comment.