-
Notifications
You must be signed in to change notification settings - Fork 364
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
[linustws] iP #95
base: master
Are you sure you want to change the base?
[linustws] iP #95
Conversation
Level-7
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.
Just a few coding convention issues but other than that good job on the OOP design and the commenting.😄
import duke.task.Deadline; | ||
import duke.task.Event; | ||
import duke.task.Todo; | ||
|
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.
Maybe this empty line could be removed
public static Command parse(String fullCommand) throws DukeException { | ||
String[] parts = fullCommand.split(" ", 2); | ||
int indexInput; | ||
switch(parts[0].trim()) { |
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.
switch cases statements do not need an indent as per the coding convention
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.
Some positive comments that I found quite helpful for myself!
src/main/java/duke/task/Event.java
Outdated
LocalDate endDate = null; | ||
LocalDateTime endDateTime = null; | ||
DateTimeFormatter[] formatters = { | ||
DateTimeFormatter.ofPattern("ddMMyyyy HHmm"), |
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.
Good job on including so many datetime formats for Event class even when it is not one of the required increments 👍
@@ -0,0 +1,142 @@ | |||
package duke.ui; |
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.
nice UI class!
src/test/java/duke/DukeTest.java
Outdated
@@ -0,0 +1,120 @@ | |||
package duke; |
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.
very comprehensive test cases that tests for almost all features
src/test/java/duke/ui/UiTest.java
Outdated
package duke.ui; | ||
|
||
import org.junit.jupiter.api.Test; | ||
|
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.
Maybe all these empty lines between import statements can be removed?
Create _config.yml and add theme
Duke (BorzAI Edition)
Let BorzAI unleash the full potential of your mind by memorising things you need to do so you don't have to. It's,
EASYSUPER EASY to useAll you have to do is,
And it is FREE!
Features:
If you Java programmer, you can use it to practice Java too. Here's the
main
method: