Regex Split fichier .txt pour remplir un Datatable

**pouletbiencuit** · 20/04/2018, 22h36

Nom : O0rbE.png
Affichages : 206
Taille : 2,8 Ko

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[] myenum = Enum.GetNames(typeof(allenum));
DataTable dt = new DataTable();
foreach (string enum in myenum)
{
    string[] lines = File.ReadAllLines(files);
    foreach (var line in lines)
    {
        string[] items = line.Split(new char[] {'|'}, StringSplitOptions.RemoveEmptyEntries);
        DataRow row = dt.NewRow();
        if (!line.Contains(enum)) continue; 
            row[items[2]] = items[3];
        dt.Rows.Add(row);
    }
}
dgVResult.DataSource = dt;

ce que j'obtiens avec le code ci dessus :

Nom : byMri.png
Affichages : 222
Taille : 3,5 Ko

**tsuji** · 22/04/2018, 14h31

Le raisonnement suggère que la ligne

Code c# :

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

DataRow row=dt.NewRow();

doit être placée devant le boucle foreach (string enum in myenum) et après la ligne DataTable dt=new DataTable(); bien entendu; et que la ligne

Code c# :

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

dt.Rows.Add(row);

doit être placée après le boucle foreach (string enum in myenum) et devant la ligne dgVResult.DataSource=dt. Voilà !

**pouletbiencuit** · 22/04/2018, 15h21

merci ça fonctionne !

juste une chose, est il possible de garder cette méthode pour plusieurs fichiers ?
pour un fichier ça fonctionne parfaitement.

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
16
17
18
19
20
21
22
23
24
25
26
27
public void trifichiers(string[] files)
        {
            string[] myenum = Enum.GetNames(typeof(allenum));
            DataTable dt = new DataTable();
 
            DataRow row = dt.NewRow();
 
            foreach (string enum in myenum)
            {
                dt.Columns.Add(enum); // je crée les en-tetes à l'aide de l'enum
                foreach (string file in files)
                {
                    string[] lines = File.ReadAllLines(file);
                    foreach (var line in lines)
                    {
                        char[] delimiter = { '|' };
                        string[] elements = line.Split(delimiter, StringSplitOptions.RemoveEmptyEntries);
 
                        if (!line.Contains(enum)) continue;
 
                        row[elements[2]] = elements[3];
                    }
                }
            }
            dt.Rows.Add((row));
            dgVResult.DataSource = dt;
        }

**tsuji** · 22/04/2018, 19h36

juste une chose, est il possible de garder cette méthode pour plusieurs fichiers ?

Oui, mais l'esquisse montrée n'est pas correcte. Je vais la réarranger pour que la logique se soit conduite correctement. Mais il y a des soucis que je ne sens pas confortable.
[1] enum est un mot-clé, je pense il ne faut pas se l'employé comme le nom d'une variable.
[2] Les données délimitées par |, soit, ok, comment vous voulez. Mais les espaces devant et après sont en danger de cassé en matchant le nom des colonnes... Peut-être vous avez arrangé bien ça, mais c'est assez hasardeux.
[3] Le critère !line.Contains(enum) est presque insensé; si ça ne casse pas, c'est par une bonne chance. Si une donnée de nom de la colonne était "paselement1", ça va conduire la fonction d'ajouter les données au DataRow et ça va résulter une erreur. On peut imaginer beaucoup de cas pour casser la fonctionnalité. Non, c'est bien fragile. Il faut absolument l'améliorer fortement après.
[4] Il n'y a pas de raison a priori que le fichier nécessairement donne au moins un data matché le nom de colonne. Pour les cas comme ça, il ne faut pas faire exécuter la ligne dt.Rows.Add(row). Je mets un critère concis pour ça mais vous devrez ajouter using System.Linq pour le faire marcher. On peut facilement imaginer les autres façons de faire la même, mais probablement en plus de lignes que montrée. J'y mets cette façon de faire pour m'amuser, c'est tout.

Voilà ce qu'il faut faire en réarranger les lignes. (Vous remettez le style que vous préférez, bien sûr.)

Code c# :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
public void trifichiers(string[] files) {
    string[] myenum = Enum.GetNames(typeof(allenum));
    DataTable dt = new DataTable();
    DataTable dt = new DataTable();
    foreach (string senum in myenum) {    //renommé senum
        dt.Columns.Add(senum);
    }
    foreach (String file in files) {
        DataRow row = dt.NewRow();
        string[] lines = File.ReadAllLines(file);
        foreach (var line in lines) {
            char[] delimiter = { '|' };
            string[] elements = line.Split(delimiter, StringSplitOptions.RemoveEmptyEntries);
            foreach (string senum in myenum) {    //même note
            if (!line.Contains(senum)) continue;
            row[elements[2].Trim()] = elements[3].Trim();
        }
    }
 
    if (!row.ItemArray.All(i => i is DBNull)) {    //ajouter using System.Linq;
        dt.Rows.Add(row);
    }
 
    //etc quelque travail pas montré avec dt comme on veut...
 
}

**pouletbiencuit** · 22/04/2018, 19h54

Merci ! effectivement avant le traitement je fait appel à une fonction permettant de supprimé tout les espaces contenus dans les fichiers à l'aide du regex

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        public void DeleteSpace(string file)
        {
            string[] lines = File.ReadAllLines(file);
            using (StreamWriter sw = new StreamWriter(file))
            {
                foreach (string line in lines)
                {
                    if (!string.IsNullOrEmpty(line) && line.Length > 1)
                    {
                        sw.WriteLine(Regex.Replace(line, @"\s+", ""));
                    }
                }
            }
        }

Je vais essayer de rendre le code plus costaud pour éviter tout problèmes
Merci encore

Regex Split fichier .txt pour remplir un Datatable

C#

Vue hybride

Discussions similaires

Partager

Partager