Impossible d'utiliser la valeur de retour de la méthode dans le contexte d'écriture

Je pense que le code suivant devrait fonctionner, mais il ne fonctionne pas (Modifié: fonctionne maintenant en PHP 5.5+) :

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

Où getError()est tout simplement:

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

Pourtant, je me retrouve avec cette erreur:

ne peut pas utiliser la valeur de retour de la méthode dans le contexte d'écriture

Qu'est-ce que ça veut dire? N'est-ce pas juste une lecture?

empty() doit accéder à la valeur par référence (afin de vérifier si cette référence pointe vers quelque chose qui existe), et PHP avant 5.5 ne supportait pas les références aux valeurs temporaires renvoyées par les fonctions.

Cependant, le vrai problème que vous avez est que vous l'utilisez empty()du tout, croyant à tort que la valeur "vide" est différente de "fausse".

Vide n'est qu'un alias pour !isset($thing) || !$thing. Lorsque la chose que vous vérifiez existe toujours (en PHP, les résultats des appels de fonction existent toujours), la empty()fonction n'est rien d'autre qu'un opérateur de négation .

PHP n'a pas de concept de vide . Les valeurs évaluées à false sont vides, les valeurs évaluées à true ne sont pas vides. C'est la même chose. Ce code:

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

et ça:

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

a toujours le même résultat, dans tous les cas, pour tous les types de données (car $xest défini empty()est redondant).

La valeur de retour de la méthode existe toujours (même si vous n'avez pas d' returninstruction, la valeur de retour existe et contient null). Donc:

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

est logiquement équivalent à:

if ($r->getError())

Remarque: Il s'agit d'une réponse très votée avec une grande visibilité, mais veuillez noter qu'elle favorise de mauvaises pratiques de codage inutiles! Voir la réponse de @ Kornel pour la bonne façon.

Note # 2: J'approuve les suggestions pour utiliser la réponse de @ Kornel . Lorsque j'ai écrit cette réponse il y a trois ans, je voulais simplement expliquer la nature de l'erreur, pas nécessairement approuver l'alternative. L'extrait de code ci-dessous n'est pas recommandé.

C'est une limitation de empty () dans les versions PHP inférieures à 5.5.

Remarque: empty () vérifie uniquement les variables car toute autre chose entraînera une erreur d'analyse. En d'autres termes, ce qui suit ne fonctionnera pas: empty (trim ($ name)).

Il faudrait changer pour ça

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

Selon les documents PHP :

empty () vérifie uniquement les variables car toute autre chose entraînera une erreur d'analyse

Vous ne pouvez pas utiliser empty()directement la valeur de retour d'une fonction. Au lieu de cela, définissez le retour de getError()sur une variable et exécutez- empty()la.

Je crée généralement une fonction globale appelée is_empty () juste pour contourner ce problème

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

Ensuite, n'importe où j'aurais normalement utilisé empty () J'utilise simplement is_empty ()

Comme souligné par d'autres, c'est une limitation (étrange) de empty ().

Pour la plupart des purproses, faire cela équivaut à appeler vide, mais cela fonctionne:

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

Le problème est le suivant, vous voulez savoir si l'erreur n'est pas vide.

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

L'ajout d'une méthode isErrorSet () résoudra le problème.

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

Maintenant, cela fonctionnera bien avec ce code sans préavis.

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

L'autre façon de vérifier si un tableau est vide pourrait être:

count($array)>0

Cela fonctionne pour moi sans cette erreur