False positive: Boolean literals should not be redundant

In the following code snippet SonarCloud reports the else as redundant:

val useDeviceTimeZone = if (isEmpty()) false else repository.readUseDeviceTimeZoneEnabled()

Here is the full function:


Hi @tobiaspreuss,

This looks like a true positive to me. Your case seems to be the same as the following line in the noncompliant example of the rule description:

booleanVariable = if (booleanMethod()) false else exp;

It therefore can be replaced as follows:

val useDeviceTimeZone = !isEmpty() && repository.readUseDeviceTimeZoneEnabled()


1 Like

Thank you. This wasn’t obvious to me.

This topic was automatically closed 7 days after the last reply. New replies are no longer allowed.