Невозможно использовать возвращаемое значение метода в контексте записи

Я думаю, что следующий фрагмент кода должен работать, но это не так (отредактировано: теперь работает в PHP 5.5+) :

if (!empty($r->getError()))

Где getError()просто:

public function getError()
{
    return $this->error;
}

Все же я в конечном итоге с этой ошибкой:

нельзя использовать возвращаемое значение метода в контексте записи

Что это значит? Разве это не просто чтение?

empty() должен получить доступ к значению по ссылке (чтобы проверить, указывает ли эта ссылка на что-то существующее), а PHP до 5.5 не поддерживал ссылки на временные значения, возвращаемые функциями.

Однако настоящая проблема, с которой вы столкнулись, заключается в том, что вы используете ее empty()вообще, ошибочно полагая, что «пустое» значение отличается от «ложного».

Пустой это просто псевдоним для !isset($thing) || !$thing. Когда проверяемая вещь всегда существует (в PHP всегда присутствуют результаты вызовов функций), empty()функция является ничем иным, как оператором отрицания .

В PHP нет понятия пустоты . Значения, которые оцениваются как ложные, являются пустыми, значения, которые оцениваются как истинные, не являются пустыми. Это то же самое. Этот код:

$x = something();
if (empty($x)) …

и это:

$x = something();
if (!$x) …

во всех случаях всегда имеет одинаковый результат для всех типов данных (поскольку $xопределяется empty()как избыточный).

Возвращаемое значение из метода всегда существует (даже если у вас нет returnоператора, возвращаемое значение существует и содержит null). Следовательно:

if (!empty($r->getError()))

логически эквивалентно:

if ($r->getError())

Примечание. Это ответ с очень высоким рейтингом и высокой видимостью, но имейте в виду, что он способствует плохим, ненужным практикам кодирования! Смотрите ответ @ Kornel для правильного пути.

Примечание № 2: Я поддерживаю предложения использовать ответ @ Kornel . Когда я написал этот ответ три года назад, я просто хотел объяснить природу ошибки, а не обязательно одобрить альтернативу. Фрагмент кода ниже не рекомендуется.

Это ограничение empty () в версиях PHP ниже 5.5.

Примечание: empty () проверяет только переменные, так как все остальное приведет к ошибке разбора. Другими словами, следующее не будет работать: пусто (trim ($ name)).

Вы должны изменить это

// Not recommended, just illustrates the issue
$err = $r->getError();
if (!empty($err))

Согласно документации PHP :

empty () проверяет только переменные, так как все остальное приведет к ошибке разбора

Вы не можете empty()напрямую использовать возвращаемое значение функции. Вместо этого установите return из getError()переменной и запустите empty()переменную.

Я обычно создаю глобальную функцию is_empty (), чтобы обойти эту проблему

function is_empty($var)
{ 
 return empty($var);
}

Тогда везде, где я бы обычно использовал empty (), я просто использую is_empty ()

Как отмечают другие, это (странное) ограничение empty ().

Для большинства целей сделать это равносильно вызову empty, но это работает:

if ($r->getError() != '')

Проблема в том, что вы хотите знать, если ошибка не пуста.

public function getError() {
    return $this->error;
}

Добавление метода isErrorSet () решит проблему.

public function isErrorSet() {
    if (isset($this->error) && !empty($this->error)) {
        return true;
    } else {
        return false;
    }
}

Теперь это будет хорошо работать с этим кодом без предварительного уведомления.

if (!($x->isErrorSet())) {
    echo $x->getError();
}

Альтернативный способ проверить, является ли массив пустым:

count($array)>0

У меня работает без этой ошибки