klicksoe
Messages postés38Date d'inscriptionvendredi 9 mars 2007StatutMembreDernière intervention23 avril 2009
-
6 juin 2008 à 11:37
nicomilville
Messages postés3472Date d'inscriptionlundi 16 juillet 2007StatutMembreDernière intervention28 février 2014
-
18 juin 2008 à 10:13
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
nicomilville
Messages postés3472Date d'inscriptionlundi 16 juillet 2007StatutMembreDernière intervention28 février 201436 18 juin 2008 à 10:13
Effectivement, j'arrive un peut tard et comme l'a si bien dit coucou747 :
- $HTTP_POST_VAR c'est très vieux, maintenant on utilise plutot : $_POST['nom']...
Et comme l'a dit aussi Morphinof :
- il n'y a aucune vérification, tu pourrai au moins vérifier si les variables existent et si elle ne sont pas vide : de cette façon :
- pour vérifier l'existance des variables : if(isset($_POST['nom_du_champs']) && isset($_POST...) {...
- pour vérifier si les variables ne sont pas vides : if(!empty($_POST['nom_du_champs']) && !empty($_POST...)
a++
PS : je préfère ne pas mettre de note !
bouztarak25
Messages postés5Date d'inscriptionlundi 25 juin 2007StatutMembreDernière intervention16 juin 2008 16 juin 2008 à 17:46
En effet, je ne suis pas très intéressé par 'FORMMAIL'!
Pourriez vous peut être un peu modifier et améliorer (test mail si il est bien parti etc..)?!
Bon courage !
webdeb
Messages postés488Date d'inscriptionsamedi 5 avril 2003StatutMembreDernière intervention31 mars 20094 6 juin 2008 à 19:07
Quelle horreur ce code....
Morphinof
Messages postés255Date d'inscriptionvendredi 20 avril 2007StatutMembreDernière intervention 9 août 20134 6 juin 2008 à 13:28
En dehors de ce qui disais très justement klicksoe au niveau du html, tu ne fais aucune vérification sur tes champs de formulaire !
Déjà au niveau sécurité c'est à revoir mais le minimum serait de vérifier le format de l'e-mail.
Quand tu utilises la fonction mail il faut tester son retour :
if(mail(...)) pour afficher un message d'erreur si jamais le mail passe pas etc.
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 6 juin 2008 à 12:15
salut
$HTTP_POST_VARS...
c'est du vieux php3... on en est a php6 actuellement
ton script n'a rien de personalisable, il n'est absolument pas generique, n'utilise aucune fonction, aucun template, etc...
klicksoe
Messages postés38Date d'inscriptionvendredi 9 mars 2007StatutMembreDernière intervention23 avril 2009 6 juin 2008 à 11:37
Salut,
quelques petites "erreurs" de HTML et de PHP...
En voilà une : (index.php)
#
-
#
ça devrait être :
#
-
#
Respect des normes W3C... Je te conseille le module HTML validator pour Firifox, il te permettra de faire ce genre de vérification.
Ensuite pour le PHP (formulaire.php)
Etant donné qu'il n'y en a pas beaucoup, je ne pas très bien voir si il y a des choses que u devrais approfondir, mais je te conseillerai de mettre des simple côte au lieu des guillemets.
18 juin 2008 à 10:13
- $HTTP_POST_VAR c'est très vieux, maintenant on utilise plutot : $_POST['nom']...
Et comme l'a dit aussi Morphinof :
- il n'y a aucune vérification, tu pourrai au moins vérifier si les variables existent et si elle ne sont pas vide : de cette façon :
- pour vérifier l'existance des variables : if(isset($_POST['nom_du_champs']) && isset($_POST...) {...
- pour vérifier si les variables ne sont pas vides : if(!empty($_POST['nom_du_champs']) && !empty($_POST...)
a++
PS : je préfère ne pas mettre de note !
16 juin 2008 à 17:46
Pourriez vous peut être un peu modifier et améliorer (test mail si il est bien parti etc..)?!
Bon courage !
6 juin 2008 à 19:07
6 juin 2008 à 13:28
Déjà au niveau sécurité c'est à revoir mais le minimum serait de vérifier le format de l'e-mail.
Quand tu utilises la fonction mail il faut tester son retour :
if(mail(...)) pour afficher un message d'erreur si jamais le mail passe pas etc.
6 juin 2008 à 12:15
$HTTP_POST_VARS...
c'est du vieux php3... on en est a php6 actuellement
ton script n'a rien de personalisable, il n'est absolument pas generique, n'utilise aucune fonction, aucun template, etc...
6 juin 2008 à 11:37
quelques petites "erreurs" de HTML et de PHP...
En voilà une : (index.php)
#
-
#
ça devrait être :
#
-
#
Respect des normes W3C... Je te conseille le module HTML validator pour Firifox, il te permettra de faire ce genre de vérification.
Ensuite pour le PHP (formulaire.php)
Etant donné qu'il n'y en a pas beaucoup, je ne pas très bien voir si il y a des choses que u devrais approfondir, mais je te conseillerai de mettre des simple côte au lieu des guillemets.
//Klick.