Discussion :

[tarharnak]

Bonjour,

Je voulais savoir quelle est la bonne syntaxe :

Code :

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

1
2
3
4
5
 
if(isset($var) == 'valeur')
{
    #code
}

ou

Code :

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

1
2
3
4
if(isset($var) AND $var == 'valeur')
{
    #code
}

Car les deux semble fonctionner

Merci

[mathieu]

la fonction isset retourne un booléen donc le 1er bout de code que vous montrez va exécuter le "if" quand la variable est définie et quelque soit sa valeur :
https://www.php.net/manual/fr/function.isset.php
https://www.php.net/manual/fr/types.comparisons.php

[tarharnak]

Très bien, merci pour cette précision.

Je suis en train de passer de PHP 7 à PHP 8 (il serait temps...) et lorsque je ne mets pas isset(), j'ai un Warning: Undefined.
J'imagine donc que je dois à chaque fois soit déclarer la variable null ou alors utiliser isset() à chaque fois ?

Par exemple voici mon ancien code :

Code :

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

1
2
3
	if($_POST['foo'] == 'bar') {
        //...
    }

Les trois différentes manières qui suivent sont elles toute bonnes ?

Code :

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

1
2
3
4
5
6
 $var = null;
    $var = $_POST['foo'];
 
	if($var === 'bar'){
        //...
    }

Code :

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

1
2
3
4
5
6
 if(isset($_POST['foo'])) $var = $_POST['foo'];
    else $var = null;
 
	if($var === 'bar'){
        //...
    }

Code :

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

1
2
3
if(isset($_POST['foo']) AND $_POST['foo'] === 'bar'){
        //...
    }

[CosmoKnacki]

lorsque je ne mets pas isset(), j'ai un Warning: Undefined.

"Undefined" mais encore? Lis le message jusqu'au bout, il est là pour te dire précisément ce qui coince.

J'imagine donc que je dois à chaque fois soit déclarer la variable null ou alors utiliser isset() à chaque fois ?

Par exemple voici mon ancien code :

Code :

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

1
2
3
	if($_POST['foo'] == 'bar') {
        //...
    }

Le problème ici est que tu ne sais pas s'il existe une clef 'foo' dans le tableau $_POST. Si cette clef n'existe pas, un warning sera émis pour te le signaler dés que tu utiliseras $_POST['foo'] dans ton code (à l'exception de isset($_POST['foo']) et dans le cas d'une utilisation avec l'opérateur null coalescent).

Les trois différentes manières qui suivent sont elles toutes bonnes ?

Code :

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

1
2
3
4
5
6
 $var = null;
    $var = $_POST['foo'];
 
	if($var === 'bar'){
        //...
    }

Non, la première ligne ne sert à rien et de toute manière une variable qui n'est pas définie sera toujours évaluée à null.
De plus tu ne testes pas l'existence de la clef 'foo', donc si elle n'existe pas => Warning: Undefined array key "foo".

Code :

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

1
2
3
4
5
6
 if(isset($_POST['foo'])) $var = $_POST['foo'];
    else $var = null;
 
	if($var === 'bar'){
        //...
    }

Ça c'est correct.

Code :

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

1
2
3
if(isset($_POST['foo']) AND $_POST['foo'] === 'bar'){
        //...
    }

et ça c'est correct aussi. C'est la même chose que précédemment sauf que tu n'utilises pas de variable intermédiaire. (Préfères utiliser && plutôt que AND, qui s'écrit d'ailleurs en minuscule, et réserve le pour les cas plus subtile de priorité des opérateurs).

Variantes:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
// Plus explicite que isset() (puisqu'il s'agit de tester l'existence d'une clef)
 
if (array_key_exists('foo', $_POST) && $_POST['foo'] === 'bar') {
    //...
}
 
// L'opérateur ternaire    condition ? valeur si vrai : valeur si faux
 
$var = isset($_POST['foo']) ? $_POST['foo'] : null;
 
if ($var === 'bar') {
    //...
}
 
// L'opérateur null coalescent (null coalescing operator).
 
$var = $_POST['foo'] ?? null;
 
if ($var === 'bar') {
    //...
}

[Bisûnûrs]

Code :

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

1
2
3
4
5
// Plus explicite que isset() (puisqu'il s'agit de tester l'existence d'une clef)
 
if (array_key_exists('foo', $_POST) && $_POST['foo'] === 'bar') {
    //...
}

Mais beauuuucoup moins performant.

[Séb.]

Généralement on évite de travailler sur $_POST, $_GET, etc. et on importe les valeurs soit directement dans une variable scalaire, soit dans un tableau. Ainsi on peut toujours revenir sur les $_POST, etc. originaux si besoin.

=> Récupère tes valeurs en début de script ou donne une valeur par défaut si inexistante. Pour cela il existe l'opérateur ??.

Code :

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

1
2
3
4
5
6
7
$foo = $_POST['foo'] ?? null; // $foo vaudra la valeur de $_POST['foo'] ou null par défaut, cela sans warning
 
if ($foo === null) {
    echo '$foo non transmis';
} else {
    ...
}

Alternative :

Code :

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

1
2
$request['post']['foo'] = $_POST['foo'] ?? null;
...

[CosmoKnacki]

Mais beauuuucoup moins performant.

Ce n'est plus le cas à partir de PHP >= 7.4.
Effectivement pour PHP 7.3, on peut voir que array_key_exists est environ 1.5 fois plus lent que isset, mais pour les versions ultérieures c'est kif-kif bourricot.