完全在 try/catch 内部的主要方法代码:这是不好的做法吗? [英] Main method code entirely inside try/catch: Is it bad practice?

问题描述

通常我将所有 Main 方法代码放在 try/catch 块中，如下所示:

Usually I put all of my Main method code inside of a try/catch block like so:

public static void Main(string[] args)
{
   try
   {
      // code
   }
   catch (Exception e)
   {
      // code
   }
}

我这样做只是为了防止任何异常设法从程序逻辑的其余部分中溜走，从而允许我做一些事情，例如将其显示到控制台，将其记录到文件等.但是，我被告知这是不好的做法.

I do this just in case any exceptions manage to slip out of the rest of the program logic, thus allowing me to do something about it, such as display it to console, log it to a file, etc. However, I have been told that this is bad practice.

你认为这是不好的做法吗?

Do you think it is bad practice?

推荐答案

在 try/catch任何段代码> 没有正当理由的阻止是不好的做法.

在 .NET 编程模型中，应为真正的异常情况或条件保留异常.您应该只尝试捕获您实际上可以做某事的异常.此外，您应该几乎永远捕获基本的 System.Exception 类(而是更喜欢捕获您可以的更具体的派生异常类)处理).如果在您的程序执行过程中遇到真正意外的异常，您实际上应该崩溃.

In the .NET programming model, exceptions should be reserved for truly exceptional cases or conditions. You should only try to catch exceptions that you can actually do something about. Furthermore, you should should hardly ever catch the base System.Exception class (but rather prefer to catch the more specific, derived exception classes you can handle). And should a truly unexpected exception be encountered during the course of your program's execution, you actually should crash.

显然，正确"的答案必须逐案确定，具体取决于 //code 占位符中的 catch> 阻止.但是，如果您要的是一般规则或最佳实践"，那么您应该始终有一个特定的理由来捕获异常，而不仅仅是将所有代码包装在一个巨大的 try/catch 中 不假思索地把块当作理所当然.

Obviously the "correct" answer would have to be made on a case-by-case basis, depending on what's going on inside that // code placeholder in your catch block. But if you're asking for a general rule or "best practice", you should always have a specific reason to catch exceptions, not just wrap all of your code in a giant try/catch block as a matter of course without thinking about it.

请注意，如果您只是为了记录或错误报告而试图捕获任何可能发生的未处理的异常，您应该使用 AppDomain.UnhandledException 事件.这是一个仅通知事件，因此它不允许您处理这些异常，但它是在应用程序崩溃后实现日志记录或错误报告功能的正确位置.

Note that if you're simply trying to catch any unhandled exceptions that might occur for the purposes of logging or error reporting, you should be using the AppDomain.UnhandledException event. This is a notification-only event, so it doesn't allow you to handle those exceptions, but it is the right place to implement your logging or error reporting functionality after your application has crashed.

当我正在阅读 Raymond Chen 的优秀博客时，"The Old New Thing"，我注意到他最近发表了一篇关于类似主题的文章.它特定于 COM，而不是 .NET Framework，但有关错误处理的一般概念同样适用于这两种环境.我想我会在这里分享这篇文章中的一些精华，以支持我的 [显然颇有争议] 的观点.

As I was catching up on my reading of Raymond Chen's excellent blog, "The Old New Thing", I noticed that he had recently published an article on a similar topic. It's specific to COM, rather than the .NET Framework, but the general concepts regarding error handling are equally applicable to both environments. I thought I'd share a couple of gems from the article here, in support of my [apparently quite controversial] opinion.

从历史上看，COM 在服务器的方法周围放置了一个巨大的 try/except.如果您的服务器遇到通常未处理的异常，巨大的 try/except 会捕获它并将其转换为错误 RPC_E_SERVERFAULT.然后它将异常标记为已处理，以便服务器保持运行，从而即使在遇到问题时也保持服务器运行来提高健壮性."

Historically, COM placed a giant try/except around your server's methods. If your server encountered what would normally be an unhandled exception, the giant try/except would catch it and turn it into the error RPC_E_SERVERFAULT. It then marked the exception as handled, so that the server remained running, thereby "improving robustness by keeping the server running even when it encountered a problem."

请注意，这实际上是一种伤害.

Mind you, this was actually a disservice.

发生未处理异常的事实意味着服务器处于意外状态.通过捕获异常并说别担心，一切都很好"，您最终会使损坏的服务器继续运行.

The fact that an unhandled exception occurred means that the server was in an unexpected state. By catching the exception and saying, "Don't worry, it's all good," you end up leaving a corrupted server running.

[ ...]

捕获所有异常并让进程继续运行假定服务器可以从意外故障中恢复.但这是荒谬的.您已经知道服务器不可恢复地吐司:它崩溃了！

Catching all exceptions and letting the process continue running assumes that a server can recover from an unexpected failure. But this is absurd. You already know that the server is unrecoverably toast: It crashed!

最好让服务器崩溃，以便可以在故障点捕获崩溃转储.现在您有机会弄清楚发生了什么.

Much better is to let the server crash so that the crash dump can be captured at the point of the failure. Now you have a fighting chance of figuring out what's going on.

您可以 [并且应该] 在他的博客上阅读整篇文章:如何关闭 COM帮助"环绕您的服务器的异常处理程序.

You can [and should] read the whole article here on his blog: How to turn off the exception handler that COM "helpfully" wraps around your server.

这篇关于完全在 try/catch 内部的主要方法代码:这是不好的做法吗?的文章就介绍到这了，希望我们推荐的答案对大家有所帮助，也希望大家多多支持IT屋！