-
Notifications
You must be signed in to change notification settings - Fork 101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CS2113T-T10-3] Mod Happy #9
Open
Ch40gRv1-Mu
wants to merge
586
commits into
nus-cs2113-AY2122S2:master
Choose a base branch
from
AY2122S2-CS2113T-T10-3:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
586 commits
Select commit
Hold shift + click to select a range
b290e03
Update confirmation to use TextUi instead of Scanner
ngys117 7d94630
Update the Regex for OptionParser
Ch40gRv1-Mu 311cd2c
Update Class Diagram for DeleteCommand
ngys117 38cab52
Update Command Diagram Note
ngys117 ddf59d2
Update UG phrasing and formatting
chooyikai a68f995
Fix Coding standards
Ch40gRv1-Mu 36750db
added javadoc
Yzkkk 2330582
Merge pull request #114 from Yzkkk/master
heekit73098 f037650
Update OptionCommand regex and refactor configuration values
heekit73098 04b484a
Edit UG based on peer review feedback
chooyikai d07a47e
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
ngys117 c767879
Change TextUi Methods
ngys117 44910c3
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 fb40b04
Fixed regex for Help Command
heekit73098 fc133fe
Update code quality
heekit73098 5a134bc
Update DG to show abstract and interfaces explicitly
ngys117 11ad550
Fixed JUnit Tests
heekit73098 d590850
Merge pull request #115 from ngys117/branch-PR-CleanUp
heekit73098 eae5bcc
Added Content page to DG
heekit73098 34ff0f1
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp
ngys117 843fd8d
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 d6f7b93
Fixed regex for tag
heekit73098 32c8286
minor edit
Yzkkk 2808796
Merge branch 'master' into minor-edits
chooyikai 1b45f9e
Added Grade and Edit feature, User stories and front matters
heekit73098 a577470
Fix class diagrams (hopefully)
chooyikai 79d007a
Add TaskDuration class and corresponding JUnit
Ch40gRv1-Mu bff52e4
Fix ModHappyParser
Ch40gRv1-Mu 11def20
Fix coding standard
Ch40gRv1-Mu cdcdff6
Update TaskDuration
Ch40gRv1-Mu e9b3a24
Update TextUiTest
Ch40gRv1-Mu 35610e7
Merge branch 'master' into branch-update-ugdg
Ch40gRv1-Mu d9f5d75
Update UG for duration
Ch40gRv1-Mu e1e05de
Merge branch 'branch-update-ugdg' of https://github.com/Ch40gRv1-Mu/t…
Ch40gRv1-Mu 6fba17e
Update StringConstants
Ch40gRv1-Mu 82ca730
Update TaskDuration
Ch40gRv1-Mu 2faf4f8
Merge pull request #124 from Ch40gRv1-Mu/branch-refactor-duration
heekit73098 b3c776f
Update DG
Ch40gRv1-Mu 11f4400
Merge pull request #118 from Ch40gRv1-Mu/branch-update-ugdg
heekit73098 02f6f20
Merge branch 'master' into minor-edits
chooyikai 70f5e66
Update DG
heekit73098 3c165c8
Fix #125, update some JUnit tests
chooyikai d6d4d2e
Fix bug and limit the available MCs
heekit73098 5535496
Update UG, fix some bugs
chooyikai 4a0504f
Merge pull request #116 from chooyikai/minor-edits
heekit73098 06b4fb2
Added Seq Diagrams
heekit73098 f684ba6
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 5c96be8
Update Code Quality
heekit73098 d2a9710
Fix Code Quality
heekit73098 996585b
Fixed Code Quality
heekit73098 f5777ac
Merge pull request #121 from heekit73098/branch-expandExceptions
chooyikai 59387bb
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 d48a6e8
Removed image
heekit73098 0dd31fd
Remove Debug
heekit73098 d570cbc
Removed debug statement
heekit73098 6b4e1b6
Merge pull request #126 from heekit73098/branch-expandExceptions
chooyikai 76b4b97
Update contents in UG
heekit73098 15a7dfa
Merge pull request #127 from heekit73098/branch-updateDG
Ch40gRv1-Mu 39bd014
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp
ngys117 36d8103
minor edits
Yzkkk 47f4018
resolved merge conflicts
Yzkkk a75baab
made some error messages more informative
Yzkkk f8c8075
made changes on documentation
Yzkkk a26d262
minor edits
Yzkkk 697c3ef
minor edits
Yzkkk 7a684a6
commented some of the test cases
Yzkkk 1fbbb85
commented junit test case
Yzkkk 3e7902f
added exception to account for spaces added as parameters
Yzkkk c8fba54
minor edits
Yzkkk 4cd906f
minor edits
Yzkkk c7bdac3
minor edits
Yzkkk 8f92607
minor edits
Yzkkk 41a61b0
Fix unhandled null exception when deleting a module that does
ngys117 a51e674
Refactor code to reduce arrowhead
ngys117 884047f
made error messages and exceptions more descriptive
Yzkkk 37689f0
minor edits
Yzkkk 0d33ed7
commented out some test cases
Yzkkk aa9152c
Fix Typos
ngys117 c512ff9
Remove number of tasks message
ngys117 282df85
Add JUnit test
ngys117 86a2924
Update UG
Ch40gRv1-Mu b4c56c8
Fix editing a non-existent module throwing a null exception
ngys117 f87fcf8
Updates user guide
Ch40gRv1-Mu 95f6f8a
Add comment to DeleteCommand
ngys117 c6001d6
Update Developer guide
Ch40gRv1-Mu 95d65fc
Merge pull request #171 from ngys117/branch-PR-BugFixes
Ch40gRv1-Mu d6b375d
Merge branch 'master' into branch-update-ug-dg-fix-bugs
Ch40gRv1-Mu 48908bd
Refactor Code for Issue #172
ngys117 6ea147e
Update User guide
Ch40gRv1-Mu cb19dd6
Merge branch 'branch-update-ug-dg-fix-bugs' of https://github.com/Ch4…
Ch40gRv1-Mu 9a04af2
Upgrade UG
Ch40gRv1-Mu 552c7d0
Update UG
Ch40gRv1-Mu e730e3a
Merge pull request #173 from Ch40gRv1-Mu/branch-update-ug-dg-fix-bugs
heekit73098 4775b20
documented regex
Yzkkk 95423fa
minor edits
Yzkkk 3240da2
Code cleanup as per TA feedback
chooyikai 5b2043b
Fix #136 and #140
chooyikai a8a20bd
fixed bugs
Yzkkk f6b005a
Merge branch 'master' into master
Yzkkk 3f15a99
minor edits
Yzkkk 2ea2877
resolved merge conflict
Yzkkk f852dba
minor edits
Yzkkk 0fb6530
Merge branch 'master' of https://github.com/Yzkkk/tp
Yzkkk 0945884
Attempt to fix UG formatting
chooyikai f58705f
Fix some inconsistencies
chooyikai df0097c
Update user input prompt to match UG
chooyikai 360d836
Add PPP draft
chooyikai f1454f2
updated code quality
Yzkkk b551a1d
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp
ngys117 17eedef
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
ngys117 e00d2c8
Merge pull request #123 from Yzkkk/master
heekit73098 54d8664
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
ngys117 16404ee
Add Product Portfolio Page
ngys117 2757d55
Merge branch 'master' into bugfix
chooyikai ab75334
Update tag op exception
ngys117 601eb9b
Add InvalidFlagException to Junit
ngys117 c3c36f7
Update ModuleList method according to suggestion
ngys117 2c8e0d6
Update Junit
ngys117 dc48f7a
Fix PPP headings
ngys117 ab07581
Update AboutUs.md
ngys117 1789b64
Update AboutUs.md
ngys117 fdcf1a8
Fix hyperlink error in about us
chooyikai 62a6437
Merge pull request #174 from ngys117/branch-PR-BugFixes
chooyikai ead01ef
Merge branch 'master' into bugfix
chooyikai b714227
Merge pull request #175 from chooyikai/bugfix
ngys117 46db8ef
Added more descriptive error messages
heekit73098 3f6c215
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 99c45f6
Update TagException
heekit73098 fbced7f
Update code quality
heekit73098 86bd02f
Add PPP of Mu Changrui
Ch40gRv1-Mu 5e86af2
update PPP
heekit73098 7f055ae
Update code based on comments
heekit73098 ae145af
update code quality
heekit73098 5ca6c2d
Merge pull request #176 from heekit73098/branch-bugfix
Yzkkk 7c6cc05
Merge pull request #178 from heekit73098/branch-PPP
heekit73098 e1494a8
Merge pull request #1 from Ch40gRv1-Mu/master
Ch40gRv1-Mu 8aea8d6
Update Task
Ch40gRv1-Mu 868686e
Update Main, Add ModHappyStorageManager
Ch40gRv1-Mu ea106fc
Update the ModHappyStorageManager
Ch40gRv1-Mu f5badfc
Update the TaskDuration
Ch40gRv1-Mu eeec022
Update the ModHappyParserTest
Ch40gRv1-Mu fecab06
Edited task duration format and added PPP
Yzkkk de19c97
Update Userguid
Ch40gRv1-Mu f81bbf7
updated UG
Yzkkk ad7232c
updated PPP
Yzkkk 0d6da5d
Modify name of PPP
Ch40gRv1-Mu d6fa91c
Add Changrui's PPP
Ch40gRv1-Mu ee3845f
Merge pull request #181 from Ch40gRv1-Mu/branch-A-PPP
heekit73098 a64c611
updated PPP
Yzkkk 6315c64
Merge branch 'master' into master
Yzkkk e4b6474
updated PPP
Yzkkk 1382049
Merge branch 'master' of https://github.com/Yzkkk/tp
Yzkkk a11a524
Merge pull request #179 from Yzkkk/master
Ch40gRv1-Mu 0bbc0be
Update DG
heekit73098 099d79e
Update formatting
heekit73098 644398d
fixed bug
Yzkkk a20f053
updated code quality
Yzkkk a7f3421
Update PPP
Ch40gRv1-Mu eef4447
Update PPP and ModHappyStorageManager
Ch40gRv1-Mu 56aaded
Fix coding standard
Ch40gRv1-Mu 27494cc
Merge branch 'master' into branch-add-ppp-fix-bug-refactoring-storage
Ch40gRv1-Mu 18e96e5
Update ModHappyStorageManager
Ch40gRv1-Mu 9b5a794
Attempt to fix github pages formatting errors
chooyikai 38c71a7
Merge pull request #184 from chooyikai/ug-bugfix-attempt
chooyikai e5f3d31
Attempt to fix github pages formatting issues... again
chooyikai 88de173
Merge pull request #185 from chooyikai/ug-bugfix-attempt
chooyikai bdb2e4d
Merge branch 'master' into branch-add-ppp-fix-bug-refactoring-storage
Ch40gRv1-Mu 66bc6a2
Update Userguide
Ch40gRv1-Mu 5191545
Update Userguide
Ch40gRv1-Mu 135d2f8
Update Userguide.md
Ch40gRv1-Mu b6296de
Merge pull request #180 from Ch40gRv1-Mu/branch-add-ppp-fix-bug-refac…
chooyikai 20045a6
Update based on comments and bug fixes
heekit73098 5165ae8
update code quality
heekit73098 c656cb2
Added javadoc and more parser bug fix
heekit73098 6525396
Merge pull request #182 from heekit73098/branch-PPP
chooyikai ded1a84
Update UG and puml
Ch40gRv1-Mu 69262ac
Update PPP
Ch40gRv1-Mu 635f30d
Update DG
Ch40gRv1-Mu b60b98a
Update the exception catch and deleted unremoved system out
Ch40gRv1-Mu 2a699fb
Remove unused import
Ch40gRv1-Mu 6672a7c
Improve code quality
Ch40gRv1-Mu 0285f8b
Updates Storage.puml
Ch40gRv1-Mu e729d4f
Basic refactoring for parsers for code quality
heekit73098 33de955
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 caf9f8e
update ppp
heekit73098 14a79aa
Merge pull request #186 from Ch40gRv1-Mu/branch-update-dg
chooyikai bd1b775
Merge pull request #187 from heekit73098/branch-PPP
Ch40gRv1-Mu 2783867
Improve Coding Quality
Ch40gRv1-Mu 22559c7
Merge pull request #188 from Ch40gRv1-Mu/branch-improve-coding-quality
chooyikai 9fd5e4f
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp
ngys117 348cbcd
Added authorship
heekit73098 a934ccd
Merge pull request #190 from heekit73098/branch-authorship
ngys117 9936d2b
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp
ngys117 1374847
Add author tags
ngys117 ab42bd6
Merge pull request #191 from ngys117/master
Ch40gRv1-Mu 2ecc3b3
Declare Authorship and fix #189
Ch40gRv1-Mu f79c427
Fix coding standard
Ch40gRv1-Mu 5e40f56
Refactoring the authorship tag
Ch40gRv1-Mu 4c23b80
Update contribution for Ui
Ch40gRv1-Mu 20320b7
Merge pull request #192 from Ch40gRv1-Mu/branch-authorship-fix-bug
chooyikai bb93231
Add reposense authorship tags
chooyikai 2bfcd62
Improve wording of some error messages
chooyikai e86a8c0
Merge pull request #193 from chooyikai/reposense
ngys117 11bff4d
Update code quality and minor bug fixes
heekit73098 a27be68
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 7ea209e
Add data verification for config file
chooyikai a76184c
Add more checks for task and module loading
chooyikai 061747e
Fix for null grade
chooyikai 26bab0d
Add reposense tags to new files
chooyikai 8ff7cee
Update DG with details on data-load error handling
chooyikai 101058b
Added Javadoc, bugfix, and allowed "" for editing a task
heekit73098 1fd476b
Update code quality
heekit73098 7a96362
update pr based on comments
heekit73098 5d3a532
Update PPP
heekit73098 e95b519
Improve code quality
chooyikai e7ea440
Merge pull request #196 from heekit73098/branch-authorship
Ch40gRv1-Mu e2cbc69
Merge branch 'master' into config-bugfix
chooyikai e4f380b
Fix error when removing task description
chooyikai 8e626ae
resolved merge conflict
Yzkkk 3e5a802
Update UG to explain removing parameters
chooyikai fb00f69
Fix typo in DG
chooyikai 5d6deec
resolved merge conflict, added authorship
Yzkkk 0e7f0da
Merge pull request #194 from chooyikai/config-bugfix
heekit73098 ecb77c0
minor edits
Yzkkk 17a1081
minor edits
Yzkkk ae861fb
minor edits
Yzkkk ea68761
no message
Yzkkk 2ccdf61
Added support for module grade removal
heekit73098 916430e
Update PPP
heekit73098 335a614
fixed code quality
heekit73098 27f9c4a
Fix code quality
heekit73098 db57bc3
Merge pull request #183 from Yzkkk/master
heekit73098 5f2786a
Merge branch 'master' of https://github.com/AY2122S2-CS2113T-T10-3/tp…
heekit73098 5931420
Merge pull request #197 from heekit73098/branch-authorship
chooyikai 387a81d
minor fixing ad added javadoc for exceptions
Yzkkk 83dffd6
updated PPP
Yzkkk b6425d7
minor fixing
Yzkkk 1638940
Fix UG/DG typos and consistency issues
chooyikai d18a951
Merge pull request #198 from Yzkkk/master
chooyikai 12b2260
Merge branch 'master' into ugdg-typos
chooyikai f1b52a2
Fix merge issues
chooyikai 25a7e50
Update UG and PPP, fix bug in config loading
chooyikai d2e710d
Merge pull request #199 from chooyikai/ugdg-typos
heekit73098 937b7a6
Fix broken table for the nth time
chooyikai 9c92802
Minor fix to DG
chooyikai abf75d4
Update DG testing instructions
chooyikai 9c7cac6
Merge pull request #200 from chooyikai/table-fix
heekit73098 190c130
Update ch40grv1-mu.md
Ch40gRv1-Mu c9cfa76
Updated ch40grv1-mu.md
Ch40gRv1-Mu e8e02ae
Merge pull request #201 from Ch40gRv1-Mu/branch-update-PPP
ngys117 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,9 @@ | ||
# About us | ||
|
||
Display | Name | Github Profile | Portfolio | ||
--------|:----:|:--------------:|:---------: | ||
![](https://via.placeholder.com/100.png?text=Photo) | John Doe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | Don Joe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | Ron John | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | John Roe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | Don Roe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
| Display | Name | Github Profile | Portfolio | | ||
|-----------------------------------------------------|:-------------:|:-----------------------------------------:|:--------------------------------:| | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Mu Changrui | [Github](https://github.com/Ch40gRv1-Mu) | [Portfolio](team/ch40grv1-mu.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Yang Zikun | [Github](https://github.com/Yzkkk) | [Portfolio](team/yzkkk.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Ng Yong Sheng | [Github](https://github.com/ngys117) | [Portfolio](team/ngys117.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Choo Yi Kai | [Github](https://github.com/chooyikai/) | [Portfolio](team/chooyikai.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Bang Hee Kit | [Github](https://github.com/heekit73098/) | [Portfolio](team/heekit73098.md) | |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,66 @@ | ||
@startuml | ||
'https://plantuml.com/class-diagram | ||
|
||
!define ABSTRACT {abstract} | ||
|
||
skinparam classAttributeIconSize 0 | ||
skinparam shadowing false | ||
skinparam classFontSize 12 | ||
skinparam classAttributeFontSize 12 | ||
hide circle | ||
|
||
package command { | ||
class "ABSTRACT\nCommand" as Command | ||
class DeleteCommand | ||
class TextUi | ||
class ExitCommand | ||
|
||
Command <|-- DeleteCommand | ||
Command <|-- ExitCommand | ||
DeleteCommand ..> CommandResult :creates > | ||
ExitCommand ..> CommandResult :creates > | ||
|
||
DeleteCommand ..> TextUi :uses > | ||
|
||
class Command { | ||
+execute(): CommandResult ABSTRACT | ||
} | ||
|
||
class DeleteCommand { | ||
-moduleCode: String | ||
-taskModule: String | ||
-taskIndex: int | ||
+DeleteCommand(moduleCode: String) | ||
+DeleteCommand(taskModule: String, index: int) | ||
+execute(): CommandResult | ||
+getUserConfirmation(): String {static} | ||
} | ||
|
||
class TextUi { | ||
+getUserCommand(): String {static} | ||
+showMessage(): void {static} | ||
} | ||
|
||
class ExitCommand { | ||
+execute(): CommandResult | ||
} | ||
|
||
class CommandResult { | ||
+CommandResult(result: String) | ||
+toString(): String | ||
} | ||
} | ||
|
||
note top of command | ||
Note: | ||
This class diagram only illustrates two | ||
example command types for simplicity. | ||
end note | ||
|
||
note left of DeleteCommand | ||
Note: | ||
Dependency on TextUi is | ||
unique to DeleteCommand. | ||
end note | ||
|
||
@enduml | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
@startuml | ||
|
||
actor user | ||
user --> [Ui] | ||
[Ui] --> [Main] | ||
[Main] --> [Parser] | ||
[Parser] --> [Command]: produces > | ||
[Main] --> [Command]: executes > | ||
[Command] --> [Main]: returns feedback to > | ||
[Command] --> [Storage]: writes to > | ||
[Command] --> [Data]: manipulates > | ||
[Main] --> [Storage]: reads from > | ||
@enduml |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,87 @@ | ||
@startuml | ||
|
||
!define ABSTRACT {abstract} | ||
|
||
skinparam classAttributeIconSize 0 | ||
skinparam shadowing false | ||
skinparam classFontSize 12 | ||
skinparam classAttributeFontSize 12 | ||
hide circle | ||
|
||
package data { | ||
class ModuleList { | ||
-- | ||
+ addModule(m: Module): Module | ||
+ removeModule(moduleCode: String): Module | ||
+ getModule(moduleCode: String): Module | ||
} | ||
|
||
class Module { | ||
- moduleCode: String | ||
- moduleDescription: String | ||
- isGeneralTask: boolean | ||
- modularCredit: int | ||
-- | ||
+ addTask(task: Task): void | ||
} | ||
|
||
class TaskList { | ||
-- | ||
+ addTask(task: Task): Task | ||
+ removeTask(index: int): Task | ||
+ addTag(tagDescription: String, index: int): Task | ||
+ deleteTag(tagDescription: String, index: int): Task | ||
} | ||
|
||
class Task { | ||
- taskName: String | ||
- taskDescription: String | ||
- isTaskDone: boolean | ||
- workingTime: String | ||
- tags: ArrayList<String> | ||
-- | ||
+ getTaskParameterStatus(): TaskParameters | ||
+ toString(): String | ||
} | ||
|
||
enum "<<enumeration>>\nTaskParameters" as TaskParameters { | ||
DESCRIPTION_AND_WORKING_TIME | ||
DESCRIPTION_ONLY | ||
WORKING_TIME_ONLY | ||
NO_DESCRIPTION_OR_WORKING_TIME | ||
} | ||
} | ||
|
||
note top of data | ||
Note: | ||
Some methods have been | ||
omitted from this class diagram. | ||
end note | ||
|
||
ModuleList --> "1..*" Module | ||
|
||
Module -r> "1" TaskList | ||
TaskList -r> "*" Task | ||
Task --u> "1" TaskParameters | ||
|
||
class Main | ||
hide Main attributes | ||
hide Main methods | ||
|
||
Main --> "1" ModuleList | ||
|
||
class Command | ||
hide Command attributes | ||
hide Command methods | ||
|
||
Command ..> ModuleList | ||
|
||
class Storage | ||
hide Storage attributes | ||
hide Storage methods | ||
|
||
Storage ..> ModuleList | ||
|
||
hide TaskParameters methods | ||
|
||
@enduml |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
@startuml | ||
|
||
skinparam classAttributeIconSize 0 | ||
skinparam shadowing false | ||
skinparam classFontSize 12 | ||
skinparam classAttributeFontSize 12 | ||
hide circle | ||
hide attributes | ||
hide methods | ||
|
||
ModuleList --> "*" Module | ||
ModuleList --> "1" TaskList | ||
Module -l> "1" TaskList | ||
TaskList -l> "*" Task | ||
|
||
@enduml |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,60 @@ | ||
@startuml | ||
|
||
!define ABSTRACT {abstract} | ||
|
||
skinparam classAttributeIconSize 0 | ||
skinparam shadowing false | ||
skinparam classFontSize 12 | ||
skinparam classAttributeFontSize 12 | ||
hide circle | ||
|
||
package parsers { | ||
class "ABSTRACT\nParser" as Parser { | ||
commandFormat: String | ||
groupNames: HashSet<String> | ||
parsedCommand: HashMap<String, String> | ||
-- | ||
+ parseString(str: String): HashMap<String, String> | ||
} | ||
|
||
class XYZParser { | ||
+ parseCommand(userInput: String): Command | ||
} | ||
|
||
class ModHappyParser { | ||
- getCommandParser(commandWord: String): Parser | ||
+ parseCommand(userInput: String): Command | ||
} | ||
} | ||
|
||
note top of XYZParser | ||
XYZParser is any command-specific | ||
parser (e.g. AddParser, TagParser) | ||
end note | ||
|
||
Parser <|-u- ModHappyParser | ||
XYZParser <. ModHappyParser: uses < | ||
ModHappyParser <-u- Main | ||
|
||
hide Main methods | ||
hide Main attributes | ||
|
||
Parser <|-- XYZParser | ||
|
||
class "ABSTRACT\nCommand" as Command { | ||
} | ||
hide Command methods | ||
hide Command attributes | ||
|
||
XYZCommand <.u. ModHappyParser: returns < | ||
Command <|-- XYZCommand | ||
|
||
hide XYZCommand methods | ||
hide XYZCommand attributes | ||
|
||
note bottom of XYZCommand | ||
XYZCommand is any command | ||
(e.g. AddCommand, TagCommand) | ||
end note | ||
|
||
@enduml |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The use of diagrams is appropriate and they are very clear and easy to follow.