Best practices isset

Bonjour,

Je me questionne sur un test et internet n'a malheureusement pas répondu à mon questionnement (ou j'ai mal cherché).
Mais il a bien répondu à la question array_key_exists :)

Je veux "optimiser" un test qui fait actuellement quelque chose comme ça :

Code:

---

if (!isset($anArray) || !isset($anArray['test']))

---

$anArray est bien entendu censé être un tableau.
Donc je me dis que ceci suffit :

Code:

---

if (!isset($anArray['test']))

---

Mais est-ce bien de tester directement la clé sans tester l'existence du tableau ?

Merci d'avance à vous

Bonjour,

Aucune notion de bien ou de mal ici, mais de logique.

Si la condition fonctionne comme voulu, c'est bon.

C'est surtout tester une clé dans un tableau qui n'existe pas qui pourrait potentiellement poser problème.

Attention, isset vérifie si la clé existe mais si elle est est vide, tu as la fonction empty() qui vérifie non seulement si la clé existe, mais aussi si elle est vide, deux fonctions en une :)

Citation:

---

Envoyé par Sylvaner

C'est surtout tester une clé dans un tableau qui n'existe pas qui pourrait potentiellement poser problème.

---

Du tout.
isset($anArray['test']) s'assure que le tableau $anArray est bien défini et que la clé 'test' est également bien définie. Bref les deux vérifications en une seule passe.
Si une des vérif échoue c'est false.

Citation:

---

Envoyé par Sylvaner

Mais est-ce bien de tester directement la clé sans tester l'existence du tableau ?

---

Comme l'indique rawsrc, isset teste à la fois l'existence du tableau et de la clé, donc tu n'as pas besoin de tester le tableau d'abord.

Remarque : isset va retourner faux si la clé existe mais que la valeur est null (par opposition à array_key_exists):

Code:

---

1 2 3 4 5 6 7

<?php $search_array = array('first' => null, 'second' => 4);   isset($search_array['first']); // retourne false   array_key_exists('first', $search_array); // retourne true ?>

---