PRoblème de lecture et récupération de données dans un fichier texte

Bonjour à tous,
j'ai un petit soucis dans mon programme que j'ai débuté.
Le but est le suivant :
en entrée je sélectionne un fichier texte en sortie je dois récupérer un fichier excel.

Dans un premier temps, je choisi mon fichier texte, puis quand je clique sur le bouton "Traitement", je parcours mon fichier texte, voici un exemple de mon code :

Code:

---

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 28 29 30 31 32 33 34 35 36 37 38

private void btn_start_ficExcel_Click(object sender, EventArgs e) { //1 selection du fichier en entrée     string FileToRead = txtbox_entree.Text; //txtbox_entree.Text correspond au chemin de mon fichier texte   string[] lines = File.ReadAllLines(FileToRead); //lecture du fichier   int lenTab = 0; //recuperer la longueur du tableau pour les sirens, c'est à dire le nombre de siren dans mon fichier texte. /* j'ai mis en pièce jointe un échantillon pour illustrer mon exemple, dans mon fichier texte, le premier numéro de siren se trouve à la ligne 3, donc dans mon tableau c'est l'index 2. Le siren suivant ( siren n°2) se trouve à la ligne 17 Dans un premier temps, dans mon code ci dessous, je lis mon fichier et je compte le nombre de siren, en incrémentant mon compteur de 13 */ for (int i = 0; i <= File.ReadAllLines(FileToRead).Length; i += 14) { lenTab++; }   string[] Tabsiren = new string[lenTab]; //j'initialise mon tableau   //récupération des sirens   //ici j'essaye de parcourir les lignes et dans tabsiren, je souhaite avoir toute la liste des sirens présente dans mon fichier, donc à la ligne 3, 17, 31 etc.. // mais je n'y arrive pas, soit j'ai des cases vide soit il me prend toutes les données de chaque lignes. for (int j=0; j<= File.ReadAllLines(FileToRead).Length;j++) { // for (int k = 2; k <= File.ReadAllLines(FileToRead).Length; k+=14) // { Tabsiren[j] = lines[j].Substring(6, 9);   // } }   }

---

le principe est qu'une fois que j'ai récupéré la liste des sirens dans un tableau, j'initialise la connexion à la base de données, puis je ferais une boucle, avec un select pour récupérer toutes les informations que je souhaite en fonction du sirend et ensuite je constituerais un fichier excel.

Merci pour votre aide .

Si tu sais que le premier SIREN est sur la ligne 3 (indice 2) et que les autres sont toutes les 14 lignes, pourquoi commencer à 0 ?
Et Pourquoi lire deux fois le fichier ?

Bonjour,
depuis hier j'ai modifié mon codage, dont voici la descritpion :

Code:

---

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 28 29 30 31 32 33 34

string[] lines = File.ReadAllLines(FileToRead); //lecture du fichier   int lenTab = 0; int compteur = 0;   //recuperer la longueur du tableau pour les siren for (int i = 0; i <= File.ReadAllLines(FileToRead).Length; i++) { lenTab++; }   //lenTab =43   string[] Tabsiren = new string[lenTab];   MessageBox.Show("lenTab: " + lenTab.ToString()); //lenTab +43 MessageBox.Show("Nombre de ligne du fichier: " + File.ReadAllLines(FileToRead).Length.ToString()); //NB DE LIGNE 42   try { for (int j = 0; j <= lenTab; j++) {   Tabsiren[compteur] = lines[j].Substring(6, 9); compteur = compteur + 1;   } } catch (Exception monerreur) {   MessageBox.Show(monerreur.ToString()); }

---

pour répondre à votre question, je ne commence pas à 2 car l'index 0 et 1 seront à null.
En modificant mon code, je parcours chaque ligne de mon fichier et j'enregistre chaque ligne dans mon tableau, comme je sais que chaque siren se trouvent toutes les 14 lignes.

Ensuite , je créé un autre tableau dans lequel je vais récupérer que les sirens se trouvant dans mon 1er tableau (tabsiren)

Code:

---

1 2 3 4 5 6 7 8 9 10 11

string[] ListeSiren = new string[lenTab]; int compteurTab = 0;   for (int i = 2; i <= Tabsiren.Length; i++) //je commence à l'index 2, comme je sais que le 1er siren se trouve à l'index 2 de mon 1er tableau et j'incremente mon compteur i de 1 { ListeSiren[compteurTab] = Tabsiren[i]; i = i + 13; compteurTab = compteurTab + 1; }

---

Vous trouverez en pièce jointe le tableau que j'obtiens " tableau listsiren" dans lequel j'ai mes 3 sirens se trouvant à l'index 0, 1 et 2 ( mon fichire test disposant de 42 ligne).
Les fichiers finaux disposeront d'environ 600 ou 700 lignes, là j'essaye de trouver le moyen de faire en sorte que le tableau listSiren ne contiennent que les siren présent dans le fichier et soit de la bonne taille, pour éviter d'avoir des valeurs nulls.
Contrairement à mon premier poste je pense que là ça doit être plus optimisé car je ne le lis qu'une fois le fichier.


Merci

Ceci est inutile car lenTab sera égal à File.ReadAllLines(FileToRead).Length, sans compter que là aussi tu relis le fichier alors que as déjà son contenu dans lines.

Citation:

---

Code:

---

1 2 3 4

for (int i = 0; i <= File.ReadAllLines(FileToRead).Length; i++) { lenTab++; }

---

---

C'est non seulement inutile et contre-productif. Mais c'est également faux puisque tu n'as pas 43 SIREN mais seulement 3

Citation:

---

Code:

---

1 2	//lenTab =43 string[] Tabsiren = new string[lenTab];

---

---

Ce qui explique pourquoi tu te retrouves avec 40 valeurs nulles dans ton tableau final.
Tu plantes parce que tu lis les lignes de 0 à 43 (à cause du inférieur ou égal) alors que le dernier indice est à 42.

Donc là, tu lis le fichier 2 fois au lieu de 3 (j'en avais oublié une tout à l'heure) et en plus tu boucle dessus 2 fois.
Tu as donc 4 itérations (au lieu de 6 comme tout à l'heure) alors qu'une seule suffit.

Donc pour être efficace.
1. Déclarer un tableau de longueur 3 (nombre de lignes de ton fichier (43) divisé par 14), ou directement une List<String> comme ça tu n'a pas à te préoccuper de sa taille.
2. Faire une seule boucle qui commence à partir de 2 jusqu'à la fin avec un pas de 14
3. Et dans cette unique boucle extraire les SIREN

Code:

---

1 2 3 4 5 6

String[] lines = File.ReadAllLines("data.txt"); List<String> sirens = new List<String>(); for (Int32 index = 2; index < lines.Count(); index+=14) { sirens.Add(lines[index].Substring(6, 9)); }

---

bonjour Popo,

merci pour ton aide, il est génial ton code, je comprend mieux .
J'ai tenté de m'inspiré de List<string> mais je n'ai jamais pratiqué.

merci *

La notation List<T> où T est un objet, ici un String est ce qu'on appelle une liste générique.
C'est en quelque sorte un tableau avec des fonctionnalités plus avancées et gérées nativement par le Framework.

Les listes génériques ont plein de mécanismes qui font le gros du travail à ta place et facilitent l'écriture et la lecture du code.
Parmi, les plus utiles :
- Pas de besoin de gérer un compteur,
- L'ajout se fait avec la méthode Append, la méthode va ajuster le compteur pour toi et ajouter l'élément à la fin
- L'insertion à un emplacement particulier se fait avec la méthode Insert, la méthode va ajuster le compteur et décaler les enregistrements pour toi
- Idem lors de la suppression, le décalage et le compteur sont adaptés automatiquement.
- Le parcours de TOUS les éléments est simplifié par l'utilisation d'une boucle foreach, (pas d'indice à gérer)

Si ton problème est résolu, n'oublie pas de cliquer sur le bouton de même nom pour l'indiquer.

J'en prends bonne note, merci.

bonjour,

j'ai un soucis dans mon programme, il refuse de tourné quand le fichire renseigné est volumineux ( environ 50k de ligne).

Suite à tes recommandations, j'ai tenté de modifier mon code, en y ajoutant des listString, voici l'extrait de code

Code:

---

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 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260

namespace Suivi_tese { public partial class Form1 : Form { public Form1() { InitializeComponent();     }   private FolderBrowserDialog folderBrowserDialog2;   private bool selfichier; private bool selrepertoire; private static string repexc = "";   private static MySqlConnection conn; private static MySqlDataReader monreader = null; protected Range Cellules; protected Range UneCellule;   List<string> numcode = new List<string>(); List<string> matadh = new List<string>(); List<string> solmon = new List<string>();   List<string> soldat = new List<string>(); List<string> adhfer = new List<string>(); List<string> raison = new List<string>(); List<string> milib = new List<string>(); List<string> emibas = new List<string>(); List<string> emicoe = new List<string>();   List<string> debdat = new List<string>(); List<string> findat = new List<string>(); List<string> datrec = new List<string>();   public string ReturnReaderValue(MySqlDataReader reader, int indexe) { if (!reader.IsDBNull(indexe)) { return reader.GetString(indexe); } else { return ""; } } private void btn_start_ficExcel_Click_1(object sender, EventArgs e) { BasicConfigurator.Configure();   string path = @"C:\Users\.........\test\log.txt"; log.Info("--------------------"); log.Info("Debut de traitement"); log.Info("--------------------");   log.Info("Chemin du fichier pour le log " + path);     if (selrepertoire == true & selfichier == true) { log.Info("Selection OK"); int compteur = 0; // LBLRES.Text = compteur.ToString();   // Set Minimum to 1 represente la 1ere ligne mabarre.Visible = true;     string FileToRead = txtbox_entree.Text; log.Info("Fichier dub choisi" + txtbox_entree.Text);     /*   RECUPERATION DES SIRENS   */ #region Recuperation_sirens int moncompteur = 0; String[] lines2 = File.ReadAllLines(FileToRead); List<String> sirens = new List<String>(); for (Int32 index = 2; index < lines2.Count(); index += 14) { if (lines2[index].Substring(0, 5) == "Info1") { sirens.Add(lines2[index].Substring(6, 14)); moncompteur++; }   } log.Info("Mon compteur :" + moncompteur); // Set Maximum - correspond au nombre de ligne du fichier mabarre.Maximum = sirens.Count; // Set the initial value of the ProgressBar. mabarre.Value = 1;   #endregion   /*   Connexion à la base mysql   */ string server = "server"; string database = "mabse"; string user = "monuser"; string password = "monpass"; string port = "2222";   string connString = "Server=" + server + "; Port=" + port + ";Database=" + database + ";Uid=" + user + ";Pwd=" + password + ";SSL Mode=None;default command timeout=0"; string marequete = "";   conn = new MySqlConnection(connString); try { conn.Open();   /*   CREATION ET OUVERTURE DU FICHIER EXCEL   */ repexc = folderBrowserDialog2.SelectedPath + "\\dossier" + DateTime.Now.ToString("yyyyMMdd_HHmmss") + ".xlsx"; Microsoft.Office.Interop.Excel.Application xla = new Microsoft.Office.Interop.Excel.Application(); xla.Visible = false; Workbook wb = xla.Workbooks.Add(XlSheetType.xlWorksheet); //WorksheetCollection worksheets = xla.Worksheets; Worksheet ws = (Worksheet)xla.ActiveSheet;   ws.Cells[1, 1] = "colonne 1"; ws.Cells[1, 2] = " colonne 2"; ws.Cells[1, 3] = " colonne 3"; ws.Cells[1, 4] = " colonne 4"; ws.Cells[1, 5] = "colonne 5"; ws.Cells[1, 6] = "colonne 6"; ws.Cells[1, 7] = "colonne 7"; ws.Cells[1, 8] = "colonne 8"; ws.Cells[1, 9] = "colonne 9"; ws.Cells[1, 10] = "colonne 10"; ws.Cells[1, 11] = "colonne 11"; ws.Cells[1, 12] = "colonne 12"; ws.Cells[1, 13] = "colonne 13"; UneCellule = ws.get_Range("A1", "M1"); ws.Columns.Font.Name = "MS Sans Serif"; ws.Columns.Font.Size = 12;     int cptCol = 2; /*   Parcours de la base de données et récupération du résultat   */   int compteurAnnexe = 0;   for (int a = 0; a < sirens.Count; a++) {   marequete = " select t0.tesccpcod, " + "t0.numero, " + "t0.identifiant, " + "t0.xxxxx, " + "t0.xxxxx, " + "t0.xxxxx, " + "t1.xxxxx," + "t2.xxxxx," + "t2.xxxxxx," + "t2.xxxxxx," + "t1.xxxxxx," + "t1.xxxxxx," + "t1.xxxxxx " + "from tes000pf t0 " + "left join tes001pf t1 on t1.ccccc = t0.ccccc " + "left join tes002pf t2 on t2.ccccc = t0.ccccc " + "left join tes003pf t3 on t3.ccccc = t0.ccccc " + "left join tes004pf t4 on t4.ccccc = t0.ccccc " + "where t0.numero ='" + sirens[a] + "'";   log.Info("Ma requete utilisé:" + marequete); log.Info("sirens :" + sirens[a]);   MySqlCommand cmd = new MySqlCommand(marequete, conn); monreader = cmd.ExecuteReader(); if (monreader.Read()) //while { //recuperation des données numcode.Add(ReturnReaderValue(monreader, 0)); //code matadh.Add(ReturnReaderValue(monreader, 1)); //siret solmon.Add(ReturnReaderValue(monreader, 2)); // matadh soldat.Add(ReturnReaderValue(monreader, 3)); adhfer.Add(ReturnReaderValue(monreader, 4)); raison.Add(ReturnReaderValue(monreader, 5)); milib.Add(ReturnReaderValue(monreader, 6)); emibas.Add(ReturnReaderValue(monreader, 7)); emicoe.Add(ReturnReaderValue(monreader, 8)); debdat.Add(ReturnReaderValue(monreader, 9)); findat.Add(ReturnReaderValue(monreader, 10)); datrec.Add(ReturnReaderValue(monreader, 11));   //enregistrement dans mon fichier excel ws.Cells[cptCol, 1] = numcode[compteurAnnexe]; ws.Cells[cptCol, 2] = matadh[compteurAnnexe]; ws.Cells[cptCol, 3] = solmon[compteurAnnexe]; ws.Cells[cptCol, 4] = soldat[compteurAnnexe]; ws.Cells[cptCol, 5] = adhfer[compteurAnnexe]; ws.Cells[cptCol, 6] = raison[compteurAnnexe]; ws.Cells[cptCol, 7] = milib[compteurAnnexe]; ws.Cells[cptCol, 8] = emibas[compteurAnnexe]; ws.Cells[cptCol, 9] = emicoe[compteurAnnexe]; ws.Cells[cptCol, 10] = debdat[compteurAnnexe]; ws.Cells[cptCol, 11] = findat[compteurAnnexe]; ws.Cells[cptCol, 12] = datrec[compteurAnnexe];   compteurAnnexe++; cptCol++; mabarre.PerformStep();   } else { ws.Cells[cptCol, 2] = sirens[a]; cptCol++; }   mabarre.PerformStep(); monreader.Close();   compteur++; LBLRES.Text = compteur.ToString();   }//fin du for   log.Info("********* FIn de la boucle - sauvegarde du fichier excel ********** " ); ws.SaveAs(repexc, Missing.Value, Missing.Value, Missing.Value, Missing.Value, Missing.Value, Missing.Value, Missing.Value, Missing.Value, Missing.Value); xla.Quit();     MessageBox.Show("Fin du traitement", "Message d'information", MessageBoxButtons.OK, MessageBoxIcon.Information); } catch (MySqlException erreur) { log.Info("******** ERREUR DANS LE TRAITEMENT :" + erreur.ToString()); MessageBox.Show(erreur.ToString());   conn.Close(); } log.Info("----------- FIN DE TRAITEMENT -----------"); } else { log.Info("Fin du traitement du traitement - KO ");     log.Error("AUcun Fichier ou dossier selectoinné"); MessageBox.Show("Merci de choisir un fichier et un répretoire de sortie", "Message d'information", MessageBoxButtons.OK, MessageBoxIcon.Exclamation); } // monstream.Close(); } }//fin méthode public parital class       }//fin du namespace

---

avec un fichire "normale" les résultats remontés dans mon fichier excel sont correct, mais pourtant je ne comprend pas pourquoi il refuse de tourné.
Logiquement il devrait tourner car meme s'il fait par exempe 1000 ligne, je ne peux qu'avoir que 50 sirens ce qui n'est rien en soit et donc mon programme devrait être fluide ....

Merci pour tes conseils .