Discussion :

[Sylvaner]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Merci d'avance à vous

[Invité]

Bonjour,

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

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

[Sylvaner]

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

[Geoffrey74]

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

[rawsrc]

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.

[Celira]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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
?>