Detector that reports duplicated code in conditional branches.

Duplicated code in branches is bad because it:

  1. Reduces Readability: Repetition makes the code harder to understand.
  2. Increases Maintenance: Changes must be made in multiple places, risking errors.
  3. Signals Poor Design: It suggests missed opportunities for cleaner, more abstract code.
if (a > 42) {
  a = 43; // bad: duplicated code
} else {
  a = 43;
}

Use instead:

if (a > 42) {
  a = inc(b); // ok
} else {
  a = 43;
}

Hierarchy (view full)

Constructors

Properties

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

  • Returns true if the identifier with the given name should not be reported by unused variables detectors.

    Parameters

    • name: string

    Returns boolean