feat(EC-515): Solve cognitive complexity sonar issue

green-code-initiative · May 30, 2024 · d75fbe9 · d75fbe9
1 parent e188254
commit d75fbe9
Showing 1 changed file with 19 additions and 18 deletions.
diff --git a/swift-lang/src/main/java/io/ecocode/ios/swift/checks/sobriety/AudioRecorderLeakCheck.java b/swift-lang/src/main/java/io/ecocode/ios/swift/checks/sobriety/AudioRecorderLeakCheck.java
@@ -39,26 +39,27 @@ public void apply(ParseTree tree) {
         }
 
         if (importExist) {
-            if (tree instanceof Swift5Parser.ExpressionContext && tree.getText().contains("record()")) {
-                id = (Swift5Parser.ExpressionContext) tree;
-                audioRecorderStarted = true;
-            }
+            findStartedButNotStoppedAudioRecord(tree);
+        }
+    }
 
-            if (tree instanceof Swift5Parser.ExpressionContext
-                    && (tree.getText().contains("stop()"))) {
-                audioRecorderStopped = true;
-            }
+    private void findStartedButNotStoppedAudioRecord(ParseTree tree) {
+        if (tree instanceof Swift5Parser.ExpressionContext && tree.getText().contains("record()")) {
+            id = (Swift5Parser.ExpressionContext) tree;
+            audioRecorderStarted = true;
+        }
 
-            if (tree instanceof TerminalNodeImpl && tree.getText().equals("<EOF>")) {
-                if (audioRecorderStarted && !audioRecorderStopped) {
-                    this.recordIssue(id.getStart().getStartIndex(), DEFAULT_ISSUE_MESSAGE);
-                }
-                audioRecorderStarted = false;
-                audioRecorderStopped = false;
-                importExist = false;
+        if (tree instanceof Swift5Parser.ExpressionContext && (tree.getText().contains("stop()"))) {
+            audioRecorderStopped = true;
+        }
+
+        if (tree instanceof TerminalNodeImpl && tree.getText().equals("<EOF>")) {
+            if (audioRecorderStarted && !audioRecorderStopped) {
+                this.recordIssue(id.getStart().getStartIndex(), DEFAULT_ISSUE_MESSAGE);
             }
+            audioRecorderStarted = false;
+            audioRecorderStopped = false;
+            importExist = false;
         }
     }
-}
-
-
+}