Строгие стандарты: только переменные должны передаваться по ссылке

$el = array_shift($instance->find(..))

Вышеприведенный код каким-то образом сообщает о строгом стандартном предупреждении, но это не будет:

function get_arr(){
    return array(1,2);
}
$el = array_shift(get_arr());

Итак, когда он все равно сообщит о предупреждении?

Ответ 1

Рассмотрим следующий код:

error_reporting(E_STRICT);
class test {
    function test_arr(&$a) {
        var_dump($a);   
    }
    function get_arr() {
        return array(1,2);  
    }
}

$t= new test;
$t->test_arr($t->get_arr());

Это сгенерирует следующий вывод:

Strict Standards: Only variables should be passed by reference in 'test.php' on line 14
array(2) {
  [0]=>
  int(1)
  [1]=>
  int(2)
}

Причина? Метод test::get_arr() не является переменной, и в строгом режиме это вызовет предупреждение. Такое поведение крайне неинтуитивно, поскольку метод get_arr() возвращает значение массива.

Чтобы обойти эту ошибку в строгом режиме, либо измените сигнатуру метода, чтобы он не использовал ссылку:

function test_arr($a) {
    var_dump($a);  
}

Поскольку вы не можете изменить сигнатуру array_shift вы также можете использовать промежуточную переменную:

$inter= get_arr();
$el= array_shift($inter);

Ответ 2

$instance->find() возвращает ссылку на переменную.

Вы получаете отчет, когда пытаетесь использовать эту ссылку в качестве аргумента функции, не сохраняя ее сначала в переменной.

Это помогает предотвратить утечки памяти и, вероятно, станет ошибкой в следующих версиях PHP.

Ваш второй код выдаст ошибку, если он будет выглядеть так (обратите внимание на сигнатуру функции & в):

function &get_arr(){
    return array(1,2);
}
$el = array_shift(get_arr());

Так что быстрое (и не очень хорошее) исправление будет:

$el = array_shift($tmp = $instance->find(..));

Обычно вы сначала делаете присвоение временной переменной и отправляете переменную в качестве аргумента.

Ответ 3

Причиной ошибки является использование внутренней функции Структуры данных программирования PHP, array_shift() [php.net/end].

Функция принимает массив в качестве параметра. Хотя в прототипе функции array_shift() в "Руководстве" указан амперсанд ", в расширенном определении этой функции нет предупреждающей документации, а также нет очевидного объяснения того, что параметр фактически передается по ссылке.

Возможно, это/понял/. Я не понял, однако, поэтому мне было трудно определить причину ошибки.

Воспроизвести код:

function get_arr()
{
 return array(1,2);
}
$array = get_arr();
$el = array_shift($array);

Ответ 4

Второй фрагмент тоже не работает и вот почему. array_shift - это функция-модификатор, которая изменяет свой аргумент, поэтому ожидает, что ее параметр является ссылкой, и вы не можете ссылаться на то, что не является переменной. См. Объяснения Расмуса здесь: Строгие стандарты: только переменные должны передаваться по ссылке

Ответ 5

Этот код:

$monthly_index = array_shift(unpack('H*', date('m/Y')));

Нужно изменить на:

$date_time = date('m/Y');
$unpack = unpack('H*', $date_time);
array_shift($unpack);

Ответ 6

Что ж, в очевидных случаях, подобных этому, вы всегда можете сказать PHP подавлять сообщения, используя "@" перед функцией.

$monthly_index = @array_shift(unpack('H*', date('m/Y')));

Возможно, это не лучшая практика программирования для подавления всех ошибок таким способом, но в некоторых случаях (например, в этом) это удобно и приемлемо.

В результате я уверен, что ваш друг SysAdmin будет доволен менее загрязненной error.log