module/Application/src/Most/Adapter/Wheelsize.php
Method getCars
has 40 lines of code (exceeds 25 allowed). Consider refactoring. Open
Open
public function getCars(Sql\Select $select, string $language): array
{
$wheel = $this->attributes['rear'];
$specService = $this->most->getSpecs();
Avoid unused parameters such as '$language'. Open
Open
public function getCars(Sql\Select $select, string $language): array
- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}
Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
syntax error, unexpected 'array' (T_ARRAY), expecting function (T_FUNCTION) or const (T_CONST)
Open
Open
protected array $attributes;
- Exclude checks
Avoid excessively long variable names like $tyreseriesValuesTable. Keep variable name length under 20. Open
Open
$tyreseriesValuesTable = $specService->getValueDataTable($tyreseries['type_id'])->getTable();
- 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++ ) {
}
}
}