[EC63] UnnecessarilyAssignValuesToVariables rule on caught exception

CHANGELOG.md

@@ -19,6 +19,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - [#128](https://github.com/green-code-initiative/ecoCode/pull/128) Adding EC35 rule for Python and PHP : EC35 rule replaces EC34 with a specific use case ("file not found" sepcific)
 - [#132](https://github.com/green-code-initiative/ecoCode/issues/132) Upgrade RULES.md: set proposed Python rule "Use numpy array instead of standard list" as refused with link to the justification
 - [#103](https://github.com/green-code-initiative/ecoCode/issues/103) Upgrade RULES.md: set proposed HTML rule "HTML page must contain a doctype tag" as refused with link to the justification
+- [#228](https://github.com/green-code-initiative/ecoCode/issues/228) Upgrade rule EC63 for java: Stop reporting caught exception even if unused
 
 ### Deleted
 

java-plugin/src/main/java/fr/greencodeinitiative/java/checks/UnnecessarilyAssignValuesToVariables.java

@@ -75,7 +75,7 @@ private void reportIfUnknow(String name, Tree tree) {
     private class GetVariableVisitor extends BaseTreeVisitor {
         @Override
         public void visitVariable(VariableTree tree) {
-            if (!tree.parent().is(Kind.METHOD)) {
+            if (!tree.parent().is(Kind.METHOD) && !tree.parent().is(Kind.CATCH)) {
                 variableList.put(tree.simpleName().name(), tree);
             }
             super.visitVariable(tree);

java-plugin/src/test/files/UnnecessarilyAssignValuesToVariablesTestCheck.java

@@ -92,4 +92,13 @@ public void testNonCompliantThrow() throws Exception {
     public void testCompliantThrow() throws Exception {
         throw new Exception("dummy");
 	}
-}
+
+    public void testCompliantCatchException() throws Exception {
+        try {
+            int i = 1;
+            System.out.println(i);
+        } catch (Exception e) {
+            System.out.println("Something went wrong.");
+        }
+    }
+}