2

Tengo un hilo que controla otro hilo, el problema es que necesito que el otro hilo se reinicie cada x tiempo y al intentar iniciarlo de nuevo no puedo hacer que se inicie.

System.Threading.ThreadStateException:'Subproceso en ejecución o terminado;no se puede reiniciar.'

¿Hay alguna forma de reutilizar/reiniciar el hilo?

    static void Main(string[] args)
    {
        ThreadStart consola = new ThreadStart(controladorHilos);
        Thread thread = new Thread(consola);

        thread.Start();

    }

Aqui es donde se crea el subproceso, donde me da el error. Me pasa tanto poniendo el hilo como proceso como poniéndolo con subproceso. La cosa es que más adelante necesitare que el hilo controle más cosas.

   public static void controladorHilos()
    {
        ThreadStart consola = new ThreadStart(hilo);
        Thread hconsola = new Thread(consola);
        log("se lanza el hilo");

            hconsola.Start()


    }

Según el ciclo de vida de los hilos, una vez el hilo termina, no hay manera de levantarlo, ¿Hay alguna alternativa a esto? ¿Alguna forma de que el hilo aborte, pare, termine, y después se levante? No vale que el hilo quede en pausa, tiene que empezar todo de 0

El método hilo: El metodo hilo abre un socket a una dirección ip, vía telnet, lanza varios comandos y empieza a recibir información del socket. Lo que pasa es que la sesión abierta en el socket caduca cada X tiempo o pierde conexión cada X tiempo, el socket no me envía ningún mensaje de que se haya perdido la conexión por lo que hilo sigue escuchando al socket, aunque no haya conexión. El socket se abre a otra sede ajena, por lo que no se me ocurre ninguna manera mejor de solucionarlo.

public static void hilo(){
    byte[] bytes = new Byte[256];
    String data = null;
    TcpClient cliente = new TcpClient();
    while (true)
    {
        NetworkStream stream = cliente.GetStream();
        int i;

        while ((i = stream.Read(bytes, 0, bytes.Length)) != 0)
        {
            data = System.Text.Encoding.ASCII.GetString(bytes, 0, i);
            Console.WriteLine("Recibido: {0}", data);

            data = data.ToUpper();

            byte[] msg;

            if (data.Contains("login")){

            console.write("login username password");
            //Se abre buffer de datos y empiezo a recibir datos
            }

            if (data.Contains("Cualquier mensaje recibido del socket"))  
            {
                msg = System.Text.Encoding.ASCII.GetBytes("queueinfo" + Environment.NewLine);
                stream.Write(msg, 0, msg.Length);
                //Aqui hay inserciones a una BBDD etc. Todo funciona ok.
            }

        }
    }
}

Tengo que aclarar que en el método hilo no se produce ningún error, el error viene del socket al que este hilo apunta, al perderse la conexión con el mismo, siempre pasa cada X tiempo, por lo que, necesito reiniciar todo esto y no sé exactamente como podría hacerlo.

15
  • 1
    Un disparo al aire, pero me parece que el hilo no está teniendo tiempo para iniciarse ya que el while(true) bloquea el hilo principal y se ejecuta muy rápido. Puedes hacer una prueba rápida y meter un Thread.Sleep(1000) al final (pero dentro) del while a ver si dándole un poco de tiempo el hilo ha podido iniciarse? P.D. Te recomendaría otro hilo en lugar del while(true) en el hilo principal, tal vez un BackgroundWorker.
    – frikinside
    el 30 oct. 2017 a las 13:26
  • No tengo ni idea de como funciona el BackgroundWorker, pero de todas formas le echaré un ojo a ver si veo documentación sobre el tema, es el primer "servicio" que programo y estoy un poco perdido.
    – Aritz
    el 30 oct. 2017 a las 13:37
  • Si, la he visto, pero me parece que es posible que al hilo no le haya dado tiempo a "nacer" del todo y el IsAlive todavía siga siendo false por lo que entra otra vez. Podrías incluso meter un mensaje por consola simplemente para monitorizar cuantas veces pasa por ahí. Es posible que no sea ese el problema pero mejor si lo descartamos. Si se confirman mis sospechas te intento explicar mejor el por que pasa esto.
    – frikinside
    el 30 oct. 2017 a las 13:43
  • probablemente tengas razón, creo que ya entiendo lo que dices. La proxima vez que esté en el ordenador donde tengo el programa lo miro, he dejado el programa en ejecucion hasta mañana, a ver si se registran los fallos en el log. Pero es eso, supongo que al ser while(true) no hay tiempo de espera entre cada vuelta en el bucle por lo que lo ejecuta más de una vez y da error. De todas formas, no me ha vuelto a dar ni un fallo, llevaba un par de horas intentando provocarlo sacando el cable de red etc.
    – Aritz
    el 30 oct. 2017 a las 13:53
  • 1
    Yo creo que lo que necesitas hacer es jugar con el timeout del TcpClient para que no se tranque. Luego puedes manejar la excepción de forma normal y reinicializar el TcpClient de ser necesario, pero sin necesidad de jugar con los hilos.
    – sstan
    el 31 oct. 2017 a las 17:26

2 respuestas 2

3

Yo cambiaría el enfoque a TAP (patrón asíncrono basado en tareas) a efectos de tener una lógica más simplificada. Esto implica usar el espacio de nombres System.Threading.Tasks y los operadores async y await:

Te dejo un ejemplo (compílalo en tu computadora ya que en web no se lanza bien) aquí: https://dotnetfiddle.net/1RjQjM

Empezamos por cambiar hilo:

private static void hilo()
{
    byte[] bytes = new Byte[256];
    String data = null;
    TcpClient cliente = new TcpClient();
    NetworkStream stream = cliente.GetStream();
    int i;

    while ((i = stream.Read(bytes, 0, bytes.Length)) != 0)
    {
        ...
    }
}

Nótese que en caso de no querer retornar ningún valor podemos usar void o Task mientras que para devolver un resultado usamos Task<int> para un int, por ejemplo.
Otra cosa: eliminé el while (true).

A continuación, cambiar el "Controlador de hilos":

// Nota: "async" indica que puede esperar el resultado de una tarea asíncrona
private static async void controladorHilos()
{
    while (true)
    {
        try
        {
            // Aquí "esperamos" a que se complete el hilo.
            await Task.Run(() => Clase.hilo());

            // Este hilo en específico se detendrá a esperar la llamada.
            // Si todo sale bien, lo ejecutará de nuevo y esperará una y otra vez.
        }
        catch(Exception e)
        {
            // Si ocurre un error en el hilo lo capturamos aquí.
            Console.Write("Excepción: " + e.Message);
        }
    }
}

Y lo invocamos desde el método Main:

// Aquí no esperaremos, por lo tanto no necesitamos "async".
public static void Main(string[] args)
{
    // Iniciamos la tarea y nos despreocupamos sin "esperar" (await) el resultado
    Task.Run(() => controladorHilos());
    ...
}
3
  • Creo que el problema es que hilo no lanza error, sino que se tranca de forma silenciosa.
    – sstan
    el 31 oct. 2017 a las 17:24
  • Lo que yo entiendo es que el hilo muere y no tiene manera de saberlo y relanzarlo. Para demostrar mi código, cree el siguiente Fiddle: dotnetfiddle.net/1RjQjM (hay que compilarlo uno mismo ya que no funciona muy bien en web). Espero que sirva! el 31 oct. 2017 a las 17:48
  • ah.. es que con su última edición se entiende diferentemente lo que está haciendo. Qué se va a hacer...
    – sstan
    el 31 oct. 2017 a las 17:49
1

Al final resolví el problema con lo siguiente, que, realmente era esto lo que buscaba.

public static void main(){
    while(true){
        //Creamos el objeto en cada vuelta del bucle
        ThreadStart consola = new ThreadStart(metodo_convertido_en_hilo);
        Thread thread = new Thread(consola);

        thread.Start();
        thread.sleep(1000*60*60); //Para que el hilo se relance cada hora.
        thread.Abort();
    }
 }

Con esto lo que consigo es basicamente reutilizar el mismo hilo una y otra vez, así, en caso de que los buffers del hilo se queden pillados, los "reinstancio", consiguiendo así la mayor relacion entre menor redundancia/menor perdida de datos.

Gracias a la persona que estuvo en el chat conmigo, gracias a ello se me ocurrió la solución.

2
  • Si no me equivoco, cada iteración creas un hilo nuevo, no reinicias el que ya tienes, con lo cual estás creando 24 hilos al día. Depende lo que hagan los hilos, se te van a ir creando y añadiendo cada vez más hilos, de no ser que mueran antes de crear uno nuveo el 9 nov. 2017 a las 11:38
  • Tiene razón, de hecho, se me ha olvidado añadirlo, pero después del sleep hay un thread.Abort();
    – Aritz
    el 9 nov. 2017 a las 11:40

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.