POST et EMPTY

Version imprimable

POST et EMPTY

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

Code:

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

14/10/2010, 20h16
ritter jack

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

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:

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.

14/10/2010, 23h39
gene69

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:

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

l'autre

Code:

filter_var('bob@example.com', FILTER_SANITIZE_EMAIL);
15/10/2010, 08h17
wattaroo

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

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:

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

15/10/2010, 16h01
wattaroo

Effectivement "empty" veut dire vide !!! :aie:

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.