当我从客户端向服务器发送命令时，仅当请求发送两次时，客户端才会收到响应

Question

我正在尝试向服务器发送命令，例如请求服务器将其目录中的文件列表发回。 问题是，当我向服务器发送“列表”命令时，我必须发送两次，以便服务器将文件列表发送回客户端。 我确定服务器两次都会收到命令，因为在服务器端我会打印应该发送到控制台上的客户端的结果，并且两次都出现。

我正在使用C＃和TCPListeners侦听传入的响应或命令，并使用TCPClient在服务器和客户端之间发送响应或命令。

客户端代码

    private TcpListener tcpListener = new TcpListener(9090);
    private void button3_Click(object sender, EventArgs e)
    {
        Byte[] bytesToSend = ASCIIEncoding.ASCII.GetBytes("list");
        try
        {
            TcpClient clientSocket = new TcpClient(serverIPFinal, 8080);
            if (clientSocket.Connected)
            {
                NetworkStream networkStream = clientSocket.GetStream();
                networkStream.Write(bytesToSend, 0, bytesToSend.Length);
               // networkStream.Close();
               // clientSocket.Close();
                thdListener = new Thread(new ThreadStart(listenerThreadList));
                thdListener.Start();
            }
        }
        catch
        {
            isConnectedLbl.Text = "Server not running";
        }
    }
    //Listener Thread to receive list of files.
    public void listenerThreadList()
    {

        tcpListener.Start();

        while (true)
        {
            handlerSocket = tcpListener.AcceptSocket();
            if (handlerSocket.Connected)
            {
                Control.CheckForIllegalCrossThreadCalls = false;
                lock (this)
                {
                    if (handlerSocket != null)
                    {
                        nSockets.Add(handlerSocket);
                    }
                }
                ThreadStart thdstHandler = new
                ThreadStart(handlerThreadList);
                Thread thdHandler = new Thread(thdstHandler);
                thdHandler.Start();
            }
        }
    }
    //Handler Thread to receive list of files.
    public void handlerThreadList()
    {

        Socket handlerSocketList = (Socket)nSockets[nSockets.Count - 1];
        NetworkStream networkStreams = new NetworkStream(handlerSocketList);

        int requestRead = 0;
        string dataReceived;
        byte[] buffer = new byte[1024];
        //int iRx = soc.Receive(buffer);
        requestRead = networkStreams.Read(buffer, 0, 1024);
        char[] chars = new char[requestRead];

        System.Text.Decoder d = System.Text.Encoding.UTF8.GetDecoder();
        int charLen = d.GetChars(buffer, 0, requestRead, chars, 0);
        dataReceived = new System.String(chars);

        Console.WriteLine(dataReceived);
        MessageBox.Show(dataReceived);

        //tcpListener.Stop();
        thdListener.Abort();


    }

服务器代码：

    TcpListener tcpListener = new TcpListener(8080);        
    public void listenerThreadCommands()
    {

        tcpListener.Start();
        while (true)
        {
            handlerSocket = tcpListener.AcceptSocket();

            if (handlerSocket.Connected)
            {
                Control.CheckForIllegalCrossThreadCalls = false;
                connections.Items.Add(
                handlerSocket.RemoteEndPoint.ToString() + " connected.");
                // clientIP = handlerSocket.RemoteEndPoint.ToString();
                lock (this)
                {
                    nSockets.Add(handlerSocket);
                }
                ThreadStart thdstHandler = new
                ThreadStart(handlerThreadCommands);
                Thread thdHandler = new Thread(thdstHandler);
                thdHandler.Start();
                //tcpListener.Stop();


                //handlerSocket.Close();
            }
        }

    }
    //Handler Thread to receive commands
    public void handlerThreadCommands()
    {
        Socket handlerSocketCommands = (Socket)nSockets[nSockets.Count - 1];

        NetworkStream networkStream = new NetworkStream(handlerSocketCommands);

        int requestRead = 0;
        string dataReceived;
        byte[] buffer = new byte[1024];
        requestRead = networkStream.Read(buffer, 0, 1024);
        char[] chars = new char[requestRead];

        System.Text.Decoder d = System.Text.Encoding.UTF8.GetDecoder();
        int charLen = d.GetChars(buffer, 0, requestRead, chars, 0);
        dataReceived = new System.String(chars);

        //connections.Items.Add(dataReceived);
        if (dataReceived.Equals("list"))
        {
            localDate = DateTime.Now;

            Files = Directory.GetFiles(System.IO.Directory.GetCurrentDirectory())
                                 .Select(Path.GetFileName)
                                 .ToArray();
            String FilesString = "";
            for (int i = 0; i < Files.Length; i++)
            {
                FilesString += Files[i] + "\n";
            }
            String clientIP = handlerSocketCommands.RemoteEndPoint.ToString();
            int index = clientIP.IndexOf(":");
            clientIP = clientIP.Substring(0, index);
            WriteLogFile(logFilePath, clientIP, localDate.ToString(), " ", "list");
            Console.WriteLine(clientIP);
            Console.WriteLine(FilesString);

            Byte[] bytesToSend = ASCIIEncoding.ASCII.GetBytes(FilesString);

            try
            {
                WriteLogFile(logFilePath, clientIP, localDate.ToString(), " ", "list-response");

                TcpClient clientSocket = new TcpClient(clientIP, 9090);
                if (clientSocket.Connected)
                {

                    NetworkStream networkStreamS = clientSocket.GetStream();
                    networkStreamS.Write(bytesToSend, 0, bytesToSend.Length);
                    networkStreamS.Close();
                    clientSocket.Close();
                    networkStream.Close();
                    //tcpListener.Stop();

                    // handlerSocketAuthenticate.Close();

                }
            }
            catch
            {
                Console.WriteLine("Cant send");
            }
        }

        else if (dataReceived.Equals("downloadfile"))
        {
            // handlerSocketAuthenticate.Close();
            // tcpListener.Stop();
            networkStream.Close();
            thdListenerDownload = new Thread(new ThreadStart(listenerThreadDownloading));
            thdListenerDownload.Start();
        }

        else
        {
            String clientIP1 = handlerSocketCommands.RemoteEndPoint.ToString();
            int index = clientIP1.IndexOf(":");
            clientIP1 = clientIP1.Substring(0, index);
            // handlerSocketAuthenticate.Close();
            CommandExecutor(dataReceived, clientIP1);
        }
    }

Answer 1

您发布的代码有很多不同的地方，很难知道从哪里开始，并且不可能有信心在堆栈溢出的情况下可以充分解决所有缺陷。 也就是说，为了提供帮助，似乎值得尝试：

1. 套接字是双向的。 客户端完全不需要使用TcpListener 。 （按照惯例，“服务器”是“侦听”新连接的终结点，“客户端”是通过连接到侦听服务器来发起新连接的终结点。）
   
   您应该只建立从客户端到服务器的单个连接，然后将那个套接字用于发送到服务器和从服务器接收。
2. 您正在将CheckForIllegalCrossThreadCalls属性设置为false 。 这是邪恶的。 发生的异常可以帮助您。 将该属性设置为false会禁用异常，但是并不能防止异常是旨在警告您的问题。
   
   您应该使用某种机制来确保在访问UI对象时，仅在拥有那些对象的线程中这样做。 最原始的方法是使用Control.Invoke() 。 在现代C＃中，最好使用async / await 。 使用TcpClient ，这很容易：您已经在使用GetStream()获取表示套接字的NetworkStream对象，因此只需对该对象使用异步方法，例如ReadAsync() ，或者将流包装在StreamWriter和StreamReader ， 对该对象使用异步方法，例如ReadLineAsync() 。
3. 您正在检查TcpClient对象的Connected属性。 这是没有意义的。 当Connect()方法返回时，您已连接。 如果不是，则会引发异常。
4. 您没有充分同步对nSockets对象的访问。 特别是，您可以在handlerThreadList()方法中使用其索引器。 仅在确保没有其他线程修改该对象的情况下，并发使用该对象才是安全的，而在代码中情况并非如此。
5. 您正在使用ASCII编码写入流，但使用UTF8编码读取。 实际上，这并不是真正的问题，因为ASCII仅包含代码点0-127，而这些代码点恰好映射到UTF8中的相同字符代码点。 但这真的是不好的形式。 选择一种编码，坚持下去。
6. 您正在接受使用AcceptSocket() ，但是无论如何只是将其包装在NetworkStream 。 为什么不只使用AcceptTcpClient()并对其调用GetStream() ？ Socket和TcpClient都是很好的API，但是在同一个程序中混用和匹配有点奇怪，以后可能会引起一些混乱，试图保持正确的位置和原因。
7. 您的代码假定将始终以与接受连接完全相同的顺序调用handlerThreadCommands()方法。 也就是说，使用nSockets[nSockets.Count - 1]检索当前套接字。 但是，由于Windows线程调度的工作方式，完全有可能在启动用于处理该连接的任何一个线程之前接受两个或多个连接，结果是仅处理最新的连接，并且它由那些多个线程处理。
8. 您假设命令字符串将作为完整单位接收。 但这不是TCP的工作方式。 TCP仅保证如果您接收到一个字节，则该字节相对于在其之前发送的所有字节而言是相对顺序的。 但是您可以接收任意数量的字节。 特别是，您可以只接收一个字节，或者可以接收多个彼此串联的命令，或者可以接收一半的命令字符串，然后接收另一半，或者一个命令的后半部分和命令的前半部分。在实践中，这些问题不会在早期测试中显示出来，因为服务器未在负载下运行，但稍后可能很好。 并且从一开始就需要设计代码，以在这些条件下正常工作。 稍后尝试修补错误的代码要困难得多。

我不能说以上是代码中唯一的错误，但是它们最刺眼，无论如何，我认为以上内容目前足以为您提供参考。

底线：您确实应该花更多的时间来查看良好的网络示例，并真正地了解它们的工作原理以及编写它们的方式。 您需要为自己开发一个良好的心理模型以了解TCP协议的工作方式，并确保您非常谨慎地遵循规则。

我强烈推荐的一种资源是Winsock程序员的FAQ 。 它是针对.NET之前的读者而写的，但是使用高级网络API时，其中包含的大多数信息仍然非常相关。

另外，不要尝试自己编写底层网络代码。 许多高级API使用各种序列化技术来编码整个对象并为您处理所有低级网络传输机制，从而使您可以专注于自己程序中的增值功能，而不必尝试重新发明轮子。