简单的最佳实践问题。
你应该嵌套 try catch 语句还是只使用方法。
例如，如果您有一个打开文件的方法确实有效并关闭文件，那么您将在 try catch 之外进行打开和关闭，或者更确切地说，在 finally 块中进行关闭。
现在，如果您的 open 方法失败，该方法会断言正确吗？那么你应该将它包装在一个 try catch 块中还是应该从另一个方法调用它，而这又是一个 try catch 块？
【问题讨论】：
标签： c# error-handling methods
取决于您要执行的操作，但在大多数情况下，嵌套的 try/catch 表示函数过于复杂（或不太了解异常如何工作的程序员！）。
在打开文件的情况下，我会使用 IDisposable 持有者和 using 子句，因此不需要任何显式的 try/catch。
【讨论】：
这是一个风格问题，但对我来说，我尽量不要在一个方法中嵌套多于一层的 try/catch/finally。在您进行嵌套尝试时，您几乎可以肯定违反了 1 function = 1 操作原则，应该使用第二种方法。
【讨论】：
在打开文件的方法的上下文中，我会使用 using 语句而不是 try catch。 using 语句确保在发生异常时调用 Dispose。
using (FileStream fs = new FileStream(file, FileMode.Open))
{
//do stuff
}
做同样的事情：
FileStream fs;
try
{
fs = new FileStream(file, FileMode.Open);
//do Stuff
}
finally
{
if(fs!=null)
fs.Dispose();
}
【讨论】：
大多数时候，我会将嵌套的 try/catch 块分解为函数。但是我有时会编写代码来捕获和记录我的应用程序抛出的所有未捕获的异常。但是如果日志记录代码失败怎么办？所以我有另一个尝试/捕捉，只是为了防止用户看到默认的 .NET 未处理异常对话框。但即使是这段代码也可以很容易地重构为函数，而不是嵌套的 try/catch 块。
try
{
try
{
DoEverything();
}
catch (Exception ex)
{
// Log the exception here
}
}
catch (Exception ex)
{
// Wow, even the log is broken ...
}
【讨论】：
现在我们有了 lambdas 和类型推断以及其他一些东西，有一个在其他语言中很常见的习语现在在 C# 中很有意义。你的例子是关于打开一个文件，对它做一些事情，然后关闭它。好吧，现在，您可以创建一个打开文件的辅助方法，并确保关闭/处置/清理，但调用您为“做的东西”部分提供的 lambda。这将帮助您将复杂的 try/catch/finally dispose/cleanup 东西放在一个地方，然后一遍又一遍地使用它。
这是一个例子：
public static void ProcessFile(string filePath, Action<File> fileProcessor)
{
File openFile = null;
try
{
openFile = File.Open(filePath); // I'm making this up ... point is you are acquiring a resource that needs to be cleaned up after.
fileProcessor(openFile);
}
finally
{
openFile.Close(); // Or dispose, or whatever.
}
}
现在，此方法的调用者不必担心如何打开文件或关闭/处置它。他们可以这样做：
Helpers.ProcessFile("C://somefile.txt", f =>
{
while(var text = f.ReadLine())
{
Console.WriteLine(text);
}
});
【讨论】：
您在哪里有不一定属于它自己的单独功能的相关代码呢？那么这会是正确的吗？
try
{
// Part 1 Code Here
try
{
// Part 2 Code Here
}
catch (Exception ex)
{
// Error from Part 2
}
}
catch (Exception ex)
{
// Error from Part 1
}
【讨论】：
try
{
----
}
catch
{
try
{
---
}
catch
{
---
}
}
【讨论】：
//create a switch here and set it to 0
try
{
DoChunk1();
//looks good. set the switch to 1
}
catch (Exception ex)
{
// Log the exception here
}
// 检查开关，如果此时它仍然为零，那么你可以在这里暂停你的程序；否则将开关设置回零并执行下一个 try catch 语句。完全同意如上所述将它们分解
试试 { DoChunk2(); //看起来不错。将开关设置为 1 } 捕捉（例外前） { // 在此处记录异常 }
【讨论】：