Discussion :

[laurent12345]

Bonjour à tous,

Je cherche à parser une chaine avec une regexp :

Code :

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

Regex reLastNumber = new Regex(@"CRYST1\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+C(\s+[\.0-9]+)+");

La chaine est soit la première, soit la seconde :

Code :

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

1
2
3
line = "CRYST1   60.009   61.046   62.671  90.00 201.60  80.00 C 1 2 1       4";
 
line = "CRYST1   60.135   61.902   60.865  90.00  102.41  90.00 C 2           4";

et je veux obtenir le dernier nombre (on peut voir que le nombre de nombre après le 'C' est variable).

ma regexp matche bien la chaine mais si je veux obtenir le dernier nombre par :

Code :

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

numberTmp = double.Parse(reLastNumber.Match(line).Groups[reLastNumber.Match(line).Groups.Count].Value);

çà plante car le dernier nombre est une chaine vide.

Auriez-vous une solution ?

Merci beaucoup par avance.

Laurent.

[BenoitM]

C'est quel nombre dans tes 2 cas que tu veux trouver?

[laurent12345]

BenoitM,

C'est le dernier nombre de la chaine que je veux trouver.

Laurent.

[toopac]

Si c'est le dernier nombre, pourquoi ne pas faire simplement ceci :

Code :

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

1
2
3
Regex reLastNumber = new Regex(@"\d+$");
 
double numberTmp = double.Parse(reLastNumber.Match(line).Value);

[BenoitM]

Methode je suis un bourin

Code :

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

1
2
string[] arr=String.Split(" ",line); 
double.Parse(arr[arr.Lenght-1]);

[laurent12345]

Merci BenoitM et toopac pour vos réponses,

En fait, les chaines que je parse sont contenues dans un fichier qui contient des milliers de lignes toutes différentes.

C'est pourquoi je suis obligé de sélectionner la ligne par ma regex :

Code :

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

1
2
 
Regex reLastNumber = new Regex(@"CRYST1\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+C(\s+[\.0-9]+)+");

Ce serait bien si je pouvais sélectionner la ligne et obtenir la dernière valeur de la ligne avec une seule regexp.

Laurent.

[toopac]

Si tu lis ton fichier ligne par ligne, je ne vois pas le problème
ex :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
string line;
Regex reLastNumber = new Regex(@"\d+$");
double numberTmp;
using (StreamReader reader = new StreamReader(@"C:\myfile.txt"))
{
    while ((line = reader.ReadLine()) != null)
    {
        numberTmp = double.Parse(reLastNumber.Match(line).Value);
        //[...]
    }
}

Sinon si tu lis le fichier d'un coup

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
string file;
using (StreamReader reader = new StreamReader(@"C:\myfile.txt"))
{
    file = reader.ReadToEnd();
}
 
Regex crystLine = new Regex(@"CRYST1\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+([\.0-9]+)\s+C(\s+[\.0-9]+)+");
Regex lastNumber = new Regex(@"\d+$");
 
MatchCollection matches = crystLine.Matches(file);
foreach (Match match in matches)
{
    numberTmp = double.Parse(lastNumber.Match(match.ToString()).Value);
    //[...]
}

[laurent12345]

toopac,

Je lis le fichier ligne par ligne.

Si tu lis ton fichier ligne par ligne, je ne vois pas le problème

Le problème est de repérer la ligne : "CRYST..."...

Je ne vois pas comment se passer de ma regexp...

Sinon ta solution avec les 2 regexp est efficace, je l'ai testée, elle marche (dans le cas d'une lecture ligne par ligne).

Mais c'est moins élégant que de faire une seule regexp ;-)

Laurent.

[laurent12345]

Ta réponse me fait revoir mes ambitions à la baisse : je pensais qu'on pouvait faire les deux.

En tout cas merci beaucoup pour ton aide et tes lumières sur les regexp !!

Je vais adopter ta solution.

Laurent.

[farfouch00]

Bonjour,

Tu peux vérifier que la ligne correspond à ta Regex avec un IsMatch, qui renvoi un booléen. Si c'est le cas, tu parses ta ligne de la fin et tu récupères le dernier chiffre...!

[laurent12345]

farfouch00,

Merci pour ton conseil.

C'est ce que je vais faire :

je recherche la ligne voulue et ensuite je recherche le dernier nombre de la ligne.

Laurent.

[toopac]

En fait j'ai lu ton premier post un peu en diagonale...
En fait t'avais presque la bonne réponse.
Le dernier élément d'une collection n'est pas tab[tab.Count], mais tab[tab.Count - 1].
Du coup numberTmp = double.Parse(reLastNumber.Match(line).Groups[reLastNumber.Match(line).Groups.Count - 1].Value); devrait fonctionner.

[laurent12345]

En fait,

les reLastNumber.Match(line).Groups[x].Value

renvoyaient des valeurs fausses pour x > 6 (après le 'C') ce qui prouvait que ma regexp était correcte pour matcher la ligne mais pas pour récupérer les valeurs...

Donc je me suis rabattu sur ta solution des 2 regexp et çà marche, donc c'est déjà bien.

Mais c'est sur, si quelqu'un a une solution avec 1 seule regexp : je suis preneur !

Laurent.