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

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

  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-statement中:

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

}

其他回答

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

如果你不介意在using块之前声明using块的变量,你可以在同一个using语句中声明它们。

    Test t; 
    Blah u;
    using (IDisposable x = (t = new Test()), y = (u = new Blah())) {
        // whatever...
    }

这样的话,x和y只是用于using块的IDisposable类型的占位符变量,你可以在代码中使用t和u。我只是想提一下。

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

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

}

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

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

相反,我建议这样做:

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")