Statements should not be on same line as curly brace

Docs > Datadog Security > Code Security > Static Code Analysis (SAST) > SAST Rules > Statements should not be on same line as curly brace

Metadata

ID: kotlin-code-style/statement-wrapping

Language: Kotlin

Severity: Notice

Category: Code Style

Description

When using a multi-line statement with a curly brace, you should always put the contained code on a separate line from the curly brace.

The one exception to this is when using a lambda expression.

Non-Compliant Code Examples

fun doTask() { if (enabled) {
        spawn()
    }
}

val stringified = when (num) { /* comment */ 10 -> "Ten"
    20 -> "Twenty"
    else -> "Other"
}

durations.filter { d > 0 }.forEach { total += d
}

Compliant Code Examples

fun doTask() {
    if (enabled) {
        spawn()
    }
}

val stringified = when (num) {
    10 -> "Ten"
    20 -> "Twenty"
    else -> "Other"
}

// Single line block statements are ok
val double: (Int) -> Int = { a -> a * 2 }
enum class Status { OK, ERROR }

class Widget { // Comments are ok
    val enabled = true
    val kind = "mechanical"
}

durations.filter { d > 0 }.forEach {
    total += d
}

// It's fine to include params on a lamdba
val add: (Int, Int) -> Int = { a, b ->
    a + b
}