tests/RollsOn/Traps/ShortRollOnPropertyTest.php
Missing class import via use statement (line '52', column '39'). Open
Open
$basePropertyNamespace = (new \ReflectionClass(BaseProperty::class))->getNamespaceName();- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
Avoid excessively long variable names like $basePropertyNamespace. Keep variable name length under 20. Open
Open
$basePropertyNamespace = (new \ReflectionClass(BaseProperty::class))->getNamespaceName();- Read upRead up
- Exclude checks
LongVariable
Since: 0.2
Detects when a field, formal or local variable is declared with a long name.
Example
class Something {
protected $reallyLongIntName = -3; // VIOLATION - Field
public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
$otherReallyLongName = -5; // VIOLATION - Local
for ($interestingIntIndex = 0; // VIOLATION - For
$interestingIntIndex < 10;
$interestingIntIndex++ ) {
}
}
}Source https://phpmd.org/rules/naming.html#longvariable
The method I_can_use_it is not named in camelCase. Open
Open
public function I_can_use_it()
{
/** @var ShortRollOnProperty $sutClass */
$sutClass = self::getSutClass();
/** @var ShortRollOnProperty $shortRollOnProperty */- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}