Discussion :

[clavier12AZQSWX]

bonjour,

ça fait un bail que cette erreur existe et je comprends pas pourquoi PHP ne la détecte pas en erreur ni en warning :

souvent, je fais un copié-coller de ma condition se trouvant dans le IF pour la remettre dans le code plus bas et j'oublie d'enlever un "égal"

Si j'ai dans mon code :

Code :

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

if ($a=="45fdfdfdf87") { // do things..}

alors ma recopie sera :

Code :

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

$a=="45fdfdfdf87";

au lieu de :

Code :

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

$a="45fdfdfdf87";

Je pense copié une affectation qui est en fait une expression de test (qui renvoie true/false) sans produire d'erreur (dans le sens Anomalie de syntaxe) .

Pourquoi PHP me laisse écrire de tel ligne ? est-ce que ça a une utilité d'écrire ce genre de code :

Code :

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

1
2
3
$b="rien";
$a=="45fdfdfdf87";
$c="truc";

PHP (son compilateur ou interpréteur) devrait me mettre un warning pour me dire que la ligne 2: $a=="45fdfdfdf87"; est une erreur non ?
ça n'a pas de sens de faire une expression de test entre deux affectations ! n'est-ce pas ?

[mathieu]

je pense que ce genre de syntaxe est valide parce qu'on peut écrire des lignes de ce genre : ($a == "225") && appel_fonction();.
si la variable ne contient pas 225, la suite de la ligne n'est pas évaluée et la fonction n'est pas appelée. c'est une espèce de test "if" mais en version compacte.

ensuite je pense que l'erreur qui arrive le plus souvent est plutôt de taper trop vite et d'oublier un "=" dans le if :

Code :

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

1
2
3
4
if ($a = "225") {
    appel_fonction();
 
}

là aussi la syntaxe est valide mais l'erreur d'écriture va provoquer un dysfonctionnement dans le code.
c'est pour ça que pour faire des comparaisons dans mon code, j'inverse souvent pour mettre la variable en 2e position if ("225" === $a) {.
ainsi en oubliant des "=", obtient cette fois une erreur php : if ("225" = $a) {.

vous remarquerez que j'ai utilisé "===" pour faire une comparaison stricte. j'utilise principalement cet opérateur pour éviter les surprises :
https://www.php.net/manual/fr/types.comparisons.php

[grunk]

La syntaxe est valide, au même titre que

Code :

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

if($a = 2)

l'est , alors que c'est en général aussi une erreur.

Le moteur PHP se contente de savoir si ce que tu écrits est valide. Il ne se préoccupe pas de savoir si dans ton contexte particulier le code à du sens ou pas.

Si tu veux être avertit de ce genre de chose il faut utilser des outils dédiés d'analyse statique.
PHPStorm par exemple intègre PHP_CodeSniffer qui alerte de ce genre de problématique. Selon ton IDE il faut trouver le bon outil qui s"intègre bien. A toi ensuite de configurer l'outil pour qu'il réponde à tes besoins.

[clavier12AZQSWX]

merci pour vos précisions.