Merge pull request #44 from max-208/67-java

GCI67 S.T.E.P Challenge2025

Author: dedece35

CHANGELOG.md

@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - upgrade libraries versions
 - correction of technical problem with Integration tests (because of Maven format in technical answer to "sonar-orchestrator-junit5" library)
 - upgrade JDK from 11 to 17
+- [#4](https://github.com/green-code-initiative/creedengo-java/issues/4) Improvement: "++i" statement is not so bad
 
 ### Deleted
 

src/it/java/org/greencodeinitiative/creedengo/java/integration/tests/GCIRulesIT.java

@@ -429,8 +429,8 @@ void testGCI67() {
         String filePath = "src/main/java/org/greencodeinitiative/creedengo/java/checks/IncrementCheck.java";
         String ruleId = "creedengo-java:GCI67";
         String ruleMsg = "Use ++i instead of i++";
-        int[] startLines = new int[]{9, 19, 38};
-        int[] endLines = new int[]{9, 19, 38};
+        int[] startLines = new int[]{9, 24, 47};
+        int[] endLines = new int[]{9, 24, 47};
 
         checkIssuesForFile(filePath, ruleId, ruleMsg, startLines, endLines);
     }

src/it/test-projects/creedengo-java-plugin-test-project/src/main/java/org/greencodeinitiative/creedengo/java/checks/IncrementCheck.java

@@ -9,40 +9,71 @@ int foo1() {
         return counter++; // Noncompliant {{Use ++i instead of i++}}
     }
 
+    private int j = 0;
+    int foo10() {
+        return this.j++; // Compliant because maybe the use case needs to return j AND increment it
+    }
+
     int foo11() {
         int counter = 0;
         return ++counter;
     }
 
-    void foo2(int value) {
+    int foo2() {
         int counter = 0;
         counter++; // Noncompliant {{Use ++i instead of i++}}
+        return counter;
     }
 
-    void foo22(int value) {
+    int foo22() {
         int counter = 0;
         ++counter;
+        return counter;
     }
 
-    void foo3(int value) {
+    int foo3() {
         int counter = 0;
         counter = counter + 197845 ;
+        return counter;
     }
 
-    void foo4(int value) {
+    int foo4() {
         int counter = 0;
         counter = counter + 35 + 78 ;
+        return counter;
     }
 
-    void foo50(int value) {
+    void foo50() {
         for (int i=0; i < 10; i++) { // Noncompliant {{Use ++i instead of i++}}
-            System.out.println(i);
+            System.out.println(i); //NOSONAR
         }
     }
 
-    void foo51(int value) {
+    void foo51() {
         for (int i=0; i < 10; ++i) {
-            System.out.println(i);
+            System.out.println(i); //NOSONAR
         }
     }
+
+    void bar61(int value) {
+        // For test purpose
+    }
+
+    int foo61() {
+        int i = 0;
+        bar61(i++); // Compliant because maybe bar61 needs the unincremented value
+        return i;
+    }
+
+    int foo62() {
+        int i = 0;
+        bar61(2 + i++); // Compliant because maybe bar61 needs the unincremented value
+        return i;
+    }
+
+    void foo71() {
+        int counter = 0;
+        int a = 2 + counter++;  // Compliant because we probably want to increment counter
+                                // then to add it to 2 to initialize a
+    }
 }

src/main/java/org/greencodeinitiative/creedengo/java/checks/IncrementCheck.java

@@ -22,7 +22,10 @@
 
 import org.sonar.check.Rule;
 import org.sonar.plugins.java.api.IssuableSubscriptionVisitor;
+import org.sonar.plugins.java.api.tree.Arguments;
+import org.sonar.plugins.java.api.tree.BinaryExpressionTree;
 import org.sonar.plugins.java.api.tree.Tree;
+import org.sonar.plugins.java.api.tree.UnaryExpressionTree;
 import org.sonar.plugins.java.api.tree.Tree.Kind;
 import org.sonarsource.analyzer.commons.annotations.DeprecatedRuleKey;
 
@@ -40,6 +43,13 @@ public List<Kind> nodesToVisit() {
 
     @Override
     public void visitNode(Tree tree) {
+        UnaryExpressionTree unaryExprTree = (UnaryExpressionTree) tree;
+
+        if (unaryExprTree.parent() instanceof BinaryExpressionTree
+            || unaryExprTree.parent() instanceof Arguments
+            || unaryExprTree.expression().is(Tree.Kind.MEMBER_SELECT)) {
+            return ;
+        }
         reportIssue(tree, MESSAGERULE);
     }
 }

src/test/files/IncrementCheck.java

@@ -15,49 +15,92 @@
  * You should have received a copy of the GNU General Public License
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
-class MyClass {
-    MyClass(MyClass mc) {
+package org.greencodeinitiative.creedengo.java.checks;
+
+private class Foo {
+    public int i; //NOSONAR
+}
+
+class IncrementCheck {
+
+    IncrementCheck(IncrementCheck mc) {
     }
 
     int foo1() {
         int counter = 0;
         return counter++; // Noncompliant {{Use ++i instead of i++}}
     }
 
+    private int j = 0;
+    int foo10() {
+        return this.j++; // Compliant because maybe the use case needs to return j AND increment it
+    }
+
     int foo11() {
         int counter = 0;
         return ++counter;
     }
 
-    void foo2(int value) {
+    int foo12() {
+        Foo f;
+        return f.i++; // Compliant because maybe the use case needs to return j AND increment it
+    }
+
+    int foo2() {
         int counter = 0;
         counter++; // Noncompliant {{Use ++i instead of i++}}
+        return counter;
     }
 
-    void foo22(int value) {
+    int foo22() {
         int counter = 0;
         ++counter;
+        return counter;
     }
 
-    void foo3(int value) {
+    int foo3() {
         int counter = 0;
         counter = counter + 197845 ;
+        return counter;
     }
 
-    void foo4(int value) {
-        int counter =0;
+    int foo4() {
+        int counter = 0;
         counter = counter + 35 + 78 ;
+        return counter;
     }
 
-    void foo50(int value) {
+    void foo50() {
         for (int i=0; i < 10; i++) { // Noncompliant {{Use ++i instead of i++}}
-            System.out.println(i);
+            System.out.println(i); //NOSONAR
         }
     }
 
-    void foo51(int value) {
+    void foo51() {
         for (int i=0; i < 10; ++i) {
-            System.out.println(i);
+            System.out.println(i); //NOSONAR
         }
     }
+
+    void bar61(int value) {
+        // For test purpose
+    }
+
+    int foo61() {
+        int i = 0;
+        bar61(i++); // Compliant because maybe bar61 needs the unincremented value
+        return i;
+    }
+
+    int foo62() {
+        int i = 0;
+        bar61(2 + i++); // Compliant because maybe bar61 needs the unincremented value
+        return i;
+    }
+
+    void foo71() {
+        int counter = 0;
+        int a = 2 + counter++;  // Compliant because we probably want to increment counter
+                                // then to add it to 2 to initialize a
+    }
 }