Why is this an issue?
Merging collapsible if
statements increases the code’s readability.
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();
}