Aide pour amelioration script PHP

Aide pour amelioration script PHP - PHP - Programmation

Marsh Posté le 23-07-2005 à 17:07:37    

Bonjour, j'ai mis sur ma page web un formaulaire de contact en PHP ---> http://membres.lycos.fr/piratland/pages/contact.html
 
Or, il se trouve que meme si le message est vide, qu'aucun champ n'est rempli, un mail m'est envoyé...
 
voici le code :
 
<?  
$email = $_POST['email'];  
$nom = $_POST['nom'];  
$message = $_POST['message'];  
 
$msg = "Nom :\t$nom\n\n";  
$msg .= "Message :\t$message\n\n";  
   
$recipient = "piratland@laposte.net";  
 
echo '<HTML><HEAD>';  
echo '<TITLE>Form Sent!</TITLE></HEAD><BODY>';  
echo '<H1 align=center><br><font color="#FFFFFF">Merci, '.$nom.'</H1>';  
 
if( mail($recipient, "Formulaire de Contact, Piratland" ,$msg, "From: $email\r\nReply-To: $email" ) )  
{  
   echo '<P align=center>Votre message est bien envoyé au webmaster.</font></P>';  
}  
else  
{  
   echo '<P align=center>Erreur !!!, le message n\'a pas pu etre envoyé.</P>';  
}  
echo '<P align=center>';  
echo '<a href="javascript:history.go(-2)">Retour</a></P>';  
echo '</BODY></HTML>';  
?>

 
Je suis débutant en PHP, et je vois pas comment faire, quelqu'un pourrait m'aider ??
merci d'avance.


---------------
.:P1R@TM4N:.
Reply

Marsh Posté le 23-07-2005 à 17:07:37   

Reply

Marsh Posté le 23-07-2005 à 17:15:13    

Test tes champs de formulaire (en javascript par exemple même si je suis plus pour la méthode php) avant d'envoyer ton message.. :)

Reply

Marsh Posté le 23-07-2005 à 19:21:30    

Test en JS insuffisant.
 
Test côté serveur (donc en PHP) conseillé. Tu peux en plus tester en JS pour améliorer la réactivité de la page, mais pas te fier la dessus.


---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Marsh Posté le 23-07-2005 à 19:48:51    

micfont999 a écrit :

Test tes champs de formulaire (en javascript par exemple même si je suis plus pour la méthode php) avant d'envoyer ton message.. :)


 
 
 

Code :
  1. <? 
  2. $email = $_POST['email']; 
  3. $nom = $_POST['nom']; 
  4. $message = $_POST['message']; 
  5. if(empty($message)) {
  6. // ajoute ici ce que tu veux comme message avec echo ! exemple ->
  7. echo "Le message ne doit pas etre vide pour pouvoir etre envoyer  ";
  8. } else {
  9. $msg = "Nom :\t$nom\n\n"; 
  10. $msg .= "Message :\t$message\n\n"; 
  11.    
  12. $recipient = "piratland@laposte.net"; 
  13. echo '<HTML><HEAD>'; 
  14. echo '<TITLE>Form Sent!</TITLE></HEAD><BODY>'; 
  15. echo '<H1 align=center><br><font color="#FFFFFF">Merci, '.$nom.'</H1>'; 
  16. if( mail($recipient, "Formulaire de Contact, Piratland" ,$msg, "From: $email\r\nReply-To: $email" ) ) 
  17.    echo '<P align=center>Votre message est bien envoyé au webmaster.</font></P>'; 
  18. else 
  19.    echo '<P align=center>Erreur !!!, le message n\'a pas pu etre envoyé.</P>'; 
  20. echo '<P align=center>'; 
  21. echo '<a href="javascript:history.go(-2)">Retour</a></P>'; 
  22. echo '</BODY></HTML>'; 
  23. }
  24. ?>

Reply

Marsh Posté le 23-07-2005 à 19:58:22    

100top a écrit :

Code :
  1. <? 
  2. $email = $_POST['email']; 
  3. $nom = $_POST['nom']; 
  4. $message = $_POST['message']; 
  5. if(empty($message)) {
  6. // ajoute ici ce que tu veux comme message avec echo ! exemple ->
  7. echo "Le message ne doit pas etre vide pour pouvoir etre envoyer  ";
  8. } else {
  9. $msg = "Nom :\t$nom\n\n"; 
  10. $msg .= "Message :\t$message\n\n"; 
  11.    
  12. $recipient = "piratland@laposte.net"; 
  13. echo '<HTML><HEAD>'; 
  14. echo '<TITLE>Form Sent!</TITLE></HEAD><BODY>'; 
  15. echo '<H1 align=center><br><font color="#FFFFFF">Merci, '.$nom.'</H1>'; 
  16. if( mail($recipient, "Formulaire de Contact, Piratland" ,$msg, "From: $email\r\nReply-To: $email" ) ) 
  17.    echo '<P align=center>Votre message est bien envoyé au webmaster.</font></P>'; 
  18. else 
  19.    echo '<P align=center>Erreur !!!, le message n\'a pas pu etre envoyé.</P>'; 
  20. echo '<P align=center>'; 
  21. echo '<a href="javascript:history.go(-2)">Retour</a></P>'; 
  22. echo '</BODY></HTML>'; 
  23. }
  24. ?>



 
Voila, j'avais pas le courage de le faire mais c'est tout à fait ça :)

Reply

Marsh Posté le 25-07-2005 à 11:37:31    

Je rajouterai juste un exit; a la fin du echo"message peut pas etre envoyé" pour etre sur que ca execute pas le reste.(Peut-être inutile mais plus prudent si jamis tu modife le script derrière)

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed