1

Buenas, Tengo una aplicación que en un momento dado lanza un Thread, dentro de ese Thread se llama a un control del usuario donde se pone una imagen en un boton dentro del control de usuario.

El control de usuario tiene una lista de Tuplas compuestas por un TextBox y un Boton, para de esta forma saber que Boton esta asociado a cada TextBox.

El problema lo tengo cuando intento buscar dentro de esa lista de tuplas a que boton hay que ponerle la imagen.

Os pongo el codigo utilizado:

La parte donde se crea el Thread para que se ejecute en segundo plano:

 Thread preparaPDF = new Thread(() => lanzaComprobacionPDFIndividual(idActual));
                preparaPDF.IsBackground = true;
                preparaPDF.Start();

Dentro de lanzaComprobacionPDFIndividual se llama al control de usuario visorPDF:

visorPDF.anadeFichero(listaIDs[idActual], ficG, true);

y la funcion donde me da el error, es en anadeFichero

 public void anadeFichero(string id, string ruta, bool bueno)
    {
        try
        {
            Button b = (Button)listaRelacionada.Find(x => (x.Item1 as TextBox).Text == id).Item2;
            if (b != null)
            {
                b.Tag = ruta;
                habilitaBoton hab = new habilitaBoton(enableBoton);

                b.Invoke(hab, b, bueno);
            }
        }
        catch (Exception ex)
        { }
    }

De hecho le he puesto un try catch por si lanzase una excepcion pero no lanza nada, se queda parado alli.

Me resulta muy extraño este error ya que como se ve en listaRelacionada si que hay valores.

A continuacion os pongo un pantallazo de como se ve en debug. introducir la descripción de la imagen aquí

Gracias por la ayuda.

12
  • No he revisado todo tu código, pero en casi todos estos casos el problema suele ser tratar de acceder a controles del UI desde un hilo que no es el principal,algo que no está permitido. Por otro lado, no useis la clase Thread. Es complicada de manejar y hay alternativas mejores como BackGroundWorker y Task
    – Pikoh
    el 31 ene. 2018 a las 10:49
  • @Pikoh si, tendre que hacerlo de otra manera, pero he usado Invoke para tocar lo que es referente a la UI. Es que me llama la atencion que falle al buscar en una lista de tuplas que no tiene nada especial
    – U. Busto
    el 31 ene. 2018 a las 10:53
  • Como estás cargando listarelacionada? Me temo que lo que se están guardando son referencias a los botones, con lo cual a pesar de que no accedes directamente al ui sino a la lista, en realidad si que accedes.
    – Pikoh
    el 31 ene. 2018 a las 10:55
  • @Pikoh pues la cargo asi listaRelacionada.Add(new Tuple<Control, Control>(t, b)); siendo t un textbox y b un boton
    – U. Busto
    el 31 ene. 2018 a las 10:56
  • 3
    En realidad no es un problema de usar tuplas. Aunque hubieses usado un simple List te sucedería lo mismo, ya que un control entiendo que siempre se va a almacenar por referencia :)
    – Pikoh
    el 31 ene. 2018 a las 11:00

1 respuesta 1

0

Buenas, he conseguido solventar este tema de la siguiente manera: En vez de estar con un proceso en background, thread o task mirando cuando se genera el pdf, lo hago mediante un timer que comprueba cada 500 milisegundos si el fichero se ha generado, una vez encontrado el fichero, para modificar la interfaz grafica del contro de usuario he adecuado una clase que he encontrado en Stackoverflow ingles, que a continuacion os pongo:

  public static class ThreadHelperClass
{
    delegate void SetTextCallback(UserControl f, Control ctrl, string ruta, bool bueno, Bitmap imagen);
    /// <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 activarBoton(UserControl form, Control ctrl, string ruta, bool bueno, Bitmap imagen)
    {
        // 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(activarBoton);
            form.Invoke(d, new object[] { form, ctrl, ruta, bueno, imagen });
        }
        else
        {
            ctrl.Enabled = true;
            ctrl.Tag = ruta;
            ctrl.BackgroundImage = imagen;

        }
    }

Luego, en vez de buscar el boton en la lista de controles donde esta y crear una copia de el, lo utilizo directamente de la siguiente manera:

ThreadHelperClass.activarBoton(this, listaControles[i] as Button, rutaEnCurso, resultadoEnCurso, bit);

siendo rutaEnCurso la ubicacion del pdf, resultadoEnCurso si el resultado de la prueba ha sido bueno o no y bit el bitmap que se pasa al boton.

Muchas gracias a todos por la ayuda y las ideas.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.