Nie można użyć wartości zwracanej przez metodę w kontekście zapisu

Sądzę, że następujący fragment kodu powinien działać, ale nie działa (edytowano: Teraz działa w PHP 5.5+) :

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

Gdzie getError()jest po prostu:

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

Jednak kończę na tym błędzie:

nie można użyć wartości zwracanej przez metodę w kontekście zapisu

Co to znaczy? Czy to nie tylko czytanie?

empty() musi uzyskać dostęp do wartości przez referencję (w celu sprawdzenia, czy ta referencja wskazuje na coś, co istnieje), a PHP przed 5.5 nie obsługiwał referencji do wartości tymczasowych zwracanych z funkcji.

Jednak prawdziwym problemem jest to, że empty()w ogóle go używasz , błędnie wierząc, że „pusta” wartość różni się od „fałszywej”.

Pusty to tylko alias !isset($thing) || !$thing. Kiedy rzecz, którą sprawdzasz, zawsze istnieje (w PHP zawsze istnieją wyniki wywołań funkcji), empty()funkcja jest tylko operatorem negacji .

PHP nie ma pojęcia pustki . Wartości, które mają wartość false, są puste, wartości, które mają wartość true, są niepuste. To jest to samo. Ten kod:

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

i to:

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

ma zawsze ten sam wynik we wszystkich przypadkach dla wszystkich typów danych (ponieważ $xjest zdefiniowany empty()jest nadmiarowy).

Zwracana wartość z metody zawsze istnieje (nawet jeśli nie masz returninstrukcji, zwracana wartość istnieje i zawiera null). W związku z tym:

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

jest logicznie równoważne z:

if ($r->getError())

Uwaga: Jest to bardzo wysoko głosowana odpowiedź z dużą widocznością, ale należy pamiętać, że promuje złe, niepotrzebne praktyki kodowania! Prawidłowy sposób zobacz odpowiedź @ Kornela .

Uwaga 2: Popieram sugestie dotyczące użycia odpowiedzi @ Kornel . Kiedy napisałem tę odpowiedź trzy lata temu, chciałem jedynie wyjaśnić naturę błędu, niekoniecznie popierając alternatywę. Poniższy fragment kodu nie jest zalecany.

Jest to ograniczenie funkcji empty () w wersjach PHP poniżej 5.5.

Uwaga: empty () sprawdza tylko zmienne, ponieważ wszystko inne spowoduje błąd analizy. Innymi słowy, następujące funkcje nie będą działać: puste (przycinanie (nazwa $)).

Musisz to zmienić

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

Według dokumentów PHP :

empty () sprawdza tylko zmienne, ponieważ wszystko inne spowoduje błąd analizy

Nie można użyć empty()bezpośrednio na wartości zwracanej przez funkcję. Zamiast tego ustaw return from getError()na zmienną i uruchom empty()na zmiennej.

Zwykle tworzę funkcję globalną o nazwie is_empty (), aby obejść ten problem

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

Wtedy gdziekolwiek normalnie użyłbym pustego () Po prostu używam is_empty ()

Jak zauważyli inni, jest to (dziwne) ograniczenie pustych ().

W przypadku większości purproses zrobienie tego jest równoznaczne z wywołaniem pustego, ale działa to:

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

Problem polega na tym, że chcesz wiedzieć, czy błąd nie jest pusty.

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

Dodanie metody isErrorSet () rozwiąże problem.

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

Teraz będzie działać dobrze z tym kodem bez powiadomienia.

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

Alternatywnym sposobem sprawdzenia, czy tablica jest pusta, może być:

count($array)>0

Działa dla mnie bez tego błędu