io.gitlab.arturbosch.detekt.rules.style.CollapsibleIfStatements.kt Maven / Gradle / Ivy

Go to download

Show more of this group Show more artifacts with this name
Show all versions of detekt-rules-style Show documentation

Static code analysis for Kotlin

The newest version!

package io.gitlab.arturbosch.detekt.rules.style

import io.gitlab.arturbosch.detekt.api.CodeSmell
import io.gitlab.arturbosch.detekt.api.Config
import io.gitlab.arturbosch.detekt.api.Debt
import io.gitlab.arturbosch.detekt.api.Entity
import io.gitlab.arturbosch.detekt.api.Issue
import io.gitlab.arturbosch.detekt.api.Rule
import io.gitlab.arturbosch.detekt.api.Severity
import org.jetbrains.kotlin.psi.KtContainerNodeForControlStructureBody
import org.jetbrains.kotlin.psi.KtExpression
import org.jetbrains.kotlin.psi.KtIfExpression
import org.jetbrains.kotlin.psi.psiUtil.getChildrenOfType

/**
 * This rule detects `if` statements which can be collapsed. This can reduce nesting and help improve readability.
 *
 * However, carefully consider whether merging the if statements actually improves readability, as collapsing the
 * statements may hide some edge cases from the reader.
 *
 * 
 * val i = 1
 * if (i > 0) {
 *     if (i < 5) {
 *         println(i)
 *     }
 * }
 * 
 *
 * 
 * val i = 1
 * if (i > 0 && i < 5) {
 *     println(i)
 * }
 * 
 */
class CollapsibleIfStatements(config: Config = Config.empty) : Rule(config) {

    override val issue = Issue(
        "CollapsibleIfStatements",
        Severity.Style,
        "Two if statements which could be collapsed were detected. " +
            "These statements can be merged to improve readability.",
        Debt.FIVE_MINS
    )

    override fun visitIfExpression(expression: KtIfExpression) {
        if (isNotElseIfOrElse(expression) && hasOneKtIfExpression(expression)) {
            report(CodeSmell(issue, Entity.from(expression), issue.description))
        }
        super.visitIfExpression(expression)
    }

    private fun isNotElseIfOrElse(expression: KtIfExpression) =
        expression.`else` == null && expression.parent !is KtContainerNodeForControlStructureBody

    private fun hasOneKtIfExpression(expression: KtIfExpression) =
        expression.then?.getChildrenOfType()?.singleOrNull().isLoneIfExpression()

    private fun KtExpression?.isLoneIfExpression() = this is KtIfExpression && `else` == null
}