[Pattern] Problème de matching

Version imprimable

Bonjour à tous,

Quelqu'un pourrait-il me dire pourquoi ma String définie ci-dessous ne vérifie pas l'expression régulière ?

Je débute dans le domaine, donc désolé si ma question vous paraît stupide... :oops:

Code:

1
2
3
4
5
6
7
8
9
public static final String MY_REG_EX = "\"666\",\"[a-zA-Z0-9]{1,8}\",\"[0-9]{5}\",\"[CDEFRT]\",\".{0,12}\",\"-{0,1}(\\d{1,16})\\.(\\d{0,2})\",\"-{0,1}(\\d{1,16})\\.(\\d{0,2})\",\"-{0,1}(\\d{1,16})\\.(\\d{0,2})\"";
 
public static void main (String[] args){
    Pattern myPattern = Pattern.compile(MY_REG_EX);
    String myCSVEntry = String myCSVEntry = "\"666\",\"TOTO4321\",\"12345\",\"C\",\"JEAN-MARC1\",\"100000000\",\"123456789.12\",\"-12345678.91\"";
 
    // Devrait retourner true mais ce n'est pas le cas
    System.out.println("Result : " + myPattern.matcher(myCSVEntry).matches());
}

13/01/2010, 14h34
lucimed

Citation:

String myCSVEntry = "666","TOTO4321","12345","C","JEAN-MARC1","100000000","123456789.12","-12345678.91";

Ceci n'est pas un String mais un tableau de String...
13/01/2010, 16h25
Mildju

Euuuh, oui, je n'avais pas formaté la String correctement... J'ai fait la modif dans mon exemple.

C'est parce qu'en fait, dans la réalité, cette String est une entrée d'un fichier CSV (d'ou mon expression régulière).
13/01/2010, 16h40
Astartee

Si dans la chaîne que tu testes, tu remplaces "100000000" par "100000000.00", le test renverra bien true...
Maintenant, à toi de voir s'il faut revoir le format de l'entrée ou modifier ton expression régulière.
13/01/2010, 21h37
Mildju

Nom de Zeus, tu as tout à fait raison Astartee ! :ccool:

Disons que les entrées de mon fichier ne changeront pas et que c'est donc l'expression régulière que je vais devoir adapter.

Pour symboliser un nombre positif ou négatif ayant une partie entière de 1 à 16 chiffres avec ou sans partie décimale de 1 ou 2 chiffres, est-ce que l'expression suivante convient ?

Code:

-{0,1}(\\d{1,16})(\\.\\d{0,2})
14/01/2010, 10h58
Uther

Il y a une petite erreur car dans ton expression le point est nécessaire même pour un nombre entier.

On peut également simplifier un peu en remplaçant replacer "{0,1}" par "?". Enfin, les parenthèse autour de \\d{1,16} ne sont pas nécessaires sauf si tu veux récupérer la valeur de ce groupe.

La regexp suivante devrait convenir:

Code:

-?\\d{1,16}(\\.\\d{1,2})?
15/01/2010, 13h47
Mildju

@Uther : effectivement, tu as raison pour le point. Et merci pour l'amélioration de mon expression régulière. ;)

Affaire classée, merci à tous ! :mrgreen: