在C#中使用语句嵌套

我正在做一个项目,并且必须比较两个文件,看看他们是否相互激动。

我的第一个草案之前,错误检查和validation了很多:

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 (StreamReader outFile = new StreamReader(outputFile.OpenRead())) using (StreamReader expFile = new StreamReader(expectedFile.OpenRead())) { ///... } 

如果对象的types相同 ,则可以执行以下操作

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

IDisposable s属于同一types时,您可以执行以下操作:

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

关于using的MSDN页面有关于此语言function的文档。

您可以执行以下操作,不pipeIDisposable是否是相同的types:

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

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

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

这样,x和y只是使用typesIDisposable的占位符variables,您可以在代码中使用t和u。 只是想我会提到。

如果要高效地比较文件,请不要使用StreamReaders,然后不需要使用 – 您可以使用低级别stream读取拉入数据的缓冲区进行比较。

您也可以先比较文件大小等事情,快速检测不同的文件,以免自己也读取所有的数据。

你也可以说:

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

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

using语句在IDisposable接口中工作,因此另一种select是创build某种types的实现IDisposable的组合类,并引用通常放在using语句中的所有IDisposable对象。 不利的一面是,你必须首先声明你的variables,并且在范围之外,这些variables在需要更多代码行的使用块内是有用的,而不是其他一些build议所需要的。

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

在这种情况下,DisposableCollection的构造函数是一个params数组,所以你可以尽可能多地进入。

没有什么奇怪的。 一旦代码块完成, using是确保对象处置的简便方法。 如果你的外部块有一个可拆卸的物体需要使用,这是完全可以接受的。

编辑:键入太慢,以显示合并的代码示例。 +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; 

除了最内部的用法,你可以省略括号:

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

我认为这是比其他几个相同types的使用,如其他人所build议的,但我相信很多人会认为这是混乱

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

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

当我编写代码的时候,这些都会出现。 你可以考虑把第二个使用语句移到另一个函数中?

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

另外,如果你已经知道path,扫描目录就没有意义了。

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

 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会将一个文件夹或文件名添加到path中,并确保path和名称之间只有一个反斜杠。

File.OpenText将打开一个文件并创build一个StreamReader

通过在@前加一个string,可以避免必须转义每个反斜杠(例如@"a\b\c"

我想我可能已经find了一个语法上更清晰的方式来声明这个使用语句,它似乎为我工作? 在using语句中使用var而不是IDisposable来使用var类似于dynamic地推断两个对象上的types,并允许我实例化这两个对象,并调用它们的属性和方法来分配类,如

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

如果有人知道这是不正确的,请让我知道