Test 003

.vscode/settings.json

@@ -0,0 +1,6 @@
+{
+    "sonarlint.connectedMode.project": {
+        "connectionId": "http-localhost-9000",
+        "projectKey": "robbiebise_test-java_AYUMz9uSsTqRf_H8TmDL"
+    }
+}

main/Main.java

@@ -9,9 +9,28 @@ public static void main(String[] arguments) {
       a = "asdf";
       if (a != null) {out.println(a);}
       //TODO: something elsee
+      //TODO: another task
+      for (;true;) { 
+         out.println("bad"); 
+         OldStuff oldStuff = new OldStuff();
+         oldStuff.oldMethod();
+      }
 
-      OldStuff oldStuff = new OldStuff();
-      oldStuff.oldMethod();
+      for (;true;) { 
+         out.println("bad"); 
+         OldStuff oldStuff = new OldStuff();
+         oldStuff.oldMethod();
+      }
    }
+
+   void removeFrom(List<String> list) {
+      // expected: iterate over all the elements of the list
+      for (int i = 0; i < list.size(); i++) {
+        if (list.get(i).isEmpty()) {
+          // actual: remaining elements are shifted, so the one immediately following will be skipped
+          list.remove(i); // Noncompliant
+        }
+      }
+    }
 }
 

main/OldStuff.java

@@ -2,9 +2,31 @@
 import static java.lang.System.out;
 
 
-// @Deprecated
+abstract class MyClass{
+
+}
+
+class AbstractLikeClass {
+
+}
+
 public class OldStuff {
    public void oldMethod(){
       out.println("old!");
    }
-}
+
+   private String color = "red";
+
+   private void doSomething(){
+   synchronized(color) {  // Noncompliant; lock is actually on object instance "red" referred to by the color variable
+      //...
+      color = "green"; // other threads now allowed into this block
+      // ...
+   }
+   synchronized(new Object()) { // Noncompliant this is a no-op.
+      // ...
+   }
+   }
+}
+
+