roistat/php-code-conventions

Противоречие примеров

Lui22 opened this issue · 1 comments

Первый пример правила «Запрещено использовать результат операции присваивания»:

Хорошо:

$bar = strlen($someVar);
$foo = $bar;

противоречит правилу «Нельзя нескольким переменным присваивать одно и то же значение»:

Плохо:

function loadUsers(array $ids) {
    $usersIds = $ids;
    // ...
}

Тут в обоих кейсах примеры надо сделать лучше.
Например, во втором кейсе с одним и тем же значением, пример вполне валидный. Так как мы не меняем переданные на вход в метод параметры, то мы должны как раз положить значение в другую переменную, там менять как нам надо и дальше использовать.
А в первом кейсе в целом этот пример можно просто удалить, чтобы не смущал.

Можете сделать PR.