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

"switch" statements should have "default" clauses

    Details

    • Type: Code Smell Detection
    • Status: Active
    • Resolution: Unresolved
    • Labels:
    • Message:
      Add a "[case default|WHEN OTHERS|Case Else]" clause to this "[switch|CASE|Select]" statement.
    • Default Severity:
      Critical
    • Impact:
      High
    • Likelihood:
      Low
    • Default Quality Profiles:
      Sonar way
    • Legacy Key:
      switch-without-default, SwitchWithoutDefault, SwitchLastCaseIsDefaultCheck, SelectWithoutCaseElse, SelectWithoutOtherwise, COBOL.EvaluateStatementUsageCheck
    • Targeted languages:
      RPG, TypeScript
    • Covered Languages:
      ABAP, C#, C, C++, Cobol, Flex, Go, Java, JavaScript, Objective-C, PHP, PL/I, PL/SQL, Ruby, T-SQL, VB.Net, VB6
    • Irrelevant for Languages:
      HTML, Kotlin, Python, Swift, XML
    • Remediation Function:
      Constant/Issue
    • Constant Cost:
      5min
    • Analysis Level:
      Syntactic Analysis
    • Analysis Scope:
      Main Sources, Test Sources
    • Common Rule:
      Yes
    • CERT:
      MSC01-C.
    • CWE:
      CWE-478
    • Checkstyle:
      DefaultComesLast, MissingSwitchDefault
    • ESLint:
      default-case
    • FindBugs:
      SF_SWITCH_NO_DEFAULT
    • PC-Lint:
      744, 1744, 9014, 9082
    • PMD:
      DefaultLabelNotLastInSwitchStmt,SwitchStmtsShouldHaveDefault

      Description

      The requirement for a final default clause is defensive programming. The clause should either take appropriate action, or contain a suitable comment as to why no action is taken.

      See

        Attachments

          Issue Links

          duplicates

          Finding Detection - RSPEC-915 A switch statement shall be a well-formed switch statement

          • Closed
          is duplicated by

          Finding Detection - RSPEC-918 The final clause of a switch statement shall be the default clause

          • Closed
          is implemented by

          New Feature - A new feature of the product, which has yet to be developed. SONARRPG-160 Rule S131: "switch" statements should have "default" clauses

          • Major - Major loss of function.
          • Open

          New Feature - A new feature of the product, which has yet to be developed. SONARSLANG-229 [Scala] Enable Rule: S131 "match" statements should have "case _" clauses

          • Major - Major loss of function.
          • Open

          New Feature - A new feature of the product, which has yet to be developed. SONARPLI-28 Rule : Avoid SELECT statement without the OTHERWISE clause

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARPLSQL-669 Rule S131: "switch" statements should have "default" clauses

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARSLANG-76 Rule S131: "switch" statements should have "default" clauses

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARSLANG-312 [Apex] Enable Rule: S131 "switch" statements should have "default" clauses

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARSWIFT-348 Rule S131: "switch" statements should have "default" clauses

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARTSQL-25 Rule: "CASE" expressions should end with "ELSE" clauses

          • Major - Major loss of function.
          • Closed

          New Feature - A new feature of the product, which has yet to be developed. SONARVBSIX-177 Rule: "[switch|CASE]" statements should end with a "[case default|WHEN OTHERS|Case Else]" clause

          • 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

          Improvement - An improvement or enhancement to an existing feature or task. CPP-2003 Adjust the behavior of S131 to not raise an issue when "default" is not at the correct position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARABAP-211 Update title and description of rule "Missing OTHERS in CASE"

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARABAP-369 Improve S131: exclude cases when "default" is not last

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARCOBOL-1510 Adjust the behavior of S131 to not raise an issue when "default" is not at the correct position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARFLEX-150 SwitchWithoutDefault(S131): Do not raise an issue when "default" is not at last position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARJAVA-2700 Adjust the behavior of S131 to not raise an issue when "default" is not at the correct position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARPHP-784 Adjust the behavior of S131 to not raise an issue when "default" is not at the correct position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARPLI-192 Adjust the behavior of S131 to not raise an issue when "default" is not at the correct position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARTSQL-128 Adjust the behavior of S131 to not raise an issue when "default" is not at the correct position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARVB-305 Adjust the behavior of S131 to not raise an issue when "default" is not at the correct position

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SONARVBSIX-276 S131: Do not raise an issue when "default" is not at the correct position

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

          Code Smell Detection - RSPEC-3532 Empty "default" clauses should be removed

          • Active

          Code Smell Detection - RSPEC-3562 "switch" statements should cover all cases

          • Active
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. CPP-2768 S131, S2816: Accept the "default:"-less exhaustive "switch"es for the SonarWay version of S2816(S131)

          • Major - Major loss of function.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. CPP-2766 S131: Use "double quotes" for the code in the message

          • Major - Major loss of function.
          • Closed

          Code Smell Detection - RSPEC-3458 Empty "case" clauses that fall through to the "default" should be omitted

          • Active
          links to

          Web Link is implemented by SonarGo/issues/160

          Web Link Is implemented by SonarTS/issues/490

            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: