Discussion :

[kururin]

Hello

Je rencontre un petit probleme que je n'arrive pas à résoudre...
J'ai une listbox qui prend comme source un datatable. Ce datatable est rempli dans un thread à part.
Voici le code pour mieux comprendre:

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
 
private void FonctionAppeleeParThread()
{
                datatable = new DataTable();
                DataColumn dc = new DataColumn("string", typeof(string));
                datatable.Columns.Add(dc);
                DataRow dr = null; 
 
                for (int i = 0; i < 3; i++)
                {
                    dr = datatable.NewRow();
                    dr.BeginEdit();
                    dr["string"] = "truc" + i.ToString();
                    dr.EndEdit();
                    datatable.Rows.Add(dr);
                }       
                _listBox.DisplayMember = "string";
                _listBox.DataSource = dt;         
...
...
}

datatable est une variable globale.
Et donc le problème vient de ce qu'affiche ma listbox au final, à savoir:

System.Data.DataRowView
System.Data.DataRowView
System.Data.DataRowView

Comme si le DisplayMember n'était pas renseigné.
Si je exactement la meme chose dans le thread principal, l'affichage est correct.

Ca fait plusieurs heures que je galère sans trop comprendre... Si quelqu'un avait une piste, ce serait chouette

Merci d'avance !!
Mathieu

[karlus]

Code :

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

_listBox.DataSource = dt;

Qu'est ce dt ? En supposant que c'est ta datatable de départ, ton problème doit venir tu fait que tu ne peux pas modifier des objets graphique en dehors du thread principal de ta form.

Essai :

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
 
public delegate void FonctionAppeleeParThreadDelegate();
 
 
private void FonctionAppeleeParThread()
    {
      if (this.InvokeRequired)
      {
        this.Invoke(new FonctionAppeleeParThreadDelegate(FonctionAppeleeParThread));
      }
      else
      {
        datatable = new DataTable();
        DataColumn dc = new DataColumn("string", typeof(string));
        datatable.Columns.Add(dc);
        DataRow dr = null;
 
        for (int i = 0; i < 3; i++)
        {
          dr = datatable.NewRow();
          dr.BeginEdit();
          dr["string"] = "truc" + i.ToString();
          dr.EndEdit();
          datatable.Rows.Add(dr);
        }
        _listBox.DisplayMember = "string";
        _listBox.DataSource = datatable;
      }
    }

[olibara]

La remarque de Karlus est tout a fait pertinente, mais j'en ai une autre en plus

J'ai une listbox qui prend comme source un datatable. Ce datatable est rempli dans un thread à part

Fait gaffe !
Je pense que la datatable n'est pas thread safe

J'ai ramé des heure a resoudre un probleme sporadique dans une config similaire avec une datatable datasource d'un dgv

En final j'ai résolu en gardant la datatable et le dgv dans le thread principal le thread secondaire ne fait que calculer les valeur a mettre dans la datatable et le passage se fait par event a chaque row a ajouter

[kururin]

Merci pour vos réponses déjà

(oui, dt = datatable)
Karlus, j'ai essayé ta proposition (j'avais essayé un truc similaire auparavant), mais mon résultat reste le même.

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
 
public delegate void SetListBoxSourceDelegate();
 
//dans le thread principal
thread2 = new Thread(new ThreadStart(FonctionAppelee));
 
private void FonctionAppelee()
{
     SetListBoxSource();
     ...
}
 
        private void SetListBoxSource()
        {
            if (this.InvokeRequired)
            {
                SetListBoxSourceDelegate d = new SetListBoxSourceDelegate(SetListBoxSource);
                this.Invoke(d, new object[] { });
            }
            else
            {
                datatable = new DataTable();
                DataColumn dc = new DataColumn("string", typeof(string));
                datatable.Columns.Add(dc);
                DataRow dr = null;
 
                for (int i = 0; i < 3; i++)
                {
                    dr = datatable.NewRow();
                    dr.BeginEdit();
                    dr["string"] = "truc" + i.ToString();
                    dr.EndEdit();
                    datatable.Rows.Add(dr);
                }
                listBox.DisplayMember = "string";
                listBox.DataSource = datatable;
            }
        }

Mais c'est pareil

Olibara, merci pour ce conseil. Est ce que tu penses que ca peut expliquer mon problème? J'ai du mal à voir comment tu as procédé de ton côté par contre... C'est peut-être abusé, mais si tu avais un bout de code, ou des explications un peu plus détaillées, ce serait génial

En tout cas merci encore!!

[kururin]

Ok c'est bon j'ai fixé mon problème !!

J'ai tout préparé au moment du load de ma form:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
//global
DataTable datatable = null;
DataView dv = null;   
 
private void Form1_Load(object sender, EventArgs e)
{
            datatable = new DataTable();
            DataColumn dc = new DataColumn("string", typeof(string));
            datatable.Columns.Add(dc);
            listBox_JiraIssues.DisplayMember = "string";
            dv = new DataView(datatable);
            listBox_JiraIssues.DataSource = dv;
}

Et la fonction appelée par le second thread ne fait que rajouter les lignes à la datatable... En gros, je crois que j'ai fait ce que vous m'avez dit, et ca marche

Donc merci encore!!!

[olibara]

Verifie quand meme que ca marche en mode release si tu ajoute plusieurs lignes (quelques dizaines)


Car sans voir ton deuxieme thread, j'imagine que tu lui passe ta datatable
Et pendant que ce deuxieme thread va modifier ta datatable, le premier va l'afficher

Si ce deuxieme thread ne sert qu'a ca, a mon avis il est inutile et meme encombrant !