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)) {
/* ... */
}
def isFileOrDirectory(file: File): Boolean = file.isFile || file.isDirectory