Skip to content

Commit

Permalink
Merge pull request #219 from yyccbb/master
Browse files Browse the repository at this point in the history
DG update
  • Loading branch information
yyccbb authored Apr 15, 2024
2 parents 3ae7f81 + 8be0f48 commit 2fdc33a
Showing 1 changed file with 17 additions and 0 deletions.
17 changes: 17 additions & 0 deletions docs/DeveloperGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,23 @@ The below sequence diagram depicts the process of a user executing the `find` co

![FindSequenceDiagram](images/FindSequenceDiagram.png)

### Stateful Utility Classes

The `ParserUtil` and `StringUtil` classes were originally classes that provide utility
APIs for parsing and string operations. Upon noticing the need that some parsing and string
operations require state information of the current model, both classes are updated to record
the state information and follow the singleton pattern. They are renamed `StatefulParserUtil` and
`StatefulStringUtil`.
<br>

For both classes,
* An `initialize()` method is provided to initialize with the current model. This method should only
be called once.
* An `getInstance()` method is provided to access the private field `model` that
captures the state information
* Some utility methods can now use the state information (e.g. the current filtered list of persons)
to achieve their functionalities.

### \[Proposed\] Undo/redo feature

#### Proposed Implementation
Expand Down

0 comments on commit 2fdc33a

Please sign in to comment.