logo标签列表

C#中的嵌套using语句

c#.netfileusing
363

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

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

  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()){} - hcp
可能是在C#中处理嵌套的"using"语句的重复问题。 - 200_success
@200_success 这个问题是在2009年被问到的，那个问题则是在2013年，所以如果有什么区别的话，我可能会更倾向于将重复标识翻转。 (2¢, fyi, 等等) - ruffin
17个回答
6

另外，如果您已经知道路径，那么扫描目录就没有意义了。

相反，我建议像这样做：

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"）。

5

为了更加清晰明了，就此情况而言，由于每个连续的语句都是单个语句（而不是代码块），所以您可以省略所有括号：

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;
有趣的解决方案；即使只在最低层使用一组括号，也可以实现与左对齐堆叠相同的目标（在我看来更加清晰），同时满足了其他人提出的美观嵌套需求以显示任何下级关系。 - galaxis
5

这很正常。 using 是一种简写方式，可以确保代码块完成后对象被正确处理。如果你的外部块中有一个可处理的对象，内部块需要使用它，这是完全可以接受的。

3

您是否也在询问是否有更好的方式来比较文件？我更喜欢为两个文件计算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[]，所以你仍然需要比较这两个数组。
2
方法应该接受一个“Stream”对象而不是一个字节数组，并调用“ReadByte”方法，直到它返回-1。这将为大文件节省大量内存。 - SLaks
那么，你将如何计算所有字节的 CRC 呢？ - TimothyP
哦，不用在意我说的话 :p 谢谢，我会在我的代码中进行更改 :p 我们只用它来处理小于1000字节的数据，所以还没有注意到问题，但是我们还是会进行更改。 - TimothyP
每次调用ReadByte时，流的位置会向前移动一个字节。因此，如果您一直调用它直到返回-1（EOF），它将为您提供文件中的每个字节。http://msdn.microsoft.com/en-us/library/system.io.stream.readbyte.aspx - SLaks
7
如果您想要多次比较多个文件，使用循环冗余校验码是很好的选择，但如果只是单次比较，则必须完整地读取两个文件以计算CRC。如果将数据分成小块进行比较，则可以在找到不同字节时立即退出比较。 - Jason Williams
ushort 不是哈希值有点短了吗？根据需要确定文件是否完全匹配或要比较的文件来自何处，我认为这种方法太容易发现哈希冲突。 - binki
3

这是正常使用的方式，而且非常完美。虽然还有其他实现方式。 几乎每个答案都已经在这个问题的回复中提到了。但是我会将它们全部列出来。

已使用

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

我觉得我可能已经找到了一种更加简洁的语法来声明这个using语句，并且看起来对我有效？在using语句中，使用var作为你的类型，而不是IDisposable，在两个对象上动态推断类型，可以让我实例化它们并调用它们分配的类的属性和方法，就像这样：

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

如果有人知道这不正确的原因，请告诉我。

1
如果所有的东西都是相同类型，那么一行上可以放置多个。不同类型必须分开使用()。但是对于var来说不起作用，您必须指定一个类型（C# 5规范，p237）。 - Chris F Carroll
3

在我的编程过程中，这种情况时常出现。你可以考虑将第二个 using 语句移入另一个函数中。

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文，
原文链接