-
Notifications
You must be signed in to change notification settings - Fork 437
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
[Gao Gui] iP #115
base: master
Are you sure you want to change the base?
[Gao Gui] iP #115
Conversation
src/main/java/Deadline.java
Outdated
return "[D]" + super.toString() + " (by: " + by.getMonth().toString().substring(0, 3) + " " + | ||
by.getDayOfMonth() + " " + by.getYear() + " " + | ||
String.format("%d:%d)", by.getHour(), by.getMinute()) + ")"; |
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.
I think placing the '+' in front of every line is better.
src/main/java/Duke/Deadline.java
Outdated
protected LocalDateTime by; | ||
|
||
/** | ||
* Constructs an event with a deadline. |
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.
It would be better to leave a line after the description.
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.
LGTM. Just some nits to be fixed.
Overall I like your coding style as it is easy to read and the method names are easy to understand.
branch-A-Assertions
branch-A-CodeQuality
branch-A-Lambdas
No description provided.