Syntaxe formulaire php

Version imprimable

Bonjour à tous, :P

J'ai réalisé un petit formulaire en php avec vérification des champs (empty), lors de différents tests toutes les fonctions ont fonctionnées (vérification, affichage du message d'erreur, envoi du mail).

Maintenant que j'ai fini mon bout de code, quand je fais un test global, le traitement reste bloqué sur mon fichier mail.php.

Je pense que c'est tout simplement un problème de syntaxe, il doit y avoir des {} mal placés, j'ai essayé plusieurs solutions, tout en me référant au tuto:
http://sylvie-vauthier.developpez.co...ulaires#LIII-1

Mais je dois avouer que je tourne en rond.

Si vous avez une idée lumineuse ou juste la possibilité de mettre le doigt sur mes erreurs, je vous en serai très reconnaissant. :ccool:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr">
<head>
<meta http-equiv="Content-Type" content="text/plain; charset=UTF-8" />
</head>
<?PHP
if (get_magic_quotes_gpc())
{
    $nom = stripslashes($_POST['nom']); 
    $email = stripslashes($_POST['email']);
    $nation = stripslashes($_POST['nation']); 
    $prenom = stripslashes($_POST['prenom']); 
    $telephone = stripslashes($_POST['telephone']); 
    $adresse = stripslashes($_POST['adresse']);
    $profession = stripslashes($_POST['profession']); 
    $statut = stripslashes($_POST['statut']); 
    $interet = stripslashes($_POST['interet']);
    $message = stripslashes($_POST['message']);  
} 
else
{  
    $nom = $_POST['nom'];
    $email = $_POST['email'];
    $nation = $_POST['nation']; 
    $prenom = $_POST['prenom']; 
    $telephone = $_POST['telephone']; 
    $adresse = $_POST['adresse'];
    $profession = $_POST['profession']; 
    $statut = $_POST['statut']; 
    $interet = $_POST['interet'];
    $message = $_POST['message'];  
 
 
 
/* Destinataire (votre adresse e-mail) */
$to = 'mail@alamaison.com';
 
// Subject
$subject = 'Demande informations concernant mon site';
/////voici la version Mine 
$headers = "MIME-Version: 1.0\r\n"; 
 
$headers .='Content-Type: text/plain; charset="utf-8"'." ";
$headers .='Content-Transfer-Encoding: 8bit'; 
/* Construction du message */
$msg  = 'Bonjour,'."\r\n\r\n";
$msg .= 'Ce mail a été envoyé depuis le site monsite.com par '.$nom.' '.$prenom."\r\n\r\n";
$msg .= 'Voici le message qui vous est adressé:'."\r\n";
$msg .= '***************************'."\r\n";
$msg .= 'Le nom est :'.$nom."\r\n\r\n";
$msg .= 'Le prénom est :'.$prenom."\r\n\r\n";
$msg .= 'La nationalité est :'.$nation."\r\n\r\n";
$msg .= 'Le numéro de téléphone est :'.$telephone."\r\n\r\n";
$msg .= 'L adresse email est :'.$email."\r\n\r\n";
$msg .= 'L adresse de domiciliation est :'.$adresse."\r\n\r\n";
$msg .= 'Sa profession est :'.$profession."\r\n\r\n";
$msg .= 'Son statut est :'.$statut."\r\n\r\n";
$msg .= 'Son intérêt est :'.$interet."\r\n\r\n";
$msg .= 'Son message est :'.$message."\r\n\r\n";
$msg .= '***************************'."\r\n\r\n";
 
 
$headers = 'From: '.$nom.' '.$prenom.' <'.$expediteur.'>'."\r\n\r\n";}
/* Envoi de l'e-mail */
 
if (empty($nom) || empty($prenom) || empty($nation) || empty($telephone) || empty($email))
    echo 'veuillez remplir tous les champs';
else
    mail($to, $subject, utf8_decode($msg), utf8_decode ($headers));
 
header('Location: http://www.monsite.com/');
</html>

Je vous remercie d'avance pour votre aide, très bonne journée.

Bonjour

Tu n'as absolument pas besoin de balise HTML pour faire le traitement que tu réalises en php. Donc
Code:

1 2 3 4 5 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr"> <head> <meta http-equiv="Content-Type" content="text/plain; charset=UTF-8" /> </head>
et

Code:

</html>

doivent virer.

Ensuite, pour ouvrir une déclaration php, la balise exacte est

Code:

<?php

Ensuite dans le bas de ta page, tu oublies de la fermer :
Code:

1 2 3 header('Location: http://www.monsite.com/'); </html> // à virer ?>
De plus, tu regarderas la syntaxe de ton if/else
Code:

1 2 3 4 5 6 if (empty($nom) || empty($prenom) || empty($nation) || empty($telephone) || empty($email)) echo 'veuillez remplir tous les champs'; else mail($to, $subject, utf8_decode($msg), utf8_decode ($headers)); header('Location: http://www.monsite.com/');
C'est ce que j'ai repéré au premier coup d'oeil

18/07/2012, 10h22
Fench

Bonjour à tous les deux,

@ledisciple
Il y a du html car il peut y avoir un echo dans le code php ;)
Sinon ok pour les balises php, manque le ?> et c'est ça qui doit merdouiller

@paikan
Peut tu développer:

Citation:

Maintenant que j'ai fini mon bout de code, quand je fais un test global, le traitement reste bloqué sur mon fichier mail.php.
18/07/2012, 10h29
ledisciple

Citation:

Envoyé par Fench

Bonjour à tous les deux,

@ledisciple
Il y a du html car il peut y avoir un echo dans le code php ;)
Sinon ok pour les balises php, manque le ?> et c'est ça qui doit merdouiller

En même temps, je n'ai pas remarqué

à quoi ça sert qu'il fasse un echo

si derrière, il y a

Code:

header('Location: http://www.monsite.com/');

Le header, n'est pas censé faire basculer sa page sur ce site, sans afficher les echo? :roll:
18/07/2012, 10h43
Fench

Citation:

à quoi ça sert qu'il fasse un echo

si derrière, il y a

Code :Sélectionner tout - Visualiser dans une fenêtre à partheader('Location: http://www.monsite.com/');

C'est sûr, et je ne m'attache pas à la structure du code ... et il n'est peut être pas complet (voir le ?> manquant alors qu'il existe peut être)

:lol: sinon +1 pour toi
18/07/2012, 10h59
paikan06

Merci beaucoup pour vos coups de main et vos réponses.
C'est effectivement ?> qui été manquant.

Donc, si j'ai bien compris le concept, si j'utilise la fonction header, je ne peux pas afficher le message pour indiquer aux visiteurs que tous les champs ne sont pas remplis......

Connaissez vous un autre moyen d'afficher l'information tout en conservant la fonction header ??? (popup, Js, etc....)

Merci d'avance :ccool:
18/07/2012, 11h11
ledisciple
Perso, j'utilise les $_SESSION[""].

Ca me permet de stocker les erreurs, d'une part :

Code:

$_SESSION["monform"]["erreur"]

et les champs d'autre part
Code:

1 2 3 $_SESSION["monform"]["champ1"] $_SESSION["monform"]["champ2"] //etc
après, je suis sur qu'il y a d'autres façon bien mieux de le faire ..
18/07/2012, 11h29
Fench

Citation:

Donc, si j'ai bien compris le concept, si j'utilise la fonction header, je ne peux pas afficher le message pour indiquer aux visiteurs que tous les champs ne sont pas remplis......

Tu as plusieurs solutions:

En js, tu peux vérifier tous les champs et sur le submit du formulaire interdire ce submit si manque un champ + afficher un message.

En php, lors du submit du formulaire, tu pars sur une page qui vérifie et affiche les champs manquants + message et enfin un post.

En php, de même sur la même page avec ajout de lignes en rouge pour les champs manquants

En jQuery encore plus de possibilité

Donc et j'en passe que de solutions ...

Citation:

Connaissez vous un autre moyen d'afficher l'information tout en conservant la fonction header

Pourquoi vouloir garder le header pour ce genre de traitement ?

Rappel sur le protocole HTTP

Les headers doivent, comme leur nom l'indique, être servis au client en tête de réponse et avant tout envoi de contenu.

En PHP, l'envoi de n'importe quel contenu, qu'il soit explicite avec echo ou implicite en envoyant du flux HTML (ou autre...) ou en rencontrant une erreur affichable, déclenche implicitement l'envoi de headers (notamment le content-type, par défaut text/html).

Egalement, l'utilisation de session_start déclenche l'envoi de header (pour le cookie de session notamment). cette fonction doit donc être appellée avant l'envoi de tout contenu.

En résumé:
Code:

1 2 3 4 5 6 7 8 9 10 <html> <head><title>Mauvais Exemple</title></head> <body> <?php session_start(); // va provoquer un erreur header("Reresh: 5;url=index.php"); // va provoquer une erreur echo "Vous allez être redirigé vers index.php"; ?> </body> </html>
Voici la bonne façon de faire:
Code:

1 2 3 4 5 6 7 8 9 10 <?php session_start(); header("Refresh: 5; url=index.php"); ?> <html> <head><title>Bon Exemple</title></head> <body> <p>Vous allez être redirigé</p> </body> </html>

Citation:

En jQuery encore plus de possibilité

Il n'y a pas "plus" de possibilités en jQuery, ce sont celles de JavaScript enrobées de sucre et joliment embalées pour que ce soit plus doux et que ça fonde sous la langue du développeur paresseux (je me fais l'avocat du diable, j'adore jQuery).

Exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 // voici un bout de code jQuery typique jQuery(function ($) { $('#mon-select').change(function () { alert($(this).val()); } }); // et voici son équivalent JavaScipt pur window.onload = function () { document.getElementById('mon-select').onchange = function () { alert(this.options[this.selectedIndex].value); } }

18/07/2012, 15h15
paikan06

Merci à tous pour votre aide,
j'ai bien compris la situation, je laisse tomber "header"

je suis en train d'essayer un petit script du style:

Code:

"javascript:history.back()"

sans trop de succès, a votre avis c'est la bonne piste ?:cry:
18/07/2012, 15h21
Benjamin Delespierre

Non, fais tes redirection normalement avec des headers, comme montré plus haut.