Impossibile utilizzare il valore restituito dal metodo nel contesto di scrittura

Penso che il seguente pezzo di codice dovrebbe funzionare, ma non funziona (modificato: ora funziona in PHP 5.5+) :

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

Dov'è getError()semplicemente:

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

Eppure finisco con questo errore:

impossibile utilizzare il valore restituito dal metodo nel contesto di scrittura

Cosa significa questo? Non è solo una lettura?

empty() deve accedere al valore per riferimento (per verificare se quel riferimento punta a qualcosa che esiste) e PHP prima della 5.5 non supportava riferimenti a valori temporanei restituiti da funzioni.

Tuttavia, il vero problema che hai è che lo usi empty()affatto, credendo erroneamente che il valore "vuoto" sia diverso da "falso".

Vuoto è solo un alias per !isset($thing) || !$thing. Quando la cosa che stai controllando esiste sempre (nei risultati PHP delle chiamate di funzione esistono sempre), la empty()funzione non è altro che un operatore di negazione .

PHP non ha il concetto di vuoto . I valori che valutano false sono vuoti, i valori che valutano true sono non vuoti. È la stessa cosa. Questo codice:

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

e questo:

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

ha sempre lo stesso risultato, in tutti i casi, per tutti i tipi di dati (perché $xdefinito empty()è ridondante).

Il valore restituito dal metodo esiste sempre (anche se non si dispone di returnun'istruzione, il valore restituito esiste e contiene null). Perciò:

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

è logicamente equivalente a:

if ($r->getError())

Nota: questa è una risposta molto votata con un'alta visibilità, ma tieni presente che promuove pratiche di codifica errate e inutili! Vedi la risposta di @ Kornel per il modo corretto.

Nota n. 2: sottoscrivo i suggerimenti per utilizzare la risposta di @ Kornel . Quando ho scritto questa risposta tre anni fa, intendevo semplicemente spiegare la natura dell'errore, non necessariamente approvare l'alternativa. Lo snippet di codice in basso non è raccomandato.

È una limitazione di empty () nelle versioni PHP sotto la 5.5.

Nota: empty () controlla solo le variabili poiché qualsiasi altra cosa provocherà un errore di analisi. In altre parole, non funzionerà: vuoto (trim ($ name)).

Dovresti cambiare in questo

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

Secondo i documenti PHP :

empty () controlla solo le variabili poiché qualsiasi altra cosa comporterà un errore di analisi

Non è possibile utilizzare empty()direttamente sul valore restituito di una funzione. Invece, imposta il ritorno da getError()su una variabile ed eseguilo empty()sulla variabile.

Di solito creo una funzione globale chiamata is_empty () solo per aggirare questo problema

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

Quindi dovunque normalmente avrei usato empty () Uso solo is_empty ()

Come sottolineato da altri, è una (strana) limitazione di empty ().

Per la maggior parte delle ipotesi, fare questo equivale a chiamare vuoto, ma funziona:

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

Il problema è questo, vuoi sapere se l'errore non è vuoto.

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

L'aggiunta di un metodo isErrorSet () risolverà il problema.

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

Ora funzionerà bene con questo codice senza preavviso.

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

Il modo alternativo per verificare se un array è vuoto potrebbe essere:

count($array)>0

Funziona per me senza quell'errore