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
PHP

PHP static code analysis

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

  • All rules 273
  • Vulnerability42
  • Bug51
  • Security Hotspot34
  • Code Smell146
Filtered: 10 rules found
confusing
    Impact
      Clean code attribute
        1. Methods should not have identical implementations

           Code Smell
        2. Array or Countable object count comparisons should make sense

           Bug
        3. A conditionally executed single line should be denoted by indentation

           Code Smell
        4. Functions should use "return" consistently

           Code Smell
        5. Ternary operators should not be nested

           Code Smell
        6. Return values from functions without side effects should not be ignored

           Bug
        7. TestCases should contain tests

           Code Smell
        8. Loops with at most one iteration should be refactored

           Bug
        9. A reason should be provided when skipping a test

           Code Smell
        10. Redundant pairs of parentheses should be removed

           Code Smell

        Array or Countable object count comparisons should make sense

        intentionality - clear
        reliability
        Bug
        • confusing

        Why is this an issue?

        The count of elements from an array or Countable object is always greater than or equal to zero. Testing it doesn’t make sense, since the result is always true.

        if (count($arr) >= 0) {...} // Noncompliant: always true
        
        $result = count($arr) >= 0; // Noncompliant always true
        

        Similarly testing that it is less than zero will always return false.

        if (count($arr) < 0) {...} // Noncompliant: always false
        

        Fix the code to properly check for emptiness if it was the intent, or remove the redundant code to keep the current behavior.

          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