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