Discussion :

[Franck.H]

Je n'ai pas vraiment l'habitude d'utiliser les expressions régulières et j'ai donc un problème, voici ma fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
        function hasMultiComma(str) {
            if (str != "") {
                var reg = new RegExp(",{1,}", "ig");
 
                ret = str.search(reg);
                if (ret > 1) {
                    return true;
                }
                else {
                    return false;
                }
            }
        }

Je dois vérifier s'il y a plus d'un caractère "," dans la chaîne passée en argument. Il s'agit en fait de valeur numérique récupérées de champs de saisie et comme il se doit, il ne peut y avoir qu'une seule virgule. Le seul hic c'est que ça marche mais pas toujours. Si j'ai une chaine comme par exemple "56, 5886, 25" il me retrouve bien deux virgules mais par exemple avec ceci "1,1,1" il ne m'en trouve qu'une seule

Quel est le problème ?

[SpaceFrog]

Code :

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

1
2
3
4
   function hasMultiComma(str) {
            if (str != "") {
               return str.split(',').length>2
        }

[edit] 2!!! pas 0 [\edit]

[Bovino]

T'aurais pas plutôt intérêt à utiliser l'attribut pattern ?

Code html :

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

<input name="float" pattern="\d+([,.]\d+)?" placeholder="Pattern" />

[Franck.H]

Code :

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

1
2
3
4
   function hasMultiComma(str) {
            if (str != "") {
               return str.split(',').length>2
        }

[edit] 2!!! pas 0 [\edit]

Oui c'est une solution possible et qui à la mérite de fonctionner, merci bien, j'ai cherché peut-être trop compliqué, je suis pas encore très à l'aise avec le JavaScript

T'aurais pas plutôt intérêt à utiliser l'attribut pattern ?

Code html :

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

<input name="float" pattern="\d+([,.]\d+)?" placeholder="Pattern" />

En fait je ne peut pas, en tous cas pas de cette manière car les champs sont créés dynamiquement par rapport à certains paramètres et dans un code VB qui est ensuite affiché sur une page aspx

[Lakhdr]

bonjour!
Votre methode cherche le premier indexe de (,) ou(,,) ou(,,....) dans la chaine passee en parametre , donc c est normal que dans "1,1,1" ca vous renvoie "1" trouve le (,) a la position "1" en partant de zero .
je vous propose une methode comme ca :

Code :

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

1
2
3
4
5
6
7
8
 
function hasMultiComma(str) {
           var ret=false;
          if (str != "") {
                ret=(str.match(/,+/g) ||[,]).length==1;
            }
          return ret;
        }

[SpaceFrog]

Code :

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

  ret=chaine.match(/,+/g).length==1

est amplement suffisant ...