Skip to content

BSL039 — Nested ternary operator

← All rules

Summary

Nested ternary operator

Identifiers

Field Value
Rule code BSL039
Compatible alias NestedTernaryOperator
Severity WARNING
Enabled by default Yes
Implemented Yes
Tags brain-overload, readability

Behavior

  • The public identifier BSL039 and alias NestedTernaryOperator are stable.
  • The rule reports the cases documented on this page.
  • Suppressions and project configuration are applied before publication.
  • The rule requires neither an external analyzer nor network access.

Configuration and suppression

BSL### is the primary stable identifier. The compatible alias is accepted in select, ignore, and compatible block suppression comments.

[tool.onec-hbk-bsl]
select = ["BSL039"]
ignore = ["NestedTernaryOperator"]

All three suppression families support both a current line and a range. When an opening comment follows code, it affects only that line. Use any one form:

  • noqa:
Value = "example";  // noqa: BSL039
  • bsl-disable:
Value = "example";  // bsl-disable: BSL039
  • compatible BSLLS form:
Value = "example";  // BSLLS:NestedTernaryOperator-off

When the same opening comment is on a line by itself, it starts a range. Close it with the matching marker from the same family:

// noqa: BSL039
// code without this diagnostic
// noqa-enable: BSL039

// bsl-disable: BSL039
// code without this diagnostic
// bsl-enable: BSL039

// BSLLS:NestedTernaryOperator-off
// code without this diagnostic
// BSLLS:NestedTernaryOperator-on

To disable the rule until the end of the file, omit the closing noqa-enable, bsl-enable, or BSLLS:…-on marker.

Opening and closing markers must belong to the same family.

Description

Use of nested ternary operators decrease code readability.

Examples

Incorrect use of ternary operators

Result = ?(X%15 <> 0, ?(X%5 <> 0, ?(X%3 <> 0, x, "Fizz"), "Buzz"), "FizzBuzz");

If ?(P.Emp_emptype = Null, 0, PageEmp_emptype) = 0 Then

      Status = "Done";

EndIf;

Possible implementation

If x % 15 = 0 Then
    Result = "FizzBuzz";
ElseIf x % 3 = 0 Then
    Result = "Fizz";
ElseIf x % 5 = 0 Then
    Result = "Buzz";
Else
    Result = x;
EndIf;

If PageEmp_emptype = Null OR PageEmp_emptype = 0 Then

      Status = "Done";

End If;