Soucis dans une expression simple

:salut:

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:

---

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 :weird:

Quel est le problème ?


:merci:

Code:

---

1 2 3 4

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

---

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

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

Code:

---

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

---

Citation:

---

Envoyé par SpaceFrog

Code:

---

1 2 3 4

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

---

[edit] 2!!! pas 0 :oops: [\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 :aie:

Citation:

---

Envoyé par Bovino

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

Code:

---

<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 :aie:

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:

---

1 2 3 4 5 6 7 8

function hasMultiComma(str) { var ret=false; if (str != "") { ret=(str.match(/,+/g) ||[,]).length==1; } return ret; }

---

Code:

---

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

---

est amplement suffisant ...