[bernicetoh] iP #207

bernicetoh · 2023-01-25T02:24:08Z

Duke

"Your mind is for having ideas, not holding them." - David Allen (source)

Free your mind and let Duke remember the things you have to do. It's,

text-based
easy to learn
~~FAST~~ SUPER FAST to use

All you need to do is,

download it from here
double-click it.
add your tasks.
let it manage your tasks for you 😆

And it is FREE!

Features:

Manage your todo list
Keep track of your deadlines
Keep track of your events

If you are a Java programmer, you can use it to practice Java too. Here's the main method:

public class Main {
    public static void main(String[] args) {
        Application.launch(MainApp.class, args);
    }
}

…hiveBaseName values

* branch-A-CodingStandard: Tweak code to comply with the coding standard

* branch-Level-9: Add search function to find a task by searching for a keyword

Use Assertion

There is a merge conflict when merging the code quality branch

Improve code quality

jishnu28 · 2023-02-05T12:17:34Z

src/main/java/duke/Duke.java

+        this.taskList = new TaskList(storage.loadData());
+
+    }
+


I think this method could be given a header comment.

jishnu28 · 2023-02-05T12:18:49Z

src/main/java/duke/DukeApp.java

+        userInput.clear();
+    }
+
+    /**


I think you could remove/replace these header comments once you're done following the instructions here.

jishnu28 · 2023-02-05T12:20:04Z

src/main/java/duke/commands/Command.java

@@ -0,0 +1,14 @@
+package duke.commands;


I think its a good call to package the different types of commands under this class.

jishnu28 · 2023-02-05T12:20:56Z

src/main/java/duke/commands/Command.java

+ */
+public abstract class Command {
+
+    public abstract boolean isExit();


I think you could include header comments for the following two lines.

jishnu28 · 2023-02-05T12:22:29Z

src/main/java/duke/parser/Parser.java

+        final String commandWord = matcher.group("commandWord");
+        final String arguments = matcher.group("arguments");
+
+        switch (commandWord) {


Good use of switch-case and appropriate indentation!

damithc and others added 30 commits July 31, 2022 17:20

Add Gradle support

556af3f

add level-1

6958076

add level-2

48c75f5

add level-3

6e286eb

add level-4

4d8d1ee

updated classes, added testing

bd8549a

changed class Task to be abstract

75c24b4

added error handling

66327ce

delete function

5b81a82

updated testing for delete

a91a441

updated testing for bye function

08c26f4

added javadocs and deleted command class

136bdf1

Add level-7

1496377

Add Level-8

d2810e0

Merge branch 'branch-Level-8'

0a73f7f

Refactor code

4281200

Divide classes into packages

7c86a39

Merge gradle

c9c2524

Add JUnit tests

6fc2dae

Reconfigure constructor of TaskList class

2374931

Add JUnit tests for TaskList class

4a54d96

Update build.gradle file to contain the correct mainClassName and arc…

96d9dc7

…hiveBaseName values

Add and tidy up JavaDoc

3b9f5d3

Tweak code to comply with the coding standard

075de79

Add search function to find a task by searching for a keyword

2da3c0d

Merge branch 'branch-A-CodingStandard'

5b764a8

* branch-A-CodingStandard: Tweak code to comply with the coding standard

Merge branch 'branch-Level-9'

9428f2a

* branch-Level-9: Add search function to find a task by searching for a keyword

Add data/duke.txt to .gitignore file

6e361a7

Create command classes and encapsulate Parser and Duke classes

50af343

Add checkstyle to detect coding style violations

0888507

bernicetoh and others added 3 commits February 5, 2023 18:12

Merge pull request #1 from bernicetoh/branch-A-Assertions

ced7dc1

Use Assertion

Remove line 36 in Parser.java

d759d30

There is a merge conflict when merging the code quality branch

Merge pull request #2 from bernicetoh/branch-A-CodeQuality

29153b4

Improve code quality

jishnu28 reviewed Feb 5, 2023

View reviewed changes

bernicetoh added 26 commits February 8, 2023 10:20

Add Tagging and Priority

1151944

Fix fonts in dialogbox and dukeapp

51f6972

Increase width of main stage

c3c3079

Fix bug when loading data

c510e4a

Fix spacing in UI

fa78a9c

Add Javadocs to classes

2c1f623

Modify layout of DukeApp

10e7b00

Update height of stage and mainlayout

ec236e0

Add User Guide

ab2539b

Add image to README.md

187743d

Fix image in README.md

90fecc7

Minor fixes in README.md

d747513

Add breaklines to README.md

05dcee7

Fix line breaks in README.md

3420222

Minor change to README.md

b89b4d2

Fix section link in README.md

bae617e

Change mainClassName in build.gradle

d20ef6a

Add catch blocks for tag and prioritize commands

d72cba7

Add catch blocks for tag and prioritize commands

6c87347

Minor fix in README.md

ac10cd5

Fix in README.md

5c79e3a

Terminate application with 'bye' command

a946628

Modify to follow coding standard

d488009

Remove unnecessary spacing and unused variables

86be507

Modify table format

cbd1515

Update starting GUI image

dcac494

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[bernicetoh] iP #207

[bernicetoh] iP #207

bernicetoh commented Jan 25, 2023 •

edited

jishnu28 Feb 5, 2023

jishnu28 Feb 5, 2023

jishnu28 Feb 5, 2023

jishnu28 Feb 5, 2023

jishnu28 Feb 5, 2023

[bernicetoh] iP #207

Are you sure you want to change the base?

[bernicetoh] iP #207

Conversation

bernicetoh commented Jan 25, 2023 • edited

Duke

jishnu28 Feb 5, 2023

Choose a reason for hiding this comment

jishnu28 Feb 5, 2023

Choose a reason for hiding this comment

jishnu28 Feb 5, 2023

Choose a reason for hiding this comment

jishnu28 Feb 5, 2023

Choose a reason for hiding this comment

jishnu28 Feb 5, 2023

Choose a reason for hiding this comment

bernicetoh commented Jan 25, 2023 •

edited