Discussion :

[Mildju]

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

Code :

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

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());
}

[lucimed]

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

Ceci n'est pas un String mais un tableau de String...

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

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

[Mildju]

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

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 :

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

-{0,1}(\\d{1,16})(\\.\\d{0,2})

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

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

-?\\d{1,16}(\\.\\d{1,2})?

[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 !