web-dev-qa-db-fra.com

Opération cross-thread non valide: Contrôle 'textBox1' accédé à partir d'un thread autre que celui sur lequel il a été créé

Je souhaite envoyer la valeur de température d'un microcontrôleur à l'aide de UART à l'interface C # et afficher la température sur Label.Content. Voici mon code de microcontrôleur:

while(1) {
   key_scan(); // get value of temp
   if (Usart_Data_Ready())
   {
      while(temperature[i]!=0)
      {
         if(temperature[i]!=' ')
         {
            Usart_Write(temperature[i]);
            Delay_ms(1000);
         }
         i = i + 1;
      }
      i =0;
      Delay_ms(2000);
   }
}

et mon code C # est:

private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
{
   txt += serialPort1.ReadExisting().ToString();
   textBox1.Text = txt.ToString();
}

mais une exception survient là-bas "Opération cross-thread non valide: La commande 'textBox1' accessible depuis un thread autre que celui sur lequel elle a été créée" Merci de me dire comment obtenir la chaîne de température de mon microcontrôleur et de la supprimer. Erreur!

c#multithreadinginvokeuart
164
Fatima Zohra

Les données reçues dans votre méthode serialPort1_DataReceived proviennent d'un autre contexte de thread que le thread d'interface utilisateur, et c'est la raison pour laquelle vous voyez cette erreur.
Pour remédier à cela, vous devrez utiliser un répartiteur tel que décrit dans l'article MSDN:
Comment: effectuer des appels en toute sécurité avec des contrôles Windows Forms

Ainsi, au lieu de définir la propriété text directement dans la méthode serialport1_DataReceived, utilisez ce modèle:

delegate void SetTextCallback(string text);

private void SetText(string text)
{
  // InvokeRequired required compares the thread ID of the
  // calling thread to the thread ID of the creating thread.
  // If these threads are different, it returns true.
  if (this.textBox1.InvokeRequired)
  { 
    SetTextCallback d = new SetTextCallback(SetText);
    this.Invoke(d, new object[] { text });
  }
  else
  {
    this.textBox1.Text = text;
  }
}

Donc dans votre cas:

private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
{
  txt += serialPort1.ReadExisting().ToString();
  SetText(txt.ToString());
}
286
Magnus Johansson

Je ne sais pas si cela suffit mais j'ai créé une classe statique ThreadHelperClass et je l'ai implémenté comme suit. Maintenant, je peux facilement définir la propriété text de divers contrôles sans trop coder.

public static class ThreadHelperClass
    {
        delegate void SetTextCallback(Form f, Control ctrl, string text);
        /// <summary>
        /// Set text property of various controls
        /// </summary>
        /// <param name="form">The calling form</param>
        /// <param name="ctrl"></param>
        /// <param name="text"></param>
        public static void SetText(Form form, Control ctrl, string text)
        {
            // InvokeRequired required compares the thread ID of the 
            // calling thread to the thread ID of the creating thread. 
            // If these threads are different, it returns true. 
            if (ctrl.InvokeRequired)
            {
                SetTextCallback d = new SetTextCallback(SetText);
                form.Invoke(d, new object[] { form, ctrl, text });
            }
            else
            {
                ctrl.Text = text;
            }
        }
    }

En utilisant le code:

 private void btnTestThread_Click(object sender, EventArgs e)
        {
            Thread demoThread =
               new Thread(new ThreadStart(this.ThreadProcSafe));
            demoThread.Start();
        }

        // This method is executed on the worker thread and makes 
        // a thread-safe call on the TextBox control. 
        private void ThreadProcSafe()
        {
            ThreadHelperClass.SetText(this, textBox1, "This text was set safely.");
            ThreadHelperClass.SetText(this, textBox2, "another text was set safely.");
        }
45
Thunder

vous pouvez simplement faire ceci.

TextBox.CheckForIllegalCrossThreadCalls = false;
31
HforHisham

Utilisez les extensions suivantes et passez l'action comme ceci:

_frmx.PerformSafely(() => _frmx.Show());
_frmx.PerformSafely(() => _frmx.Location = new Point(x,y));

Classe d'extension:

public static class CrossThreadExtensions
{
    public static void PerformSafely(this Control target, Action action)
    {
        if (target.InvokeRequired)
        {
            target.Invoke(action);
        }
        else
        {
            action();
        }
    }

    public static void PerformSafely<T1>(this Control target, Action<T1> action,T1 parameter)
    {
        if (target.InvokeRequired)
        {
            target.Invoke(action, parameter);
        }
        else
        {
            action(parameter);
        }
    }

    public static void PerformSafely<T1,T2>(this Control target, Action<T1,T2> action, T1 p1,T2 p2)
    {
        if (target.InvokeRequired)
        {
            target.Invoke(action, p1,p2);
        }
        else
        {
            action(p1,p2);
        }
    }
}
22
Miguel Merayo Regueras

Dans le même sens que les réponses précédentes, mais un ajout très court qui permet d’utiliser toutes les propriétés du contrôle sans exception d’appel croisé.

Méthode d'assistance

    /// <summary>
    /// Helper method to determin if invoke required, if so will rerun method on correct thread.
    /// if not do nothing.
    /// </summary>
    /// <param name="c">Control that might require invoking</param>
    /// <param name="a">action to preform on control thread if so.</param>
    /// <returns>true if invoke required</returns>
    public bool ControlInvokeRequired(Control c,Action a)
    {
        if (c.InvokeRequired) c.Invoke(new MethodInvoker(delegate { a(); }));
        else return false;

        return true;
    }

Exemple d'utilisation

    // usage on textbox
    public void UpdateTextBox1(String text)
    {
        //Check if invoke requied if so return - as i will be recalled in correct thread
        if (ControlInvokeRequired(textBox1, () => UpdateTextBox1(text))) return;
        textBox1.Text = ellapsed;
    }

    //Or any control
    public void UpdateControl(Color c,String s)
    {
        //Check if invoke requied if so return - as i will be recalled in correct thread
        if (ControlInvokeRequired(myControl, () => UpdateControl(c,s))) return;
        myControl.Text = s;
        myControl.BackColor = c;
    }
12
Mike

Utilisez un conteneur partagé pour transférer des données entre des threads.

7
rotator