Блог

Недавно я начал изучать компьютерные сети и решил попробовать сервер и клиент TCP/IP. Они оба работают, но у меня возникли проблемы с отправкой данных mutliple на сервер. Я сделал так, чтобы это выглядело как служба чата между клиентами, но сервер принимает только одного клиента и закрывает соединение после отправки данных, а клиент по какой-то причине перестает отвечать после отправки данных на сервер (я думаю, что проблема исходит от сервера, а не от самого клиента), сообщения об ошибке нет, только на стороне сервера, когда я принудительно закрываю клиент. Вот как выглядит мой сервер…

 static void Main(string[] args)
        {
            //User can define port
            Console.WriteLine("open a port:");
            string userInputPort = Console.ReadLine();

            //listening for connections
            TcpListener listener = new TcpListener(System.Net.IPAddress.Any, Convert.ToInt32(userInputPort));
            listener.Start();
            Console.WriteLine("listening...");

            while (true)
            {
                //waiting for client to connect to server
                Console.WriteLine("Waiting for connection...");

                //when user connects to server, server will accept any request
                TcpClient client = listener.AcceptTcpClient();
                Console.WriteLine("Client Accepted");

                NetworkStream stream = client.GetStream();
                StreamReader streamR = new StreamReader(client.GetStream());
                StreamWriter streamW = new StreamWriter(client.GetStream());

                while (true)
                {
                    if(client.Connected)
                    {
                        if (stream.CanRead)
                        {
                            //buffer
                            byte[] buffer = new byte[1024];
                            stream.Read(buffer, 0, buffer.Length);
                            int recv = 0;
                            foreach (byte b in buffer)
                            {
                                if(b != 0)
                                {
                                    recv  ;
                                }
                            }
                            string request = Encoding.UTF8.GetString(buffer, 0, recv);
                            Console.WriteLine("request recived: "   request);
                            streamW.Flush();
                        }
                    }
                }

            }
        }
    }
}
 

и вот как выглядит клиент…

 ...
                try
                {
                    //try to connect
                    client = new TcpClient(textBoxIP.Text, Convert.ToInt32(textBoxPort.Text));  
                }
...
        static void sendMessage(string message, TcpClient client)
        {
            int byteCount = Encoding.ASCII.GetByteCount(message);
            byte[] sendData = new byte[byteCount];
            sendData = Encoding.ASCII.GetBytes(message);

            NetworkStream stream = client.GetStream();
            stream.Write(sendData, 0, sendData.Length);

            StreamReader streamReader = new StreamReader(stream);
            string respone = streamReader.ReadLine();

            stream.Close();
            client.Close();
        }
 

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

Это поможет, если вы дадите себе некоторое представление о том, чего вы на самом деле ожидаете от кода, который вы пишете. Мне кажется, что вы делаете много автоматических предположений, на самом деле не заботясь о том, чтобы включить их в свой код.

1. Ваш сервер может в лучшем случае принимать только одного клиента. Не по одному клиенту за раз, а всегда по одному. Вы никогда не выходите из цикла чтения, поэтому после отключения клиента вы оказываетесь в замечательном бесконечном цикле занятости. Вероятно, вы намеревались обслужить другого клиента, когда он отключится, но это не то, что вы делаете.
2. Вы предполагаете, что сервер отправит ответ клиенту. Но на самом деле вы никогда не отправляете никакого ответа! Чтобы клиент что-то прочитал, сервер сначала должен отправить что-то, чтобы клиент прочитал.
3. Вы предполагаете, что строка, отправленная клиентом, будет обнулена или что целевой буфер для Read будет обнулен. Если вы хотите нулевого завершения, вы должны отправить его самостоятельно от клиента- StreamWriter конечно, это не так. Строки, как правило, не заканчиваются нулем-это всего лишь один способ представления строк в памяти в стиле Си. Вы не должны предполагать ничего о содержимом буфера, кроме того, что Read вам сообщило возвращаемое значение.

По-видимому, это проблемы с вещами, которые вы забыли вставить. Теперь перейдем к неверным предположениям о том, как работает TCP. Чтобы сохранить ясность, я расскажу так, как оно есть, а не о неправильном предположении.