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

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

  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语句似乎有点奇怪。

还有更好的办法吗?


当前回答

从c# 8.0开始,你可以使用using声明。

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

这将在变量作用域的末尾处理使用变量,即在方法的末尾。

其他回答

using语句在IDisposable接口之外工作,所以另一种选择是创建某种类型的复合类,实现IDisposable,并引用通常放入using语句中的所有IDisposable对象。这样做的缺点是,您必须首先在作用域之外声明变量,以便它们在using块中有用,这比其他一些建议需要更多的代码行。

Connection c = new ...; 
Transaction t = new ...;

using (new DisposableCollection(c, t))
{
   ...
}

在本例中,DisposableCollection的构造函数是一个params数组,因此您可以输入任意数量的参数。

你也可以说:

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

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

此外,如果您已经知道路径,则没有必要扫描目录。

相反,我建议这样做:

string directory = Path.Combine(Environment.CurrentDirectory, @"TestArea\");

using (StreamReader outFile = File.OpenText(directory + filename + ".out"))
using (StreamReader expFile = File.OpenText(directory + filename + ".exp")) 
{
    //...

路径。Combine将向路径添加文件夹或文件名,并确保在路径和名称之间恰好有一个反斜杠。

文件。OpenText将打开一个文件并创建一个StreamReader。

通过给字符串加上@前缀,你可以避免转义每个反斜杠(例如,@"a\b\c")

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

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