phpAnonyme
Messages postés392Date d'inscriptionmercredi 28 octobre 2009StatutMembreDernière intervention23 mars 2012
-
17 mars 2011 à 19:57
Xstoudi
Messages postés19Date d'inscriptionmercredi 20 octobre 2010StatutMembreDernière intervention 2 juillet 2011
-
17 mars 2011 à 20:42
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
Xstoudi
Messages postés19Date d'inscriptionmercredi 20 octobre 2010StatutMembreDernière intervention 2 juillet 2011 17 mars 2011 à 20:42
Merci, je vais tenter d'améliorer pour les prochains code sources!
phpAnonyme
Messages postés392Date d'inscriptionmercredi 28 octobre 2009StatutMembreDernière intervention23 mars 201255 17 mars 2011 à 20:34
Juste 2-3 exemples :
- Les instructions imbriqués successifs en interne if if if else else else juste pour vérifier la validité des 3 valeurs...c'est laid
- Utiliser htmlspecialchars() sur le nombre de mail a envoyé c'est laid...il y a intval()
- Tu fais des vérifications...autant les faire jusqu'au bout, non ? ! Tu ne vérifie pas même pas si l'adresse email à un format valide, la seule chose de faite c'est une conversion de caractères spéciaux !!
En bonus :
/////////////
# mail($destinataire, $sujet, $message);
# echo('Message n°' . $nombre . ' a été envoyé
');
////////////
Comment peut-tu prétendre que le mail est envoyé alors que tu ne vérifie pas la valeur de retour pour mail()
Voilà en quoi je dis que c'est mal codé !
Xstoudi
Messages postés19Date d'inscriptionmercredi 20 octobre 2010StatutMembreDernière intervention 2 juillet 2011 17 mars 2011 à 20:00
Je ne vois pas en quoi c'est mal coder, de plus, c'est mon premier code php
phpAnonyme
Messages postés392Date d'inscriptionmercredi 28 octobre 2009StatutMembreDernière intervention23 mars 201255 17 mars 2011 à 19:57
############
-Qui souhaitent regarder la communication formulaire-variable
-Qui souhaitent étudier la boucle "for"
-Qui souhaitent étudier la fonction mail() de php
###########
Ah bon ! Ben fallait faire un tuto dans ce cas ! non ?
A part l'incitation aux spams...ce code n'a rien à faire là, en plus c'est mal codé !
17 mars 2011 à 20:42
17 mars 2011 à 20:34
- Les instructions imbriqués successifs en interne if if if else else else juste pour vérifier la validité des 3 valeurs...c'est laid
- Utiliser htmlspecialchars() sur le nombre de mail a envoyé c'est laid...il y a intval()
- Tu fais des vérifications...autant les faire jusqu'au bout, non ? ! Tu ne vérifie pas même pas si l'adresse email à un format valide, la seule chose de faite c'est une conversion de caractères spéciaux !!
En bonus :
/////////////
# mail($destinataire, $sujet, $message);
# echo('Message n°' . $nombre . ' a été envoyé
');
////////////
Comment peut-tu prétendre que le mail est envoyé alors que tu ne vérifie pas la valeur de retour pour mail()
Voilà en quoi je dis que c'est mal codé !
17 mars 2011 à 20:00
17 mars 2011 à 19:57
-Qui souhaitent regarder la communication formulaire-variable
-Qui souhaitent étudier la boucle "for"
-Qui souhaitent étudier la fonction mail() de php
###########
Ah bon ! Ben fallait faire un tuto dans ce cas ! non ?
A part l'incitation aux spams...ce code n'a rien à faire là, en plus c'est mal codé !