Skip to content

BSL265 — Useless ternary operator

← All rules

Summary

Useless ternary operator

Identifiers

Field Value
Rule code BSL265
Compatible alias UselessTernaryOperator
Severity INFORMATION
Enabled by default Yes
Implemented Yes
Tags redundant, readability

Behavior

  • The public identifier BSL265 and alias UselessTernaryOperator 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 = ["BSL265"]
ignore = ["UselessTernaryOperator"]

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: BSL265
  • bsl-disable:
Value = "example";  // bsl-disable: BSL265
  • compatible BSLLS form:
Value = "example";  // BSLLS:UselessTernaryOperator-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: BSL265
// code without this diagnostic
// noqa-enable: BSL265

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

// BSLLS:UselessTernaryOperator-off
// code without this diagnostic
// BSLLS:UselessTernaryOperator-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

The placement of Boolean constants "True" or "False" in the ternary operator indicates poor code thoughtfulness.

Examples

Useless operators

A = ?(B = 1, True, False);

A = ?(B = 0, False, True);

Suspicious operators (both branches are the same boolean constant, the result does not depend on the condition)

A = ?(B = 1, True, True);

A = ?(B = 0, False, False);