背景:
我有以下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);
//}
}
答案 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);