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
Java

Java static code analysis

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

  • All rules 733
  • Vulnerability60
  • Bug175
  • Security Hotspot40
  • Code Smell458

  • Quick Fix 65
Filtered: 17 rules found
design
    Impact
      Clean code attribute
        1. Circular dependencies between classes across packages should be resolved

           Code Smell
        2. Circular dependencies between classes in the same package should be resolved

           Code Smell
        3. The Singleton design pattern should be used with care

           Code Smell
        4. Methods should not perform too many tasks (aka Brain method)

           Code Smell
        5. Classes should not depend on an excessive number of classes (aka Monster Class)

           Code Smell
        6. Constructors of an "abstract" class should not be declared "public"

           Code Smell
        7. Tests should use fixed data instead of randomized data

           Code Smell
        8. Tests should be stable

           Code Smell
        9. Spring components should use constructor injection

           Code Smell
        10. Constructor injection should be used instead of field injection

           Bug
        11. "ThreadGroup" should not be used

           Code Smell
        12. Classes without "public" constructors should be "final"

           Code Smell
        13. Public methods should not contain selector arguments

           Code Smell
        14. Double-checked locking should not be used

           Bug
        15. Two branches in a conditional structure should not have exactly the same implementation

           Code Smell
        16. String literals should not be duplicated

           Code Smell
        17. Utility classes should not have public constructors

           Code Smell

        Tests should use fixed data instead of randomized data

        intentionality - complete
        maintainability
        Code Smell
        • tests
        • design
        • confusing

        Why is this an issue?

        How can I fix it?

        More Info

        Randomness in test code, whether introduced intentionally to cover multiple scenarios or unintentionally through non-deterministic library functions, undermines the principles of effective testing. In most cases, randomness leads to problems, resulting in code that is unreliable and difficult to debug. Consequently, deterministic and reproducible tests are preferred, primarily for the following reasons:

        • When a test fails, the ability to reproduce the conditions that led to the failure is crucial for effective debugging. Randomness can make it difficult or even impossible to pinpoint the root cause, as subsequent runs may not exhibit the same failure.
        • Being able to replay a scenario allows us to easily compare logs between different test runs.
        • Determinism gives us confidence that a bug is fixed when it no longer appears in tests. If they behave randomly, a passing test after a fix might be coincidental due to a specific random input, rather than a genuine resolution of the underlying problem.
        • Flaky tests, which pass or fail intermittently without any code changes, are a significant problem for CI pipelines (continuous integration). They erode confidence in the CI system, lead to unnecessary investigations and reruns, and ultimately slow down the development and release process. A stable CI pipeline relies on deterministic test outcomes.

        This rule raises an issue when new Random() or UUID.randomUUID() are called in test code.

          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 Community BuildAnalyze code in your
          on-premise CI
          Available Since
          9.1
        • 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