afficher dans un DataGridView à partir d'une classe Fermé

Question

Bonjour,  je me permet de poser ma question car j'ai une classe qui me permet de récupérer des valeurs à partir d'un fichier texte, j'aimerai pouvoir les renvoyer sur ma form d'origine qui elle contient mon DataGrid et pouvoir justement les afficher dans le DataGrid, j'aimerais aussi savoir si le DataGrid me permettais d'afficher une grande quantité de valeurs. 
Merci à vous ^^



Configuration: Windows / Chrome 80.0.3987.149

Utilisateur anonyme · Answer

Bonjour

Attention un Datagrid et un Datagridview sont 2 contrôles différents.

Commence par là
https://codes-sources.commentcamarche.net/faq/1291-utilisation-du-binding-au-travers-de-l-objet-databindingsource

Melman · Answer

C'est bien un DataGridView je vais suivre le tuto je me tournerais vers toi si cela ne fonctionne pas, Merci !

Melman · Answer

J'ai pas trouver la réponse à ma question mon datagridview est bien bind à ma classe je retourne bien les valeurs après avoir trier mais je sais pas si je les affiches correctement
string result;            foreach (string chemin in cheminFichierOk)                i++;            for (int j = 0; j < i; j++)            {                IEnumerable<string> lines = File.ReadLines(cheminFichierOk[j]);                result = Path.GetFileName(cheminFichierOk[j]);                foreach (var item in lines)                {                    LogParse leLog = LogParse.Parse(item, result);                    lesLogs.Add(leLog);                }            }
je pense que mon soucis vien d'ici, comment je peux afficher à partir d'une list ?

Melman · Answer

Aucune des deux méthodes fonctionne je dois mal m'y prendre mais mon DataGridView est bind à ma classe qui me permet de trier et quand j'ai finis le trie je lui retourne les valeurs en fonction des colonnes

return new LogParse
                    {
                        Datation = date,
                        Identificateur = "Nom",
                        Module = ligneArray[1],
                        Type = ligneArray[2],
                        Info = ligneArray[3],
                        Details = ligneArray[4],
                    };

Utilisateur anonyme · Answer

Bonjour, 

ce code est-il dans le même formulaire que le datagridview?

string result;
foreach (string chemin in cheminFichierOk)
i++;
for (int j = 0; j < i; j++)
{
  IEnumerable<string> lines = File.ReadLines(cheminFichierOk[j]);
  result = Path.GetFileName(cheminFichierOk[j]);
  foreach (var item in lines)
  {
    LogParse leLog = LogParse.Parse(item, result);
    lesLogs.Add(leLog);
  }
}

Utilisateur anonyme · Answer

Et après l’exécution de ce code, y a t il truc du genre
MonDatagridview.DataSource = lesLogs; 
Ou
LogParseBindingSource.DataSource = lesLogs; 
?

Melman · Answer

Oui je passe par la première option,
dtgLog.DataSource = lesLogs;
je sais pas si c'est important mais je passe par un BackGroundWorker

Utilisateur anonyme · Answer

Ha oui, y'a des chances.
Je ne sais pas quelle est le type de ta collection LesLogs, dans un premier temps essaye de mettre ObservableCollection<LogParse> (t'auras peut être un using à ajouter).

Si ne marche pas, il faudra que tu montres plus de code

Melman · Answer

C'est une list, le Observable n'a rien donner du coup voici ma form, namespace AnalyseTool { public partial class AnalyseForm : Form { public string dateDebut; public string dateFin; private string lcPath = ConfigurationManager.AppSettings["LogisticCorePath"]; private string TraficPath = ConfigurationManager.AppSettings["TraficPath"]; public AnalyseForm() { InitializeComponent(); } private void AnalyseForm_Load(object sender, EventArgs e) { } private List lesLogs = new List(); public void ChargerFichier(int i, List cheminFichierOk) { string result; foreach (string chemin in cheminFichierOk) i++; for (int j = 0; j < i; j++) { IEnumerable lines = File.ReadLines(cheminFichierOk[j]); result = Path.GetFileName(cheminFichierOk[j]); foreach (var item in lines) { LogParse leLog = LogParse.Parse(item, result); lesLogs.Add(leLog); } } lesLogs = lesLogs.OrderBy(x => x.Datation).ToList(); dtgLog.DataSource = lesLogs; } public void dtpDebut_ValueChanged(object sender, EventArgs e) { } private void btnType_Click(object sender, EventArgs e) { Type type = new Type(); type.ShowDialog(); } private void btnExport_Click(object sender, EventArgs e) { Export export = new Export(); export.ShowDialog(); } private void btnPlay_Click(object sender, EventArgs e) { if (!backgroundWorker1.IsBusy) { backgroundWorker1.RunWorkerAsync(); } } private void dtpFin_ValueChanged_1(object sender, EventArgs e) { } private void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e) { } private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e) { } private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e) { DateTime dateDebut = this.dtpDebut.Value, dateFin = this.dtpFin.Value; int i = 0; List cheminFichier; AnalyseLog analyseLog = new AnalyseLog(); cheminFichier = analyseLog.IntervaleFichierGlobal(lcPath, dateDebut, dateFin); analyseLog.ChargerFichier(i, cheminFichier); cheminFichier = analyseLog.IntervaleFichierGlobal(TraficPath, dateDebut, dateFin); analyseLog.ChargerFichier(i, cheminFichier); } public void DateTimePicher_ValueChanged(object sender, EventArgs e) { } private void lbxAnalyse_SelectedIndexChanged(object sender, EventArgs e) { } } } je pense qu'il ya de mauvaise pratique si tu en vois n'hésite pas à me corriger pour que je puisse m'améliorer ^^ ici ma classe pour les chemins public class AnalyseLog : AnalyseForm { private AnalyseForm analyseForm = new AnalyseForm(); private List cheminFichierOk = new List(); public List IntervaleFichierGlobal(string path, DateTime dateDebut, DateTime dateFin) { string[] fichiers = Directory.GetFiles(path, "*.log", SearchOption.TopDirectoryOnly); DateTime[] creationTimes = new DateTime[fichiers.Length]; for (int i = 0; i < fichiers.Length; i++) { creationTimes[i] = new FileInfo(fichiers[i]).CreationTime; if (creationTimes[i] >= dateDebut && creationTimes[i] <= dateFin) { cheminFichierOk.Add(fichiers[i].ToString()); } } return cheminFichierOk; } }

Melman · Answer

ici je fait le trie dans mes fichiers internal class LogParse { public DateTime Datation { get; set; } public string Identificateur { get; set; } public string Module { get; set; } public string Type { get; set; } public string Info { get; set; } public string Details { get; set; } public static LogParse Parse(string lignes, string nomFichier) { if (nomFichier.StartsWith("Logis")) { string[] ligneArray = lignes.Split(' '); //sépare les chaînes à partir Match m = Regex.Match(lignes, @"(?\d\d?)?/(?\d\d?)?/(?\d{4})?\s+(?\d\d?)?:(?\d\d?)?:(?\d\d?)?\s+(?[AP]M)?:(?\d{1,3})?"); if (m.Success) { int jour = int.Parse(m.Groups["jour"].Value); int mois = int.Parse(m.Groups["mois"].Value); int annee = int.Parse(m.Groups["annee"].Value); int heure = int.Parse(m.Groups["heure"].Value); int minute = int.Parse(m.Groups["minute"].Value); int secondes = int.Parse(m.Groups["seconde"].Value); string MS = m.Groups["ms"].Value; int ms = 0; if (!string.IsNullOrWhiteSpace(MS)) { ms = int.Parse(MS); if (MS.Length == 1) ms *= 100; else if (MS.Length == 2) ms *= 10; } DateTime date = new DateTime(annee, mois, jour, heure, minute, secondes, ms); if (m.Groups["ampm"].Value == "PM") date = date.AddHours(12); return new LogParse { Datation = date, Identificateur = "Nom", Module = ligneArray[1], Type = ligneArray[2], Info = ligneArray[3], Details = ligneArray[4], }; } return null; }

Utilisateur anonyme · Answer

Je ne fais plus de winform depuis longtemps, et à l'époque je ne me servais pas des observablecollection.
Ça marche très bien en WPF, faut croire qu'en Winform non.

Il va me falloir un peu de temps pour étudier ton code et je ne peux pas t'en accorder avant le début de soirée.

Utilisateur anonyme · Answer

un truc peut-être public void ChargerFichier(int i, List cheminFichierOk) { string result; foreach (string chemin in cheminFichierOk) i++; for (int j = 0; j < i; j++) { IEnumerable lines = File.ReadLines(cheminFichierOk[j]); result = Path.GetFileName(cheminFichierOk[j]); foreach (var item in lines) { LogParse leLog = LogParse.Parse(item, result); lesLogs.Add(leLog); } } lesLogs = lesLogs.OrderBy(x => x.Datation).ToList(); dtgLog.DataSource = null; // ajoute ça dtgLog.DataSource = lesLogs; }

Utilisateur anonyme · Answer

Tout à l'heure, je me suis dit qu'il allait me falloir décrypter ta méthode Parse pour me faire un fichier avec une ou 2 lignes d'exemple.
Mais non, je l'ai reconnue cette méthode!
Et donc j'avais conservé ton fichier log.

J'ai pas encore compilé et testé, dans tout ce que tu as donné, il me manque des classes, j'essaye de voir au fur et à mesure si elles ont un intérêt pour moi.
Je pense que Export et ConfigurationManager je n'en ai pas besoin.


J'ai déjà vu pleins de petits trucs qui font perdre du temps à ton traitement, mais ça on verra après

Peut-être que j'ai la raison de ton problème, mais je ne l'ai pas testé.

Tu as dérivé ton formulaire pour une méthode, donc tu as 2 classes quasi similaires.

Je pense que tu as affiché une instance de AnalyseForm, mais dans la méthode du backgroundworker c'est à une instance de AnalyseLog, mais tu ne l'affiche pas.

DateTime dateDebut = this.dtpDebut.Value, dateFin = this.dtpFin.Value;
            int i = 0;
            List<string> cheminFichier;
            AnalyseLog analyseLog = new AnalyseLog();//nouvelle instance de AnalyseLog
            cheminFichier = analyseLog.IntervaleFichierGlobal(lcPath, dateDebut, dateFin);
            analyseLog.ChargerFichier(i, cheminFichier);
            cheminFichier = analyseLog.IntervaleFichierGlobal(TraficPath, dateDebut, dateFin);
            analyseLog.ChargerFichier(i, cheminFichier);

//pas de analyseLog.Show() donc elle n'est pas visible.

Utilisateur anonyme · Answer

Un problème à la fois tu veux?

du coup je doit afficher analyselog pour pouvoir affchier mon datagridview ? 
oui ou non, ça dépend de toi
Si tu veux un formulaire par fichier, disons oui, mais le datagridview de AnalyseForm va rester vide.

Pour vérifier, tu peux faire un analyseLog.Show(), voir si ça s'affiche bien dans un formulaire par fichier et après en fonction de ce que préfèrerais voir on avise.

Utilisateur anonyme · Answer

Et donc est ce que là  avec analyseLog.Show() ça s'affiche?

Utilisateur anonyme · Answer

Hum, je m’y repenche dans la soirée

Utilisateur anonyme · Answer

Bon j'ai fait pas mal de changements D'abord, j'ai viré la form dérivée. J'ai mis la méthode IntervaleFichierGlobal et private List cheminFichierOk = new List() dans AnalyseForm Puisque que tu ne veux pas avoir plusieurs formulaire, elle ne présente aucun intérêt et même avec plusieurs formulaires, c'était très bof puisque ta forme mère restait vide. Ensuite j'ai supprimé foreach (string chemin in cheminFichierOk) i++; for (int j = 0; j < i; j++) que j'ai remplacé par cette seule ligne for (int j = 0; j < cheminFichierOk.Count - i; j++)plus concis et plus rapide à l'exécution ensuite, j'ai viré string result; et result = Path.GetFileName(cheminFichierOk[j]); et remplacé LogParse leLog = LogParse.Parse(item, result); par LogParse leLog = LogParse.Parse(item, Path.GetFileName(cheminFichierOk[j])); Et surtout, j'ai affecté bindé le datagridview en dehors du thread, en effet seul le thread d'affichage a le droit d'accéder à un contrôle. Au final le code de ChargerFichier est public void ChargerFichier(int i, List cheminFichierOk) { for (int j = 0; j < cheminFichierOk.Count - i; j++) { IEnumerable lines = File.ReadLines(cheminFichierOk[j]); foreach (var item in lines) { LogParse leLog = LogParse.Parse(item, Path.GetFileName(cheminFichierOk[j])); lesLogs.Add(leLog); } } lesLogs = lesLogs.OrderBy(x => x.Datation).ToList(); } Et pour le binding, je me suis servi de l'évènement de fin du backgroundworker. private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e) { dtgLog.DataSource = lesLogs; } Quand à la méthode abonnée à DoWOrk c'est "juste" adapté aux changements. Ça c'est ce qui marche chez moi private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e) { DateTime dateDebut = this.dtpDebut.Value, dateFin = this.dtpFin.Value; int i = 0; List cheminFichier = new List { "LogisticCore300120.log" };//ça c'est pour que ça marche chez moi //cheminFichier = IntervaleFichierGlobal(lcPath, dateDebut, dateFin); ChargerFichier(i, cheminFichier); //cheminFichier = IntervaleFichierGlobal(TraficPath, dateDebut, dateFin); //ChargerFichier(i, cheminFichier); } ET ce qui devrait chez toi private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e) { DateTime dateDebut = this.dtpDebut.Value, dateFin = this.dtpFin.Value; int i = 0; List cheminFichier = cheminFichier = IntervaleFichierGlobal(lcPath, dateDebut, dateFin); ChargerFichier(i, cheminFichier); cheminFichier = IntervaleFichierGlobal(TraficPath, dateDebut, dateFin); ChargerFichier(i, cheminFichier); }

Melman · Answer

Bonne nouvelle ça fonctionne super bien, seulement pour les petits fichiers quand les fichiers sont gros ça pose problème, le processus prend 7 minutes et à la fin je reçois l'erreur comme quoi x pour la datation est null alors que le format de l'heure et des lignes du fichiers devrais être trier correctement donc je sais pas si cela est du à leur taille. Petite question aussi pour la listbox, la classe logGroup permet de la trier ?

Utilisateur anonyme · Answer

Pour les gros fichiers, il est possible que ça finisse effectivement par poser problème.
Cela dit je me serais plutôt attendu à un OutOfMemory ou un StackOverFlow.
Donc la date null pourrait avoir une autre raison.
Peux tu poster une capture d'écran

Petite question aussi pour la listbox, la classe logGroup permet de la trier je n'ai pas vu de listbox dans le code que tu m'as donné, mais tu peux trier la liste lesLogs avec la méthode OrderBy

Melman · Answer

lesLogs = lesLogs.OrderBy(x => x.Datation).ToList();
Et l'erreur : System.NullReferenceException : 'La référence d'objet n'est pas définie à une instance d'un objet' x a été null.

Et pour la list box je veux au préalable choisir ce que je vais afficher, (Erreur, Warning, Info) mais dans ceux la il y avais les logs que toi tu avais affichés tous séparément, j'aimerai les encapsuler ex: N_ERREUR ce retrouve dans Erreur qui pourrait être dans ma list box mais seulement si je l'ai demander avant, donc je pense que ça limiterait grandement l'affichage et donc l'apparition d'erreur OutOfMemory ou StackOverFlow?

Utilisateur anonyme · Answer

Pour le null, la méthode Parse peut retourner un Log null si le format de la date n'est pas reconnu, ou si le nom de fichier ne commence par par Logis

Option 1, tu ignores ce log

        public void ChargerFichier(int i, List<string> cheminFichierOk)
        {
            for (int j = 0; j < cheminFichierOk.Count - i; j++)
            {
                IEnumerable<string> lines = File.ReadLines(cheminFichierOk[j]);
                foreach (var item in lines)
                {
                    LogParse leLog = LogParse.Parse(item, Path.GetFileName(cheminFichierOk[j]));
                    if(leLog != null)//ici
                        lesLogs.Add(leLog);
                }
            }
            lesLogs = lesLogs.OrderBy(x => x.Datation).ToList();
        }

option 2, il faut modifier Parse pour retourner un log par défaut et / ou partiellement remplir

par exemple

        public static LogParse Parse(string lignes, string nomFichier)
        {
            if (nomFichier.StartsWith("Logis"))
            {
                string[] ligneArray = lignes.Split('\t'); //sépare les chaînes à partir
                Match m = Regex.Match(lignes, @"(?<mois>\d\d?)?/(?<jour>\d\d?)?/(?<annee>\d{4})?\s+(?<heure>\d\d?)?:(?<minute>\d\d?)?:(?<seconde>\d\d?)?\s+(?<ampm>[AP]M)?:(?<ms>\d{1,3})?");
                if (m.Success)
                {
                    int jour = int.Parse(m.Groups["jour"].Value);
                    int mois = int.Parse(m.Groups["mois"].Value);
                    int annee = int.Parse(m.Groups["annee"].Value);
                    int heure = int.Parse(m.Groups["heure"].Value);
                    int minute = int.Parse(m.Groups["minute"].Value);
                    int secondes = int.Parse(m.Groups["seconde"].Value);

                    string MS = m.Groups["ms"].Value;
                    int ms = 0;
                    if (!string.IsNullOrWhiteSpace(MS))
                    {
                        ms = int.Parse(MS);
                        if (MS.Length == 1)
                            ms *= 100;
                        else if (MS.Length == 2)
                            ms *= 10;
                    }
                    DateTime date = new DateTime(annee, mois, jour, heure, minute, secondes, ms);
                    if (m.Groups["ampm"].Value == "PM")
                        date = date.AddHours(12);
                    return new LogParse
                    {
                        Datation = date,
                        Identificateur = "Nom",
                        Module = ligneArray[1],
                        Type = ligneArray[2],
                        Info = ligneArray[3],
                        Details = ligneArray[4],
                    };
                }
                return new LogParse
                    {
                        Identificateur = "Date Invalide",
                        Module = ligneArray[1],
                        Type = ligneArray[2],
                        Info = ligneArray[3],
                        Details = ligneArray[4],
                    };
            }
          return new LogParse
                    {
                        Identificateur = "Fichier Invalide"
                    };
                }

Melman · Answer

Petit soucis, quand je veux analyser mon fichier plus volumineux il me remet l'erreur de la date nul mais quand un fichier qui fonctionne a été analyser avant on arrive sur l'erreur OutOfMemory

Utilisateur anonyme · Answer

Une chose à la fois.
Pour la date null, as tu juste modifié la regex ou aussi appliqué l'une ou l'autre des mes options.
Si tu as appliqué l'une ou l'autre de mes options alors, peut-être faut il mettre une date par défaut dans les 2 return partiels

Si après avoir testé ça plantevencore, peux tu poster une impression d'écran du moment où ça plante, avec si possible des espions des variables utiles.

Utilisateur anonyme · Answer

Le out of memory, c'est que ton fichier est trop gros.
On verra plus tard.

Le  IndexOutOfRangeException, c'est parce que le tableau ligneArray à moins de 5 items.
Pour savoir pourquoi, il faut regarder le contenu de la ligne au complet.
Et ensuite modifier le code en conséquence.
Si tu ne sais pas modifier, poste le contenu de la ligne.

Utilisateur anonyme · Answer

Je te demande la ligne qui est lue au moment où ça plante.

Utilisateur anonyme · Answer

Tu peux essayer, quand le split ne retourne pas assez d’items, de concaténer avec la ligne suivante

MelmanM · Answer

Je retombe toujours sur la même erreur mes tests ne sont pas concluant ^^'

Utilisateur anonyme · Answer

Je tache d'y regarder demain

Utilisateur anonyme · Answer

Ha pardon, j'avais mal lu/compris ton message.
J'ai cru que tu avais un log écrit sur 2 lignes.
Mais en fait, tu as une ligne vide.

Je te propose donc 2 choses:

ignorer les lignes vides.
ignorer les lignes qu'on ne sait pas lire (sans planter) et les logguer dans un fichier.

A la fin, signaler s'il y a eu des erreurs de lecture, afin d'analyser les lignes et adapter le code si besoin

La méthode LogParse.Parse

        public static LogParse Parse(string lignes, string nomFichier)
        {
            if (nomFichier.StartsWith("Logis"))
            {
                string[] ligneArray = lignes.Split('\t'); //sépare les chaînes à partir
                if (ligneArray.Length != 5)
                    return null;

                Match m = Regex.Match(lignes, @"(?<mois>\d\d?)?/(?<jour>\d\d?)?/(?<annee>\d{4})?\s+(?<heure>\d\d?)?:(?<minute>\d\d?)?:(?<seconde>\d\d?)?\s+(?<ampm>[AP]M)?:(?<ms>\d{1,3})?");
                if (m.Success)
                {
                    int jour = int.Parse(m.Groups["jour"].Value);
                    int mois = int.Parse(m.Groups["mois"].Value);
                    int annee = int.Parse(m.Groups["annee"].Value);
                    int heure = int.Parse(m.Groups["heure"].Value);
                    int minute = int.Parse(m.Groups["minute"].Value);
                    int secondes = int.Parse(m.Groups["seconde"].Value);

                    string MS = m.Groups["ms"].Value;
                    int ms = 0;
                    if (!string.IsNullOrWhiteSpace(MS))
                    {
                        ms = int.Parse(MS);
                        if (MS.Length == 1)
                            ms *= 100;
                        else if (MS.Length == 2)
                            ms *= 10;
                    }
                    DateTime date = new DateTime(annee, mois, jour, heure, minute, secondes, ms);
                    if (m.Groups["ampm"].Value == "PM")
                        date = date.AddHours(12);
                    return new LogParse
                    {
                        Datation = date,
                        Identificateur = "Nom",
                        Module = ligneArray[1],
                        Type = ligneArray[2],
                        Info = ligneArray[3],
                        Details = ligneArray[4],
                    };
                }
            }
            return null;
        }

La méthode ChargeFichier

        public void ChargerFichier(int i, List<string> cheminFichierOk, out bool Erreur, string FichierErreur)
        {
            Erreur = false;
            for (int j = 0; j < cheminFichierOk.Count - i; j++)
            {
                string[] lines = File.ReadAllLines(cheminFichierOk[j]);
                for (int k = 0; k < lines.Length; k++)
                {
                    string item = lines[k];
                    if (string.IsNullOrWhiteSpace(item))
                        continue;//on ignore les lignes vides

                    LogParse leLog = LogParse.Parse(item, Path.GetFileName(cheminFichierOk[j]));
                    if (leLog != null)
                        lesLogs.Add(leLog);
                    else
                    {
                        Erreur = true;
                        if (!File.Exists(FichierErreur))
                            File.AppendAllText(FichierErreur, "Fichier;Ligne;Texte\r\n");//création du fichier avec la ligne d'entête

                            File.AppendAllText(
                            FichierErreur, 
                            string.Format("{0};{1};{2}\r\n", cheminFichierOk[j],k,item)
                            );
                    }
                }
            }
            lesLogs = lesLogs.OrderBy(x => x.Datation).ToList();
        }

Et la méthode du backgroundworker

        private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            DateTime dateDebut = this.dtpDebut.Value, dateFin = this.dtpFin.Value;
            int i = 0;
            //List<string> cheminFichier = new List<string> { "LogisticCore300120.log" };//ça c'est pour que ça marche chez moi
            bool Erreur = false;
            string erreurLog = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.Desktop), "Erreurs.csv");

            cheminFichier = IntervaleFichierGlobal(lcPath, dateDebut, dateFin);
            ChargerFichier(i, cheminFichier, out Erreur, erreurLog);
            cheminFichier = IntervaleFichierGlobal(TraficPath, dateDebut, dateFin);
            ChargerFichier(i, cheminFichier, out Erreur, erreurLog);

            if (Erreur)
            {
                Process.Start(erreurLog);//on affiche le fichier
            }
        }

J'ai sauté une ligne dans le fichier, (ça s'est pas loggué, mais on pourrait si tu veux), modifié une date et supprimé une info dans une autre ligne.
Le fichier erreurs.csv est

Fichier;Ligne;Texte
LogisticCore300120.log;6;1/30/2020 4:01:27 	LOGMON	N_DEBUG	commandLine [L:55]  	CommandLine ok
LogisticCore300120.log;12;1/30/2020 4:01:27 PM:581	LOGMON	N_DEBUG	startWebServer [L:264]

Utilisateur anonyme · Answer

Bon

alors pour le fichier volumineux, il n'y a pas le choix, il faut le lire en plusieurs fois.
Peux tu mettre un point d'arrêt après  string[] lines = File.ReadAllLines(cheminFichierOk[j]); pour voir si cette étape passe.
Si oui c'est "facile" il suffit de dire je traite d'abord X milliers de lignes, puis dans un second temps X milliers de nouvelles lignes etc...
Si non, il faudra lire le fichier en mode séquentiel.

Utilisateur anonyme · Answer

si on commençait par le début tu veux bien.

Utilisateur anonyme · Answer

Il faut que tu comprennes qu'avec un fichier trop volumineux ça peut tomber n'importe ou dans le code.
Dés que tu atteints la quantité de mémoire disponible pour ton logiciel ça plante.
Donc ça n'est pas une ligne spécifique qui génère un OutOfMemory, c'est l'accumulation. 

Ma question était est ce que ça va dépasse cette étape là:
string[] lines = File.ReadAllLines(cheminFichierOk[j]);

De ta réponse, je comprends que oui.
Maintenant, je voudrais savoir le nombre d'items dans string[] lines, tu devrais pouvoir le voir avec les outils de débugage, ou en tapant lines.Lenght dans la fenêtre d'exécution.

Utilisateur anonyme · Answer

OK même opération avec ton fichier qui fonctionne

Utilisateur anonyme · Answer

Ha oui y'a un rapport 33...

Bon, reprends le fichier trop gros.

Modifie le code ainsi

string[] lines = File.ReadAllLines(cheminFichierOk[j]);
                for (int k = 0; k < lines.Length; k++)
                {
                    if(k > 313000 && k % 1000 == 0)
                          Console.WriteLine(k);

string item = lines[k];
                    if (string.IsNullOrWhiteSpace(item))
                        continue;//on ignore les lignes vides

Laisse aller jusqu'à planter, et à ce moment là regarde la dernière valeur de k et dit la moi.

Utilisateur anonyme · Answer

Ok donc pour ce fichier, au bout d'un peu plus de 2,5 millions de lignes ton logiciel n'arrive plus à instancier des LogParse.
Alors je ne te parle même pas de les afficher.
En plus une instance de LogParse n'a pas une taille fixe, car elle est composée de strings dont la taille dépend du nombre de caractères.
On va faire un test avec les 2 premières millions de lignes.

Modifie ceci
for (int k = 0; k < 2000000; k++)
Si ça passe à mon avis on n'est pas loin du clash, si ça ne passe pas, essaye 1 500 000 puis 1 000 000, 500 000 jusqu'à ce que ça passe et ré-augmente petit à petit.

Utilisateur anonyme · Answer

OK, alors on va considerer qu'environ 2M de lignes c'est déjà bien.

 Après                 string[] lines = File.ReadAllLines(cheminFichierOk[j]);

tu vas ajouter

File.WriteAllLines("test2M.log", File.ReadAllLines(cheminFichierOk[j].Take(2000000));
return;
tu exécutes une fois.
Tu vas vérifier dans le dossier de ton exe si le fichier test2M.log et bien créé.
SI oui, tu regardes sa taille (en octet, se sera plus simple pour la suite) et tu peux supprimer les 2 lignes ajoutée.

L'idée c'est de trouver la taille à partir de laquelle, il faudra faire un traitement "par morceau".

Utilisateur anonyme · Answer

Je l'ai tapé de tête , j'ai p'tet écris une boulette.
Bon maintenant, il faut faire un choix.


Option 1
avant de lire le fichier, on regarde sa taille, et si elle dépasse disons 240 000 000 d'octets, on signale à l'opérateur que seules les 2M de premières lignes (environs) vont être afficher.
Il faudra alors une action de sa part pour afficher les 2M suivantes.

Option 2
on lit systématiquement par parquet, plus petits 2M de ligne c'est déjà énorme.

Option 3
Considérant que l'opérateur ne peut pas faire grand chose de 2M de lignes, je suppose qu'ensuite tu effectues des filtres. Je te propose de voir les choses dans l'autre sens, l'opérateur fait ses filtres et on ne va lire dans le fichier que les lignes concernées.

Option 4
je sais pas mais y'a certainement encore pleins de façon de voir les choses.

Utilisateur anonyme · Answer

oui ça peut être ça

Afficher dans un DataGridView à partir d'une classe

38 réponses

Discussions similaires