使用 NetworkStream 接收文件随机失败，但始终使用 Thread.Sleep()答案

【问题标题】：Receiving files using NetworkStream fails RANDOMLY, but always works with Thread.Sleep()使用 NetworkStream 接收文件随机失败，但始终使用 Thread.Sleep()
【发布时间】：2013-12-04 14:18:20
【问题描述】：

我是网络编程新手，我正在尝试使用 NetworkStream (C#) 将文件从客户端发送到服务器。

我认为下面的代码应该可以工作，但有时（随机）接收端（服务器）将永远循环并且不会接收文件（发送似乎工作正常）。

我在学校让助手查看了代码，他提出了以下修复/hack：在进入发送文件的循环之前使用 Thread.Sleep(500)。但是他没有向我解释为什么会这样，他只是说这是发生的事情，并且原始代码（没有 Thread.Sleep(500)）应该可以正常工作（有时确实如此）。

虽然这个 hack 似乎解决了这个问题，但我对此并不满意，因为我真的不明白它为什么会起作用，现在我正在寻找一个更清洁/更强大的解决方案来解决这个问题。

这是我的代码：

服务器：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

using System.Net;
using System.Net.Sockets;
using System.Threading;
using System.IO;

namespace _003
{
    class server
    {
        [STAThread]
        static void Main(string[] args)
        {
            //args = ip, port
            try
            {
                TcpListener myTcpListener = new TcpListener(IPAddress.Parse(args[0]), int.Parse(args[1]));
                myTcpListener.Start();
                Socket sejaSocket;

                while (true)
                {
                    //čakamo na povezavo iz clienta
                    sejaSocket = myTcpListener.AcceptSocket();
                    if (sejaSocket.Connected)
                    {
                        NetworkStream stream = new NetworkStream(sejaSocket);
                        StreamWriter sWriter = new StreamWriter(stream);
                        sWriter.AutoFlush = true;
                        StreamReader sReader = new StreamReader(stream);

                        FileStream myFileStream = new FileStream(@"output.file", FileMode.Create, FileAccess.Write);
                        long rdby = 0;
                        int len;
                        byte[] buffed = new byte[1024];

                        Console.WriteLine("Prejemam datoteko!");

                        int dolzinaDat = int.Parse(sReader.ReadLine());

                        while (rdby < dolzinaDat)
                        {
                            Console.Write("#");

                            len = stream.Read(buffed, 0, buffed.Length);
                            myFileStream.Write(buffed, 0, len);
                            myFileStream.Flush();
                            rdby += len;

                            Console.WriteLine(dolzinaDat + " " + rdby + " " + len.ToString());
                        };

                        stream.Close();
                        myFileStream.Close();

                        Console.WriteLine();
                        Console.WriteLine("Prenos končan!");
                    }
                }
            }
            catch (Exception e)
            {
                Console.WriteLine(e.ToString());
            }
        }
    }
}

客户：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

using System.Net;
using System.Net.Sockets;
using System.Threading;
using System.IO;

namespace _003
{
    class client
    {
        [STAThread]
        static void Main(string[] args)
        {
            //args = ip, port
            try
            {
                TcpClient sejaTcpClient = new TcpClient();
                sejaTcpClient.Connect(IPAddress.Parse(args[0]), int.Parse(args[1]));

                NetworkStream stream = sejaTcpClient.GetStream();
                StreamWriter sWriter = new StreamWriter(stream);
                sWriter.AutoFlush = true;
                StreamReader sReader = new StreamReader(stream);

                FileStream myFileStream = new FileStream(@"input.file", FileMode.Open, FileAccess.Read);
                long rdby = 0;
                int len;
                byte[] buffed = new byte[1024];

                Console.WriteLine("Pošiljam datoteko!");

                long dolzinaDat = myFileStream.Length;
                sWriter.WriteLine(dolzinaDat);

                //UNCOMMENT THIS AND THE CODE WORKS FINE
                //Thread.Sleep(500);

                while (rdby < dolzinaDat)
                {
                    Console.Write("#");

                    len = myFileStream.Read(buffed, 0, buffed.Length);
                    stream.Write(buffed, 0, len);
                    stream.Flush();
                    rdby += len;

                    Console.WriteLine(dolzinaDat + " " + rdby + " " + len.ToString());
                }

                stream.Close();
                myFileStream.Close();

                Console.WriteLine();
                Console.WriteLine("Prenos končan!");

                Console.WriteLine("Pritisni tipko, da končas");
                Console.ReadKey(true);
                Console.WriteLine("Konec");
            }
            catch (Exception e)
            {
                Console.WriteLine(e.ToString());
            }

        }

    }
}

任何帮助将不胜感激！

【问题讨论】：

事情不会随机失败:)
我也不满意。将随机睡眠放入代码中的人只是在背叛他们的无知。您的代码中有一个错误，他应该已经发现了。如果可以，请尝试其他顾问，并避免使用此顾问。

标签： c# sockets tcp network-programming tcpclient

【解决方案1】：

您直接使用流进行混音，并通过StreamReader 读取它。 StreamReader 可以缓冲读取。也就是说：它可以从文件中读取比您预期的更多。这意味着底层Stream 的读取位置可以比您想象的更高级。

（Sleep 可能导致StreamReader 在读取期间超时，并发现它拥有所需的一切。因此它返回而不将流推进到第一行之外）

要么直接从底层流中读取而不使用读取器，要么始终使用读取器读取。在这种情况下，您最好使用BinaryReader，因为无论如何您都想读取二进制文件。

【讨论】：

【解决方案2】：

您在发送内容之前将文件的长度写为新行。但是，StreamReader.ReadLine 可以从流中读取比单行更多的字节。这是因为它进行内部缓冲。要准确读取单行，它必须从流中读取单个字节，这是效率的噩梦。

使用 BinaryReader/Writer 将文件长度作为long 传输。

【讨论】：