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
  • ShellShell
  • 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 674
  • Vulnerability13
  • Bug139
  • Security Hotspot19
  • Code Smell503

  • Quick Fix 91
Filtered: 24 rules found
since-c++17
    Impact
      Clean code attribute
        1. The optional init-statement in a control statements should only be used to declare variables

           Code Smell
        2. Assigning to an optional should directly target the optional

           Bug
        3. "try_emplace" should be used with "std::map" and "std::unordered_map"

           Code Smell
        4. "auto" should be used for non-type template parameter

           Code Smell
        5. Use "std::variant" instead of unions with non-trivial types.

           Code Smell
        6. "std::optional" member function "value_or" should be used

           Code Smell
        7. "std::byte" should be used when you need byte-oriented memory access

           Code Smell
        8. Inline variables should be used to declare global variables in header files

           Code Smell
        9. "if constexpr" should be preferred to overloading for metaprogramming

           Code Smell
        10. "[*this]" should be used to capture the current object by copy

           Code Smell
        11. "std::uncaught_exception" should not be used

           Code Smell
        12. "static_assert" with no message should be used over "static_assert" with empty or redundant message

           Code Smell
        13. Redundant class template arguments should not be used

           Code Smell
        14. "std::filesystem::path" should be used to represent a file path

           Code Smell
        15. "std::string_view" should be used to pass a read-only string to a function

           Code Smell
        16. Fold expressions should be used instead of recursive template instantiations

           Code Smell
        17. [[nodiscard]] should be used when the return value of a function should not be ignored

           Code Smell
        18. "as_const" should be used to make a value constant

           Code Smell
        19. Structured binding should be used

           Code Smell
        20. "if" and "switch" initializer should be used to reduce scope of variables

           Code Smell
        21. "std::visit" should be used to switch on the type of the current value in a "std::variant"

           Code Smell
        22. "std::scoped_lock" should be created with constructor arguments

           Bug
        23. "std::scoped_lock" should be used instead of "std::lock_guard"

           Code Smell
        24. Concise syntax should be used for concatenatable namespaces

           Code Smell

        "std::visit" should be used to switch on the type of the current value in a "std::variant"

        adaptability - modular
        maintainability
        Code Smell
        • since-c++17
        • clumsy

        Why is this an issue?

        std::variant is a type-safe union that can hold values of a type out of a fixed list of types.

        Depending on the current alternative inside a variant, it is common to execute dedicated code. There are basically two ways to achieve that:

        • Writing code that checks the current alternative, then getting it and running specific code
        • Letting std::visit perform the check and select the code to run by using overload resolution with the different alternatives

        The second option is usually preferable:

        • It requires less boilerplate code.
        • It is easy to handle multiple similar alternatives together if desired.
        • It is usually more robust: if a new alternative is added to the variant, but the visitor does not support it, it will not compile.

        This rule raises an issue when variant::index is called, or when variant::holds_alternative or variant::get_if is used in a series of if - else if (calling one of these functions in isolation can be an acceptable lightweight alternative to std::visit in some cases).

        Note: When defining the visitor of a variant, it can be nicer to use a series of lambdas by making use of the overloaded pattern

        Noncompliant code example

        using Variant = std::variant<int, float, string>;
        void printType1(Variant const &v) {
            switch(v.index()) { // Noncompliant
                case 0: cout << "int " <<get<int>(v) << "\n"; break;
                case 1: cout << "float " <<get<float>(v) << "\n"; break;
                case 2: cout << "string " <<get<string>(v) << "\n";break;
            }
        }
        void printType2(Variant const &v) {
            if(auto p = get_if<int>(&v)) { // Noncompliant
                cout << "int " << *p << "\n";
            } else if (auto p = get_if<float>(&v)) {
                cout << "float " << *p << "\n";
            } else if (auto p = get_if<string>(&v)) {
                cout << "string " << *p << "\n";
            }
        }
        

        Compliant solution

        using Variant = std::variant<int, float, string>;
        
        struct VariantPrinter {
            void operator() (int i) { cout << "int " << i << "\n"; }
            void operator() (float f) { cout << "float " << f << "\n"; }
            void operator() (std::string const &s) { cout << "string " << s << "\n"; }
        };
        
        void printType3(Variant const &v) {
            std::visit(VariantPrinter{}, v);
        }
        
        // Same principle, but using the overloaded pattern
        void printType4(Variant const &v) {
            std::visit(overloaded{
                [](int i){cout << "int " << i << "\n";},
                [](float f){cout << "float " << f << "\n";},
                [](std::string const &s){cout << "string " << s << "\n";}
            }, v);
        }
        
          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