Correction d'un regex complexe

Bonjour!

Je souhaite obtenir un regex un peu particulier.
Il doit commencer par A ou B ou C, suivi de '(', une suite de chiffre séparé par des virgule et terminée par un ')'.

Voici le regex auquel je pense:

Code:

---

1 2	^(A|B|C){1} \\( ([0-9]+,?){1,12} \\)$

---

Le ^ de début et le $ de fin pour éviter les caractères non désirés.
Le '(A|B|C){1}' pour qu'il y ait juste une fois A ou B ou C
Le '([0-9]+,?){1,12}' pour avoir de 1 à 12 fois des chiffres avec ou non virgule.

Donc je veux que ça accepte par exemple: A(1) ou B(1, 2 , 55,3 )

Mais pour le moment je n'arrive à faire accepter aucune chaine à mon pattern :cry:

Quelqu'un pourrait m'aider?

Bonjour,

En général, pour des listes d'éléments, on utilise plutôt la forme :

Code:

---

X(, X)*

---

au lieu de

Code:

---

(X,?)+

---

qui accepte des éléments sans séparateur.

Sinon, je pencherais pour un problème d'espaces qui ne semblent pas gérés.

Peux-tu nous mettre un bout de code qui utilise ton expression et qui ne reconnaît pas une chaîne 'valide'?

Le code ressemble a ça:

Code:

---

1 2 3 4 5 6 7 8 9

Pattern p; Matcher m;   p = Pattern.compile("(A|B|C){1} \\( [0-9](, [0-9]){0,11} \\)"); m = p.matcher("A(1)");   if (!m.matches()){ ...

---

Avec ta modification ça va un peu mieux, ça accepte maintenant:
'A ( 1 )' ou 'A ( 1, 1 )' (en mettant bien les espaces de partout)

Reste donc à supprimer le fait que les espaces soient obligatoires.

Salut,

Citation:

---

Envoyé par nestra

Le '(A|B|C){1}' pour qu'il y ait juste une fois A ou B ou C

---

Ou plus simplement [ABC] ;)

Citation:

---

Envoyé par nestra

Le '([0-9]+,?){1,12}' pour avoir de 1 à 12 fois des chiffres avec ou non virgule.

---

Il serait plus propre d'utiliser quelques chose comme cela :

[0-9]+(,[0-9]+){0,11}

ou bien

\\d+(,\\d+){0,11}

:arrow: Un premier nombre obligatoire, suivi de 0 à 11 autres nombres précédé par une virgule

Citation:

---

Envoyé par nestra

Mais pour le moment je n'arrive à faire accepter aucune chaine à mon pattern :cry:

---

Cela vient des espaces. Si tu met un espace dans ton pattern, il devra obligatoirement être présent dans la chaine.

Pour garder une bonne lisibilité malgré tout, tu peux découper la chaine en sous-chaine :

Code:

---

1 2 3 4 5 6 7

String regexp = "^" // Début de chaine + "[ABC]" // A, B ou C + "\\(" // parenthèse ouvrante + "\\d+" // Un nombre + "(,\\d+){0,11}" // Une virgule suivi d'un nombre (0 à 11 fois) + "\\)" // parenthèse fermante + "$"; // fin de chaine

---

Mais ici les chaines ne doivent contenir aucun espace. Il faut donc rajouter des "\\s*" partout où il peut y avoir un espace ;)


a++

[edit] un peu grillé :aie:

Merci de votre aide!

En reprenant ma solution précédente, j'ai autorisé les espaces ou je le voulais avec [ ]* et ça devient:

Code:

---

1 2	"(A|B|C){1}[ ]*\\([ ]*[0-9]([ ]*,[ ]*[0-9]){0,11}[ ]*\\)"

---

Avec [ABC], n'accepterai-je pas aussi la chaine 'AB (1)'? ' (ce qui est une erreur)

Citation:

---

Envoyé par nestra

Avec [ABC], n'accepterai-je pas aussi la chaine 'AB (1)'? ' (ce qui est une erreur)

---

Non, c'est équivalent (et plus clair à mon avis), ça veut bien dire A ou B ou C (avec des 'ou' exclusifs).

Citation:

---

Envoyé par nestra

Avec [ABC], n'accepterai-je pas aussi la chaine 'AB (1)'? ' (ce qui est une erreur)

---

Non : [ABC] signifie A ou B ou C


a++