Skip to content

[Ria Khaitan] iP#328

Open
riakhaitan wants to merge 39 commits into
nus-cs2103-AY2122S2:masterfrom
riakhaitan:master
Open

[Ria Khaitan] iP#328
riakhaitan wants to merge 39 commits into
nus-cs2103-AY2122S2:masterfrom
riakhaitan:master

Conversation

@riakhaitan
Copy link
Copy Markdown

@riakhaitan riakhaitan commented Feb 6, 2022

Halloumi ChatBot ^_^

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

The Halloumi ChatBot keeps track of your tasks and progress, all in one place. It's,

  • easy SUPER EASY to use ⏩
  • text-based 📝
  • offline 🌳

If you're 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);
    }
}

All you need to do is,

  1. Sign up here for free
  2. Complete the demo
  3. Start adding your tasks

Features:

  • Task management
  • Managing deadlines (coming soon)
  • Reminders (coming soon)

Some commands to get you started with Halloumi are:

list
todo <insert task here>
mark 1

Copy link
Copy Markdown

@rahulprasad01 rahulprasad01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall really clean code

Comment thread src/main/java/Duke.java Outdated
@@ -1,10 +1,108 @@
import exceptions.DukeExceptions;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the import statements be ordered differently?

Comment thread src/main/java/Duke.java Outdated
System.out.println("|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|");
}

public static String makeDesc(String[] text, int len) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps a more intuitive variable name here?

Comment thread src/main/java/Task.java Outdated
}
}

class ToDo extends Task {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like how the code has been indented

Copy link
Copy Markdown

@markbiju markbiju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A good effort at following Java coding standards, with appropriate method naming and use of camelCase!

Comment thread src/main/java/Duke.java Outdated
File f = new File("/Users/riakhaitan/iP/ip/data/duke.txt");
f.createNewFile();

boolean directory = f.getParentFile().mkdirs();
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a suggestion, perhaps you could use a name for this variable that is more boolean-sounding such as isDirectory? Good use of mkdir() here, creating a directory to store data

Comment thread src/main/java/Task.java Outdated
when = date;
}
public String toString() {
return "[D]" + super.toString() + " (: " +
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A good practice would be to place a line break before an operator! Good use of line break to improve readability

Comment thread src/main/java/Duke.java Outdated
String[] textSplit = textSplitOne[0].split(" ");
String fullDesc = makeDesc(textSplit, textSplit.length);
try {
switch (textSplit[0]) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For case clauses, there is no indentation

Comment thread src/main/java/Duke.java Outdated
System.out.println("You have " + lists.size() + " tasks left now! ^_^");
lineTwo();
break;
case "deadline":
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logical units within a block should be separated by a line, perhaps a space after each case would be ideal!

riakhaitan and others added 11 commits February 13, 2022 19:51
…cies

To implement JavaFX and incorporate gui in our code, we need to add the related dependencies for it.

We add the corresponding dependencies to our build.gradle so we are able to use JavaFX and incorporate gui in our code.

We need to add dependencies to our build.gradle so we can use gradle to build our gui, this is the most efficient way.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants