@cyqjoseph We did an automated analysis of your code to detect potential areas to improve the code quality. We are sharing the results below, to help you improve the iP code further.
IMPORTANT: Note that the script looked for just a few easy-to-detect problems only, and at-most three example are given i.e., there can be other areas/places to improve.
Aspect: Tab Usage
No easy-to-detect issues ๐
Aspect: Naming boolean variables/methods
No easy-to-detect issues ๐
Aspect: Brace Style
Example from src/main/java/services/TaskList.java
lines 175-176
:
// For other tasks, compare by name only
else {
Suggestion: As specified by the coding standard, use egyptian style braces.
Aspect: Package Name Style
No easy-to-detect issues ๐
Aspect: Class Name Style
No easy-to-detect issues ๐
Aspect: Dead Code
No easy-to-detect issues ๐
Aspect: Method Length
Example from src/main/java/services/parser/Parser.java
lines 204-251
:
public LocalDateTime parseDate(String dateString) throws DukeException {
assert dateString != null && !dateString.isEmpty() : "dateString cannot be null or empty";
List<DateTimeFormatter> dateTimeFormatters = Arrays.asList(
DateTimeFormatter.ofPattern("d/M/yyyy HHmm"),
DateTimeFormatter.ofPattern("d-M-yyyy HHmm"),
DateTimeFormatter.ofPattern("yyyy-M-d HHmm"),
DateTimeFormatter.ofPattern("d/M/yyyy HH:mm"),
DateTimeFormatter.ofPattern("M/d/yyyy h:mm a"),
DateTimeFormatter.ofPattern("yyyy-M-d HHmm"),
DateTimeFormatter.ofPattern("d MMM yyyy h:mma"),
DateTimeFormatter.ofPattern("MMM d, yyyy HH:mm"),
DateTimeFormatter.ofPattern("yyyyMMdd h:mm a"),
DateTimeFormatter.ofPattern("d-M-yyyy HH:mm"),
DateTimeFormatter.ofPattern("yyyy/M/d h:mma"),
DateTimeFormatter.ofPattern("d MMMM yyyy HHmm"),
DateTimeFormatter.ofPattern("yyyy-M-d h:mm a")
);
List<DateTimeFormatter> dateFormatters = Arrays.asList(
DateTimeFormatter.ofPattern("d/M/yyyy"),
DateTimeFormatter.ofPattern("M/d/yyyy"),
DateTimeFormatter.ofPattern("yyyy/M/d"),
DateTimeFormatter.ofPattern("d-M-yyyy"),
DateTimeFormatter.ofPattern("M-d-yyyy"),
DateTimeFormatter.ofPattern("yyyy-M-d"),
DateTimeFormatter.ofPattern("d MMM yyyy"),
DateTimeFormatter.ofPattern("MMM d, yyyy"),
DateTimeFormatter.ofPattern("d MMMM yyyy"),
DateTimeFormatter.ofPattern("yyyyMMdd")
);
for (DateTimeFormatter formatter : dateTimeFormatters) {
try {
return LocalDateTime.parse(dateString, formatter);
} catch (DateTimeParseException e) {
// Continue to try the next format
}
}
for (DateTimeFormatter formatter : dateFormatters) {
try {
return LocalDate.parse(dateString, formatter).atStartOfDay();
} catch (DateTimeParseException e) {
// Continue to try the next format
}
}
throw new DukeException("Invalid Date and time format");
}
Example from src/main/java/services/parser/Parser.java
lines 260-303
:
public static Task parseTaskFromString(String taskString) throws DukeException {
assert taskString != null && !taskString.isEmpty() : "taskString cannot be null or empty";
String[] parts = taskString.split(" \\| ");
assert parts.length >= 3 : "Task string format is invalid";
String taskType = parts[0];
boolean isDone = parts[1].trim().equals("1");
String description = parts[2].trim();
String additionalInfo = parts.length > 3 ? parts[3].trim() : null;
switch (taskType) {
case "T":
ToDo todo = new ToDo(description);
if (isDone) {
todo.markAsDone();
}
return todo;
case "D":
if (additionalInfo == null) {
throw new DukeException("Invalid tasks.Deadline format in file");
}
LocalDateTime by = LocalDateTime.parse(additionalInfo);
Deadline deadline = new Deadline(description, by);
if (isDone) {
deadline.markAsDone();
}
return deadline;
case "E":
String[] times = additionalInfo.split(" to ");
if (times.length < 2) {
throw new DukeException("Invalid tasks.Event time format in file.");
}
LocalDateTime start = LocalDateTime.parse(times[0].trim());
LocalDateTime end = LocalDateTime.parse(times[1].trim());
Event event = new Event(description, start, end);
if (isDone) {
event.markAsDone();
}
return event;
default:
return null;
}
}
Suggestion: Consider applying SLAP (and other abstraction mechanisms) to shorten methods e.g., extract some code blocks into separate methods. You may ignore this suggestion if you think a longer method is justified in a particular case.
Aspect: Class size
No easy-to-detect issues ๐
Aspect: Header Comments
No easy-to-detect issues ๐
Aspect: Recent Git Commit Message
possible problems in commit ee190cf
:
Done C-DetectDuplicates
This commit introduces a significant enhancement to the project to
filter for duplicates.
Added isDuplicateTask method in TaskList.
- For Deadline tasks, it checks for duplicates based on both name and deadline time.
- For Event tasks, it verifies if either the start or end time, along with the task name, is identical to an existing task.
- For other task types, a simple name-based comparison is performed.
Modified the addTask method to prevent addition of duplicate tasks.
- body not wrapped at 72 characters: e.g.,
- For Deadline tasks, it checks for duplicates based on both name and deadline time.
possible problems in commit 8c00bb0
:
Done A-Asssertions
In this commit, assertions have been integrated into the project to improve the overall reliability and maintainability of the code. These additions are focused on ensuring that key assumptions in the codebase are explicitly checked during runtime.
- body not wrapped at 72 characters: e.g.,
In this commit, assertions have been integrated into the project to improve the overall reliability and maintainability of the code. These additions are focused on ensuring that key assumptions in the codebase are explicitly checked during runtime.
possible problems in commit 58622c3
:
Implement Assertions in Various Classes
This commit introduces the use of Java assertions across multiple classes in the project to validate assumptions about the program's state. Assertions are added to ensure that the program operates under expected conditions, enhancing the robustness and reliability of the code during development.
Changes made:
1. TaskList Class:
- Added assertions to check if task-related operations are
performed with valid indices and non-null task objects.
2. Storage Class:
- Inserted assertions to validate non-null task lists and the
existence of the file after creation.
3: Parser Class:
- Added assertions in date parsing methods to ensure non-null
and properly formatted date strings.
- Implemented assertions to verify non-empty and non-null user
input strings.
- body not wrapped at 72 characters: e.g.,
This commit introduces the use of Java assertions across multiple classes in the project to validate assumptions about the program's state. Assertions are added to ensure that the program operates under expected conditions, enhancing the robustness and reliability of the code during development.
Suggestion: Follow the given conventions for Git commit messages for future commits (do not modify past commit messages as doing so will change the commit timestamp that we used to detect your commit timings).
Aspect: Binary files in repo
No easy-to-detect issues ๐
โน๏ธ The bot account used to post this issue is un-manned. Do not reply to this post (as those replies will not be read). Instead, contact [email protected]
if you want to follow up on this post.