La operación entre hilos no es válida: se accede al control desde un hilo que no sea el hilo en el que se creó

584

Tengo un escenario (Windows Forms, C #, .NET)

  1. Hay un formulario principal que aloja algún control de usuario.
  2. El control de usuario realiza una operación de datos pesados, de modo que si llamo directamente al UserControl_Loadmétodo, la interfaz de usuario deja de responder durante la ejecución del método de carga.
  3. Para superar esto, cargo datos en diferentes hilos (tratando de cambiar el código existente lo menos que puedo)
  4. Utilicé un subproceso de trabajo en segundo plano que cargará los datos y cuando termine notificará a la aplicación que ha realizado su trabajo.
  5. Ahora vino un verdadero problema. Toda la interfaz de usuario (formulario principal y sus controles de usuario secundarios) se creó en el subproceso principal principal. En el método LOAD del control de usuario, estoy obteniendo datos basados ​​en los valores de algún control (como el cuadro de texto) en userControl.

El pseudocódigo se vería así:

CÓDIGO 1

UserContrl1_LoadDataMethod()
{
    if (textbox1.text == "MyName") // This gives exception
    {
        //Load data corresponding to "MyName".
        //Populate a globale variable List<string> which will be binded to grid at some later stage.
    }
}

La excepción que dio fue

Operación de subprocesos no válida: control al que se accede desde un subproceso distinto del subproceso en el que se creó.

Para saber más sobre esto, busqué en Google y surgió una sugerencia como usar el siguiente código

CÓDIGO 2

UserContrl1_LoadDataMethod()
{
    if (InvokeRequired) // Line #1
    {
        this.Invoke(new MethodInvoker(UserContrl1_LoadDataMethod));
        return;
    }

    if (textbox1.text == "MyName") // Now it wont give an exception
    {
    //Load data correspondin to "MyName"
        //Populate a globale variable List<string> which will be binded to grid at some later stage
    }
}

PERO PERO PERO ... parece que he vuelto al punto de partida. La aplicación vuelve a dejar de responder. Parece ser debido a la ejecución de la línea # 1 si la condición. La tarea de carga se realiza nuevamente por el subproceso principal y no por el tercero que generé.

No sé si percibí esto bien o mal. Soy nuevo en el enhebrado.

¿Cómo resuelvo esto y también cuál es el efecto de la ejecución de la Línea # 1 si se bloquea?

La situación es la siguiente : quiero cargar datos en una variable global basada en el valor de un control. No quiero cambiar el valor de un control desde el hilo secundario. Nunca lo haré desde un hilo secundario.

Por lo tanto, solo acceda al valor para que los datos correspondientes se puedan obtener de la base de datos.

Prerak K
fuente
Para mi caso particular de este error, encontré la solución para usar un BackgroundWorker en el formulario para manejar las partes del código que requieren muchos datos. (es decir, coloque todo el código del problema en el método backgroundWorker1_DoWork () y llámelo a través de backgroundWorker1.RunWorkerAsync ()) ... Estas dos fuentes me señalaron en la dirección correcta: stackoverflow.com/questions/4806742/… youtube.com/ watch? v = MLrrbG6V1zM
Giollia

Respuestas:

433

Según el comentario de actualización de Prerak K (ya eliminado):

Supongo que no he presentado la pregunta correctamente.

La situación es esta: quiero cargar datos en una variable global basada en el valor de un control. No quiero cambiar el valor de un control desde el hilo secundario. Nunca lo haré desde un hilo secundario.

Por lo tanto, solo acceda al valor para que los datos correspondientes se puedan obtener de la base de datos.

La solución que desea entonces debería verse así:

UserContrl1_LOadDataMethod()
{
    string name = "";
    if(textbox1.InvokeRequired)
    {
        textbox1.Invoke(new MethodInvoker(delegate { name = textbox1.text; }));
    }
    if(name == "MyName")
    {
        // do whatever
    }
}

Realice su procesamiento serio en el hilo separado antes de intentar volver al hilo del control. Por ejemplo:

UserContrl1_LOadDataMethod()
{
    if(textbox1.text=="MyName") //<<======Now it wont give exception**
    {
        //Load data correspondin to "MyName"
        //Populate a globale variable List<string> which will be
        //bound to grid at some later stage
        if(InvokeRequired)
        {
            // after we've done all the processing, 
            this.Invoke(new MethodInvoker(delegate {
                // load the control with the appropriate data
            }));
            return;
        }
    }
}
Jeff Hubbard
fuente
1
Ha pasado un tiempo desde que hice la programación de C #, pero según el artículo de MSDN y mi conocimiento irregular, parece que sí.
Jeff Hubbard
1
La diferencia es que BeginInvoke () es asíncrono mientras que Invoke () se ejecuta sincrónicamente. stackoverflow.com/questions/229554/…
frzsombor
179

Modelo de subprocesos en la interfaz de usuario

Lea el Modelo de subprocesos en aplicaciones de IU (el enlace VB anterior está aquí ) para comprender los conceptos básicos. El enlace navega a la página que describe el modelo de subprocesos WPF. Sin embargo, Windows Forms utiliza la misma idea.

El hilo de la interfaz de usuario

  • Solo hay un subproceso (subproceso de interfaz de usuario), que puede acceder a System.Windows.Forms.Control y sus miembros de subclases.
  • Intentar acceder al miembro de System.Windows.Forms.Control desde un subproceso diferente al subproceso de la interfaz de usuario provocará una excepción entre subprocesos.
  • Como solo hay un subproceso, todas las operaciones de IU se ponen en cola como elementos de trabajo en ese subproceso:

ingrese la descripción de la imagen aquí

ingrese la descripción de la imagen aquí

Métodos BeginInvoke e Invoke

  • La sobrecarga informática del método que se invoca debe ser pequeña, así como la sobrecarga informática de los métodos de manejo de eventos porque el subproceso de la interfaz de usuario se usa allí, el mismo responsable de manejar la entrada del usuario. Independientemente de si se trata de System.Windows.Forms.Control.Invoke o System.Windows.Forms.Control.BeginInvoke .
  • Para realizar operaciones costosas de computación, utilice siempre hilos separados. Desde .NET 2.0 BackgroundWorker se dedica a realizar operaciones costosas de cómputo en Windows Forms. Sin embargo, en las nuevas soluciones, debe usar el patrón async-wait como se describe aquí .
  • Use los métodos System.Windows.Forms.Control.Invoke o System.Windows.Forms.Control.BeginInvoke solo para actualizar una interfaz de usuario. Si los usa para cálculos pesados, su aplicación bloqueará:

ingrese la descripción de la imagen aquí

Invocar

ingrese la descripción de la imagen aquí

BeginInvoke

ingrese la descripción de la imagen aquí

Solución de código

Lea las respuestas a la pregunta ¿Cómo actualizar la GUI desde otro hilo en C #? . Para C # 5.0 y .NET 4.5, la solución recomendada está aquí .

Ryszard Dżegan
fuente
72

Solo desea usar Invokeo BeginInvokepara el mínimo trabajo requerido para cambiar la interfaz de usuario. Su método "pesado" debe ejecutarse en otro hilo (por ejemplo, vía BackgroundWorker) pero luego usar Control.Invoke/ Control.BeginInvokesolo para actualizar la interfaz de usuario. De esa manera, su hilo de IU será libre de manejar eventos de IU, etc.

Vea mi artículo de subprocesos para ver un ejemplo de WinForms , aunque el artículo fue escrito antes de BackgroundWorkerllegar a la escena, y me temo que no lo he actualizado a ese respecto. BackgroundWorkersimplemente simplifica un poco la devolución de llamada.

Jon Skeet
fuente
aquí en esta condición mía Ni siquiera estoy cambiando la interfaz de usuario. Solo estoy accediendo a sus valores actuales desde el hilo secundario. cualquier sugerencia para implementar
Prerak K
1
Todavía necesita reunir el hilo de la interfaz de usuario incluso para acceder a las propiedades. Si su método no puede continuar hasta que se acceda al valor, puede usar un delegado que devuelva el valor. Pero sí, ve a través del hilo de la interfaz de usuario.
Jon Skeet
Hola Jon, creo que me estás dirigiendo en la dirección correcta. Sí, necesito el valor sin él, no puedo continuar. Por favor, ¿podría eloborar ese 'Uso de un delegado que devuelve un valor'? Gracias
Prerak K
1
Utilice un delegado como Func <string>: string text = textbox1.Invoke ((Func <string>) () => textbox1.Text); (Eso supone que está usando C # 3.0; de lo contrario, podría usar un método anónimo)
Jon Skeet
45

Sé que es demasiado tarde ahora. Sin embargo, incluso hoy si tiene problemas para acceder a los controles de subprocesos cruzados? Esta es la respuesta más corta hasta la fecha: P

Invoke(new Action(() =>
                {
                    label1.Text = "WooHoo!!!";
                }));

Así es como accedo a cualquier control de formulario desde un hilo.

Bravo
fuente
1
Esto me da Invoke or BeginInvoke cannot be called on a control until the window handle has been created. Lo resolví aquí
rupweb
42

He tenido este problema con FileSystemWatchery descubrí que el siguiente código resolvió el problema:

fsw.SynchronizingObject = this

El control luego usa el objeto de formulario actual para lidiar con los eventos y, por lo tanto, estará en el mismo hilo.

Peter C
fuente
2
Esto salvó mi tocino. En VB.NET utilicé.SynchronizingObject = Me
codificación codificación
20

Encuentro que el código de verificación e invocación que debe estar lleno de todos los métodos relacionados con los formularios es demasiado detallado e innecesario. Aquí hay un método de extensión simple que te permite eliminarlo por completo:

public static class Extensions
{
    public static void Invoke<TControlType>(this TControlType control, Action<TControlType> del) 
        where TControlType : Control
        {
            if (control.InvokeRequired)
                control.Invoke(new Action(() => del(control)));
            else
                del(control);
    }
}

Y luego simplemente puedes hacer esto:

textbox1.Invoke(t => t.Text = "A");

No más perder el tiempo, simple.

Rob
fuente
qué es 't' aquí
Rawat
@Rawat ten este caso será textbox1- se pasa como argumento
Rob
17

Los controles en .NET generalmente no son seguros para subprocesos. Eso significa que no debe acceder a un control desde un hilo que no sea el que vive. Para evitar esto, debe invocar el control, que es lo que intenta su segunda muestra.

Sin embargo, en su caso, todo lo que ha hecho es devolver el método de larga ejecución al hilo principal. Por supuesto, eso no es realmente lo que quieres hacer. Necesita repensar esto un poco para que todo lo que esté haciendo en el hilo principal sea establecer una propiedad rápida aquí y allá.

Joel Coehoorn
fuente
10

Una nueva apariencia usando Async / Await y callbacks. Solo necesita una línea de código si mantiene el método de extensión en su proyecto.

/// <summary>
/// A new way to use Tasks for Asynchronous calls
/// </summary>
public class Example
{
    /// <summary>
    /// No more delegates, background workers etc. just one line of code as shown below
    /// Note it is dependent on the XTask class shown next.
    /// </summary>
    public async void ExampleMethod()
    {
        //Still on GUI/Original Thread here
        //Do your updates before the next line of code
        await XTask.RunAsync(() =>
        {
            //Running an asynchronous task here
            //Cannot update GUI Thread here, but can do lots of work
        });
        //Can update GUI/Original thread on this line
    }
}

/// <summary>
/// A class containing extension methods for the Task class 
/// Put this file in folder named Extensions
/// Use prefix of X for the class it Extends
/// </summary>
public static class XTask
{
    /// <summary>
    /// RunAsync is an extension method that encapsulates the Task.Run using a callback
    /// </summary>
    /// <param name="Code">The caller is called back on the new Task (on a different thread)</param>
    /// <returns></returns>
    public async static Task RunAsync(Action Code)
    {
        await Task.Run(() =>
        {
            Code();
        });
        return;
    }
}

Puede agregar otras cosas al método de Extensión, como envolverlo en una instrucción Try / Catch, lo que permite que la persona que llama le diga qué tipo devolver después de la finalización, una devolución de llamada de excepción a la persona que llama:

Agregar Try Catch, Auto Exception Logging y CallBack

    /// <summary>
    /// Run Async
    /// </summary>
    /// <typeparam name="T">The type to return</typeparam>
    /// <param name="Code">The callback to the code</param>
    /// <param name="Error">The handled and logged exception if one occurs</param>
    /// <returns>The type expected as a competed task</returns>

    public async static Task<T> RunAsync<T>(Func<string,T> Code, Action<Exception> Error)
    {
       var done =  await Task<T>.Run(() =>
        {
            T result = default(T);
            try
            {
               result = Code("Code Here");
            }
            catch (Exception ex)
            {
                Console.WriteLine("Unhandled Exception: " + ex.Message);
                Console.WriteLine(ex.StackTrace);
                Error(ex);
            }
            return result;

        });
        return done;
    }
    public async void HowToUse()
    {
       //We now inject the type we want the async routine to return!
       var result =  await RunAsync<bool>((code) => {
           //write code here, all exceptions are logged via the wrapped try catch.
           //return what is needed
           return someBoolValue;
       }, 
       error => {

          //exceptions are already handled but are sent back here for further processing
       });
        if (result)
        {
            //we can now process the result because the code above awaited for the completion before
            //moving to this statement
        }
    }
John Peters
fuente
10

Esta no es la forma recomendada de resolver este error, pero puede suprimirlo rápidamente, hará el trabajo. Prefiero esto para prototipos o demos. añadir

CheckForIllegalCrossThreadCalls = false

en Form1()constructor.

Özgür
fuente
9

Siga la forma más simple (en mi opinión) para modificar objetos de otro hilo:

using System.Threading.Tasks;
using System.Threading;

namespace TESTE
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            Action<string> DelegateTeste_ModifyText = THREAD_MOD;
            Invoke(DelegateTeste_ModifyText, "MODIFY BY THREAD");
        }

        private void THREAD_MOD(string teste)
        {
            textBox1.Text = teste;
        }
    }
}
Vanderley Maia
fuente
Simple ! Gracias .
Ali Esmaeili
7

Encontré una necesidad de esto mientras programaba un controlador de aplicación monotáctil iOS-Phone en un proyecto de prototipo de estudio visual winforms fuera de xamarin stuidio. Prefiriendo programar en VS sobre xamarin studio tanto como sea posible, quería que el controlador se desacople por completo de la estructura del teléfono. De esta forma, implementar esto para otros marcos como Android y Windows Phone sería mucho más fácil para futuros usos.

Quería una solución en la que la GUI pudiera responder a los eventos sin la carga de lidiar con el código de conmutación de subprocesos cruzados detrás de cada clic de botón. Básicamente, permita que el controlador de clase maneje eso para mantener el código del cliente simple. Posiblemente podría tener muchos eventos en la GUI donde, como si pudiera manejarlo en un lugar de la clase, sería más limpio. No soy un experto en múltiples temas, avíseme si esto es defectuoso.

public partial class Form1 : Form
{
    private ExampleController.MyController controller;

    public Form1()
    {          
        InitializeComponent();
        controller = new ExampleController.MyController((ISynchronizeInvoke) this);
        controller.Finished += controller_Finished;
    }

    void controller_Finished(string returnValue)
    {
        label1.Text = returnValue; 
    }

    private void button1_Click(object sender, EventArgs e)
    {
        controller.SubmitTask("Do It");
    }
}

El formulario GUI no sabe que el controlador está ejecutando tareas asincrónicas.

public delegate void FinishedTasksHandler(string returnValue);

public class MyController
{
    private ISynchronizeInvoke _syn; 
    public MyController(ISynchronizeInvoke syn) {  _syn = syn; } 
    public event FinishedTasksHandler Finished; 

    public void SubmitTask(string someValue)
    {
        System.Threading.ThreadPool.QueueUserWorkItem(state => submitTask(someValue));
    }

    private void submitTask(string someValue)
    {
        someValue = someValue + " " + DateTime.Now.ToString();
        System.Threading.Thread.Sleep(5000);
//Finished(someValue); This causes cross threading error if called like this.

        if (Finished != null)
        {
            if (_syn.InvokeRequired)
            {
                _syn.Invoke(Finished, new object[] { someValue });
            }
            else
            {
                Finished(someValue);
            }
        }
    }
}
RandallTo
fuente
6

Aquí hay una forma alternativa si el objeto con el que está trabajando no tiene

(InvokeRequired)

Esto es útil si está trabajando con el formulario principal en una clase distinta del formulario principal con un objeto que está en el formulario principal, pero no tiene InvokeRequired

delegate void updateMainFormObject(FormObjectType objectWithoutInvoke, string text);

private void updateFormObjectType(FormObjectType objectWithoutInvoke, string text)
{
    MainForm.Invoke(new updateMainFormObject(UpdateObject), objectWithoutInvoke, text);
}

public void UpdateObject(ToolStripStatusLabel objectWithoutInvoke, string text)
{
    objectWithoutInvoke.Text = text;
}

Funciona igual que antes, pero es un enfoque diferente si no tiene un objeto con invokerequired, pero tiene acceso al MainForm

Ashitakalax
fuente
5

En la misma línea que las respuestas anteriores, pero una adición muy corta que permite usar todas las propiedades de Control sin tener una excepción de invocación de hilos cruzados.

Método de ayuda

/// <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;
}

Uso de muestra

// 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;
}
Miguel
fuente
5
this.Invoke(new MethodInvoker(delegate
            {
                //your code here;
            }));
Hamid Jolany
fuente
5

Por ejemplo, para obtener el texto de un control del subproceso de la interfaz de usuario:

Private Delegate Function GetControlTextInvoker(ByVal ctl As Control) As String

Private Function GetControlText(ByVal ctl As Control) As String
    Dim text As String

    If ctl.InvokeRequired Then
        text = CStr(ctl.Invoke(
            New GetControlTextInvoker(AddressOf GetControlText), ctl))
    Else
        text = ctl.Text
    End If

    Return text
End Function
UrsulRosu
fuente
3

La misma pregunta: cómo-actualizar-la-gui-de-otro-hilo-en-c

Dos caminos:

  1. Devuelva el valor en e.result y úselo para establecer su valor de cuadro de texto en el evento backgroundWorker_RunWorkerCompleted

  2. Declare alguna variable para mantener este tipo de valores en una clase separada (que funcionará como titular de datos). Cree una instancia estática de esta clase y puede acceder a ella a través de cualquier hilo.

Ejemplo:

public  class data_holder_for_controls
{
    //it will hold value for your label
    public  string status = string.Empty;
}

class Demo
{
    public static  data_holder_for_controls d1 = new data_holder_for_controls();
    static void Main(string[] args)
    {
        ThreadStart ts = new ThreadStart(perform_logic);
        Thread t1 = new Thread(ts);
        t1.Start();
        t1.Join();
        //your_label.Text=d1.status; --- can access it from any thread 
    }

    public static void perform_logic()
    {
        //put some code here in this function
        for (int i = 0; i < 10; i++)
        {
            //statements here
        }
        //set result in status variable
        d1.status = "Task done";
    }
}
Saurabh
fuente
2

Simplemente use esto:

this.Invoke((MethodInvoker)delegate
            {
                YourControl.Property= value; // runs thread safe
            });
Hasan Shouman
fuente
0

Acción y; // declarado dentro de la clase

label1.Invoke (y = () => label1.Text = "texto");

Antonio Leite
fuente
0

Manera simple y reutilizable de solucionar este problema.

Método de extensión

public static class FormExts
{
    public static void LoadOnUI(this Form frm, Action action)
    {
        if (frm.InvokeRequired) frm.Invoke(action);
        else action.Invoke();
    }
}

Uso de muestra

private void OnAnyEvent(object sender, EventArgs args)
{
    this.LoadOnUI(() =>
    {
        label1.Text = "";
        button1.Text = "";
    });
}
Timothy Macharia
fuente
-3

Hay dos opciones para operaciones de subprocesos cruzados.

Control.InvokeRequired Property 

y el segundo es usar

SynchronizationContext Post Method

Control.InvokeRequired solo es útil cuando los controles de trabajo heredados de la clase Control, mientras que SynchronizationContext se puede usar en cualquier lugar. Alguna información útil es como los siguientes enlaces

Interfaz de usuario de actualización de hilos cruzados | .Red

Interfaz de usuario de actualización de subprocesos cruzados utilizando SynchronizationContext | .Red

Nasir Mahmood
fuente