app/Http/Controllers/API/DevicesController.php
Avoid using App\Http\Controllers\API\count() function in for loops. Open
Open
for ($i = 0; $i < count($devicesForCurrentUser); $i++) {
$appliance = [
'actions' => $actions,
'additionalApplianceDetails' => (object)[],
'applianceId' => $devicesForCurrentUser[$i]->id,
- Read upRead up
- Exclude checks
CountInLoopExpression
Since: 2.7.0
Using count/sizeof in loops expressions is considered bad practice and is a potential source of many bugs, especially when the loop manipulates an array, as count happens on each iteration.
Example
class Foo {
public function bar()
{
$array = array();
for ($i = 0; count($array); $i++) {
// ...
}
}
}
Source https://phpmd.org/rules/design.html#countinloopexpression
Line exceeds 120 characters; contains 154 characters Open
Open
public function __construct(IDeviceRepository $deviceRepository, IDeviceActionInfoBroker $deviceInformationBroker, MessagePublisher $messagePublisher)
- Exclude checks
Line exceeds 120 characters; contains 135 characters Open
Open
'header' => $this->createHeader($request, DeviceActions::actionToConfirmationName($action), 'Alexa.ConnectedHome.Control'),
- Exclude checks