Discussion :

[olibara]

Bonjour

Je pense que je m'emele un peu les pinceaux

J'ai une classe form Toute simple qui instancie une classe qui va gerer une communications serie
J'utilise un delegate pour recuperer dans la form le resutat de la com
Mais je dois faire une betise qq part dans la communication entre la form et la classe

Voici 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
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
using System;
using System.Linq;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;
 
namespace MCOpticon
{
  public partial class Form1 : Form
  {
    C_BlueTooth BL;
 
    public Form1()
    {
      InitializeComponent();
    }
 
    private void button1_Click(object sender, EventArgs e)
    {
      BL.OpenS();
    }
 
    private void Form1_Load(object sender, EventArgs e)
    {
      BL = new C_BlueTooth();
      BL.ShowScan += this.ShowText;
    }
    void ShowText(string Txt)
    {
      this.txbScan.Text = Txt;
    }
  }
}

et la Classe

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
56
57
58
using System;
using System.Linq;
using System.Collections.Generic;
using System.Text;
using System.Net.Sockets;
using System.Threading;
using System.IO.Ports;
 
namespace MCOpticon
{
  public class C_BlueTooth
  {
    public delegate void ShowScanHandler(string AA);
    public event ShowScanHandler ShowScan;
    private event ShowScanHandler TransferScan;
 
    SerialPort sp = null;
 
    public C_BlueTooth()
    {
      sp=new SerialPort("COM5");
//      sp.PortName = "COM5";
      sp.BaudRate = 9600;
      sp.Parity = Parity.None;
      sp.StopBits = StopBits.One;
      sp.Handshake = Handshake.None;
      this.TransferScan = new ShowScanHandler(AddDataMethod);
    }
 
    public void AddDataMethod(String myString)
    {
      if (ShowScan != null)
      {
        ShowScan(myString);
      }
    }
    public void OpenS()
    {
 
      sp.DataReceived += new SerialDataReceivedEventHandler(serialPort1_DataReceived);
 
      sp.Open(); //opens the port
      sp.ReadTimeout = 200;
 
 
    }
 
    private void serialPort1_DataReceived(object sender, SerialDataReceivedEventArgs e)
    {
 
       string DispString = sp.ReadExisting();
       TransferScan(DispString);
 
    }
 
 
  }
}

Lorsque des données arrivent sur le Serial un event appelle serialPort1_DataReceived
Ensuite j'utilise le delegate TransferScan pour envoyer le resultat a la form via un autre delegate ShowScan
Mais a l'arrivée dans la form la methode ShowText m'envoie un message d'erreur

"Control.Invoke must be used to interact with controls created on a separate thread.

Comment faut il faire : je n'ai pas mis asser de delegates ?

Merci de votre Aide

[Pragmateek]

serialPort1_DataReceived est invoqué par un thread quelconque, sans doute issue du pool.

Puis tu as cette chaîne d'invocation toujours sur ce même thread :

Code :

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

serialPort1_DataReceived -> TransferScan -> AddDataMethod -> ShowScan -> ShowText

(au passage je ne comprends pas l'utilité de l'indirection via l'évènement TransferScan, un évènement est destiné à l'extérieur...)

Or seul le thread UI a le droit de manipuler les composants graphiques => il faut lui transférer l'opération :

Code :

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

1
2
3
4
void ShowText(string Txt)
{
    this.txbScan.Invoke(() => this.txbScan.Text = Txt);
}

[olibara]

Merci Pragmateek

Tu a raison l'indirection ne sert a rien, je m'étais embrouillé la tete en confondant Invoke et indirection

Cela etant comme d'une part je ne suis pas du tout fan des Lambda expression et qu'en plus je pense qu'elles ne sont pas autorisée su le Compact Framework, comment tu traduirait to exemple d'Invoke en utilisant un delegate Explicite ?

[Dudule70]

Bonjour,
Essayer ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
        delegate void SetLabel(string msg);
        void SetLabelMethod(string msg)
        {
            txbScan.Text = msg;
        }
 
        void ShowText(string txt)
        {
            this.Invoke(new SetLabel(SetLabelMethod), txt);
        }

Cordialement

[Pragmateek]

Ou alors pour être le plus light possible quelque chose comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
void ShowText(string Txt)
{
    if (this.txbScan.InvokeRequired)
    {
        this.txbScan.Invoke(ShowText, Txt);
        return;
    }
 
    this.txbScan.Text = Txt;
}

(pas testé mais l'idée est là)