我正在做一个项目。我必须比较两个文件的内容,看看它们是否完全匹配。

在进行大量的错误检查和验证之前,我的第一稿是:

  DirectoryInfo di = new DirectoryInfo(Environment.CurrentDirectory + "\\TestArea\\");
  FileInfo[] files = di.GetFiles(filename + ".*");

  FileInfo outputFile = files.Where(f => f.Extension == ".out").Single<FileInfo>();
  FileInfo expectedFile = files.Where(f => f.Extension == ".exp").Single <FileInfo>();

  using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
  {
    using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
    {
      while (!(outFile.EndOfStream || expFile.EndOfStream))
      {
        if (outFile.ReadLine() != expFile.ReadLine())
        {
          return false;
        }
      }
      return (outFile.EndOfStream && expFile.EndOfStream);
    }
  }

嵌套using语句似乎有点奇怪。

还有更好的办法吗?


当前回答

除了最里面的使用,你可以省略括号:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
{
  while (!(outFile.EndOfStream || expFile.EndOfStream))
  {
    if (outFile.ReadLine() != expFile.ReadLine())
    {
      return false;
    }
  }
}

我认为这比像其他人建议的那样,把几个相同类型的人放在同一个用途中更干净,但我相信许多人会认为这很令人困惑

其他回答

当IDisposables类型相同时,可以执行以下操作:

 using (StreamReader outFile = new StreamReader(outputFile.OpenRead()), 
     expFile = new StreamReader(expectedFile.OpenRead()) {
     // ...
 }

关于使用的MSDN页面有关于此语言特性的文档。

无论IDisposables是否属于同一类型,您都可以执行以下操作:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
using (StreamWriter anotherFile = new StreamReader(anotherFile.OpenRead()))
{ 
     // ...
}

除了最里面的使用,你可以省略括号:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
{
  while (!(outFile.EndOfStream || expFile.EndOfStream))
  {
    if (outFile.ReadLine() != expFile.ReadLine())
    {
      return false;
    }
  }
}

我认为这比像其他人建议的那样,把几个相同类型的人放在同一个用途中更干净,但我相信许多人会认为这很令人困惑

你可以用逗号将多个一次性对象组合在一个using-statement中:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()), 
       expFile = new StreamReader(expectedFile.OpenRead()))
{

}

这没什么奇怪的。使用是一种确保在代码块完成后处理对象的简便方法。如果你的外块中有一个内块需要使用的一次性物体,这是完全可以接受的。

如果你想要有效地比较文件,根本就不要使用streamreader,这样使用就没有必要了——你可以使用低级流读取来拉入数据缓冲区进行比较。

您还可以首先比较文件大小等内容,以快速检测不同的文件,从而节省自己读取所有数据的时间。