-
Notifications
You must be signed in to change notification settings - Fork 248
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
[CS2113-T15-1] CLIAlgo #33
Open
ong-ck
wants to merge
772
commits into
nus-cs2113-AY2223S2:master
Choose a base branch
from
AY2223S2-CS2113-T15-1: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 250 commits
Commits
Show all changes
772 commits
Select commit
Hold shift + click to select a range
135689c
Add design and implementation for remove and update add sequence diagram
nicholas132000 5c5f20e
Fix issues #169, #169, #171
nikkiDEEE c1ddd95
Hide footbox for Ui and Help seq diagrams
nikkiDEEE b805720
Merge pull request #218 from nikkiDEEE/branch-PE-D-Issues-v2.0
heejet bc9c81f
Update PPP name for it to be checked by grading scripts
heejet 808bd6e
Add break for exceptions for step 2 visualisation under Ui section in DG
nikkiDEEE 7ad463b
Refactor sequence diagram spacing
nikkiDEEE fae1b49
Add printWithBox() method to Ui
heejet 0f61516
Refactor code according to comments
heejet dee7818
Resolve lifeline extending after object deletion
heejet 432b03d
Update names of sequence diagram
heejet 16f574e
Update sequence diagrams to remove footbox
heejet 3ea4dfe
Merge pull request #226 from heejet/branch-refactor-code-v2.1
heejet 34a2ebe
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nikkiDEEE b13924b
Refactor according to PR review
nicholas132000 d8c03e5
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 ada088a
Refactor according to PR review
nicholas132000 047dabc
Update DeveloperGuide.md
heejet 899ca5d
Merge pull request #224 from nicholas132000/branch-v2.0b-DG
heejet 1641c97
Fix magic keywords/numbers, move printing stuff from FilterByImportan…
nikkiDEEE b1c0e76
Fix whitespace issue in Ui
nikkiDEEE 7c4ea13
Add: PPP
lohjooh 393e4a8
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
lohjooh d6bc2a0
Fix: addressed code review comments
lohjooh e37baaf
Refactor break to alt in Ui sequence diagram
nikkiDEEE e54933c
Add: justification for file manager
lohjooh 3b84d7f
Merge pull request #229 from lohjooh/branch-PPP-v2.1
lohjooh 295541e
Add export to HelpCommand sequence diagram
nikkiDEEE 0d25740
Refactor Ui diagram to fit DG example
nikkiDEEE fb1581a
Add FileManagerStub
heejet 3231ceb
Add JUnit test for TopicManager
heejet db6a1af
Add type to sequence diagram
nikkiDEEE 9da1b62
Add JUnit test for getAllCS2040CFilesBeforeTopic() method
heejet c27985a
Add JUnit test for getAllFilesAsFiles()
heejet d992663
Add: JUnit test for SingleFile, ExitTestModeCommand, FileDecoder
lohjooh 3748288
Add: JUnit tests for Buffer
lohjooh 4869885
Add: Javadoc for ExitTestModeCommand
lohjooh 7b1abb9
Fix: HeadlessException in BufferTest
lohjooh 9239540
Fix: checkstyle
lohjooh 4319d44
Fix: attempt to fix run failing on Github
lohjooh d8280b0
Fix: attempt to fix JUnit tests
lohjooh 7f8f1a3
Fix: checkstyle
lohjooh c7f0482
Fix: typo
lohjooh 4baadf8
Fix: checkstyle
lohjooh d782b8a
Fix: attempt to fix JUnit
lohjooh 51741d6
Fix: attempt to fix JUnit tests
lohjooh 570ca00
Fix: SingleFileTest
lohjooh f6b0b04
Fix: attempt to fix JUnit tests
lohjooh 92f5b94
Fix: attempt to fix JUnit tests
lohjooh 4a296a6
Fix: change implementation such that it works
lohjooh b4307de
Fix: attempt to fix JUnit tests
lohjooh 957bd56
Fix: attempt to change folder for FileManagerTest
lohjooh fddfa53
Fix: commented out suspected faulty code
lohjooh ba84395
Fix: checkstyle
lohjooh f7d8f92
Fix: revert to last known working configuration
lohjooh 16ef52e
Add Parser JUnit test
heejet 4044024
Add documentation to ParserTest
heejet ce54338
Refactor FileManagerStub
heejet 7aa660c
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nikkiDEEE 5aef41e
Merge pull request #225 from nikkiDEEE/branch-PE-D-Issues-v2.0
nikkiDEEE 43e6164
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nikkiDEEE f409f6f
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet 642d15d
Refactor Ui sequence diagram
nikkiDEEE b8f572b
Add: Constructor to instantiate TopoCommand with custom LinkedHashMap
ong-ck 1bc0aea
Add: TopoCommand JUnit test
ong-ck c84d3ac
Add: author tags and remove Queue.txt
ong-ck e975752
Fix checkstyle
ong-ck a2650b8
Remove excessive string constants
nikkiDEEE 414490e
Add Junit test to AddCodeCommand
nicholas132000 0a135b7
Merge pull request #228 from nikkiDEEE/branch-Code-Review-Fix-v2.1
heejet 1f18247
Merge pull request #232 from ong-ck/branch-JUnit-TopoCommand-v2.0
heejet 3291bb4
Add PPP
nikkiDEEE 9d83171
Fix: made requested changes
lohjooh d8ec8e3
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
lohjooh cc1a1a7
Merge pull request #234 from nikkiDEEE/branch-PPP-v.2.1
heejet c11f991
Merge pull request #230 from lohjooh/branch-JUnit-v2.1
heejet 7071d5e
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet 7896de0
Refactor according to PR review
nicholas132000 4299c83
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 81f8e27
Add: reclaim some code
lohjooh 7cb72e8
Add: remove assertions that are not supposed to be there
lohjooh 6504a85
Merge pull request #236 from lohjooh/branch-assetions-v2.1
lohjooh 20855f7
Add: image tags for UML diagrams to resize
ong-ck 270b8ae
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
ong-ck bacf50d
Fix Code Review Issues
ong-ck c7ef5ff
Fix: cleaned up "magic" string
lohjooh b1c3038
Refactor and address mathan code reviews
nicholas132000 6f18f79
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 89e06c5
Add PPP
nicholas132000 9da674f
Merge pull request #237 from ong-ck/branch-Update-DG-v2.1
heejet faf4887
Merge pull request #238 from lohjooh/branch-FixMagicString-v2.1
heejet 42efd55
Rename PPP filename
nicholas132000 1313e25
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 07719f1
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet c77a833
Update gitignore
heejet 8b565d7
Fix checkstyle violations
heejet c94529b
Refactor code in Parser
heejet 9a4437c
Merge pull request #240 from heejet/branch-TopicManager-JUnit-v2.1
lohjooh 9df3a9c
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 85477cf
Update lifeline note in DG
heejet b8a7fcc
Refactor code into packages
heejet 57921e4
Add description to design of Logic
heejet 82cc339
Update architecture diagram to include logic
heejet 1bff763
Refactor code due to checkstyle violations
heejet d323d2c
Fix: changed Note to CS2040CFile
lohjooh ea358a0
Fix: error in ExportClass
lohjooh df18909
Add TopicManager class diagram
heejet 8c4b0bb
Update table of contents in DG
heejet c15a071
Merge pull request #241 from lohjooh/branch-UpdateClassDiagram-v2.1
heejet 26714b1
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet ade8ae7
Rename notes to CS2040CFile in Glossary of DeveloperGuide
ong-ck dca3f7a
Add: PPP
ong-ck 022139a
Merge pull request #242 from ong-ck/branch-Add-PPP-v2.1
heejet 30c18d3
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet bfe133f
Refactor according to PR review
nicholas132000 2c082e5
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 428ffc7
Add junit for AddNoteCommand
nicholas132000 788aae0
Fix checkstyle end with a newline
nicholas132000 4846687
Add Logic sequence diagram
heejet d7f4841
Add implementation of Logic component to DG
heejet 5579b1f
Refactor DG
heejet e326b51
Add acknowledgements to DG
heejet 488b697
Add acknowledgements to UG
heejet f01d5c2
Refactor code to reduce complicated expressions
nicholas132000 9d66ac4
Update architecture sequence diagram
heejet 9b51e79
Update Parser class diagram in DG
heejet 4118b78
Update Add and Remove description in DG
heejet b322d5d
Update Buffer design in DG
heejet f8b03ea
Refactor grammar
nicholas132000 c105734
Update add implementation in DG
heejet 4d9d40f
Update remove implementation in DG
heejet 0936538
Update TopoSort implementation in DG
heejet 5a48323
Refactor DG
heejet 26ea54f
Fix developer guide
nicholas132000 cd264d2
Update author tags
heejet 0d2fbf9
Refactor junit test
nicholas132000 a6fb054
Add JUnit Tests
nikkiDEEE 24d181e
Refactor to gradle checks
nikkiDEEE 741ba87
Refactor to gradle checks
nikkiDEEE 1f44ccc
Refactor to gradle checks
nikkiDEEE 5b1f611
Merge pull request #233 from nicholas132000/branch-v2.0b-DG
heejet e5dee65
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp
heejet 772cdfb
Fix checkstyle violations
heejet bdc8e92
Merge pull request #244 from heejet/branch-Logic-documentation-v2.1
heejet 21d52b1
Update author tags
heejet cbbdc4e
Update docs README
heejet 2fe8985
Refactor gradle checks
nikkiDEEE 6ef612a
Merge branch 'master' into branch-More-JUnit-Tests-v.2.1
nikkiDEEE 1fdddd2
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nikkiDEEE b585cb1
Merge pull request #245 from nikkiDEEE/branch-More-JUnit-Tests-v.2.1
nikkiDEEE 87021e0
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nikkiDEEE e2ff4e8
Update _config.yml
nikkiDEEE d869bf7
Edit UG
nikkiDEEE f2dcbe7
Merge branch 'branch-Jekyll-CSS-v.2.1' of https://github.com/nikkiDEE…
nikkiDEEE 0e04483
Add: JUnit tests for FilterCommand
ong-ck 34a2a4a
Add: iuml file to standardise style in uml diagrams
ong-ck 62002ef
Fix: sequence diagram for AddEntry
lohjooh 635cf82
Update uml diagram png files
ong-ck fbd852a
Merge pull request #247 from ong-ck/branch-UML-style-v2.1
lohjooh 7c338d9
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
lohjooh bc0c629
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
lohjooh c4e9320
Merge branch 'branch-SequenceDiagram-v2.1' of https://github.com/lohj…
lohjooh 9fa076a
Add: style for puml diagrams
lohjooh 5224bd5
Add junit test for FilterByImportanceCommand
nicholas132000 64f44b8
Add junit for note and code
nicholas132000 59fe5ff
Merge pull request #248 from lohjooh/branch-SequenceDiagram-v2.1
heejet 7e59b08
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet cd63674
Add colours to sequence diagram
heejet b127ebf
Add color to class diagrams
heejet 2277615
Update PPP
heejet 18f9e8b
Refactor diagrams
nicholas132000 2ec1f80
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 d51c65c
Fix bug where file does not detect in mac OS
heejet 2ee1229
Update UG to remove redundant warning
heejet db0e49a
Merge pull request #257 from heejet/branch-update-documentation-v2.1
heejet 3b3c8d7
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nikkiDEEE 2535728
Refactor add diagrams to include color and update PPP
nicholas132000 4f16250
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nicholas132000 5149748
Fix to comply with mac OS
nicholas132000 9c2fabf
Merge pull request #250 from nicholas132000/branch-v2.0b-DG
heejet 19022de
Update PPP
heejet 0e16d57
Add color to sequence and class diagrams
nikkiDEEE a640fd4
Update UG to the jar file name convention
nikkiDEEE 3480504
Update PPP
nikkiDEEE 62d51c4
Update PPP
nikkiDEEE af4a7b7
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
ong-ck 3af6625
Refactor UG
nikkiDEEE b5492ed
Add: colours to activation box
lohjooh 774b1cd
Refactor UG
nikkiDEEE 184d3f6
Change font of CLIAlgo ascii art
nikkiDEEE 0389272
Merge pull request #259 from lohjooh/branch-AddColourAddSecurityExcep…
heejet 31b804f
Add: JUnit tests for FilterByTopicCommand
ong-ck 02c6541
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
ong-ck 80a4a62
Fix newline at end of file checkstyle
ong-ck c51b65a
Update colour for TopoCommand UML Class Diagram
ong-ck 1eae726
Update TopoSort UML Sequence Diagram
ong-ck 60870e6
Update PPP
ong-ck f05d0eb
Merge pull request #258 from nikkiDEEE/branch-Diagrams-v2.1
nikkiDEEE 37b4059
Update CLIAlgo logo in README
ong-ck 419b3da
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet add408e
Update size of UML Diagrams in DeveloperGuide
ong-ck 9999e34
Update Links in DeveloperGuide
ong-ck 4830899
Merge pull request #246 from ong-ck/branch-JUnit-Filter-v2.1
heejet 2fd6e0c
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
ong-ck 75d1dd5
Merge pull request #261 from ong-ck/branch-UML-colour-v2.1
heejet 0f9b98f
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
heejet 3b57350
Add page breaks to DG
heejet 611f49a
Reformat UG with page breaks
heejet 6bcb4b6
Reformat DG
heejet 756c062
Reformat UG and DG
heejet c7e303a
Add page breaks to UG
heejet aa660fd
Add refactor export section in UG
heejet 7822315
Add page breaks to UG
heejet f7cba8d
Add page break to UG
heejet 433e2dc
Add page break to UG
heejet 348d931
Reformat UG
heejet 11c60d7
Reformat UG
heejet 1ccae4e
Update formatting of UG
heejet 43b4c17
Reformat UG
heejet 553b3ec
Reformat DG
heejet 59d4105
Reformat DG
heejet 070986a
Update PPP
heejet eb89f96
Add page breaks to PPP
heejet 8df9e92
Merge pull request #262 from heejet/branch-code-refactoring-v2.1
heejet 3c232e9
Fix typos
heejet 2e021da
Merge pull request #263 from heejet/branch-check-documentation-v2.1
heejet d2cdeda
Refactor DG and UG
nicholas132000 b3881d6
Merge pull request #264 from nicholas132000/branch-v2.0b-DG
heejet 0898f99
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
nikkiDEEE 16c3c30
Refactor UG
ong-ck 97d16f6
Edit PPP and add author tag
nikkiDEEE fd6d2de
Update UG Command Summary table
ong-ck 3f3cc80
Refactor author tag
nikkiDEEE 9ea770f
Merge pull request #265 from nikkiDEEE/branch-Final-Edits-v2.1
heejet 86fd6eb
Refactor DG and PPP
ong-ck 491e4c8
Merge branch 'master' of https://github.com/AY2223S2-CS2113-T15-1/tp …
ong-ck e1abb90
Merge pull request #267 from ong-ck/branch-Final-Refactor-v2.1
heejet 0b66adf
Add colon to feature title
heejet 0f08092
Reformat DG
heejet b09f40a
Add documentation to command package
heejet e9e2f59
Add documentation to exception and file packages
heejet 4aacaec
Add documentation to top level classes
heejet fd6fe9e
Fix checkstyle violations
heejet 311502a
Merge pull request #268 from heejet/branch-final-changes-v2.1
heejet 0e2c0d0
Fix portfolio link in AboutUs
ong-ck f9fe107
Merge pull request #269 from ong-ck/branch-Final-Changes-v2.1
heejet 2d665f4
Reformat table in AboutUs.md
heejet 62a1db0
Merge pull request #270 from heejet/branch-edit-AboutUs-v2.1
heejet 478d276
Fix bug in TopoCommand
heejet cf0c9cd
Merge pull request #271 from heejet/branch-fix-bug-v2.1
heejet 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,12 @@ | ||
# 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://avatars.githubusercontent.com/u/88279614?s=400&u=3271621fb9d98584e92a54677e4e386854d94ec5&v=4) | Ong Hee Jet | [Github](https://github.com/heejet) | [Portfolio](team/ongheejet.md) | ||
![](https://avatars.githubusercontent.com/u/88373092?v=4) | Loh Joo Hoe | [Github](https://github.com/lohjooh) | [Portfolio](team/lohjoohoe.md) | ||
![](https://avatars.githubusercontent.com/u/88139349?s=400&u=1a6d496e41aae8ff748ef6aad040be452b531cd4&v=4) | Nikhil Shashidhar | [Github](https://github.com/nikkiDEEE) | [Portfolio](team/nikhil.md) | ||
![](https://avatars.githubusercontent.com/u/85627680?s=400&u=07e24badd619ed48cab013038fe25c7201e111a0&v=4) | Ong Chuan Kai | [Github](https://github.com/ong-ck) | [Portfolio](team/ongchuankai.md) | ||
![](https://avatars.githubusercontent.com/u/88270205?v=4) | Nicholas H Goh Maowen | [Github](https://github.com/nicholas132000) | [Portfolio](team/nicholas.md) | ||
|
||
|
Large diffs are not rendered by default.
Oops, something went wrong.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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,57 @@ | ||
@startuml | ||
'https://plantuml.com/sequence-diagram | ||
|
||
autonumber | ||
|
||
@startuml | ||
skinparam classAttributeIconSize 0 | ||
hide circle | ||
|
||
abstract class Command { | ||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
} | ||
|
||
class AddCommand { | ||
#name: String | ||
#ui: String | ||
#topic: String | ||
+getName(): String | ||
+getTag(): String | ||
+checkFileType(): FileType | ||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
} | ||
|
||
class AddCodeCommand { | ||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
} | ||
|
||
class AddNoteCommand { | ||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
} | ||
|
||
class TopicManager { | ||
} | ||
|
||
class Ui { | ||
} | ||
|
||
class Buffer { | ||
} | ||
|
||
class FileManager { | ||
} | ||
|
||
Command <|-- AddCommand | ||
AddCommand <|-- AddCodeCommand | ||
AddCommand <|-- AddNoteCommand | ||
|
||
TopicManager .. AddCommand: gets reference from < | ||
Ui .. AddCommand: gets reference from < | ||
FileManager .. AddCommand: gets reference from < | ||
Buffer .. AddCommand: gets reference from < | ||
|
||
@enduml | ||
|
||
|
||
|
||
@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,59 @@ | ||
@startuml | ||
skinparam classAttributeIconSize 0 | ||
hide empty members | ||
|
||
class user { | ||
|
||
} | ||
|
||
package clialgo <<rectangle>> { | ||
class CLIAlgo #lightgreen { | ||
|
||
} | ||
|
||
class Parser #lightblue{ | ||
} | ||
|
||
class Ui #red { | ||
} | ||
|
||
class TopicManager #purple { | ||
} | ||
|
||
class Topic #magenta{ | ||
} | ||
|
||
class command #grey { | ||
} | ||
|
||
class storage #beige { | ||
} | ||
|
||
class Buffer #pink { | ||
} | ||
|
||
class CS2040CFile { | ||
} | ||
} | ||
package harddisk { | ||
} | ||
user --> Ui | ||
CLIAlgo --> Ui | ||
CLIAlgo -->TopicManager | ||
CLIAlgo --> Parser | ||
CLIAlgo ..> storage | ||
Parser ..> Ui | ||
Parser ..> TopicManager | ||
Parser ..> command | ||
storage ..> harddisk | ||
TopicManager --> Topic | ||
storage --> CS2040CFile | ||
command ..> Ui | ||
command ..> TopicManager | ||
command ..> storage | ||
command ..> CS2040CFile | ||
command ..> Buffer | ||
Topic ..> CS2040CFile | ||
|
||
hide circle | ||
@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,59 @@ | ||
@startuml | ||
skinparam classAttributeIconSize 0 | ||
hide circle | ||
|
||
abstract class Command { | ||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
} | ||
|
||
class FilterCommand { | ||
#keyWord: String | ||
#topic: String | ||
|
||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
} | ||
|
||
class FilterByTopicCommand { | ||
#keyWord: String | ||
#topic: String | ||
|
||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
-printAllTopics(topicManager : TopicManager, ui : Ui) : void | ||
-printSingleTopic(topicManager : TopicManager, ui : Ui) : void | ||
} | ||
|
||
class FilterByImportanceCommand { | ||
#keyWord: String | ||
#topic: String | ||
|
||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
-printAllTopics(topicManager : TopicManager, ui : Ui) : void | ||
-printSingleTopic(topicManager : TopicManager, ui : Ui) : void | ||
} | ||
|
||
class TopicManager { | ||
} | ||
|
||
class Ui { | ||
} | ||
|
||
class FileManager { | ||
} | ||
|
||
class Buffer { | ||
} | ||
|
||
Command <|-- FilterCommand | ||
FilterCommand <|-- FilterByTopicCommand | ||
FilterCommand <|-- FilterByImportanceCommand | ||
|
||
TopicManager .. FilterCommand | ||
Ui .. FilterCommand | ||
FileManager .. FilterCommand | ||
Buffer .. FilterCommand | ||
|
||
@enduml | ||
|
||
|
||
|
||
@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,43 @@ | ||
@startuml | ||
'https://plantuml.com/sequence-diagram | ||
|
||
autonumber | ||
|
||
@startuml | ||
skinparam classAttributeIconSize 0 | ||
hide circle | ||
|
||
abstract class Command { | ||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager): void | ||
} | ||
|
||
class HelpCommand { | ||
-command: String | ||
+getCommand(): String | ||
+execute(topicManager: TopicManager, ui: Ui, fileManager: FileManager, buffer: Buffer): void | ||
} | ||
|
||
class TopicManager { | ||
} | ||
|
||
class Ui { | ||
} | ||
|
||
class FileManager { | ||
} | ||
|
||
class Buffer { | ||
} | ||
|
||
Command <|-- HelpCommand | ||
|
||
|
||
TopicManager .. HelpCommand | ||
Ui .. HelpCommand | ||
FileManager .. HelpCommand | ||
Buffer .. HelpCommand | ||
@enduml | ||
|
||
|
||
|
||
@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.
Should the command class also have two blank sections for attribute and method like other classes?