datagrid et couche métier WPF

**snay13** · 15/12/2012, 09h02

Bonjour,

je sollicite vos conseils afin d'accélérer le chargement de ma datagrid tout en gardant la facilté de binding sur ma datagrid.

Actuellement, pour charger 10 000 enregistrements dans ma datagrid, cela me prend environ 8 secondes

Voici mon code

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
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
 
<DataGrid x:Name="dgdEnregistrement" Margin="8,150,8,8" VerticalAlignment="Top" Style="{DynamicResource MyDataGridStyle}" 
                  ColumnHeaderStyle="{DynamicResource MyDataGridColumnHeaderStyle}" RowStyle="{DynamicResource MyDataGridRowStyle}" 
                  CellStyle="{DynamicResource MyDataGridCellStyle}">
 
			<DataGrid.Columns>
                <DataGridTextColumn Header="Code Id" Binding="{Binding id_util}"/>
                <DataGridTextColumn Header="Nom" Binding="{Binding nom_util}">
                    <DataGridTextColumn.ElementStyle>
                        <Style TargetType="TextBlock">
                            <Setter Property="TextWrapping" Value="Wrap" />
                            <Setter Property="MaxWidth" Value="150" />
                        </Style>
                    </DataGridTextColumn.ElementStyle>
                </DataGridTextColumn>
				<DataGridTextColumn Header="Prénom" Binding="{Binding prn_util}">
                    <DataGridTextColumn.ElementStyle>
                        <Style TargetType="TextBlock">
                            <Setter Property="TextWrapping" Value="Wrap" />
                            <Setter Property="MaxWidth" Value="150" />
                        </Style>
                    </DataGridTextColumn.ElementStyle>
                </DataGridTextColumn>
                <DataGridTextColumn Header="Email" Binding="{Binding mail_util}">
                    <DataGridTextColumn.ElementStyle>
                        <Style TargetType="TextBlock">
                            <Setter Property="TextWrapping" Value="Wrap" />
                            <Setter Property="MaxWidth" Value="150" />
                        </Style>
                    </DataGridTextColumn.ElementStyle>
                </DataGridTextColumn>
				<DataGridTextColumn Header="Identifiant" Binding="{Binding nom_conn_util}">
                    <DataGridTextColumn.ElementStyle>
                        <Style TargetType="TextBlock">
                            <Setter Property="TextWrapping" Value="Wrap" />
                            <Setter Property="MaxWidth" Value="150" />
                        </Style>
                    </DataGridTextColumn.ElementStyle>
                </DataGridTextColumn>
				<DataGridTextColumn Header="Groupe" Binding="{Binding groupe.lib_grp}">
                    <DataGridTextColumn.ElementStyle>
                        <Style TargetType="TextBlock">
                            <Setter Property="TextWrapping" Value="Wrap" />
                            <Setter Property="MaxWidth" Value="150" />
                        </Style>
                    </DataGridTextColumn.ElementStyle>
                </DataGridTextColumn>
				<DataGridTextColumn Header="Configuration" Binding="{Binding groupe.acces_param}"/>
				<DataGridTextColumn Header="Mise à jour" Binding="{Binding groupe.acces_maj}"/>
                <DataGridTextColumn Header="Manuel" Binding="{Binding groupe.acces_manuel}"/>
                <DataGridTextColumn Header="Prodotec" Binding="{Binding groupe.acces_gestProd}"/>
                <DataGridTextColumn Header="Impression" Binding="{Binding groupe.acces_gestImp}"/>
			</DataGrid.Columns>
 
		</DataGrid>

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
private void tbxRecherche_KeyDown(object sender, System.Windows.Input.KeyEventArgs e)
        {
            //Touche ENTER
            if (e == null || e.Key == Key.Enter)
            {
                if (a == null)
                {
                    a = new Thread(new ParameterizedThreadStart(LoadData));
                    a.Start(tbxRecherche.Text);
 
                }
            }
        }

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
28
29
30
31
32
33
34
35
36
37
 
 public void LoadData(Object param)
        {
            Application app = System.Windows.Application.Current;
            if (app != null)
                app.Dispatcher.BeginInvoke(DispatcherPriority.Background, (Action)delegate
                {
                    x = new MyMessageBoxWait("Mise à jour des enregistrements");
                    x.ShowDialog();
                });
 
 
            listUtilisateur = daoUtilisateur.FindAll(param.ToString());
 
 
 
            if (app != null)
                app.Dispatcher.BeginInvoke(DispatcherPriority.Background, (Action)delegate
                {
                    if (listUtilisateur != null)
                    {
                        if (listUtilisateur != null && listUtilisateur.Count > 1)
                            lblEnregistrement.Content = listUtilisateur.Count + " Enregistrements";
                        else
                            lblEnregistrement.Content = listUtilisateur.Count + " Enregistrement";
                    }
                    else
                        lblEnregistrement.Content = "0 Enregistrement";
 
                    dgdEnregistrement.ItemsSource = listUtilisateur;
                    dgdEnregistrement.SelectedValue = listUtilisateur;
                    x.Visibility = Visibility.Hidden;
                    x.Close();
                });
            a = null;
 
        }

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
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
 
 public ObservableCollection<MdlUtilisateur> FindAll(String name = "")
        {
            //Requete
            string query = "SELECT id_util, nom_util, prn_util, mail_util, nom_conn_util, mdp_conn_util, id_grp "
                + "FROM ls_util WHERE nom_util LIKE CONCAT('%', @param1, '%') AND vis_util = 1 ORDER BY nom_util";
 
            MySqlCommand cmd = null;
            MySqlDataAdapter ad = null;
            DataTable dt = null;
            DaoGroupe daoGroupe = null;
            ObservableCollection<MdlUtilisateur> listUtilisateur;
            try
            {
                //Preparation de la requete
                cmd = new MySqlCommand(query, this.connection);
                cmd.Parameters.AddWithValue("@param1", name);
                ad = new MySqlDataAdapter();
                ad.SelectCommand = cmd;
                dt = new DataTable();
                ad.Fill(dt);
 
                //Resultat
                if (dt.Rows.Count != 0)
                {   
                    daoGroupe = FactoryDao.getDaoGroupe();
                    listUtilisateur = new ObservableCollection<MdlUtilisateur>();
                    foreach (DataRow dr in dt.Rows)
                    {
                        listUtilisateur.Add(new MdlUtilisateur(Convert.ToInt64(dr[0]), dr[1].ToString(), dr[2].ToString(),
                            dr[3].ToString(), dr[4].ToString(), dr[5].ToString(), daoGroupe.findById(Convert.ToInt64(dr[6]))));
                    }
 
                    return listUtilisateur;
                }
                else
                    return null;
            }
            catch (Exception ex)
            {
                UtilsUi.ShowMessageError(ex);
                return null;
            }
            finally
            {
                //Liberation des ressources
                cmd.Dispose();
                ad.Dispose();
                dt.Dispose();
                daoGroupe = null;
                listUtilisateur = null;
            }
        }

Ce qui me prends du temps, ce n'est pas l'éxécution de la requête en elle même mais la génération du modele métier dans la méthode FindAll

Est ce la bonne méthode ?
Comment puis je accélérer tout ça ?

Merci

**infosam76** · 15/12/2012, 10h30

Bonjour,

Pour ce que je vois dans votre code pour charger vos données dans un DataTable via la méthode Fill du MySqlDataAdapter.

Ensuite vous repasser tous les enregistrements chargés pour les placer dans une ObservableCollection que vous liez à votre Datagrid.

Pourquoi ne pas lier directement votre DataGrid au DataTable et donc éviter le remplissage de la collection ?

**snay13** · 15/12/2012, 11h28

Bonjour,

Je passe par une collection afin d'utiliser la facilité de binding pour mes objets

Par exemple :

- Pour les DataGridTextColumn dans mon code xaml ci dessus

- Pour les liens avec les clés étrangères

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
28
29
30
31
32
 
public class MdlUtilisateur
    {
        /*Constructeurs*/
        public MdlUtilisateur() {}
 
        public MdlUtilisateur(String nom_util, String prn_util, String mail_util, String nom_conn_util,
            String mdp_conn_util, ModeleGroupe groupe) 
        {
            this.nom_util = nom_util;
            this.prn_util = prn_util;
            this.mail_util = mail_util;
            this.nom_conn_util = nom_conn_util;
            this.mdp_conn_util = mdp_conn_util;
            this.groupe = groupe;
        }
 
        public MdlUtilisateur(long id_util, String nom_util, String prn_util, String mail_util, String nom_conn_util,
            String mdp_conn_util, ModeleGroupe id_grp) : this(nom_util, prn_util, mail_util, nom_conn_util, mdp_conn_util, id_grp)
        {
                this.id_util = id_util;
        }
 
        /*Propriétés*/
        public long id_util { get; set; }
        public String nom_util { get; set; }
        public String prn_util { get; set; }
        public String mail_util { get; set; }
        public String nom_conn_util { get; set; }
        public String mdp_conn_util { get; set; }
        public ModeleGroupe groupe { get; set; }
    }

- Pour récupérer la ligne sélectionné de ma DataGrid, je récupère directement l'objet

Code :

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

1
2
 
utilisateur = (MdlUtilisateur)dgdEnregistrement.SelectedValue;

Mais effectivement c'est ce transfert de ma DataTable vers ma Collection qui prends du temps

**youtpout978** · 15/12/2012, 13h17

Pourquoi utiliser une datatable, tu peux mettre tes objets directement dans une observableCollection lors de l'accès à ta BDD en utilisant un datareader.

**snay13** · 15/12/2012, 17h13

Effectivement, mais avec un DataReader, j'obtiens une erreur car pour chaque ligne, je lance une requête sur la clé étrangère afin de créer mon modèle pour le groupe.

Du coup, j'ai revu mon architecture et j'utilise une jointure dans ma requête et les performances sont nettement meilleures et même acceptable (environ 1s)

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
public ObservableCollection<MdlUtilisateur> FindAll(String name = "")
        {
            //Requete
            string query = "SELECT id_util, nom_util, prn_util, mail_util, nom_conn_util, mdp_conn_util, lib_grp, note_grp, acces_param, acces_maj, acces_manuel, acces_gestProd, acces_gestImp "
                + "FROM ls_util, ls_grp WHERE nom_util LIKE CONCAT('%', @param1, '%') AND vis_util = 1 AND ls_util.id_grp = ls_grp.id_grp ORDER BY nom_util";
 
            MySqlCommand cmd = null;
            MySqlDataReader dr = null;
            DaoGroupe daoGroupe = FactoryDao.getDaoGroupe();;
            ObservableCollection<MdlUtilisateur> listUtilisateur;
            try
            {
                //Preparation de la requete
                cmd = new MySqlCommand(query, this.connection);
                cmd.Parameters.AddWithValue("@param1", name);
 
                dr = cmd.ExecuteReader();
 
                if(dr.HasRows) {
                    listUtilisateur = new ObservableCollection<MdlUtilisateur>();
                    while (dr.Read())
                    {
                        listUtilisateur.Add(new MdlUtilisateur(dr.GetInt64(0), dr.GetString(1), dr.GetString(2), dr.GetString(3), dr.GetString(4),
                            dr.GetString(5), new ModeleGroupe(dr.GetString(6), dr.GetString(7), dr.GetInt32(8), dr.GetInt32(9), dr.GetInt32(10), dr.GetInt32(11), dr.GetInt32(12))));
                    }
                    return listUtilisateur;
                }
                else
                    return null;
            }
            catch (Exception ex)
            {
                UtilsUi.ShowMessageError(ex);
                return null;
            }
            finally
            {
                //Liberation des ressources
                cmd.Dispose();
                dr.Close();
                daoGroupe = null;
                listUtilisateur = null;
            }
        }

Est ce la bonne solution ?
Existe il une meilleure solution ?

**youtpout978** · 15/12/2012, 18h50

Justement c'est une jointure qu'il faut faire et non deux requête séparé, tu as quand même bien amélioré les performances.

Après ton code n'est pas le plus claire non plus mais c'est ça dans l'idée qu'il faut faire (le getstring directement dans le add c'est pas le plus propre non plus et le faire par numéro de colonne ca peux poser des pb).
Pour ton nom s'il est vide pourquoi utilisé un Like dans ce cas, je pense que ça te fait perdre du temps autant faire deux méthodes (une avec le like une sans).

datagrid et couche métier WPF

C#

Discussions similaires

Partager

Partager