使用c#语句嵌套

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

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

  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语句似乎有点奇怪。

还有更好的办法吗?

109846 次浏览
小开

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

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

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

小开

当__abc是相同类型时，你可以执行以下操作:

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

using上的MSDN页面有关于此语言特性的文档。

不管__abc0的类型是否相同，都可以执行以下操作:

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

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

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


}
小开
最佳答案

最好的方法是在最后一个using语句之后只放一个开大括号{，如下所示:

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

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

小开

除了最里面的使用，你可以省略括号:

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()))
{
...
}

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

小开

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

相反，我建议这样做:

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

小开

如果你想要有效地比较文件，根本就不要使用streamreader，这样使用就没有必要了——你可以使用低级流读取来拉入数据缓冲区进行比较。

您还可以首先比较文件大小等内容，以快速检测不同的文件，从而节省自己读取所有数据的时间。

小开

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

为了增加清晰度，在这种情况下，因为每个连续的语句都是一个语句(而不是一个块)，你可以省略所有的括号:

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块之前声明using块的变量，你可以在同一个using语句中声明它们。

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

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

小开

using语句在IDisposable接口之外工作，所以另一种选择是创建某种类型的复合类，实现IDisposable，并引用通常放入using语句中的所有IDisposable对象。这样做的缺点是，您必须首先在作用域之外声明变量，以便它们在using块中有用，这比其他一些建议需要更多的代码行。

Connection c = new ...;
Transaction t = new ...;


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

在本例中，DisposableCollection的构造函数是一个params数组，因此您可以输入任意数量的参数。

小开

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

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

如果有人知道为什么这是不正确的，请告诉我

小开

这是正常的使用方式，工作完美。尽管还有其他实现方法。 几乎每个答案都已经出现在这个问题的回答中。但是这里我把它们都列在一起了

已经使用

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

因为c# 8.0，你可以使用使用声明。

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

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


提交答案