Swift

Swift

Made by DeepSource

Fallthrough should be avoided SW-W1012

Bug risk
Minor

Using the fallthrough keyword in switch statements can lead to subtle and hard-to-find bugs in Swift. The fallthrough keyword allows the control flow to move to the next case in a switch statement. This can be useful in some cases, but it can also make the code harder to read and increase the risk of unexpected behavior.

Avoid using fallthrough in switch statements whenever possible. Instead, consider using separate cases for each condition in the switch statement. This makes the code more explicit and easier to understand. In cases where fallthrough is necessary, make sure to add comments that explain why the fallthrough is being used and what its expected behavior is.

Bad Practice

switch someInteger {
case 1:
    print("One")
    fallthrough
case 2:
    print("Two")
default:
    print("Something else")
}

Recommended

switch someInteger {
case 1:
    print("One")
case 2:
    print("Two")
default:
    print("Something else")
}