Detects opportunities for simplifying code by eliminating redundant boolean expressions and statements.

Redundant code can make programs less efficient and harder to read. Simplifying such code improves readability, maintainability, and can prevent potential logical errors.

What it checks:

  • if statements that return boolean literals directly based on a condition.
  • Comparisons of boolean expressions with boolean literals (true or false).
  • Conditional expressions (ternary operators) that return boolean literals.
// Redundant 'if' statement:
if (condition) {
    return true;
} else {
    return false;
}
// Simplify to:
return condition;

// Redundant comparison:
return a == true;
// Simplify to:
return a;

// Redundant conditional expression:
return b ? true : false;
// Simplify to:
return b;

Hierarchy (view full)

Constructors

Properties

severity: Severity = Severity.LOW

Gets the severity of the detector.

Accessors

  • get id(): string
  • Gets the short identifier of the detector, used in analyzer warnings.

    Returns string

    The unique identifier of the detector.

  • get shareImportedWarnings(): WarningsBehavior
  • Defines the behavior of warnings generated by this detector when working with multiple projects within a single Tact configuration.

    Here are the available options:

    1. "union" Leave this value if you don't care about warnings generated in other projects.
    2. "intersect" If the warning is generated for some source location of the imported file, it should be generated by each of the projects. Example: Constants from an imported file should not be reported iff they are unused in all the projects, so you need "intersect".

    Returns WarningsBehavior

  • get usesSouffle(): boolean
  • Checks whether this detector needs the Soufflé binary to be executed.

    Returns boolean

Methods