[PHP-JS] Comment mieux tester l'existence de plusieurs variables

Bonjour les amis(es)

Depuis PHP4, il y a des erreurs lorsqu'une varaiable n'est pas testée avant son utilisation. Modifier le php.ini pour masquer les erreurs n'étant pas conseillé, nous sommes obligés de faire à chaque fois ceci:

Code:

---

1 2 3 4 5 6

<?php if(isset($_POST['nom'])){ $nom = $_POST['nom']; } ?>

---

Mais qu'en est-il si vous avez plus de 20 variables à traiter?
Il y a pas un risque d'augmenter considérablement le temps d'exécution en posant tous ces ifs?

Que fera un bon développeur?

D'avance je vous remercie

Un formulaire comportant 20 champs commence à être conséquent à mon gout.
Au dela un bon développeur ira consulter son ergonome (ou des cours faute de mieux) pour trouver une solution.

Et 20 conditions à traiter en php, à moins d'avoir un serveur datant de l'avant guerre c'est très rapide (plus que le temps de latence de la connexion je suis sur ;))

Un bon développeur pour moi s'assure qu'il n'y a pas d'erreur avant tout, et privilégie robustesse à rapidité (sans toutefois négliger ce dernier point).

Merci beaucoup.

J'apprend bcp avec toi. Donc en conclusion, il y a pas moyen de faire autrement. Je suis donc sur la bonne voie. Parce que c'est comme ça que je fais. Je voudrais savoir s'il y avait mieux.

Citation:

---

Envoyé par Alexandrebox

Bonjour les amis(es)

Depuis PHP4, il y a des erreurs lorsqu'une varaiable n'est pas testée avant son utilisation. Modifier le php.ini pour masquer les erreurs n'étant pas conseillé, nous sommes obligés de faire à chaque fois ceci:

Code:

---

1 2 3 4 5 6

<?php if(isset($_POST['nom'])){ $nom = $_POST['nom']; } ?>

---

Mais qu'en est-il si vous avez plus de 20 variables à traiter?
Il y a pas un risque d'augmenter considérablement le temps d'exécution en posant tous ces ifs?

Que fera un bon développeur?

D'avance je vous remercie

---

Je rajoute juste une toute petite chose...
Mieu vaut initialiser tes variables avant.

Code:

---

1 2 3 4 5 6 7

<?php $nom = ""; if(isset($_POST['nom'])){ $nom = $_POST['nom']; } ?>

---

Après perso je l'écrit comme cela :

Code:

---

1 2	$nom = isset($_POST['nom']) ? $_POST['nom'] : "";

---

Comme cela si j'en ai une vingtaine à faire c'est relativement rapide. Mais bon sa ne change pas le fait que c'est vraiment très fastidieux ces déclarations.

bye

initialiser les variables avant?!
Ben non, sinon ton isset ne sert à rien! Il renverra toujours true, puisqu'il sert justement à vérifier qu'une variable est définie ou non.

Mettez vous d'accord les mecs.

Là je suis embrouillé. MDR

En tout cas merci pour vos efforts

L'exemple de kaymak peut s'avérer dangereux, en particulier si le register_globals est à on dans le php.ini

Je plussoie haltabush

Citation:

---

initialiser les variables avant?!
Ben non, sinon ton isset ne sert à rien! Il renverra toujours true, puisqu'il sert justement à vérifier qu'une variable est définie ou non.

---

Ok donc, si le isset ne passe pas que fais tu ?
Tu executes ton script avec des variables qui n'existent pas ?
Dans ce cas à quoi sert il de vérifier que $_post[une clef] existe, si la variable qui recoit la donnée n'est pas systèmatiquement initialisée ?
C'est un peu comme piss** dans un violon si tu me pardonnes l'expression.

Fin moi je dis sa, je dit rien....

Citation:

---

L'exemple de kaymak peut s'avérer dangereux, en particulier si le register_globals est à on dans le php.ini

---

Dangereux ?? J'aimerais bien voir comment.

bye

Si le isset ne passe pas c'est que le formulaire n'a pas été envoyé (ou mal) et donc que tu n'as pas à effectuer le traitement. Tu passes donc ton chemin.
Typiquement si tu postes sur la meme page je ne vois pas vraiment d'autre moyen de faire la différence (ou bien si : demander au client d'autoriser l'activeX permettant d'écrire avec du javascript sur le poste client, écrire un cookie quand le formulaire est posté et tester ensuite en php :mrgreen:)

Le danger ? avec un register_globals on $_POST['toto'] équivaut à $toto si je ne m'abuse, si tu l'initialises avant ta condition d'existence sera donc systématiquement remplie :aie:

Ils ont raison kaymak

Tu ne peux pas déclarer une variable et ensuite demander si elle existe.
Bien sûr qu'elle existe puisque tu viens de la déclarer. :D

Merci pour ta participation. Merci à tous

Ce qu'il faut par contre faire au cas où ta variable n'existerait pas ce serait ça:

Code:

---

1 2 3 4 5 6 7 8 9 10

<?php if(isset($_POST['nom'])){ $nom = $_POST['nom']; } else{ $nom = ""; } ?>

---

Comme ça si plus loin tu essaie d'utiliser la variable $nom, tu n'auras pas d'erreur au cas où la variable $nom ne serait pas envoyée par $_POST

Citation:

---

Envoyé par Alexandrebox

Ils ont raison kaymak

Tu ne peux pas déclarer une variable et ensuite demander si elle existe.
Bien sûr qu'elle existe puisque tu viens de la déclarer. :D

Merci pour ta participation. Merci à tous

---

Je ne vois pas ou il teste une variable déclaré ?

Il faut savoir que $_POST['var'] n'est pas égal à $var et qu'il faut perdre l'habitude du register_global (je rappel au passage que le register_global n'existera plus du tout dans PHP 6) Et que t'en qu'a prendre les habitudes autant les prendres maintenant.

Et oui avec PHP 6, plus de register_global, plus de magic_quote, haaa enfin les developpeurs devrons se forcer à coder proprement

Oui je suis entièrement d'accord pour le register_global mais une erreur classique c'est de se dire "bah maintenant personne le fait", de meme qu'effectuer ses controles seulement en javascript et pas en php ;)

Citation:

---

Envoyé par ratapapa

Oui je suis entièrement d'accord pour le register_global mais une erreur classique c'est de se dire "bah maintenant personne le fait", de meme qu'effectuer ses controles seulement en javascript et pas en php ;)

---

Et comme ca si le javascript est désactivé t'es bai*** sans compté les problèmes de sécurité si la suite de son script dépend de l'intégrité des valeures

Citation:

---

Envoyé par |PaRa-BoL

Je ne vois pas ou il teste une variable déclaré ?

---

Lorsqu'il fait ceci:

Code:

---

1 2 3 4 5 6 7

<?php $nom =""; if(isset($_POST['nom'])){ $nom = $_POST['nom']; } ?>

---

Il teste la variable $nom après l'avoir déclarer

Heu ha bon ? nan franchement là je vois pas xD où tu vois ca ? je vois qu'une seul comparaison et c'est pas sur $nom...:nono:

Citation:

---

Envoyé par |PaRa-BoL

Et comme ca si le javascript est désactivé t'es bai*** sans compté les problèmes de sécurité si la suite de son script dépend de l'intégrité des valeures

---

C'est ce que je dis pour moi partir du principe que register_global est désactivé c'est comme ne faire les controles qu'en javascript : c'est ne pas prévoir que le cas peut se produire ;)

Ok donc on est d'accord xD

Mais dans son code il utilise $_POST['var'] et non $var donc il suppose que register_global est à off et modifier $var ne modifie pas $_POST['var'] (je suppose)

C'est vrai j'ai eu tord......







... Que d'avoir voulu lutter devant tant d'opiniatrete à continuer d'utiliser de si mauvaises méthodes.

Sur ce,

bye

Bah non moi je te défend au contraire :(

Oui c'est vrai, mais j'en ai lu de tellement verte et de tellement pas mûre dans ce thread.
Que tes commentaires, complétement censés, me sont totalement sortit de l'esprit.
Ils manquaientt surement de cette singularité si partiulière que j'ai pu observer dans les posts de ratapapa ou Alexandrebox ...
A mon plus grand plaisir ;)

Désolé mais je reste sur mon opinion.

J'y reste après avoir vu tant de fois "mon site marchait ne marche plus" seulement parce qu'on a changé d'hébergeur et que le nouveau possède le register_global activé.
J'y reste après avoir vu tant de fois "j'ai des erreurs dans les données de ma base" parce que le dev n'avait pas envisagé le fait qu'un formulaire puisse etre posté en sautant le controle javascript.

Enfin bon moi je dis ca c'est pas pour moi je continuerai à utiliser ma méthode :mrgreen:

Pour ma part :

Register_global off
Magic_quotes off

Évidement la vérification PHP des formulaire, ca va de soit :)

Bonjour,
Bon je passerais sur le saut d'humeur ("mais j'en ai lu de tellement verte e..."), mais j'ai encore 1 ou 2 commentaires à faire.
Tout d'abord, c'est vrai que le bout de code que nous a proposé au début n'était pas terrible, quand les register_globals sont à On, ce qui est le cas sur la plupart des serveurs mutualisés.
En revanche, ton code $nom=isset($_POST['nom'])?$_POST['nom']:"" est tout à fait correct...
Tu assure dans la page précédante que j'utilise dans mon code des variables non initialisées. Ben non! En gros, j'utilise isset toujours pour les même choses.
Soit la variable est définie, alors je fais mon traitement (pour le traitement des formulaire, principalement). Dans d'atures cas, si la variable n'est pas définie, alors je me "casse" avec header("location:...") (cas des sessions non initialiées)

Tu fais comme tu veux, je l'ai dit je laisse tomber.
Mais bon je préfére valider un formulaire sur la validité des valeurs plutôt que sur leur existence...

Ceci dit, je maintient fermement que dans le code précédent, à savoir :

Code:

---

1 2 3

if( isset($_post["nom"]) ) $nom = $_post["nom"];

---

N'intialise pas nécessairement $nom, et que donc c'est inutile de faire ces deux instructions.
Autant utiliser directement $_post.

bye

Salut,

pour ma part j'utilse

$nom = @$_post["nom"];
qui équivaut à:


if( isset($_post["nom"]) )
$nom = $_post["nom"];
{