Uploaded image for project: 'Rules Repository'
  1. Rules Repository
  2. RSPEC-1862

Related "if/else if" statements should not have the same condition

    Details

    • Type: Bug Detection
    • Status: Active
    • Resolution: Unresolved
    • Labels:
    • Message:
      This [branch|case] duplicates the one on line n.
    • Highlighting:
      Hide
      • primary: 2nd duplicate condition
      • secondary: 1st condition
        • message: Original

      Only the characters part of the conditions should be underlined.

      Show
      primary: 2nd duplicate condition secondary: 1st condition message: Original Only the characters part of the conditions should be underlined.
    • Default Severity:
      Major
    • Impact:
      Low
    • Likelihood:
      High
    • Default Quality Profiles:
      Sonar way
    • Targeted languages:
      Flex, PL/I, VB6
    • Covered Languages:
      ABAP, C#, C, C++, Cobol, Go, Java, JavaScript, Kotlin, Objective-C, PHP, PL/SQL, Python, RPG, Ruby, Scala, Swift, T-SQL, TypeScript, VB.Net
    • Irrelevant for Languages:
      HTML, XML
    • Remediation Function:
      Constant/Issue
    • Constant Cost:
      10min
    • Analysis Level:
      Syntactic Analysis
    • Analysis Scope:
      Main Sources, Test Sources
    • Common Rule:
      Yes
    • CERT:
      MSC12-C.
    • CPPCheck:
      duplicateIf, multiCondition
    • ESLint:
      no-duplicate-case
    • ESLint-SonarJS:
      no-identical-conditions
    • PC-Lint:
      137, 1137, 142, 1142
    • PVS-Studio:
      V3003
    • TSLint-SonarTS:
      no-identical-conditions

      Description

      A chain of if/else if statements is evaluated from top to bottom. At most, only one branch will be executed: the first one with a condition that evaluates to true.

      Therefore, duplicating a condition automatically leads to dead code. Usually, this is due to a copy/paste error. At best, it's simply dead code and at worst, it's a bug that is likely to induce further bugs as the code is maintained, and obviously it could lead to unexpected behavior.

      See

      • CERT, MSC12-C. - Detect and remove code that has no effect or is never executed

        Attachments

          Issue Links

          contributes to

          MMF - Minimum Marketable Feature MMF-887 First pack of Typescript rules available as a TSLint extension

          • Major - Major loss of function.
          • Closed
          deprecates

          Bug Detection - RSPEC-1950 Cases in a "switch" should not have the same condition

          • Deprecated
          is implemented by

          New Feature - A new feature of the product, which has yet to be developed. SONARFLEX-71 Rule S1862: Related "if/else if" statements and "cases" in a "switch" should not have the same condition

          • Major - Major loss of function.
          • Open

          New Feature - A new feature of the product, which has yet to be developed. SONARPLI-180 Rule: Related "if/else if" statements should not have the same condition

          • Major - Major loss of function.
          • Open

          New Feature - A new feature of the product, which has yet to be developed. SONARVBSIX-260 Rule: Related "if/else if" statements should not have the same condition

          • Major - Major loss of function.
          • Open

          New Feature - A new feature of the product, which has yet to be developed. CPP-709 Rule: Duplicate conditions in "if" and related "else if"

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARCOBOL-1188 Rule: Conditions in related "IF/ELSE IF" statements and "WHEN" clauses in "EVALUATE" statements should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARFLEX-66 Rule: Related "if/else if" statements and "cases" in a "switch" should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARPHP-430 Rule: Related "if/else if" statements and "cases" in a "[switch|Select Case]" should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARPLSQL-600 Rule: Related "IF/ELSIF" statements and "WHEN" clauses in a "CASE" should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARRPG-116 Rule: Related "IF/ELSEIF" statements and "WHEN" clauses in a "SELECT" should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARSLANG-155 Enable in Ruby rule S1862 (IdenticalConditionsCheck)

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARSLANG-218 [Scala] Enable Rule: S1862 Related "if/else if" statements should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARSLANG-322 [Apex] Enable Rule: S1862 Related "if/else if" statements should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARSWIFT-60 Rule: Conditions in related "if/else if" statements and "cases" in a "switch" should not have the same condition

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARTSQL-15 Rule: Related "IF"/"ELSE IF" statements and "WHEN" clauses in a "CASE" should not have the same condition

          • Major - Major loss of function.
          • Closed

          Task - A task that needs to be done. CPP-707 Create Objective-C rules repository containing targeted rules

          • Major - Major loss of function.
          • Closed

          Task - A task that needs to be done. SONARSLANG-426 [Go] Enable rules relying on Conditions

          • Major - Major loss of function.
          • Closed
          is related to

          Improvement - An improvement or enhancement to an existing feature or task. CPP-2510 S3923: Improve code equivalence checking in case of macro expansion

          • Major - Major loss of function.
          • Open

          Bug Detection - RSPEC-1763 All code should be reachable

          • Active

          Code Smell Detection - RSPEC-901 Dead code should be removed

          • Closed

          Code Smell Detection - RSPEC-2760 Sequential tests should not check the same condition

          • Active
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. SONARPHP-1086 S1862: Add a message on the secondary location

          • Major - Major loss of function.
          • Closed
          links to

          Web Link Is implemented by SONARJAVA-579

          Web Link Is Implemented in SonarTS by #11

          1.
          		 Java RSPEC-1863 Language-Specification Active Unassigned
          2.
          		 C-Family RSPEC-1864 Language-Specification Active Unassigned
          3.
          		 JavaScript: Related "if/else if" statements should not have the same condition RSPEC-1865 Language-Specification Active Unassigned
          4.
          		 PHP: Related "if/else if" statements and "cases" in a "switch" should not have the same condition RSPEC-1866 Language-Specification Active Unassigned
          5.
          		 PL/SQL: Related "IF/ELSIF" statements and "WHEN" clauses in a "CASE" should not have the same condition RSPEC-1868 Language-Specification Active Unassigned
          6.
          		 Cobol: Conditions in related "IF/ELSE IF" statements and "WHEN" clauses in "EVALUATE" statements should not have the same condition RSPEC-1972 Language-Specification Active Unassigned
          7.
          		 Swift: Related "if/else if" statements and "cases" in a "switch" should not have the same condition RSPEC-2628 Language-Specification Active Unassigned
          8.
          		 Python RSPEC-2680 Language-Specification Active Unassigned
          9.
          		 C# RSPEC-2731 Language-Specification Active Unassigned
          10.
          		 RPG: Related "IF/ELSEIF" statements and "WHEN" clauses in a "SELECT" should not have the same condition RSPEC-2867 Language-Specification Active Unassigned
          11.
          		 ABAP RSPEC-3702 Language-Specification Active Unassigned
          12.
          		 VB.NET: Related "If/ElseIf" statements should not have the same condition RSPEC-3713 Language-Specification Active Unassigned
          13.
          		 T-SQL: Related "IF"/"ELSE IF" statements and "WHEN" clauses in a "CASE" should not have the same condition RSPEC-4097 Language-Specification Active Unassigned
          14.
          		 Go RSPEC-4515 Language-Specification Active Unassigned
          15.
          		 Ruby: Related "if/elsif" statements and "when" in a "case" should not have the same condition RSPEC-4765 Language-Specification Active Unassigned
          16.
          		 Scala: Related "if"/"else if" statements and "case" in a "match" should not have the same condition RSPEC-4934 Language-Specification Active Unassigned
          17.
          		 Apex: Related "if"/"else if" statements and "when" in a "switch" should not have the same condition RSPEC-5013 Language-Specification Active Unassigned

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                freddy.mallet Freddy Mallet (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: