SonarSource Rules
  • Products

    In-IDE

    Code Quality and Security in your IDE with SonarQube Ide

    IDE extension that lets you fix coding issues before they exist!

    Discover SonarQube for IDE

    SaaS

    Code Quality and Security in the cloud with SonarQube Cloud

    Setup is effortless and analysis is automatic for most languages

    Discover SonarQube Cloud

    Self-Hosted

    Code Quality and Security Self-Hosted with SonarQube Server

    Fast, accurate analysis; enterprise scalability

    Discover SonarQube Server
  • SecretsSecrets
  • ABAPABAP
  • AnsibleAnsible
  • ApexApex
  • AzureResourceManagerAzureResourceManager
  • CC
  • C#C#
  • C++C++
  • CloudFormationCloudFormation
  • COBOLCOBOL
  • CSSCSS
  • DartDart
  • DockerDocker
  • FlexFlex
  • GitHub ActionsGitHub Actions
  • GoGo
  • HTMLHTML
  • JavaJava
  • JavaScriptJavaScript
  • JSONJSON
  • JCLJCL
  • KotlinKotlin
  • KubernetesKubernetes
  • Objective CObjective C
  • PHPPHP
  • PL/IPL/I
  • PL/SQLPL/SQL
  • PythonPython
  • RPGRPG
  • RubyRuby
  • RustRust
  • ScalaScala
  • SwiftSwift
  • TerraformTerraform
  • TextText
  • TypeScriptTypeScript
  • T-SQLT-SQL
  • VB.NETVB.NET
  • VB6VB6
  • XMLXML
  • YAMLYAML
C

C static code analysis

Unique rules to find Bugs, Vulnerabilities, Security Hotspots, and Code Smells in your C code

  • All rules 315
  • Vulnerability13
  • Bug76
  • Security Hotspot19
  • Code Smell207

  • Quick Fix 19
Filtered: 18 rules found
brain-overload
    Impact
      Clean code attribute
        1. "#undef" should be used with caution

           Code Smell
        2. Object declarations should contain no more than 2 levels of pointer indirection

           Code Smell
        3. "goto" statement should not be used

           Code Smell
        4. Cognitive Complexity of functions should not be too high

           Code Smell
        5. "goto" statements should not be used to jump into blocks

           Code Smell
        6. Structures should not have too many fields

           Code Smell
        7. The ternary operator should not be used

           Code Smell
        8. Cyclomatic Complexity of functions should not be too high

           Code Smell
        9. "switch" statements should not have too many "case" clauses

           Code Smell
        10. Functions/methods should not have too many lines

           Code Smell
        11. Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply

           Code Smell
        12. "switch case" clauses should not have too many lines of code

           Code Smell
        13. Functions should not contain too many return statements

           Code Smell
        14. Magic numbers should not be used

           Code Smell
        15. Functions should not have too many parameters

           Code Smell
        16. Expressions should not be too complex

           Code Smell
        17. Files should not have too many lines of code

           Code Smell
        18. A function should have a single point of exit at the end of the function

           Code Smell

        Functions should not have too many parameters

        adaptability - focused
        maintainability
        Code Smell
        • brain-overload

        Why is this an issue?

        More Info

        Functions with a long parameter list are difficult to use because maintainers must figure out the role of each parameter and keep track of their position.

        void setCoordinates(int x1, int y1, int z1, int x2, int y2, int z2) { // Noncompliant
            // ...
        }
        

        The solution can be to:

        • Split the function into smaller ones
        // Each function does a part of what the original setCoordinates function was doing, so confusion risks are lower
        void setOrigin(int x, int y, int z) {
           // ...
        }
        
        void setSize(int width, int height, int depth) {
           // ...
        }
        
        • Find a better data structure for the parameters that group data in a way that makes sense for the specific application domain
        struct Point {  // In geometry, Point is a logical structure to group data
            int x;
            int y;
            int z;
        };
        
        void setCoordinates(Point p1, Point p2) {
            // ...
        }
        

        This rule raises an issue when a function has more parameters than the provided threshold.

          Available In:
        • SonarQube IdeCatch issues on the fly,
          in your IDE
        • SonarQube CloudDetect issues in your GitHub, Azure DevOps Services, Bitbucket Cloud, GitLab repositories
        • SonarQube ServerAnalyze code in your
          on-premise CI
          Developer Edition
          Available Since
          9.1

        © 2008-2025 SonarSource SA. All rights reserved.

        Privacy Policy | Cookie Policy | Terms of Use