-
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
[dsja612] iP #239
base: master
Are you sure you want to change the base?
[dsja612] iP #239
Conversation
Great job dude 👍 |
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.
Overall, I find your code easy to read! For the most part, it looks like coding standards were upheld! Great Job!
src/main/java/views/UI.java
Outdated
* Displays a list of possible commands | ||
* to the user |
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.
Might it be better to end with a fullstop?
src/main/java/tasks/Event.java
Outdated
* @param name Name of the task | ||
* @param startDate Starting date of the task | ||
* @param endDate Ending date of the task |
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.
Might it be better to add a fullstop behind these lines?
src/main/java/storage/Storage.java
Outdated
} | ||
case "D": { | ||
String by = inputList[2]; | ||
LocalDateTime deadline = LocalDateTime.parse(by, DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm")); |
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.
Is this line a bit too long?
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.
Well done! Your code looks fantastic!
|
||
/** | ||
* Constructor for DukeException that stores an error message. | ||
* @param message Main body of error message |
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.
Perhaps this needs capitalising as well!
src/main/java/storage/Storage.java
Outdated
|
||
/** | ||
* Loads tasks from a local file into a TaskList, creates one if one does not exist. | ||
* @throws IOException If fail to load/create a storage file |
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.
Perhaps this needs capitalising as well!
src/main/java/storage/Storage.java
Outdated
String[] inputList = input.split(","); | ||
String taskType = inputList[0]; | ||
String taskName = inputList[1]; |
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.
Your code here looks extremely readable! 👓
The GUI does not automatically close after the bye command is entered. The size of the window is also too small to display the initial list of supported commands. The GUI will now automatically close after a 3 second delay. Also, the GUI window is a bit bigger to accommodate long lists of text in the future if needed.
Add assertions for multiple files
Change Find command to use streams
Improve on code quality
Add command for user to find closest free day
Duke
Yet another school project??
Duke is a task manager written in Java, in an object-oriented manner. This is all so you can live your life in a task-oriented manner 🤓
"Pfft, why would I use Duke instead of Google Calendar??" - 🤓
And it is FREE!
Features
Download it here!
If you want to learn Java & OOP:
Feel free to check out how Duke is organized. Duke starts running from this code block in
Duke.java
: