ThreadPool bloque un mode release

Version imprimable

Bonjour

J'ai toujours pas bien digéré l'usage des Thread

Dans l'exemple ci-apres depuis une form j'appelle une methode dans un nouveau thread en utilisant Treadpool

Ma methode contient deux event permettant de mettre eu datagridview a jour dans la form

En debug, ca marche tres bien.
En release ca bloque en cours de traitement et je dois fermer l'application
Je sais en tout cas que je n'arrive pas au bout de la boucle de traitement

Un oeil de lynx repere-t-il une grosse betise ?

Voici le code d'appel

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
..
      DistMatrix dm = new DistMatrix();
      dm.VV = VV;
      dm.AddRow += new DistMatrix.StepRowDelegateHandler(Dist_StepRow);
      dm.AddCell += new DistMatrix.StepCellDelegateHandler(Dist_StepCell);
      ThreadPool.QueueUserWorkItem(new WaitCallback(Process),dm);
    }
 
    public delegate void StepCellDelegateHandler(int a, int b);
    public delegate void StepRowDelegateHandler(DataTable dt);
 
    // *****************************************************************************************
    void Process(Object DM)
    {
      DataTable dtn = ((DistMatrix)DM).getMatrix(dtAdr);
    }
 
    // *****************************************************************************************
    private void Dist_StepCell(int a, int b)
    {
      if (this.InvokeRequired)
      {
        try
        {
          this.Invoke(new StepCellDelegateHandler(StepCell), new object[] { a, b });
          return;
        }
        catch (Exception e)
        {
          MessageBox.Show(e.Message);
        }
      }
      else
      {
        this.StepCell(a, b);
      }
    }

Voici la synthese du traitement

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
  // ***********************************************************************************
    public DataTable getMatrix(DataTable dtIn)
    {
      DataTable dtMatrix=CreateTable(dtIn);
      for (i = 0; i < dtIn.Rows.Count; i++)
      {
        DataRow cRowY = dtIn.Rows[i];
        string fromY = cRowY["addrId"].ToString();
 
        for (j = 0; j < dtIn.Rows.Count; j++)
        {
          DataRow cRowX = dtIn.Rows[j];
          addDtCell(dtMatrix,cRowX["addrId"].ToString(),fromY,5);
          if (AddCell != null)
          {
            AddCell(j,i);
          }
        }
      }
 
      if (AddRow != null)
      {
        AddRow(dtMatrix);
      }
 
      if (AddCell != null)
      {
        AddCell(-1, -1);
      }
 
      return dtMatrix;
    }
    // ***********************************************************************************
    private void addDtCell(DataTable DstMatrix, string rX, string rY, double dist)
    {
      DataView dv = new DataView(DstMatrix);
      dv.Sort = "Start";
      int idx = dv.Find(rY);
      DataRow cRow = null;
      if (idx < 0)
      {
        cRow = DstMatrix.Rows.Add();
        cRow["Start"] = rY;
        if (AddRow != null)
        {
          AddRow(DstMatrix);
        }
      }
      else
      {
        cRow = dv[idx].Row;
      }
      if (dist > 0.01)
      {
        cRow[rX] = dist;
      }
    }

15/11/2008, 19h27
tomlev

A mon avis le problème est dans Dist_StepRow et Dist_StepCell... que fais-tu dans ces méthodes ? n'oublie pas qu'elles vont s'exécuter sur le thread de traitement, pas sur le thread de l'UI (puisque tu appelles AddRow et AddCell pendant le traitement). Donc si Dist_StepRow et Dist_StepCell accèdent à l'interface graphique, ça foire...

Tu n'aurais pas fait un truc comme ça par hasard ?

Code:

Control.CheckForIllegalCrossThreadCalls = false;

Avec ça, ça peut passer en debug et pas en release...

Salut Tomlev

Ci joint plus en détail l'usage des deux methode

En fait le but du jeux c'est precisément de mettre a jour le UI a certaines étapes de mon traitement dans le thread appelé

C'est pour ca que je gere deux event

Mais bon je reve peut etre et ce n'est peut etre pas possible (mais ca m'etonne un peu)

Dans le cas contraire, comment aurais-je du faire ?

NB :

Je n'ai pas utilisé ceci mais peut-etre est-ce implicite en mode debug.

Citation:

Control.CheckForIllegalCrossThreadCalls = false;

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
    // *****************************************************************************************
    private void Dist_StepCell(int a, int b)
    {
      if (this.InvokeRequired)
      {
        try
        {
          this.Invoke(new StepCellDelegateHandler(StepCell), new object[] { a, b });
          return;
        }
        catch (Exception e)
        {
          MessageBox.Show(e.Message);
        }
      }
      else
      {
        this.StepCell(a, b);
      }
    }
    // *****************************************************************************************
 
    private void StepCell(int X,int Y)
    {
      if (X == -1)
      {
        MessageBox.Show("Call -1");
        btn_print.Enabled = true;
        btn_export.Enabled = true;
        toolStrip1.Enabled = true;
        return;
      }
      lbl_Progress.Text = string.Format("{0}-{1} / {2}", Y, X,dtAdr.Rows.Count);
    }
 
    // *****************************************************************************************
    private void Dist_StepRow(DataTable dt)
    {
      if (this.InvokeRequired)
      {
        try
        {
          this.Invoke(new StepRowDelegateHandler(AddRow), new object[] { dt });
          return;
        }
        catch (Exception e)
        {
          MessageBox.Show("Row:"+e.Message);
        }
      }
      else
      {
        this.AddRow(dt);
      }
    }
 
    // *****************************************************************************************
    private void AddRow(DataTable Dt)
    {
      if (dgv_Dist.DataSource == null)
      {
        dgv_Dist.DataSource = Dt;
        foreach (DataGridViewColumn dgvc in dgv_Dist.Columns)
        {
          DataGridViewCellStyle style = new DataGridViewCellStyle();
          style.Format = "N2";
          style.Alignment = DataGridViewContentAlignment.MiddleRight;
          dgvc.DefaultCellStyle = style;
        }
      }
      dgv_Dist.Refresh();
    }

15/11/2008, 21h25
tomlev

bizarre, je vois pas ce qui pose problème...

sinon essaie avec un BackgroundWorker, en utilisant l'évènement ProgressChanged pour mettre à jour l'UI
15/11/2008, 22h54
olibara

Salut Tomlev

BakcroundWorker .... :?

Berk !
J'ai déja donné !

J'avais précisément envie de me faire la main avec les Thread

Bon je continue mon investigation

Ce que je sais c'est que ca plante +/- au 240eme appel de StepCell
En debug j'en fais 400
15/11/2008, 23h04
tomlev

Citation:

Envoyé par olibara

BakcroundWorker .... :?

Berk !
J'ai déja donné !

qu'est-ce que tu lui reproches ? moi je trouve ça plus pratique que les threads...
15/11/2008, 23h39
olibara

Salut

C'est peut etre un peu plus facile a mettre en oeuvre que les Thread (quoique)

Mais c'est certainement moins souple
En tout cas ca ne m'a pas convaincu !

Dans le present cas, je suis en train de cerner qu'il y a peut etre conflit entre l'appel AddCell et AddRow liés a des manip du DGV

En tout cas, si je supprime addrow ou si je lui impose un return sans traitement ca ne bloque plus !

Je continue !
16/11/2008, 00h29
olibara

Il doit y avoir un gros probleme qu'on ne voit pas

En chipotant un peu sur l'activité dgv.refresh() de addrow j'arrive a aller jusqu'au bout du process

Mais ca plante quand meme si je veux fermer la form !

Je veux me battre pour trouver le probleme dans le concept de ce que je fais car c'est pas normal !!

Ou alors ca veux dire qu'on ne peux pas faire d'event dans un thread pour invoquer un process d'un autre thread et ca se saurait !

Donc pas question d'user d'autres artifices (BackRoundWorker ou autre sans comprendre ce qui se passe ici)