DataGridView en simple

Version imprimable

25/12/2010, 16h13
fredele

DataGridView en simple

Bonjour a tous !
J' ai fait une petite application qui utilise un DataGridView en remplissant les champs simplement avec les méthodes DataGridView.Rows.Add() puis en accèdant aux Rows avec leur index ...
Je me suis apercu vers la fin que cela posant un gros problèmes de performances si on charge quelques 2000 lignes :cry:(quasiment inutilisable ..)alors qu' avec 15 cela reste fluide ...

Suis- je obliger de tout refaire avec une BDD ? en utilisant une liste d' objets ?8O

Y a t' il une option que j' ai manquée ??

Merci pour votre lecture

NB : je charge en fait des fichiers de sous-titre srt, donc les données ne sont pas énormes ...
25/12/2010, 19h09
champomy62

Salut,

Poste ton code stp

Voici le 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
 
 private void remplir(string fichier, int append_time)
        {
 
            StreamReader reader = new StreamReader(fichier, System.Text.Encoding.Default);
            string ligne = string.Empty;
            string item = string.Empty;
 
 
            while ((ligne = reader.ReadLine()) != null)
            {
                 try
            {
 
                if (utils.IsNumeric(ligne) == true && item != string.Empty)
                {
                    // 1 00:01:46,400 --> 00:01:52,905 ALEXANDRE
 
                    string[] splititem = item.Split(new string[] { "\n", "-->", "\n", }, System.StringSplitOptions.None);
 
                    string item_numero = splititem[0];
                    string item_debut = splititem[1];
                    string item_fin = splititem[2];
 
                    string item_texte = null;
                    for (int i = 3; i < splititem.Count(); i++)
                    {
                       item_texte += splititem[i] + "\n";
                    }
                    item_texte = utils.RemoveEmptyLines(item_texte);
 
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Numero"].Value = splititem[1];
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Debut"].Value = splititem[1];
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Fin"].Value = splititem[1];
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Texte"].Value = splititem[1];
 
 
                    item = null;
                    item += ligne + "\n";
                   // Application.DoEvents();
 
                }
                else
                {
                    item += ligne + "\n";
                }
            } catch (System.Exception m)
             {
                 item = null;
                 item += ligne + "\n";
                 Debug.WriteLine("remplir : " + m.Message);
 
             }
 
 
 
            } // FIN while ..
 
 
            //deselect_rows();
            reader.Close();
 
 
            Grid.Sort(new RowComparerDebut_milliseconds(SortOrder.Ascending)); renum();          
 
            this.Debut_UpDown.Maximum = Convert.ToInt32(soustitre.soustitreliste[soustitre.soustitreliste.Count -1 ].Fin_milliseconds);
            this.Fin_UpDown.Maximum = Convert.ToInt32(soustitre.soustitreliste[soustitre.soustitreliste.Count -1 ].Fin_milliseconds);
            this.Duree_UpDown.Maximum = Convert.ToInt32(soustitre.soustitreliste[soustitre.soustitreliste.Count -1 ].Fin_milliseconds);
 
 
       }

Il s' agit donc bien d' une "Unbound Grid" ...

NB : à l' import , le split est ausi très lent (mettons 1 minute pour importer un simple fichier ...:calim2:)mais la navigation en elle-même devient impossible tellement c' est lent .

Pardon, il fallait lire ceci !

Code:

1
2
3
4
5
 this.Grid.Rows.Add();
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Numero"].Value = splititem[0];
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Debut"].Value = splititem[1];
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Fin"].Value = splititem[2];
                    this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Texte"].Value = splititem[3];

Désolé ...:aie:

25/12/2010, 20h28
champomy62

Utils...C'est quoi ?

Je teste simplement pour voir si la ligne correspond a un chiffre valide .
Chaque item correspond donc à un sous-titre comprenant son numero (increment), le debut, la fin (de l' affichage ..) et le texte ...

Code:

1
2
3
4
5
6
7
 public static bool IsNumeric(string chkNumeric)
        {
            int intOutVal;
            bool isValidNumeric = false;
            isValidNumeric = int.TryParse(chkNumeric, out intOutVal);
            return isValidNumeric;
        }

Pour info, voila à quoi ressemble un fichier srt :

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
1
00:01:46,400 --> 00:01:52,905
ALEXANDRE
 
2
00:02:10,924 --> 00:02:15,553
"La chance sourit aux audacieux."
VIRGILE - L'Enéide
 
3
00:02:19,766 --> 00:02:24,020
BABYLONE, PERSE - JUIN 323 AV. J-C
 
4
00:03:02,100 --> 00:03:04,852
Alexandre, l'armée va se diviser.
 
5
00:03:05,020 --> 00:03:08,105
Les satrapes se révolteront.
Sans ordres, il y aura la guerre.
 
6
00:03:08,357 --> 00:03:10,775
Nous t'en prions. Dis-nous qui.
 
7
00:04:08,125 --> 00:04:10,584
ALEXANDRIE, EGYPTE 285 AV. J-C
40 ANS PLUS TARD

Je précise bien que même si l' import est lent , la navigation l' est tout autant ..

25/12/2010, 21h01
champomy62

Essaie de voir ou se situe le goulot d’étranglement(IO)... Mets un point d’arrêt sur ta condition et un autre sur ta boucle et ensuite regarde ou ça met du temps.

Ensuite quand tu dis que l'importation est longue... Situes nous un ordre de grandeur stp

Il faut compter environ 10 secondes pour un fichier de 2000 sous-titre ..
Lors de la navigation, je sélectionne les lignes entières et met simplement les valeurs des colonnes dans des textbox et des NumericUpDown ..
Code:

1 2 3 4 this.Debut_UpDown.Value = Convert.ToInt32(Grid.Rows[Grid.SelectedRows[0].Index].Cells["Debut_milliseconds"].Value.ToString()); this.Fin_UpDown.Value = Convert.ToInt32(Grid.Rows[Grid.SelectedRows[0].Index].Cells["Fin_milliseconds"].Value.ToString()); this.Duree_UpDown.Value = this.Fin_UpDown.Value - this.Debut_UpDown.Value; this.textBox.Text = Grid.Rows[Grid.SelectedRows[0].Index].Cells["texte"].Value.ToString();
C' est tout ce qu' il y a dans l' evenement CellClick...mais ca prends 4-5 secondes pour que les données soient rapatriées .

25/12/2010, 21h36
champomy62

Pour l'importation 10 secondes je trouve ça plutôt correct...

En fait le problème si j'ai bien compris c'est que tu veux cliquer sur une ligne de la grille et que les données s'affiche dans des textbox et numericupdow?

Citation:
Envoyé par fredele
Il faut compter environ 10 secondes pour un fichier de 2000 sous-titre ..
Lors de la navigation, je sélectionne les lignes entières et met simplement les valeurs des colonnes dans des textbox et des NumericUpDown ..
Code:

1 2 3 4 this.Debut_UpDown.Value = Convert.ToInt32(Grid.Rows[Grid.SelectedRows[0].Index].Cells["Debut_milliseconds"].Value.ToString()); this.Fin_UpDown.Value = Convert.ToInt32(Grid.Rows[Grid.SelectedRows[0].Index].Cells["Fin_milliseconds"].Value.ToString()); this.Duree_UpDown.Value = this.Fin_UpDown.Value - this.Debut_UpDown.Value; this.textBox.Text = Grid.Rows[Grid.SelectedRows[0].Index].Cells["texte"].Value.ToString();
C' est tout ce qu' il y a dans l' evenement CellClick...mais ca prends 4-5 secondes pour que les données soient rapatriées .
Je penses que c'est normal. Avant de pouvoir s'afficher il refait une importation et seuelement après il affiche.

EDIT : Créer une procedure pour celà et mets ce code dedans

25/12/2010, 21h40
fredele

Le problème, c' est uniquement la lenteur ...
Avec un fichier de 15 sous-titre ca réagit au tac-o-tac, mais avec 2000, il faut attende 4-5 secondes pour que la ligne soit sélectionnée et pour que l' utilisateur puisse modifier les données . C' est pas super je trouve ...

et si tu fais comme ça ?

Code:

1
2
 
this.textBox.Text = Grid.SelectedRows[0].Cells["texte"].Value.ToString();

25/12/2010, 22h03
fredele

Oui effectivement, c' est mieux écrit mais non, ca change rien ..
Je pense que c' est parce que je ne crée pas d' objet en tant que Datasource,

Quels sont les alternatives pour essayer de changer de stratégie? XML ? ..
C' est juste que là ca devient tout de suite compliquer sachant qu' il faut des Handlers et tout et tout juste pour ca ..
25/12/2010, 22h55
fredele

Je viens d' essayer de faire des objets sous-titre, et là le chargement est beaucoup plus rapide, la navigation parfaite, mais le code est a refaire..:mouarf:

Encore Merci
26/12/2010, 09h54
olibara

Citation:

Envoyé par fredele

Je pense que c' est parce que je ne crée pas d' objet en tant que Datasource,

Je pense que effectivement il est beaucoup plus propre et rationnel de charger des données dans une Datatable et d'utiliser cette datatable en Datasource de ton DGV tu vas passer de 10 secondes a presque rien et tu aura une isolation beaucoup plus saine de ta vue et de tes données
Un datagridview n'est pas destiné a etre un container de donnée

Ah oui, j' ai cheché ce matin avant ton post et suit tombé sur la même solution . C' est vue juste et effectivement tout se déroule de facon super fluide ..

je viens juste de comprendre comment faire pour ajouter mes données et mon code devient ...

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
 private void remplir(string fichier, int append_time)
        {
            DataTable table = new DataTable();
            this.Grid.DataSource = table;
            table.Columns.Add("Numero", System.Type.GetType("System.String"));
            table.Columns.Add("Debut", System.Type.GetType("System.String"));
            table.Columns.Add("Fin", System.Type.GetType("System.String"));
            table.Columns.Add("Debut_milliseconds", System.Type.GetType("System.String"));
            table.Columns.Add("Fin_milliseconds", System.Type.GetType("System.String"));
            table.Columns.Add("Texte", System.Type.GetType("System.String"));
 
            StreamReader reader = new StreamReader(fichier, System.Text.Encoding.Default);
            string ligne = string.Empty;
            string item = string.Empty;
 
 
            while ((ligne = reader.ReadLine()) != null)
            {
                 try
            {
 
                if (utils.IsNumeric(ligne) == true && item != string.Empty)
                {
                    // 1 00:01:46,400 --> 00:01:52,905 ALEXANDRE
 
                    string[] splititem = item.Split(new string[] { "\n", "-->", "\n", }, System.StringSplitOptions.None);
 
                    string item_numero = splititem[0];
                    string item_debut = splititem[1];
                    string item_fin = splititem[2];
 
                    string item_texte = null;
                    for (int i = 3; i < splititem.Count(); i++)
                    {
                       item_texte += splititem[i] + "\n";
                    }
                    item_texte = utils.RemoveEmptyLines(item_texte);
 
                    table.Rows.Add(splititem[0], splititem[1], splititem[2], Convert.ToString(utils.milliseconds_from_hours(splititem[1])), Convert.ToString(utils.milliseconds_from_hours(splititem[2])), splititem[3]);
                    item = null;
                    item += ligne + "\n";
 
 
 
                }
                else
                {
                    item += ligne + "\n";
                }
            } catch (System.Exception m)
             {
                 item = null;
                 item += ligne + "\n";
                 Debug.WriteLine("remplir : " + m.Message);
 
             }
 
 
 
            } // FIN while ..
 
 
 
            //deselect_rows();
            reader.Close();
 
 
            Grid.Sort(new RowComparerDebut_milliseconds(SortOrder.Ascending)); renum();
 
            this.Debut_UpDown.Maximum =Convert.ToInt32( this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Fin_milliseconds"].Value);
            this.Fin_UpDown.Maximum = Convert.ToInt32(this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Fin_milliseconds"].Value);
            this.Duree_UpDown.Maximum = Convert.ToInt32(this.Grid.Rows[this.Grid.Rows.Count - 1].Cells["Fin_milliseconds"].Value);
 
 
       }

Là j' ai plus le temps de voir , mas j' ai remarqué que certaines fonctions fonctionnent quand même alors que je n' ai rien changé d' autre dans le reste du code.

En effet, pour tout le reste je reprends bien les valeurs de la DataGridView et non celle de ta datatable.

Question : Dois-je modifier le reste de mon code quand même ?, la table est-elle mise-à-jour automatiquement à partir de la DataGridView ?
J' y gagne quoi ?

Oui, je sais, je deviens fainéant, vue que je devrais chercher moi-même ...

Merci a tous

26/12/2010, 15h53
fredele

Oui, je confirme : en accèdant simplement à la GridView la table se met à jour automatiquement et mon code n' est pas perdu : les performances sont à présent tout autre ...

Encore Merci a vous tous !