MAIL BOMBER EDUCATIF

phpAnonyme Messages postés 392 Date d'inscription mercredi 28 octobre 2009 Statut Membre Dernière intervention 23 mars 2012 - 17 mars 2011 à 19:57
Xstoudi Messages postés 19 Date d'inscription mercredi 20 octobre 2010 Statut Membre Derniè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.

https://codes-sources.commentcamarche.net/source/52951-mail-bomber-educatif

Xstoudi Messages postés 19 Date d'inscription mercredi 20 octobre 2010 Statut Membre Derniè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és 392 Date d'inscription mercredi 28 octobre 2009 Statut Membre Dernière intervention 23 mars 2012 55
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és 19 Date d'inscription mercredi 20 octobre 2010 Statut Membre Derniè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és 392 Date d'inscription mercredi 28 octobre 2009 Statut Membre Dernière intervention 23 mars 2012 55
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é !