Syntaxe formulaire php

**paikan06** · 18/07/2012, 09h23

Bonjour à tous,

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.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

**ledisciple** · 18/07/2012, 09h44

Bonjour

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

</html>

doivent virer.

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

<?php

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
header('Location: http://www.monsite.com/');
</html> // à virer
?>

De plus, tu regarderas la syntaxe de ton if/else

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**Fench** · 18/07/2012, 10h22

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:

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

**ledisciple** · 18/07/2012, 10h29

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Le header, n'est pas censé faire basculer sa page sur ce site, sans afficher les echo?

**Fench** · 18/07/2012, 10h43

à 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)

sinon +1 pour toi

**paikan06** · 18/07/2012, 10h59

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

**ledisciple** · 18/07/2012, 11h11

Perso, j'utilise les $_SESSION[""].

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

et les champs d'autre part

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ..

**Fench** · 18/07/2012, 11h29

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 ...

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 ?

**Benjamin Delespierre** · 18/07/2012, 12h24

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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>

**Benjamin Delespierre** · 18/07/2012, 12h29

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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);
  }
}