diff --git a/build.gradle b/build.gradle index 885198fcfa..9b533f55d8 100644 --- a/build.gradle +++ b/build.gradle @@ -2,12 +2,18 @@ plugins { id 'java' id 'application' id 'com.github.johnrengelman.shadow' version '5.1.0' + id 'checkstyle' } repositories { mavenCentral() } + +checkstyle { + toolVersion = '10.2' +} + dependencies { testImplementation group: 'org.junit.jupiter', name: 'junit-jupiter-api', version: '5.5.0' testRuntimeOnly group: 'org.junit.jupiter', name: 'junit-jupiter-engine', version: '5.5.0' diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml new file mode 100644 index 0000000000..e8ee76467b --- /dev/null +++ b/config/checkstyle/checkstyle.xml @@ -0,0 +1,429 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + \ No newline at end of file diff --git a/config/checkstyle/suppressions.xml b/config/checkstyle/suppressions.xml new file mode 100644 index 0000000000..135ea49ee0 --- /dev/null +++ b/config/checkstyle/suppressions.xml @@ -0,0 +1,10 @@ + + + + + + + + \ No newline at end of file diff --git a/src/main/java/jarvis/Jarvis.java b/src/main/java/jarvis/Jarvis.java index 1b425ab9a9..f6365b0b87 100644 --- a/src/main/java/jarvis/Jarvis.java +++ b/src/main/java/jarvis/Jarvis.java @@ -32,8 +32,7 @@ public static void main(String[] args) throws JarvisException { e.printStackTrace(); } parser.farewell(); - } - catch (JarvisException e) { + } catch (JarvisException e) { System.out.println((e)); } } diff --git a/src/main/java/jarvis/Storage.java b/src/main/java/jarvis/Storage.java index b134b8c6ca..d87204ce6a 100644 --- a/src/main/java/jarvis/Storage.java +++ b/src/main/java/jarvis/Storage.java @@ -41,7 +41,6 @@ public List load() throws IOException { if (next.charAt(4) == '1') { taskList.get(taskList.size() - 1).mark(); } - } if (next.charAt(0) == 'E') { String description = next.substring(8, divisor - 1); diff --git a/src/main/java/jarvis/exception/JarvisException.java b/src/main/java/jarvis/exception/JarvisException.java index e2d9910f77..ca4f0cc77b 100644 --- a/src/main/java/jarvis/exception/JarvisException.java +++ b/src/main/java/jarvis/exception/JarvisException.java @@ -4,7 +4,6 @@ public class JarvisException extends Exception{ public JarvisException(String errorMessage) { super(errorMessage); } - @Override public String toString() { return "OOPS!!! " + super.getMessage(); diff --git a/src/main/java/jarvis/task/Task.java b/src/main/java/jarvis/task/Task.java index 9005745b0a..04532e4d8d 100644 --- a/src/main/java/jarvis/task/Task.java +++ b/src/main/java/jarvis/task/Task.java @@ -6,7 +6,6 @@ public abstract class Task { public String description; public boolean isDone; - public static int count = 0; /**