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
  • GroovyGroovy
  • 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
Groovy

Groovy static code analysis

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

  • All rules 21
  • Bug5
  • Code Smell16
 
Tags
    Impact
      Clean code attribute
        1. Range methods should be used appropriately to avoid unexpected behavior

           Code Smell
        2. Property names should use camelCase

           Code Smell
        3. Simple "@Grab" annotations should use shorthand notation

           Code Smell
        4. GString expressions should not be used as map keys

           Bug
        5. Empty strings should not be used for type conversion

           Code Smell
        6. Static imports should appear before regular imports

           Code Smell
        7. Method names should not use reserved keywords

           Code Smell
        8. Do not use "sort(false)" to avoid confusion

           Code Smell
        9. Semicolons should be omitted in Groovy

           Code Smell
        10. Control structures should use braces

           Code Smell
        11. Duplicate import statements should be removed

           Code Smell
        12. Star imports should be replaced with explicit imports

           Code Smell
        13. AST transformation classes should be annotated with "@CompileStatic"

           Code Smell
        14. "@TimedInterrupt" should not be used on static methods

           Bug
        15. File operations should specify charset encoding

           Code Smell
        16. Test methods should contain assertions

           Code Smell
        17. Method names should follow camelCase naming conventions

           Code Smell
        18. Null checks should use correct logical operators

           Bug
        19. Classes with a "clone()" method should implement "Cloneable"

           Bug
        20. "wait()" calls should be inside "while" loops

           Bug
        21. Groovy parser failure

           Code Smell

        Method names should not use reserved keywords

        intentionality - clear
        maintainability
        Code Smell
        • confusing
        • naming

        This rule raises an issue when a method is defined with a reserved keyword name surrounded by quotes.

        Why is this an issue?

        How can I fix it?

        More Info

        While Groovy allows defining methods with reserved keyword names by surrounding them with quotes, this practice creates several problems:

        • Reduced readability: Quoted method names look unusual and can confuse developers who are not familiar with this Groovy feature.
        • Special calling syntax: Methods with quoted keyword names require qualified calls using the this. prefix, making the calling code more verbose and inconsistent.
        • Maintenance burden: Code becomes harder to understand and maintain when method names don’t clearly express their purpose.
        • Inconsistent style: Mixing quoted and unquoted method names creates inconsistent code style within a project.

        The Groovy documentation itself states that "using such names might be confusing and is often best to avoid." This feature was primarily intended for specific Java integration scenarios and Domain Specific Language (DSL) use cases, not for general application development.

        What is the potential impact?

        Using reserved keywords as method names can lead to:

        • Confusion for team members: Developers may not understand why certain methods require special calling syntax
        • Increased cognitive load: Reading and understanding code becomes more difficult
        • Maintenance issues: Future developers may struggle to work with such code
        • Inconsistent codebase: Mixed naming conventions reduce overall code quality
          Available In:
        • SonarQube CloudDetect issues in your GitHub, Azure DevOps Services, Bitbucket Cloud, GitLab repositories

        © 2026 SonarSource Sàrl. All rights reserved.

        Privacy Policy | Cookie Policy | Terms of Use