【问题标题】:Nested using statements in C#在 C# 中嵌套 using 语句
【发布时间】:2010-11-22 17:14:37
【问题描述】:

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

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

  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 ()){}
  • @200_success 这个问题在 2009 年和 2013 年问过,所以我可能会翻转重复识别。 (2¢,仅供参考等)

标签: c# .net file using


【解决方案1】:

如果对象属于相同类型,您可以执行以下操作

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

【讨论】:

  • 好吧,如果它们都是 IDisposable 的话,它们都是同一类型,也许演员可以工作?
  • @jpierson 确实有效,是的,但是当您从 using 块内部调用 IDisposable 对象时,我们无法调用任何类成员(没有演员表,这击败了imo)。
  • IDisposable 是一种类型,因此只需将其用作具有混合类型列表的类型,如其他一些答案所示。
【解决方案2】:

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

【讨论】:

    【解决方案3】:

    IDisposables 为同一类型时,您可以执行以下操作:

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

    using 上的 MSDN 页面包含有关此语言功能的文档。

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

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

    【讨论】:

      【解决方案4】:

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

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

      【讨论】:

        【解决方案5】:

        这样做的首选方法是只在最后一个 using 语句之后放置一个左大括号 {,如下所示:

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

        【讨论】:

        • 清洁剂?也不会强迫你使用相同的类型。即使类型匹配可读性和一致性,我也总是这样做。
        • @Hardryv:Visual Studio 的自动格式将其删除。这个想法是看起来像一个变量声明列表。
        • 不确定我是否觉得它更具可读性。如果有的话,它会破坏嵌套代码的外观。看起来好像第一个 using 语句是空的且未使用。但是,我猜有什么用...:/
        • @Bryan Watts,“逆势者”可能表达了真正的偏好。如果嵌套被推荐,很可能会有不同的开发者组反对。唯一知道的方法是在平行宇宙中再次运行实验。
        • @fmuecke:这不是很正确;它会起作用的。 IDisposable 的规则规定两次调用 Dispose() 应该什么都不做。该规则仅适用于写得不好的一次性用品。
        【解决方案6】:

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

        【讨论】:

          【解决方案7】:

          您可以省略除最内层之外的所有括号:

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

          我认为这比像其他人建议的那样将多个相同类型放在同一个用途中更干净,但我相信很多人会认为这很混乱

          【讨论】:

            【解决方案8】:

            你也可以说:

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

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

            【讨论】:

              【解决方案9】:

              另外,如果您已经知道路径,那么扫描目录是没有意义的。

              相反,我会推荐这样的东西:

              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 将打开一个文件并一次性创建一个StreamReader

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

              【讨论】:

                【解决方案10】:

                如果您想有效地比较文件,请不要使用 StreamReaders,这样就不需要使用 - 您可以使用低级别的流读取来拉入数据缓冲区进行比较。

                您还可以先比较文件大小之类的内容,以快速检测不同的文件,从而省去读取所有数据的麻烦。

                【讨论】:

                • 是的,检查文件大小是个好主意,可以节省您的时间或读取所有字节。 (+1)
                • 这似乎没有解决问题,即关于嵌套 using 语句(不管 using 语句包含什么)。相反,它应该是一条评论。
                【解决方案11】:

                您是否还询问是否有更好的方法来比较文件? 我更喜欢为这两个文件计算 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 类, 但是计算出的哈希是一个字节[],所以你仍然需要比较这两个数组。

                【讨论】:

                • 该方法不应采用字节数组,而是采用Stream 对象并调用ReadByte 方法,直到它返回-1。这将为大文件节省大量内存。
                • 你将如何计算所有字节的 crc?
                • 哦,别管我说什么了 :p Thnx,我会在我的代码中改变它 :p 我们只将它用于
                • 每次调用ReadByte,流的位置都会增加一个字节。因此,如果你一直调用它直到它返回 -1 (EOF),它会给你文件中的每个字节。 msdn.microsoft.com/en-us/library/system.io.stream.readbyte.aspx
                • 如果您想多次比较多个文件,则使用 CRC 非常好,但对于单个比较,您必须完整读取两个文件以计算 CRC - 如果您比较小块的数据那么一旦发现不同的字节就可以退出比较。
                【解决方案12】:

                为了清楚起见,在这种情况下,由于每个连续的语句都是单个语句(而不是块),您可以省略所有括号:

                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;  
                

                【讨论】:

                • 有趣的解决方案;这样做/甚至使用最低级别的一组括号可能会实现与将它们左对齐(更清洁的 IMO)堆叠相同的目标,同时解决其他人提到的显示任何从属关系的装饰性嵌套需求。
                【解决方案13】:

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

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

                这样,x 和 y 只是 IDisposable 类型的占位符变量,供 using 块使用,您在代码中使用 t 和 u。只是想我会提到。

                【讨论】:

                • 我觉得这会让查看您的代码的新开发人员感到困惑。
                • 这可能是个坏习惯;它有一个副作用,即使在非托管资源被释放后,变量仍然存在。根据 Microsoft 的 C# 参考,“您可以实例化资源对象,然后将变量传递给 using 语句,但这不是最佳做法。在这种情况下,对象在控制离开 using 块后仍保留在范围内,即使它会可能无法再访问其非托管资源。”
                • @RobertAltman 你是对的,在实际代码中我会使用另一种方法(可能是来自 Gavin H 的方法)。这只是一个不太受欢迎的选择。
                • 你可以在 using with typecasts 中移动声明。这样会更好吗?
                【解决方案14】:

                using 语句在 IDisposable 接口之外工作,因此另一种选择可能是创建某种类型的复合类,该类实现 IDisposable 并引用您通常放入 using 语句中的所有 IDisposable 对象。不利的一面是,您必须首先声明变量并在范围之外声明它们才能在 using 块中使用,这需要比其他一些建议所需的更多代码行。

                Connection c = new ...; 
                Transaction t = new ...;
                
                using (new DisposableCollection(c, t))
                {
                   ...
                }
                

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

                【讨论】:

                  【解决方案15】:

                  我想我可能已经找到了一种在语法上更简洁的方式来声明这个 using 语句,并且它似乎对我有用?在 using 语句中使用 var 作为类型而不是 IDisposable 似乎可以动态推断两个对象的类型,并允许我实例化我的两个对象并调用它们所分配的类的属性和方法,如

                  using(var uow = new UnitOfWorkType1(), uow2 = new UnitOfWorkType2()){}.

                  如果有人知道为什么这是不对的,请告诉我

                  【讨论】:

                  • 如果所有东西都属于同一类型,那么一行中的几个就可以了。混合类型必须拆分为单独的 using()。但它不适用于 var,你必须指定一个类型(C# 5 规范,p237)
                  【解决方案16】:

                  这是正常的使用方式,效果很好。尽管还有其他一些实现方式。 几乎每个答案都已经出现在这个问题的回复中。但在这里我将它们全部列出来。

                  已使用

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

                  选项 1

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

                  选项 2

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

                  【讨论】:

                    【解决方案17】:

                    由于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 var outFile = new StreamReader(outputFile.OpenRead()), var expFile = new StreamReader(expectedFile.OpenRead()); 也可以
                    猜你喜欢
                    • 2011-03-21
                    • 1970-01-01
                    • 2018-11-18
                    • 2014-04-04
                    • 1970-01-01
                    • 2022-12-12
                    • 2014-06-05
                    • 1970-01-01
                    相关资源
                    最近更新 更多