Discussion :

[fleedalfire]

Bonjour.
J'ai fait une condition avec une regex PCRE pour vérifier le numéro de téléphone et je ne comprend pas où ça bug.
L'expression régulière me semble juste. le but c'est que ça permette une numéro comme ceci: 022 772 89 34
avec soit des point, soit des espaces, soit des tiret et un / entre 022 et 772

Code :

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

1
2
3
4
5
6
7
if(FALSE === isset($_POST["telephone"])){
        $_POST['telephone'] = htmlspecialchars($_POST['telephone']);
        $message_erreur[] = "Pas de numéro de télépnone reçu";
    }
elseif (preg_match("#^02[1-9][-. /]?[1-9]{3}([-. ]?[0-9]{2}){4}$#", $_POST["telephone"])){
        $message_erreur[] = "Vous devez entré un numero de telephone correcte";
    }

Code :

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

1
2
<input id="telephone" name="telephone" type="text" class="validate"/>
<label for="telephone" class="grey-text text-darken-4">Téléphone</label>

[CosmoKnacki]

Tout d'abord:

Code :

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

1
2
3
4
5
6
if(FALSE === isset($_POST["telephone"])){ // pas besoin d'une égalité stricte, isset renvoie soit true soit false
                                          // donc if ( !isset($_POST['telephone']) ) { est suffisant
    $_POST['telephone'] = htmlspecialchars($_POST['telephone']); // À quoi sert cette ligne? 
                                                                 // $_POST['telephone'] n'est pas défini, tu ne peux pas l'utiliser.
    $message_erreur[] = "Pas de numéro de télépnone reçu";
}

Ensuite pour ta pattern, tu dis vouloir vérifier un numéro avec le format 022 772 89 34, alors pourquoi répéter 4 fois ([-. ]?[0-9]{2}){4}? Deux fois devrait suffire.
Autre chose, quand tu utilises l'ancre $ pour la fin de chaîne, ajoute aussi le modificateur D, ou bien remplace $ par \z car sinon un caractère \n pourrait traîner à la fin de la chaîne et la pattern n'y verrait que du feu.

Code :

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

1
2
3
elseif ( !preg_match("#^02[1-9][-. /]?[1-9]{3}([-. ]?[0-9]{2}){2}$#D", $_POST["telephone"]) ) {
    $message_erreur[] = "Vous devez entrer un numéro de téléphone correct";
}

Est-ce que tu confirmes que le zéro n'est pas autorisé dans le deuxième groupe de trois chiffres et est-ce que le numéro peut être entré sans aucun séparateur?

[fleedalfire]

// donc if ( !isset($_POST['telephone']) ) { est suffisant

c'est comme ça que j'ai appris, mais je sais que ! suffit.

$_POST['telephone'] = htmlspecialchars($_POST['telephone']);

sert a ce que j'ai vus empêcher de mettre des caractère spécial comme <> (il me semble)

$_POST['telephone'] n'est pas défini, tu ne peux pas l'utiliser.

si il est défini dans l'input, j'ai pas mis tout le formulaire.

Est-ce que tu confirmes que le zéro n'est pas autorisé dans le deuxième groupe de trois chiffres et est-ce que le numéro peut être entré sans aucun séparateur?

Je confirme.

[fleedalfire]

pourtant voilà ce que je peux envoyer comme numéro de téléphone, vus le regex je ne devrais pas pouvoir.
Il devrait me donner le message d'erreur

Vous devez entrer un numéro de téléphone correct

[CosmoKnacki]

Dans tous les cas, tu n'as pas besoin d'utiliser htmlspecialchars où que ce soit car quand $_POST['telephone'] n'est pas défini, il n'y a rien à échapper, et quand il l'est, tu connais les caractères présents dans la chaîne puisque tu l'as contrôlée avec preg_match. Tu peux enlever cette ligne, elle ne sert à rien et provoquera un undefined offset.

Ce n'est pas parce que tu as une balise <input name="telephone" type="text" /> dans ton formulaire qu'arrivé dans ton code php, $_POST['telephone'] va être forcément défini. Garde en tête que rien ne prouve que les données reçues par ton script PHP proviennent bien de ton formulaire. D'ailleurs si c'était le cas, pourquoi tester avec isset?

[CosmoKnacki]

Je viens de me rendre compte que le elseif teste ( preg_match... ) au lieu de ( !preg_match... ). Voilà le problème.

[fleedalfire]

ben je sais pas où est le problème, alors je te met tout le code php.

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
 
if($_SERVER['REQUEST_METHOD'] == 'GET'){
    // variable booleene qui indiquera si on affiche ou pas le formulaire
    $affiche_formulaire = true;
    //je cree une variable vide qui apparaitra si il y des messages d'erreur
    $message = "";
}
else if($_SERVER['REQUEST_METHOD'] == 'POST'){
/*Je vérifie si l'adresse de l'e-mail n'a pas ete reçu*/
    if(FALSE === isset($_POST["telephones"])){
 
        $message_erreur[] = "Pas de numéro de télépnone reçu";
 
    }/*Je vérifie si le numéro est correctment rentré*/
    elseif (preg_match("#^02[1-9][-. /]?[1-9]{3}([-. ]?[0-9]{2}){2}$#D", $_POST["telephones"])){
 
        $message_erreur[] = "Vous devez entrer un numéro de téléphone correct";
 
    }
if(empty($message_erreur)){
 
        $telephone = ($_POST["telephones"]);
        $message_recapitulatif[] .= "Telephone: ".$telephone;
 
        $message = 'Voici le récapitulatif de votre résevation <br />'.implode('<br />', $message_recapitulatif);
 
        // le formulaire ne sera pas réaffiché
        $affiche_formulaire = true;
 
    } // sinon
    else {
 
        // on fabrique un message d'erreur
        $message = implode("<br/> ", $message_erreur);
 
        // on reaffiche le formulaire
        $affiche_formulaire = TRUE;
 
    }}

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
<?php echo $message; ?>
        <?php if($affiche_formulaire){ ?>
        <form action="menus.php" method="post" enctype="multipart/form-data">
<div class="row">
                <div class="input-field col s12 m6 l6">
                    <i class="material-icons prefix">phone</i>
                    <input id="telephone" name="telephones" type="text" class="validate"/>
                    <label for="telephone" class="grey-text text-darken-4">Téléphone</label>
                </div>
                <div class="input-field col s12 m6 l6">
                    <i class="material-icons prefix">email</i>
                    <input id="email" name="mails" type="email" class="validate"/>
                    <label for="email" data-error="wrong" data-succes="right" class="grey-text text-darken-4">E-mail</label>
                </div>
            </div>
</form>
<?php } ?>

[gene69]

sans répondre sur la validité de l'expression réguliere (pour mettre au point, ya des outils en ligne qui sont utiles parce que ça nous débarasse du code de gestion de l'appel preg_match)

justement quand je compare le manuel de l'expression preg_match

preg_match() retourne 1 si le pattern fourni correspond, 0 s'il ne correspond pas, ou FALSE si une erreur survient.

et ta condition, je me dit que la ligne 17 de ton exemple ne contient pas le bon commentaire.

En plus tu passes par une variable relais (ici $message_erreur oubien c'est $message, j'ai pas compris) et un empty() sur un tableau ou une variable non initialisée. Pas terrible. Si tu tiens à mettre une variable relais au lieux de d'appeler une fonction (ou des else imbriqués), utilise un bouleen initialisé par défaut et une 2e variable pour le message.

Enfin, une fois que tu auras résolu le bug de la ligne de la ligne 17, réfléchi à reprendre ton code avec un filter_var ou filter_array, j'aime pas trop manipuler $_POST dans mon code, comme il faut partir du principe que c'est une entree utilisateur, j'ai pas à la modifier mais j'ai à m'en méfier (donc je la recopie en la nétoyant (suppression des symboles binaires notement).

par exemple je fais:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function sanitize_form1( $unsafe ){
  return filter_array($unsafe, ...);
}
//et ensuite pour récuperer sur POST
$safe=sanitize_form1($_POST);
 
//et si je veux supporter  $_GET
$safe=sanitize_form1($_GET);
 
//et si je veux les deux
$safe=sanitize_form1($_REQUEST);

[fleedalfire]

(pour mettre au point, ya des outils en ligne qui sont utiles parce que ça nous débarasse du code de gestion de l'appel preg_match)

Est-ce que tu aurais un outils en ligne a me conseiller.

[gene69]

ya plein,

https://regex101.com/
https://fr.functions-online.com/preg_match.html
https://www.regextester.com/
http://preg_match.onlinephpfunctions.com/

[fleedalfire]

merci pour vos conseil et votre aide,
Je me suis rendu compte que ce qu'il me manquait dans ma regex pour qu'elle fonctionne c'est ==FALSE.
Je met en dessous la regex qui marche au cas où ça intéresserai quelqu'un.

Code :

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

1
2
3
elseif ((preg_match("#^0[27][1-9][-. /]?[1-9]{3}([-. ]?[0-9]{2}){2}$#D", $_POST["telephone"])) == FALSE){
        $message_erreur[] = "Vous devez entrer un numéro de téléphone correct";
    }

[CosmoKnacki]

Oui, c'est peu ou prou ce que j'ai écrit dans le message #6 (et corrigé dans le message #2).
preg_match renvoie false quand la recherche génère une erreur, quand la correspondance n'est pas trouvée, preg_match retourne 0 (cf le message #8 de gene69). Donc si tu choisis d'écrire explicitement une égalité, mieux vaut le faire avec une égalité stricte à 0:if ( preg_match(...) === 0 ) {.
Dans cette perspective écrire: if ( preg_match(...) == false ) { marchera, car le 0 est transtypé en booléen, mais n'a pas trop de sens. Ceci dit, écrire if ( !preg_match(...) ) { fait exactement la même chose (le transtypage de 0 à false), mais le mot false n'apparaît pas (ce qui met le lecteur à l'abri de pensées subversives). Mais ce n'est pas grave d'un point de vue fonctionnel car ce genre de pattern ne provoquera jamais d'erreurs et ne renverra jamais false.

[fleedalfire]

Pourquoi dans ma regex

[27]

le 7 ne marche pas, mais uniquement 2.
Alors que c'est une condition "ou".

[Celira]

Définis "le 7 ne marche pas". Ta regex matche "071 851 12 22" pour moi.

[fleedalfire]

ben chez moi il me donne le message d'erreur si je tape 079 444 77 28
par contre si je tape 022 793 72 43 il me le passe comme correcte.

[Celira]

Le code suivant fonctionne pour moi :

Code :

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

1
2
3
4
5
6
7
$re = '@^0[27][1-9][-. /]?[1-9]{3}([-. ]?[0-9]{2}){2}$@D';
$str = '079 444 77 28';
 
preg_match_all($re, $str, $matches);
 
// Print the entire match result
var_dump($matches);

Tu tapes la chaîne à la main, sans espaces en trop ? ce n'est pas un copier-coller bancal ?

[fleedalfire]

non je tape la chaine à la main.

Merci quand même.