<p>in some cases are multiple if statement whereas both the if statement can be merged together , so collapsible if statement should be merged it will improve the readability and maintainability of the codebase.</p>
<h2 id="non-compliant-code">Non Compliant Code</h2>
<p><img src="https://cdn.hashnode.com/res/hashnode/image/upload/v1631093548565/U4DefFYyw.png" alt="image.png" /></p>
<h2 id="compliant-code">Compliant Code</h2>
<p><img src="https://cdn.hashnode.com/res/hashnode/image/upload/v1631093647248/falD7Iz0y.png" alt="screenshot_15.png" /></p>
<h2 id="example-from-sonar-source">Example from sonar Source</h2>
<h3 id="noncompliant-code-example">Noncompliant Code Example</h3>
<pre><code><span class="hljs-selector-tag">if</span> (file != null) {
  <span class="hljs-selector-tag">if</span> (file.isFile() || file.isDirectory()) {
    <span class="hljs-comment">/* ... */</span>
  }
}
</code></pre><h3 id="compliant-solution">Compliant Solution</h3>
<pre><code><span class="hljs-keyword">if</span> (file != <span class="hljs-keyword">null</span> &amp;&amp; isFileOrDirectory(file)) {
  <span class="hljs-comment">/* ... */</span>
}

<span class="hljs-function"><span class="hljs-keyword">private</span> <span class="hljs-keyword">static</span> <span class="hljs-keyword">boolean</span> <span class="hljs-title">isFileOrDirectory</span><span class="hljs-params">(File file)</span> </span>{
  <span class="hljs-keyword">return</span> file.isFile() || file.isDirectory();
}
</code></pre>

in some cases are multiple if statement whereas both the if statement can be merged together , so collapsible if statement should be merged it will improve the readability and maintainability of the codebase.


## Non Compliant Code


![image.png](https://cdn.hashnode.com/res/hashnode/image/upload/v1631093548565/U4DefFYyw.png)



## Compliant Code 
![screenshot_15.png](https://cdn.hashnode.com/res/hashnode/image/upload/v1631093647248/falD7Iz0y.png)



## Example from sonar Source

### Noncompliant Code Example

```
if (file != null) {
  if (file.isFile() || file.isDirectory()) {
    /* ... */
  }
}

```

### Compliant Solution
```
if (file != null && isFileOrDirectory(file)) {
  /* ... */
}

private static boolean isFileOrDirectory(File file) {
  return file.isFile() || file.isDirectory();
}

```

SonarLint : Java - Collapsible "if" statements should be merged (squid:S1066)

Non Compliant Code

Compliant Code

Example from sonar Source

Noncompliant Code Example

Compliant Solution