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

Question

Je pensais que le code suivant devrait fonctionner, mais ce n'est pas le cas. (Modifié : fonctionne maintenant en PHP 5.5+) :

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

Donde getError() est 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 cela signifie ? Ce n'est pas juste une lecture ?

Answer 1

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 retournées par les fonctions.

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

Empty est juste 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 fonction empty() La fonction est rien d'autre qu'un opérateur de négation .

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

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

et ceci :

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

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

La valeur de retour de la méthode existe toujours (même si vous ne disposez pas de return la valeur de retour existe et contient null ). Par conséquent :

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

est logiquement équivalent à :

if ($r->getError())

Answer 2

Note : Il s'agit d'une réponse très votée avec une grande visibilité, mais veuillez noter qu'elle encourage des pratiques de codage mauvaises et inutiles ! Voir Réponse de @Kornel pour le bon chemin.

Note #2 : J'approuve les suggestions d'utiliser 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, sans nécessairement approuver l'alternative. L'extrait de code ci-dessous n'est pas recommandé.

---

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

Note : empty() vérifie seulement les variables comme toute autre méthode entraînera une erreur d'analyse erreur. En d'autres termes, la commande suivante ne fonctionnera pas : empty(trim($name)).

Vous devriez changer pour ceci

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

Answer 3

Avant PHP 5.5, l'option la documentation PHP avait l'habitude de dire :

empty() ne vérifie que les variables car toute autre vérification entraînerait une erreur d'analyse.

En PHP < 5.5, vous ne pouviez pas utiliser la fonction empty() directement sur la valeur de retour d'une fonction. Au lieu de cela, vous pouvez affecter le retour de getError() à une variable et exécutez empty() sur la variable.

En PHP >= 5.5, cela n'est plus nécessaire.

Answer 4

J'ai l'habitude de créer une fonction globale appelée is_empty() juste pour contourner ce problème.

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

Ensuite, partout où j'aurais normalement utilisé empty(), j'utilise is_empty().

Answer 5

Comme d'autres l'ont souligné, c'est une limitation (bizarre) de empty().

Pour la plupart des bourses, faire cela revient à dire qu'elles sont vides, mais cela fonctionne :

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