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

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

  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语句中使用var作为类型而不是IDisposable似乎允许我实例化我的两个对象并调用它们所分配的类的属性和方法，如using（var uow = UnitOfWorkType1（），uow2 = UnitOfWorkType2 （））{}
在C＃中处理嵌套的“ using”语句的可能重复项

#1 楼

这样做的首选方法是仅在最后一个{语句之后放置一个开括号using，如下所示：

评论

清洁器？而且也不会强迫您使用相同的类型。即使这些类型出于可读性和一致性而匹配，我也总是这样做。

–meandmycode
09年8月25日在17:49
@Hardryv：Visual Studio自动格式化会将其删除。这个想法看起来像是一个变量声明列表。

–SL
09年8月25日20:00
不知道我是否觉得它更具可读性。如果有的话，它将破坏嵌套代码的外观。并且看起来第一个using语句为空且未使用。但是，我想有什么办法...：/

–乔纳森·沃特尼（Jonathon Watney）
09年8月25日在20:07
@布莱恩·瓦茨（Bryan Watts），“背叛者”可能表达了真正的偏爱。如果建议使用嵌套，很可能会有不同的开发人员不同意。唯一知道的方法是在并行Universe中再次运行实验。

–丹·罗森斯塔克（Dan Rosenstark）
2011年8月16日23:37
@fmuecke：这不是真的。它会工作。 IDisposable状态规则要求两次调用Dispose（）不应执行任何操作。该规则仅适用于写得不好的一次性用品。

–SL
2012年8月8日15:04


#2 楼

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

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


评论

好吧，如果它们都是IDisposable的，它们都是同一类型，也许强制转换会起作用？

– jpierson
2012年1月17日17:43
@jpierson确实起作用，是的，但是当您从using块内部调用IDisposable对象时，我们无法调用任何类成员（没有强制转换，这会破坏imo点）。

–康奈尔
13年3月22日在9:58
IDisposable是一种类型，因此只需将其用作类型即可获得混合类型的列表，如其他一些答案所示。

–克里斯·罗林斯
18年1月13日在20:14

#3 楼

当IDisposable属于同一类型时，您可以执行以下操作： br不论using的类型是否相同，都可以执行以下操作：

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


#4 楼

如果您不介意在using块之前声明using块的变量，则可以在同一using语句中声明所有变量。 

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


这样，x和y只是IDisposable类型的占位符变量，供using块使用，并且您在代码中使用t和u。只是以为我会提及。

评论

我觉得这会使新开发人员查看您的代码感到困惑。

–扎克
2015年1月6日14:40
这可能是一个坏习惯；这样做的副作用是，即使释放了非托管资源，这些变量仍将存在。根据Microsoft的C＃参考，“您可以实例化资源对象，然后将变量传递给using语句，但这不是最佳实践。在这种情况下，对象将在控制权离开using块之后仍保留在范围内可能再也无法访问其非托管资源。”

– Robert Altman
15年8月18日在13:30
@RobertAltman你是对的，在实际代码中，我将使用另一种方法（可能是Gavin H的方法）。这只是一个不太可取的选择。

– Botz3000
15年8月19日在7:01
您可以只在带有typecast的using内部移动声明。会更好吗？

–提莫西·布莱斯戴尔（Timothy Blaisdell）
19/12/11在23:48

#5 楼

using语句在IDisposable接口的基础上工作，因此另一种选择是创建某种类型的复合类，该复合类实现IDisposable并引用您通常会在using语句中放置的所有IDisposable对象。不利的一面是，您必须先在变量的作用域之外声明变量，以使其在use块中有用，与其他一些建议相比，它们需要更多的代码行。 
Connection c = new ...; 
Transaction t = new ...;

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


在这种情况下，DisposableCollection的构造函数是params数组，因此您可以随意输入任意数量。

#6 楼

如果您想高效地比较文件，则完全不需要使用StreamReader，那么就不需要使用它了-您可以使用低级流读取来提取数据缓冲区以进行比较。

您还可以先比较文件大小之类的内容，以快速检测不同的文件，以免自己也不得不读取所有数据。

评论

是的，检查文件大小是一个好主意，可以节省您的时间或读取所有字节。 （+1）

– TimothyP
09年8月25日在17:50
这似乎没有解决这个问题，该问题与嵌套using语句有关（无论using语句包含什么内容）。它应该是一条评论。

– TylerH
20/09/10在15:28

#7 楼

从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);


这将在变量作用域的末尾（即，在方法。

评论

使用var outFile = new StreamReader（outputFile.OpenRead（）），var expFile = new StreamReader（expectedFile.OpenRead（））;也可以

– Chazt3n
20 Sep 9 '20 at 12:23

#8 楼

您也可以说：

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


但是有些人可能会觉得很难读。顺便说一句，作为对问题的优化，为什么在逐行处理之前不先检查文件大小是否相同？

#9 楼

您可以使用除最里面的括号以外的所有括号：

提出了建议，但我相信很多人会认为这令人困惑

#10 楼

您可以在一个使用语句中用逗号将多个一次性对象分组：

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

}


#11 楼

而且，为了更加清楚起见，在这种情况下，由于每个连续的语句都是单个语句（而不是块），因此可以省略所有方括号：

评论

有趣的解决方案；做到这一点/甚至使用最低级别的一组括号，都可以实现与将它们左对齐（更清洁的IMO）堆叠在一起的目的，同时满足其他人提到的化妆品嵌套的需求，以表明从属关系。

–galaxis
17-10-29在14:55


#12 楼

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

#13 楼

您是否还在问是否有更好的方法来比较文件？
我更喜欢为这两个文件计算CRC或MD5并进行比较。 ：

public static class ByteArrayExtender
    {
        static ushort[] CRC16_TABLE =  { 
                      0X0000, 0XC0C1, 0XC181, 0X0140, 0XC301, 0X03C0, 0X0280, 0XC241, 
                      0XC601, 0X06C0, 0X0780, 0XC741, 0X0500, 0XC5C1, 0XC481, 0X0440, 
                      0XCC01, 0X0CC0, 0X0D80, 0XCD41, 0X0F00, 0XCFC1, 0XCE81, 0X0E40, 
                      0X0A00, 0XCAC1, 0XCB81, 0X0B40, 0XC901, 0X09C0, 0X0880, 0XC841, 
                      0XD801, 0X18C0, 0X1980, 0XD941, 0X1B00, 0XDBC1, 0XDA81, 0X1A40, 
                      0X1E00, 0XDEC1, 0XDF81, 0X1F40, 0XDD01, 0X1DC0, 0X1C80, 0XDC41, 
                      0X1400, 0XD4C1, 0XD581, 0X1540, 0XD701, 0X17C0, 0X1680, 0XD641, 
                      0XD201, 0X12C0, 0X1380, 0XD341, 0X1100, 0XD1C1, 0XD081, 0X1040, 
                      0XF001, 0X30C0, 0X3180, 0XF141, 0X3300, 0XF3C1, 0XF281, 0X3240, 
                      0X3600, 0XF6C1, 0XF781, 0X3740, 0XF501, 0X35C0, 0X3480, 0XF441, 
                      0X3C00, 0XFCC1, 0XFD81, 0X3D40, 0XFF01, 0X3FC0, 0X3E80, 0XFE41, 
                      0XFA01, 0X3AC0, 0X3B80, 0XFB41, 0X3900, 0XF9C1, 0XF881, 0X3840, 
                      0X2800, 0XE8C1, 0XE981, 0X2940, 0XEB01, 0X2BC0, 0X2A80, 0XEA41, 
                      0XEE01, 0X2EC0, 0X2F80, 0XEF41, 0X2D00, 0XEDC1, 0XEC81, 0X2C40, 
                      0XE401, 0X24C0, 0X2580, 0XE541, 0X2700, 0XE7C1, 0XE681, 0X2640, 
                      0X2200, 0XE2C1, 0XE381, 0X2340, 0XE101, 0X21C0, 0X2080, 0XE041, 
                      0XA001, 0X60C0, 0X6180, 0XA141, 0X6300, 0XA3C1, 0XA281, 0X6240, 
                      0X6600, 0XA6C1, 0XA781, 0X6740, 0XA501, 0X65C0, 0X6480, 0XA441, 
                      0X6C00, 0XACC1, 0XAD81, 0X6D40, 0XAF01, 0X6FC0, 0X6E80, 0XAE41, 
                      0XAA01, 0X6AC0, 0X6B80, 0XAB41, 0X6900, 0XA9C1, 0XA881, 0X6840, 
                      0X7800, 0XB8C1, 0XB981, 0X7940, 0XBB01, 0X7BC0, 0X7A80, 0XBA41, 
                      0XBE01, 0X7EC0, 0X7F80, 0XBF41, 0X7D00, 0XBDC1, 0XBC81, 0X7C40, 
                      0XB401, 0X74C0, 0X7580, 0XB541, 0X7700, 0XB7C1, 0XB681, 0X7640, 
                      0X7200, 0XB2C1, 0XB381, 0X7340, 0XB101, 0X71C0, 0X7080, 0XB041, 
                      0X5000, 0X90C1, 0X9181, 0X5140, 0X9301, 0X53C0, 0X5280, 0X9241, 
                      0X9601, 0X56C0, 0X5780, 0X9741, 0X5500, 0X95C1, 0X9481, 0X5440, 
                      0X9C01, 0X5CC0, 0X5D80, 0X9D41, 0X5F00, 0X9FC1, 0X9E81, 0X5E40, 
                      0X5A00, 0X9AC1, 0X9B81, 0X5B40, 0X9901, 0X59C0, 0X5880, 0X9841, 
                      0X8801, 0X48C0, 0X4980, 0X8941, 0X4B00, 0X8BC1, 0X8A81, 0X4A40, 
                      0X4E00, 0X8EC1, 0X8F81, 0X4F40, 0X8D01, 0X4DC0, 0X4C80, 0X8C41, 
                      0X4400, 0X84C1, 0X8581, 0X4540, 0X8701, 0X47C0, 0X4680, 0X8641, 
                      0X8201, 0X42C0, 0X4380, 0X8341, 0X4100, 0X81C1, 0X8081, 0X4040 };


        public static ushort CalculateCRC16(this byte[] source)
        {
            ushort crc = 0;

            for (int i = 0; i < source.Length; i++)
            {
                crc = (ushort)((crc >> 8) ^ CRC16_TABLE[(crc ^ (ushort)source[i]) & 0xFF]);
            }

            return crc;
        }


完成后，比较文件就很容易了：

public bool filesAreEqual(string outFile, string expFile)
{
    var outFileBytes = File.ReadAllBytes(outFile);
    var expFileBytes = File.ReadAllBytes(expFile);

    return (outFileBytes.CalculateCRC16() == expFileBytes.CalculateCRC16());
}


您可以使用内置的System.Security.Cryptography.MD5类，但是计算的哈希值是byte []，因此您仍然必须比较这两个数组。

评论

该方法应采用Stream对象并调用ReadByte方法，直到返回-1为止，而不是采用字节数组。这将为大文件节省大量内存。

–SL
09年8月25日在17:53
然后，您将如何计算所有字节上的crc？

– TimothyP
09年8月25日在17:56
噢，别介意我说的：p Thnx，我将在我的代码中更改：p我们仅将其用于<1000个字节的数据，因此尚未注意到任何问题，但是无论如何都会改变

– TimothyP
09年8月25日18:00
每次调用ReadByte时，流的位置前进一个字节。因此，如果一直调用它直到返回-1（EOF），它将为您提供文件中的每个字节。 msdn.microsoft.com/zh-CN/library/system.io.stream.readbyte.aspx

–SL
09年8月25日18:00
如果要多次比较多个文件，则使用CRC很好，但是对于一次比较，您必须完整地读取两个文件才能计算出CRC-如果您比较数据块很小，则可以退出比较一旦找到一个不同的字节。

–詹森·威廉姆斯（Jason Williams）
2009年8月25日19:51

#14 楼

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

相反，我会推荐这样的内容：

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

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


Path.Combine将在路径中添加文件夹或文件名，并确保路径和名称之间恰好有一个反斜杠。 />
通过在@前面加上字符串，可以避免必须转义每个反斜杠（例如File.OpenText）

#15 楼

我想我可能已经找到了一种语法更简洁的方法来声明此using语句，它似乎对我有用吗？在using语句中使用var作为类型而不是IDisposable似乎可以动态推断两个对象的类型，并允许我实例化两个对象并调用它们所分配的类的属性和方法，例如using(var uow = new UnitOfWorkType1(), uow2 = new UnitOfWorkType2()){}. 这不对，请让我知道

评论

如果所有事物都属于同一类型，则可以在一行上运行多个。混合类型必须划分为单独的using（）。但这不适用于var，您必须指定类型（C＃5规范，p237）

–克里斯·卡罗尔（Chris F Carroll）
2014年11月27日在17:03

#16 楼

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

#17 楼

它的正常使用方式和完美的作品。尽管还有其他实现方法。
几乎所有的答案都已经存在于该问题的答案中。但是在这里，我将它们全部列出。 />
选项2 

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;
      }
    }
  }