echec et match

**JLC83** · 15/09/2010, 22h23

Bonjour,
Un seul commentaire

En xHTML la première ligne de la page est ici :

Code :

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

<?xml version="1.0" encoding="ISO-8859-1" ?>

Car nous voulons suivre les standards - plus pour longtemps sans doute...
Pour tester quelques champs avant de l'envoyer à PHP nous utilisons une vérification via expression régulière

Code :

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

1
2
 this.ReReg['Nom'] = /^[ \-\w\x80-\xFF]+$/;
this.ReReg['Prenom'] = /^[ \-\w\x80-\xFF]+$/;

Figurez vous que :

Code :

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

1
2
var valeur = 'Jean';
return valeur.match(this.ReReg['Prenom']);

Renvoie false

Tout comme 'DR' mais pas 'BARDOL' (true) allez donc savoir pourquoi ?
Merci de vos aides !

**SpaceFrog** · 15/09/2010, 22h47

EN dehors du fais que je vois deux regexp identique, ce qui m'echappe...

Je pense que vous ne réinitialisez le pointeur de la regexp entre deux tentatives de match.
essayez de faire un match sur une chaine vide entre chaque match ...

Sinon matchez sur la reg elle même:

Code :

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

1
2
3
 
var valeur = 'Jean';
return valeur.match(/^[ \-\w\x80-\xFF]+$/);

Mais en tout état de cause dans l'optique de tester un pattern il vaut mieux utiliser test ...

Code :

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

1
2
3
4
5
6
7
 
prenom="Jean"
alert(/^[ \-\w\x80-\xFF]+$/.test(prenom))
truc="DR"
alert(/^[ \-\w\x80-\xFF]+$/.test(truc))
machin="BARDOL"
alert(/^[ \-\w\x80-\xFF]+$/.test(machin))

**Eric2a** · 15/09/2010, 23h08

Salut,

Juste pour réagir sur le prologue XML...

Ce dernier est :

facultatif pour les pages servies en "text/html".

Le fichier sera toujours analysé comme un fichier HTML "classique".
Mais IE bascule en quircks mode (modèle de mise en page erroné).
obligatoire pour les pages servies en "application/xhtml+xml"

Le fichier sera considéré comme un "vrai" fichier XML.
IE (encore lui) ne les gère même pas.

**Watilin** · 15/09/2010, 23h13

Quand je regarde les caractères de \x80 à \xFF, je vois des trucs qui n'ont rien à faire dans un nom ou un prénom... Mais selon la spécif, JavaScript utilise ISO-8859-15 et pas ISO-8859-1, le problème est peut-être là.

Par contre quand la regexp n'a pas de flag g, la méthode test() n'a pas d'effet de bord, on peut l'utiliser sans surprises. Et je conseille de n'avoir qu'une regexp et la réutiliser, car derrière toute instanciation de regexp il y a une compilation, potentiellement longue.

Quant au standard... A l'heure actuelle, un prologue XML dans un document servi comme du HTML fait passer quelques navigateurs en mode quirks, quand d'autres plus anciens ne savent carrément pas le traiter. Il vaut mieux se contenter d'un bon vieux DOCTYPE.

**JLC83** · 16/09/2010, 11h02

Bonjour,
Merci de tous ces conseils et propositions.
Je m'attèle à modifier et améliorer.
Evidemment les deux patterns sont différents dans le code, j'ai juste mal fait le copier coller dans le message

J'espère que tout va vite rentrer dans l'ordre.
Merci

**JLC83** · 16/09/2010, 15h08

Voici le code de la méthode qui vérifie les champs :

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
onSoumet.prototype.lesReg = function(champ, valeur){
        this.ReReg['Cheque'] = /^[,0-9]+$/;
        this.ReReg['Titre'] = /^[^-][[:alpha:]-][^-]+$/;
        this.ReReg['Nom'] = /^[^-][[:alpha:]-][^-]+$/;
        this.ReReg['pre'] = /^[^-][[:alpha:]-][^-]+$/;
        this.ReREg['ad1'] = /^[^<>*]+$/;
        this.ReREg['ad2'] = /^[^<>*]+$/;
        this.ReREg['cp'] = /^[ 0-9\.]+$/;
        this.ReReg['Tel1'] = /^[ 0-9\.]+$/;
        this.ReReg['Tel2'] = /^[ 0-9\.]+$/;
        this.ReReg['Tel3'] = /^[ 0-9\.]+$/;
        this.ReReg['Port1'] = /^[ 0-9\.]+$/;
        this.ReReg['Fax'] = /^[ 0-9\.]+$/;
        this.ReReg['email1'] = /^([a-z0-9_]|\\-|\\.)+@(([a-z0-9_]|\\-)+\\.)+[a-z]{2,4}\$/;
        this.ReReg['email2'] = /^([a-z0-9_]|\\-|\\.)+@(([a-z0-9_]|\\-)+\\.)+[a-z]{2,4}\$/;
        return (this.ReReg[champ].test(valeur))
    };

Et bien, firebugg signale

this.ReREg is undefined this.ReREg['ad1'] = /^[^<>*]+$/; line148

J'ai bien sûr effectué le test de cette seule expression ailleurs et là tout est Ok.
Comprenez vous pourquoi ? Car moi :-/
Merci

**SpaceFrog** · 16/09/2010, 15h10

essaye d'echapper < ou > \< \>

**JLC83** · 16/09/2010, 16h02

Hélas, même en échappant, l'erreur survient.
Vous arrive-t-il parfois de désespérer avec javascript et les différents niveaux du DOM (+ différences entre navigateurs) car ici

Merci

**SpaceFrog** · 16/09/2010, 16h06

le desespoir e la compatibilité est sans fond ^^

Code :

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

1
2
3
var reg=  /^[^<>*]+$/;
var chaine="mjmljmlkj"
alert(reg.test(chaine))

pas de souci avec ça chez moi ...

**SpaceFrog** · 16/09/2010, 16h08

et accessoirement

Code :

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

1
2
 
ReREg != ReReg

**JLC83** · 16/09/2010, 16h46

Ici aussi tout va bien dans le simple test :

Code :

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

1
2
3
var reg=  /^[^<>*]+$/;
var chaine="mjmljmlkj"
alert(reg.test(chaine))

C'est lorsque nous bouclons les champs

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
onSoumet.prototype.fpm = function(){
        var that = this;
        var validons = true;
        var tb = [];
        tb[0] = 'Titre'; tb[1]='Nom' ;tb[2]='pre' ; tb[3] = 'ad1';
        tb[4] = 'ad2'; tb[5] = 'cp'; tb[6]= 'Ville';
        tb[7]='Tel1' ; tb[8]='Tel2' ; tb[9]='Tel3' ; tb[10] = 'Fax';
        tb[11]= 'Port1'; tb[12]= 'Port2'; tb[13]= 'email1'; tb[14]='email2' ;
        for(var i=0; i<15; i++){
            var champ = document.getElementById(tb[i]);
            if(champ != null){
                var item = document.getElementById(tb[i]).value;
                if((item != null)&&(item.length > 1)){
                    if(! that.valide(tb[i], item)){
                        alert(that.leMessage);
                        validons = false;
                        i = 11;
                    }
                }
            }
        }
        return validons;
    };