Нет, это не плохая практика.На самом деле, это хорошая практика.
Когда аргументы функций должны передаваться по ссылке, это должно быть указано в определении функции.Вы можете видеть, что это так на странице справки PHP для shuffle
.
Передача по ссылке во время вызова (как, например, shuffle(&$arr)
) - плохая практика.Начиная с версии PHP 5.3, он устарел и выдает предупреждение.Это связано с тем, что это может вызвать всевозможные странные проблемы, если функция не ожидает своего аргумента в качестве ссылки.