invalid Operation Exception

Version imprimable

Bonjour

je tente d'écrire dans un textbox un caractère reçu par une liaison série
et la mise a jour du textbox me met cette erreur.
Je ne comprend rien à la notion de thread et encore moins d'accès thread safe dont parle l'aide.

Les solutions proposées avec 50 lignes de code me paissent trop démesurées avec une simple écriture dans une zone de texte pour être adaptées a ce que je fais...
Code:

1 2 3 4 5 private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e) { int data = serialPort1.ReadChar(); textBox1.Text += data.ToString(); }
Pouvez vous m'expliquer mon erreur svp

Merci par avance

27/10/2010, 11h31
Pol63

il faudrait savoir sur quelle ligne ca plante, et tu faire du pas à pas pour comprendre ce qui plante
parce que c'est surement la lecture sur le serialport qui plante

sur une application on peut avoir plusieurs threads, chaque thread s'exécutant simultanément
ca permet de gagner en performance sur certains traitement ou de ne pas bloquer l'interface

le terme thread safe utilisé dans msdn signifie qu'une classe ou un membre ne pose pas de soucis à être utilisé par plusieurs threads en même temps
par exemple sur un list<T> on peut avoir des problème, si un thread fait un for each sur la collection et qu'un autre thread fait .Add ca va planter dans le for each car la collection n'est plus la même
m'enfin je ne pense pas que ton problème vient d'un thread, du moins avec le code que tu nous montres

Bonjour

ca plante sur TextBox1.Text...

ca me met :

Citation:

Opération inter-threads non valide : le contrôle 'textBox1' a fait l'objet d'un accès à partir d'un thread autre que celui sur lequel il a été créé.

Je mets ci dessous tout le code de mon appli, elle est tres simple pour le moment, facile de s'y retrouver.

1 bouton qui envoi un caractère par la liaison série
L'appareil branché en face renvoi un caractère e retour
Mon appli doit afficher dans le textebox1 ce qu'elle reçoit

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
namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
 
        private void Form1_Load(object sender, EventArgs e)
        {
        }
 
        private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
        {
                int data = serialPort1.ReadChar();
                textBox1.Text += data.ToString();   //c'est ici que ca merdoit
        }
 
        private void Form1_Activated(object sender, EventArgs e)
        {
            serialPort1.Open();
        }
 
        private void Form1_Deactivate(object sender, EventArgs e)
        {
            serialPort1.Close();
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            // Makes sure serial port is open before trying to write
            try
            {
                if (!(serialPort1.IsOpen))
                    serialPort1.Open();
                serialPort1.Write("a");
            }
            catch (Exception ex)
            {
                MessageBox.Show("Erreur ouverture port serie :: " + ex.Message, "Error!");
            } 
        }
    }
}

Merci

27/10/2010, 11h52
Pol63

ah ok

l'évènement DataReceived est déclenché sur un thread secondaire
et seul le thread principal a le droit d'accéder aux controles

pour contourner ce soucis, il faut repasser sur le thread principal via

Code:

this.invoke(le délégué qui pointe vers un void qui lui s'occupera de modifier le textbox)

Les évènements de la classe SerialPort se produisent sur un thread secondaire, pas sur celui de l'UI. Or, seul le thread de l'UI peut accéder aux éléments de l'UI. Il faut donc utiliser la méthode Invoke pour faire la mise à jour de la TextBox sur le thread de l'UI :
Code:

1 2 3 4 5 6 7 8 9 10 private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e) { int data = serialPort1.ReadChar(); Invoke(AppendReceivedText, data.ToString()); } private void AppendReceivedText(string text) { textBox1.Text += text; }

27/10/2010, 12h45
petitours

Merci, j'ai pigé le problème des threads.

Je dois par contre avoir un problème de déclaration maintenant
faut il déclarer la méthode AppendReceivedText quelquepart ?

Le compilo me dit que les arguments du Invoke ne sont pas bons

Citation:

Erreur 1 La méthode surchargée correspondant le mieux à 'System.Windows.Forms.Control.Invoke(System.Delegate, params object[])' possède des arguments non valides
Erreur 2 Argument '1'*: impossible de convertir de 'groupe de méthodes' en 'System.Delegate'
Erreur 3 Argument '2'*: impossible de convertir de 'string' en 'object[]'

ca y est j'ai réussi

avec ceci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
        private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
        {
                int data = serialPort1.ReadChar();
                textBox1.Invoke(new TextBoxInvokeHandler(FonctionTextBox),data.ToString());
        }
 
        private delegate void TextBoxInvokeHandler(string text);
 
        private void FonctionTextBox(string text)
        {
            textBox1.Text += text;
        }

Merci beaucoup !

27/10/2010, 14h38
tomlev

Citation:

Envoyé par petitours

Le compilo me dit que les arguments du Invoke ne sont pas bons

Ah oui pardon... Plutôt que de créer un delegate spécifique, tu peux utiliser Action<string> :

Code:

textBox1.Invoke(new Action<string>(FonctionTextBox),data.ToString());