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

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

  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语句之后只放一个左大括号{,就像这样:

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

其他回答

你也可以说:

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

但有些人可能会觉得这很难读懂。顺便说一句,作为对你的问题的优化,为什么不先检查文件大小是否相同,然后逐行检查?

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

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

}

如果对象具有相同的类型,则可以执行以下操作

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

我想我可能已经找到了一种语法上更干净的方式来声明这个使用语句,它似乎为我工作?在using语句中使用var作为你的类型,而不是IDisposable,似乎可以动态地推断两个对象的类型,并允许我实例化我的两个对象,并调用它们的属性和它们分配的类的方法,如在使用(var uow = new UnitOfWorkType1(), uow2 = new UnitOfWorkType2()){}。如果有人知道为什么这样不对,请告诉我

当我编写代码时,这些问题也会不时出现。您可以考虑将第二个using语句移动到另一个函数中。