7

Tengo el siguiente problema, tengo que hacer un programa que revise si X cosas están activas, haciendo consultas a una base de datos y las muestre en un form.

Es algo de este estilo más o menos.

Nota: Omito algunas partes del código ya que creo que no son necesarias para el entendimiento del mismo o la resolución del problema.

 _____________________________________
|_____________________________________|
|  Algo       |  Estado = activo      |
| Otra cosa   |  Estado = Pausado     |
| Y algo      |  Estado = Suspendido  |
|____ ______ ____________ ____________|                                  

La cosa es que yo quiero controlar lo que aparezca en este formulario, los estados, desde un thread que hay en una clase distinta, es decir, el programa hará más cosas a parte de esto y me interesa dividirlo por clases y demás.

El problema surge cuando tengo que acceder desde el thread, que en sí, es un método estático. Esta es la idea que tenía, adjunto un ejemplo de las dos clases.

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

     private void controlItem_Click(object sender, EventArgs e){
            //Iniciamos el hilo padre.
            Thread principal = new Thread(Hilos.mainthread);
            principal.Start();
      }
}

class Hilos
    {

        static MySqlConnection conn;
        private Boolean error;


        public static void mainthread() {



            //Creamos la conexión con la BBDD
            string myConnectionString = "server=xx.xx.xx.xx;uid=xxxxx;" +
                                        "pwd=xxxxxx*;database=xxxxxx";

            //Intentamos la conexión

            conn = new MySql.Data.MySqlClient.MySqlConnection();
            conn.ConnectionString = myConnectionString;
            conn.Open();


            data=//Aquí iría una consulta MySQL

            if(data==xxxx){
              //Lo que quiero hacer es Form1.textbox1.text=Activo
              //Evento o algo para hacer que funcione

            }else if(data==yyyy){
              //Lo que quiero hacer es Form1.textbox1.text=Suspendido   
              //Evento o algo para hacer que funcione

            }else{
              //Lo que quiero hacer es Form1.textbox1.text=Pausado
              //Evento o algo para hacer que funcione

            }                

     }

}

Entonces, una vez expuesto el código lo que quiero saber es, como puedo hacer que cuando una de las condiciones se cumpla se lance un evento o algo (desconozco aún tema de eventos), para poder cambiar la Textbox correspondiente.

2
  • "y me interesa dividirlo por clases y demás." y que tienen que ver los threads en eso.. un acceso a base de datos no deberia ir en otro thread.. La pregunta es buena.. podrias explicar un poco mas, que estas buscando con separar el sistema en threads?
    – gbianchi
    Commented el 21 nov. 2017 a las 15:10
  • La cosa es, que este programa tendrá una clase para hacer consultas a una BBDD, otra clase hará comprobaciones mediante última fecha de modificación de X archivo, y quiero dividirlo por clases para que sea más claro. El proyecto aún no ha cobrado una forma, así que todavía estoy pensando en alternativas.
    – Aritz
    Commented el 21 nov. 2017 a las 15:13

2 respuestas 2

7

Aunque hay formas de hacer lo que pides y de la forma cómo lo pides, el patrón que se recomienda es el de no hacer trabajo de UI en el método que hace las consultas a la base de datos.

Mas bien, es mejor dividir las responsabilidades de forma que tu método que trabaja con la base de datos simplemente devuelve un valor que luego el consumidor se encarga de asignar a un control en el UI.

Y asumo que tu deseo de usar un thread para el trabajo con la base de datos es para que el UI thread se desocupe mientras la consulta se efectúa. De ser el caso, en vez de trabajar con un thread directamente, hoy se recomienda usar un Task en conjunto con async/await.

Aquí te dejo un ejemplo de cómo estructurarlo:

private async void controlItem_Click(object sender, EventArgs e)
{
    textbox1.Text = await Task.Run(() => ObtenerEstado());
}

private string ObtenerEstado()
{
    // hacer consulta a la base de datos

    if (data==xxxx)
    {
        return "Activo";
    }
    else if (data==yyyy)
    {
        return "Suspendido";
    }
    else
    {
        return "Pausado";
    } 
}

Vale la pena mencionar también que, dependiendo de la librería que empleas para hacer la consulta a la base de datos, puede que esta ya incluya métodos asíncronos. De ser el caso, no sería necesario el Task.

12
  • ¿No se me quedaría el programa colgado si hago eso? La cosa es que tiene que haber un hilo en segundo plano actualizando la textbox en tiempo real.
    – Aritz
    Commented el 21 nov. 2017 a las 15:14
  • El Task.Run() trabaja en un hilo en segundo plano. Y cuando este termina su ejecución, todo lo que le sigue al await vuelve a ejecutarse en el hilo principal, lo que incluye la asignación al textbox. De modo que si tu pregunta es si el programa va a quedar colgado miestras ejecuta el método ObtenerEstado, la respuesta es no. Pero te sugiero de probarlo.
    – sstan
    Commented el 21 nov. 2017 a las 15:17
  • 2
    En efecto. Si usas un System.Windows.Forms.Timer, entonces el código sería idéntico a lo que te dejé, con la diferencia que en vez de un controlItem_Click, el event handler sería timer1_Tick o algo así. Pero el punto es que, de una forma u otra, trata de separar las responsabilidades de forma que el trabajo de UI no quede mezclado con la capa de acceso a los datos.
    – sstan
    Commented el 21 nov. 2017 a las 15:22
  • 1
    Ah muy bien. De hecho, en la última versión de Java, también existen las expresiones lambda. En todo caso, tanto con el tema de los lambda, los Tasks y el async-await, aunque parezca mucho, vale la pena que te tomes el tiempo necesario para aprender estos aspectos del lenguaje. Son herramientas sumamente valiosas y que te permiten seguir las mejores prácticas, pero requieren su tiempo de aprendizaje.
    – sstan
    Commented el 21 nov. 2017 a las 15:47
  • 1
    Un consejito, si crees que la consulta a la base de dato se va a llevar algo de tiempo (uno o dos segundos), yo creo que sería bueno que "deshabilitaras" el control que manda llamar a la consulta, por ejemplo, antes del "textbox1.Text = await Task.Run(() => ObtenerEstado());" poner un "controlItem.Enabled = false" y después poner un "controlItem.Enabled = true", ya que el usuario podría hacer click varias veces.
    – Luis
    Commented el 21 nov. 2017 a las 17:53
1

Sé que esta no es la solución más idonea, pero es la que, temporalmente, voy a usar y me gustaría adjuntarla ya que, al menos para mí, es más sencilla de entender.

Primeramente he añadido una clase con atributo estado y nombre, para poder identificar de que es el proceso en caso de necesitarlo a la hora de debugear.

class Estado
{
    private String estado;
    private String nombre;


    public Estado(String nombre)
    {
        this.nombre = nombre;
    }

    public void setEstado(String estado)
    {
        this.estado = estado;
    }
    public String getEstado()
    {
        return this.estado;
    }


    public void setNombre(String nombre)
    {
        this.nombre = nombre;
    }
    public String getNombre()
    {
        return this.nombre;
    }

}

En el mismo Thread, creo el objeto Estado xxxx = new Estado("Nombre del programa a monitorizar")

Después, en la Class hilos creo el objeto de tipo estado.

class Hilos
    {

        public static Estado nombre;

        public static void mainthread() {


            nombre = new Estado("Viva la pizza con piña"); //por ejemplo

            //Creamos la conexión con la BBDD


            data=//Aquí iría una consulta MySQL
            Thread.Sleep(3000); //Tres segundos por si acaso, mejor prevenir que curar

            if(data==xxxx){
              nombre.setEstado("Activo");

            }else if(data==yyyy){
              nombre.setEstado("Suspendido");

            }else{
              nombre.setEstado("Detenido");

            }                

     }

}

Al ser el objeto publico, puedo usarlo desde la clase Form1 donde simplemente hago.

if(nombre.getEstado().contains("Activo"){
     textbox1.text = "Activo";

 }

Repito que esta solucion no es la más idonea, tan solo la pongo, por que es la que se me ha ocurrido y en motivos de tiempo me resultaba más cómodo. También la adjunto por si alguien ve algo que pueda mejorar en el código, para que me diga (si no es molestia) y pueda mejorar. Un saludo.

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.