No se puede usar el valor de retorno del método en el contexto de escritura

Creo que el siguiente fragmento de código debería funcionar, pero no funciona (Editado: ahora funciona en PHP 5.5+) :

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

Donde getError()es simplemente:

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

Sin embargo, termino con este error:

no se puede usar el valor de retorno del método en el contexto de escritura

¿Qué significa esto? ¿No es esto solo una lectura?

empty() necesita acceder al valor por referencia (para verificar si esa referencia apunta a algo que existe), y PHP antes de 5.5 no admitía referencias a valores temporales devueltos por funciones.

Sin embargo, el verdadero problema que tiene es que lo usa empty()en absoluto, creyendo erróneamente que el valor "vacío" es diferente de "falso".

Vacío es solo un alias para !isset($thing) || !$thing. Cuando lo que está comprobando siempre existe (en PHP los resultados de las llamadas a funciones siempre existen), la empty()función no es más que un operador de negación .

PHP no tiene el concepto de vacío . Los valores que evalúan a falso están vacíos, los valores que evalúan a verdadero no están vacíos. Es lo mismo. Este código:

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

y esto:

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

siempre tiene el mismo resultado, en todos los casos, para todos los tipos de datos (porque $xestá definido empty()es redundante).

El valor de retorno del método siempre existe (incluso si no tiene una returndeclaración, el valor de retorno existe y contiene null). Por lo tanto:

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

es lógicamente equivalente a:

if ($r->getError())

Nota: Esta es una respuesta muy votada con una alta visibilidad, pero tenga en cuenta que promueve prácticas de codificación innecesarias. Ver la respuesta de @ Kornel para la forma correcta.

Nota # 2: Respaldo las sugerencias para usar la respuesta de @ Kornel . Cuando escribí esta respuesta hace tres años, solo tenía la intención de explicar la naturaleza del error, no necesariamente respaldar la alternativa. No se recomienda el siguiente fragmento de código.

Es una limitación de empty () en versiones de PHP inferiores a 5.5.

Nota: empty () solo verifica las variables, ya que cualquier otra cosa dará como resultado un error de análisis. En otras palabras, lo siguiente no funcionará: vacío (recortar ($ nombre)).

Tendrías que cambiar a esto

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

De acuerdo con los documentos PHP :

empty () solo verifica las variables, ya que cualquier otra cosa dará como resultado un error de análisis

No se puede usar empty()directamente en el valor de retorno de una función. En su lugar, establezca el retorno desde getError()a una variable y ejecútelo empty()en la variable.

Por lo general, creo una función global llamada is_empty () solo para solucionar este problema

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

Luego, en cualquier lugar donde normalmente hubiera usado empty (), solo uso is_empty ()

Como señalaron otros, es una limitación (extraña) de empty ().

Para la mayoría de los purproses, hacer esto es igual a llamar a empty, pero esto funciona:

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

El problema es este, desea saber si el error no está vacío.

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

Agregar un método isErrorSet () resolverá el problema.

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

Ahora esto funcionará bien con este código sin previo aviso.

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

La forma alternativa de verificar si una matriz está vacía podría ser:

count($array)>0

Funciona para mí sin ese error