POST et EMPTY

**wattaroo** · 14/10/2010, 20h09

Bonjour à tous,
pour un formulaire de contact, j'utilise ce code :

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
<?php
    //création de noms abrégés pour les vatiables
    $nom = $_POST['nom'];
    $mail = $_POST['mail'];
    $sujet = $_POST['sujet'];
    $commentaires = $_POST['commentaires'];
 
    //initialisation d'informations
    $adresse_dest = "nom@fai.fr";
    $contenu_message = "Nom du client : ".$nom."\n"."Email du client : ".$mail."\n"."Commentaires du client : \n".$commentaires."\n";
    $adresse_exp = "From : ".$mail." ";
 
    //verification de l'envoi des données et appel de la fonction mail()
    if(!empty($_POST['nom']) && !empty($_POST['mail']) && !empty($_POST['sujet']) && !empty($_POST['commentaires'])){
        echo "Merci de remplir tous les champs du formulaire";
    }
    else{
        mail($adresse_dest,$sujet,$contenu_message,$adresse_exp);
    }
?>

et il me semblait que le if avec la condition "n'est pas remplie (!empty)" pour chaque POST enverrait sur le message d'erreur si un seul des champs n'est pas remplis.

Mais ce n'est pas le cas et le formulaire est transmis dans tous les cas, même sans information.

Que faudrait_il faire pour que la condition soit remplie ??

Merci

**ritter jack** · 14/10/2010, 21h16

Salut,

dans ta condition utilise des $nom ( comme tu l'as définie avant ) à la place de $_POST['nom']

et essaie à la place de !empty de mettre != NULL comme ca :

if($nom != NULL && .... )

**RunCodePhp** · 14/10/2010, 22h17

Salut

Il faut en 1er vérifier l'existence de tous les éléments, c'est après qu'il faudra vérifier 1 par 1 s'ils contiennent rien, et en dernier initialiser les variables $nom, $email, etc ...
Tu fais tout l'inverse.

Puis tu dois avoir un bouton submit, avec un nom et une valeurs, vérifier son existence est utile aussi, c'est un petit plus.

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
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
 
<?php
$erreur = false;
$msg_erreurs = array();
//
if (isset($_POST['Envoyer'], $_POST['nom'], $_POST['mail'], $_POST['sujet'])) {
    if (empty($_POST['nom'])) {
        $erreur = true;
        $msg_erreurs[] = 'Veuillez saisir votre nom';
    }
    //
    if (empty($_POST['mail'])) {
        $erreur = true;
        $msg_erreurs[] = 'Veuillez saisir votre adresse e-mail';
    }
    //
 
    ... etc ...
 
    if ($erreur == false) {
        $nom = $_POST['nom'];
 
        ... etc ...
 
        $verif = mail($adresse_dest,$sujet,$contenu_message,$adresse_exp);
        if ($verif == false) {
            $erreur = true;
            $msg_erreurs[] = 'Mail non envoyé ???';
        }
        else {
            // On effectue une redirection (la même page éventuellement).
            header('Location: page.php?sendmail=envoye');
        }
    }
}
 
// PARTIE HTML
?>
<!DOCTYPE ... etc ...>
<html>
 
... etc ...
 
<?php
// Affichage des éventuels erreurs
if ($erreur === true) {
    foreach ($msg_erreurs as $erreur) {
        echo $erreur.'<br />';
    }
}
else if (isset($_GET['sendmail']) && $_GET['sendmail'] == 'envoye') {
    echo 'Le mail a bien été envoyé ...<br />';
}
?>
 
<!-- Formulaire de contact -->
... etc ...
 
</html>

Je n'est pas testé (et pas du tout complet), c'est juste pour exemple.
Un empty() est malgré tout très peu restrictif, ça ne veut pas dire que l'adresse e-mail est valide par exemple, pour ça faut pousser plus loin normalement.

**gene69** · 15/10/2010, 00h39

si tu laisses ton script en l'état, donne moi url, que j'envoie du spam avec.

il y a en php une fonction qui vérifie un mail, elle est buildin, elle est conforme à la RFC, elle permet d'utiliser les adresses du genre gene69+php@ , ce qui est tout à fait autorisé, et en plus c'est un feature chez google.

l'une

Code :

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

filter_var('bob@example.com', FILTER_VALIDATE_EMAIL);

l'autre

Code :

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

filter_var('bob@example.com', FILTER_SANITIZE_EMAIL);

**wattaroo** · 15/10/2010, 09h17

Merci à tous pour vos réponse.

RunCodePhp, je vais essayer ton code cet après-midi et je reviens te dire si ça marche.

Merci

**antoinev2** · 15/10/2010, 09h36

Bonjour,

si j'ai bien lu, tu as simplement inversé les instructions...
Car "!empty" signifie "n'est pas vide", et non pas "n'est pas rempli".

Donc là, tu dis que si chacun de tes champs contient une valeur, il faut afficher le message qui demande de remplir tous les champs...

c'est l'inverse qu'il faudrait faire :

Code :

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

1
2
3
4
5
6
7
8
//verification de l'envoi des données et appel de la fonction mail()
    if(!empty($_POST['nom']) && !empty($_POST['mail']) && !empty($_POST['sujet']) && !empty($_POST['commentaires'])){
 
        mail($adresse_dest,$sujet,$contenu_message,$adresse_exp);
    }
    else{
        echo "Merci de remplir tous les champs du formulaire";
    }

**wattaroo** · 15/10/2010, 17h01

Effectivement "empty" veut dire vide !!!

Mais juste inverser les actions ne change rien, ça envoi directement le mail.

Je vais donc prendre du temps pour bien étoffer ce code, comme me l'a proposé RunCodePhp et gene69.

Merci pour votre aide.

POST et EMPTY

Langage PHP

Discussions similaires

Partager

Partager