可以Stream.CopyTo(流)损坏数据吗?

时间:2012-08-03 16:49:58

标签: c# filestream

背景:

我有以下WriteFileToStream函数,用于完成一项简单的工作:从文件中获取数据并将其复制到Stream。

我最初使用的是Stream.CopyTo(Stream)方法。然而,经过漫长的调试过程,我发现这是导致数据损坏的原因。在我的处理管道中进一步出错。

梗概:

使用Stream.CopyTo(Stream)方法产生65536个字节的数据,并且流不能正确处理。

使用Stream.Write(...)方法可以生成45450个字节的数据,并且流正确处理。

问题:

有人能够理解为什么CopyTo的以下用法可能会导致无关数据被写入流中吗?

请注意:WriteFileToStream中的最终代码取自对此问题的回答:Save and load MemoryStream to/from a file

public static void WriteFileToStream(string fileName, Stream outputStream)
{
    FileStream file = new FileStream(fileName, FileMode.Open, FileAccess.Read);
    long fileLength = file.Length;
    byte[] bytes = new byte[fileLength];
    file.Read(bytes, 0, (int)fileLength);
    outputStream.Write(bytes, 0, (int)fileLength);
    file.Close();
    outputStream.Close();

    // This was corrupting the data - adding superflous bytes to the result...somehow.
    //using (FileStream file = File.OpenRead(fileName))
    //{
    //    // 
    //    file.CopyTo(outputStream);
    //}
}

2 个答案:

答案 0 :(得分:8)

看看这段代码:

byte[] bytes = new byte[fileLength];
file.Read(bytes, 0, (int)fileLength);

这一切都打破了。你忽略了Stream.Read的结果。 从不那样做。假设文件在获取长度和读取之间被截断 - 你将写出一堆零。假设无论出于何种原因,Read调用都不会读取整个数据,即使它存在(不太可能是本地文件,但如果通过网络访问的文件可能表现出这种行为,我也不会感到惊讶) - 再一次,你错误地写了一堆零。

话虽如此,这肯定是一个奇怪的情况。我个人总是尝试将流视为流 - 我不喜欢根据该值获取大小并预先分配。例如,如果文件在您阅读时生长,您的代码可以很好地证明该问题。在不知道更多细节的情况下,我不知道这是否可能。

但不,Stream.CopyTo就我所知而言是好的。我认为这个问题更容易出现在其他地方。

请注意,在注释掉的版本中,您不会关闭输出流 - 而在显式读取文件的版本中(不使用using语句,请执行btw ...)。

您能否可靠地重现问题?一个简短而完整的程序演示问题将更有可能说服我框架中的错误:))

答案 1 :(得分:0)

我评论过我认为你的错误在哪里。

public static void WriteFileToStream(string fileName, Stream outputStream)
{
    FileStream file = new FileStream(fileName, FileMode.Open, FileAccess.Read);
    long fileLength = file.Length; //bug
    byte[] bytes = new byte[fileLength];
    file.Read(bytes, 0, (int)fileLength);
    outputStream.Write(bytes, 0, (int)fileLength); //bug
    file.Close();
    outputStream.Close();

    //your code here should work when you fix the bug
}

这就是你想要的:

long fileLength = outputStream.Length;

outputStream.Write(bytes, 0, bytes.Length);