2.3.41. Suggestions

This ruleset focuses on possibly better syntax than the one currently used. Those may be code modernization, alternatives, more efficient solutions, or simply left over from older versions.

Total : 128 analysis

• While(List() = Each())

• Function Subscripting, Old Style

• ** For Exponent

• Too Many Children

• Empty With Expression

• list() May Omit Variables

• Unreachable Code

• Overwritten Exceptions

• Return With Parenthesis

• Strict Comparison With Booleans

• Logical Should Use Symbolic Operators

• Could Use self

• Preprocess Arrays

• Repeated print()

• Echo With Concat

• No Parenthesis For Language Construct

• Unused Interfaces

• Avoid Substr() One

• PHP7 Dirname

• preg_match_all() Flag

• Already Parents Interface

• Could Use __DIR__

• Should Use Coalesce

• Could Use Alias

• Drop Else After Return

• Unitialized Properties

• Should Use array_column()

• Randomly Sorted Arrays

• No Return Used

• Could Make A Function

• Use session_start() Options

• Mismatched Ternary Alternatives

• Isset Multiple Arguments

• Should Use Foreach

• Substring First

• Use List With Foreach

• Slice Arrays First

• Parent First

• Never Called Parameter

• Should Use array_filter()

• Reuse Existing Variable

• Should Use Math

• Could Use Compact

• Could Use array_fill_keys

• Use Recursive count()

• Too Many Parameters

• Should Preprocess Chr()

• Possible Increment

• Drop Substr Last Arg

• One If Is Sufficient

• Could Use array_unique

• Nonexistent Variable In compact()

• Should Use Operator

• Could Be Static Closure

• Use is_countable

• Detect Current Class

• Avoid Real

• Use json_decode() Options

• Closure Could Be A Callback

• Add Default Value

• Named Regex

• Could Use Try

• Use Basename Suffix

• Don’t Loop On Yield

• Should Have Destructor

• Directly Use File

• Isset() On The Whole Array

• Multiple Usage Of Same Trait

• array_key_exists() Speedup

• Should Deep Clone

• Multiple Unset()

• Implode One Arg

• Useless Default Argument

• No Need For get_class()

• Substr To Trim

• Complex Dynamic Names

• Use DateTimeImmutable Class

• Set Aside Code

• Use Array Functions

• Use The Case Value

• Should Use Url Query Functions

• Too Long A Block

• Static Global Variables Confusion

• Possible Alias Confusion

• Too Much Indented

• Avoid Compare Typed Boolean

• Abstract Away

• Large Try Block

• Cancel Common Method

• Useless Typehint

• Could Use Promoted Properties

• Use get_debug_type()

• Use str_contains()

• Unused Exception Variable

• Searching For Multiple Keys

• Long Preparation For Throw

• No Static Variable In A Method

• Declare Static Once

• Could Use Match

• Could Use Null-Safe Object Operator

• This Could Be Iterable

• Multiple Similar Calls

• Could Be Ternary

• Use File Append

• Could Use Existing Constant

• Could Use array_sum()

• Too Many Stringed Elseif

• Could Be Spaceship

• Throw Raw Exceptions

• Lowered Access Level

• Could Set Property Default

• Could Be Enumeration

• Magic Method Returntype Is Restricted

• Could Be Abstract Method

• Could Use Class Operator

• Could Use Namespace Magic Constant

• Json_encode() Without Exceptions

• Class Could Be Readonly

• Use str_ends_with()

• Use str_starts_with()

• Blind Variable Used Beyond Loop

• Could Be array_combine()

• Multiline Expressions

• Could Cast To Array

• Check After Null Safe Operator

• Could Use strcontains()

• Could Drop Variable

• Could Be Readonly Property

2.3.41.1. Specs

Short name	Suggestions
Available in	Entreprise Edition, Exakat Cloud
Reports	Diplomat, Ambassador