Discussion :

[RedShift2]

Bonjour à tous,

Voila, j'ai le code suivant :

Code :

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

1
2
3
if (document.post.message.value.length < 5) {  
      formErrors = "{L_EMPTY_MESSAGE}";  
   }

qui sert à tester si le message que l'utilisateur veut poster est suffisament long et je voudrais l'améliorer avec une expression régulière pour que ça teste si le message posté est un post monosmilesque.
Alors, j'ai trouvé l'expression régulière sans trop de problème (elle peut être améliorée mais c'est pas ça qui m'intéresse ici) et donc j'ai écris le code suivant qui ne marche pas et qui fait même planter le code précédent :

Code :

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

1
2
3
4
var re = new RegExp(^:[a-zA-Z0-9]{3,9}:); 
    if (document.post.message.value.length < 2 || document.post.message.value.match(re)) { 
        formErrors = "{L_EMPTY_MESSAGE}"; 
    }

Quelqu'un a une idée de ce qu'il faut changer pour que ça marche ?

[Huntress]

Bienvenue sur developpez.net

Pas de chance tu tombes sur moi, j'suis pas très douée en JS, pis en plus ça fait longtemps que j'en ai pas fait (sérieusement) ...
Ca serait pas tout simplement un soucis de parenthèses au niveau de ta condition... genre il en manque ?

Toutes mes excuses si c'est pas ça, ça m'apprendra à venir sur le forum JS.

[javatwister]

en fait, si tu utilises cette syntaxe RegExp, place le masque entre guillemets;

ou alors, passe-toi de la déclaration de variable et définis l'expression directement en argument de match;

de plus, pourquoi commences-tu par ^? tu analyses uniquement le début de chaîne?
tu veux un seul smiley par message? alors pourquoi utiliser match, qui renverra un résultat dès la première correspondance?

[RedShift2]

Merci à vous deux pour votre aide.

Ca marche, voilà, c'était tout con, il fallait mettre l'expression régulière entre guillemets. Comme je m'étais basé sur un script d'exemple où l'expression régulière était récupérée dans un formulaire, les guillemets étaient absents. Ca m'avait induit en erreur.
Pour l'expression régulière en elle-même, je vais la retravailler mais c'est pas ça qui me causait souci.
Encore merci
A+