[CS2113T-T09-4] traKCAL#29
Conversation
|
Maybe you guys should change the title of the project to CS2113T-T09-4. The current title is missing a T. |
|
|
||
| **trakCAL** is made up of mainly 6 components. | ||
|
|
||
| `trakCAL`: tracKCAL starts here |
There was a problem hiding this comment.
Should the user talk to the UI component instead? What is the meaning of the trackCAL component? Is it something like the main() function? Maybe the team should explain more
|
|
||
|  | ||
|
|
||
| *Figure 2. Diagram for logic component* |
There was a problem hiding this comment.
Maybe you can change the arrowhead to another type to make it more clear that is an association.
|
|
||
|  | ||
|
|
||
| *Figure 2. Diagram for logic component* |
There was a problem hiding this comment.
Perhaps can specify that it is a UML diagram? Or is it not?
|
|
||
| ### 2.3 Model component | ||
|
|
||
|  |
There was a problem hiding this comment.
I like the arrowhead in the blue circle, the one in the red circle might cause confusion.
There was a problem hiding this comment.
I think can include multiplicity in the diagram to make it clear the ActivityList contains multiple Activity objects.
|
|
||
| **API**: Model.java | ||
|
|
||
| In the Model component, |
|
|
||
| The sequence diagram below shows how the components will react to a new user or for a returning user. | ||
|
|
||
|  |
There was a problem hiding this comment.
Is there an error in the red circle? Should the incoming arrow from the left go to the name box as it is a constructor? And which function calls the data for user profile function?
| The graph implementation shows the progress of the daily net | ||
| calories over the period of 7 days. The GraphProperty class extracts the available days from the | ||
| hashmap. |
There was a problem hiding this comment.
I think this part should be written after the diagram is introduced.
| is created, the properties of the graph are then stored and calculated in setProperty function. | ||
|
|
||
| Next, the graphDrawing object is created and uses the properties calculated earlier to print out the graph. | ||
| > In the case where there are less than 7 days stored, all the days will be displayed. |
There was a problem hiding this comment.
I like the way you present this extra information.
|
|
||
| The following sequence diagram shows how the `move` command is executed, where index1 is the position to be moved from and index 2 is the position to be moved below. | ||
|
|
||
|  |
|
|
||
| The following sequence diagram shows how the lastSeenList is set after a “list date" command where date is of YYYY-MM-DD or after a “list” command where the date is the current date. | ||
|
|
||
|  |
update UG + DG + PPP
improve code quality
update Developer Guide with diagrams
include intro
fixed faq numbering
correct github u/n
update indentation
…into owen-DG # Conflicts: # docs/UserGuide.md
update UG for CS2101
update UG for CS2101
7 participants
PR subject: traKCAL
Description: traKCAL helps university students to track their calories over a period of time.