Discussion :

[nestra]

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 :

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

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

Quelqu'un pourrait m'aider?

[Deaf]

Bonjour,

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

Code :

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

X(, X)*

au lieu de

Code :

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

(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'?

[nestra]

Le code ressemble a ça:

Code :

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

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.

[adiGuba]

Salut,

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

Ou plus simplement [ABC]

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}

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

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

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 :

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

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é

[nestra]

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 :

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

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)

[Deaf]

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).

[adiGuba]

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++