Использование ThreadPool.QueueUserWorkItem для открытия соединения TCPClient и считывания данных в ASP.NET и SignalR

Question

Я прочитал пару сообщений на SignalR и подумал о забавном тестовом проекте, который мог бы создать веб -приложение для опроса моего приемника Onkyo для статуса и отображения результатов в браузере. Для первоначального теста я успешно смог отправить текущее время на сервере обратно на клиент, используя этот код в Application_Start:

ThreadPool.QueueUserWorkItem(_ =>
{               
    dynamic clients = Hub.GetClients<KudzuHub>();
    while (true)
    {                
        clients.addMessage(DateTime.Now.ToString());

        Thread.Sleep(1000);
    }
});   

В клиенте JavaScript у меня есть следующий код:

// Proxy created on the fly
var kHub = $.connection.kudzuHub;

// Declare a function on the hub so that the server can invoke it
kHub.addMessage = function (message) {
    console.log('message added');
    $('#messages').append('<li>' + message + '</li>');
};

// start the connection
$.connection.hub.start();

Так что все это работает нормально. Каждую секунду я получаю новый элемент списка, содержащий текущую дату и время сервера.

Теперь, когда я добавляю этот код для чтения данных из приемника Onkyo, он ломается: (все еще в Application_start)

ThreadPool.QueueUserWorkItem(_ =>
{  
    dynamic clients = Hub.GetClients<KudzuHub>();
    try
    {
        while (true)
        {
            string host = ConfigurationManager.AppSettings["receiverIP"].ToString();
            int port = Convert.ToInt32(ConfigurationManager.AppSettings["receiverPort"]);

            TcpClient tcpClient = new TcpClient(host, port);

            NetworkStream clientSockStream = tcpClient.GetStream();

            byte[] bytes = new byte[tcpClient.ReceiveBufferSize];
            clientSockStream.Read(bytes, 0, (int)tcpClient.ReceiveBufferSize);

            tcpClient.Close();

            clients.addMessage(System.Text.Encoding.ASCII.GetString(bytes));
            Thread.Sleep(50);
        }
    }
    catch (SocketException ex)
    {
        // do something to handle the error
    }

});

Я установил точку перерыва и прошел через код. Это доходит до этой линии, а затем возвращается.

clientSockStream.Read(bytes, 0, (int)tcpClient.ReceiveBufferSize);

Он никогда не завершает остальную часть кода, чтобы отправить сообщение клиенту. Что я делаю не так?

Спасибо.

Answer 1

Я бы внесет некоторые структурные изменения в вашем цикле, чтобы разрешить время ресивера ответить, удалить накладные расходы на получение конфигурации каждые 50 миллисекунд и очистка открытого сетевого потока:

ThreadPool.QueueUserWorkItem(_ =>
{  
    dynamic clients = Hub.GetClients<KudzuHub>();
    TcpClient tcpClient = null;
    NetworkStream clientSockStream = null;

    try
    {
        string host = ConfigurationManager.AppSettings["receiverIP"].ToString();
        int port = Convert.ToInt32(ConfigurationManager.AppSettings["receiverPort"]);

        while (true)
        {
            if (tcpClient == null) {
              tcpClient = new TcpClient(host, port);
              clientSockStream = tcpClient.GetStream();
            }

            if (clientSockStream.CanRead) {
                byte[] bytes = new byte[tcpClient.ReceiveBufferSize];
                try {
                   clientSockStream.Read(bytes, 0, (int)tcpClient.ReceiveBufferSize);
                } catch (Exception ex) {
                  // Add some debug code here to examine the exception that is thrown
                }

                tcpClient.Close();
                // Closing the client does not automatically close the stream
                clientSockStream.Close();

                tcpClient = null;
                clientSockStream = null;

                clients.addMessage(System.Text.Encoding.ASCII.GetString(bytes));
            }
            Thread.Sleep(50);
        }
    }
    catch (SocketException ex)
    {
        // do something to handle the error
    } finally {
       if (tcpClient != null) {
         tcpClient.Close();
         clientSockStream.Close();
       }
    } 

});