c# - 将 try-catch 放入循环中，直到 try block 中的所有语句都执行完，没有任何异常，这是一种好习惯吗？

Question

我正在尝试开发一个多播接收器程序，套接字初始化如下所示:

    public void initializeThread()
    {
        statuscheckthread = new Thread(SetSocketOptions);
        statuscheckthread.IsBackground = true;
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        rxsock = new Socket(AddressFamily.InterNetwork, SocketType.Dgram, ProtocolType.Udp);
        iep = new IPEndPoint(IPAddress.Any, 9191);
        rxsock.Bind(iep);
        ep = (EndPoint)iep;

        initializeThread();
        statuscheckthread.Start();
    }

    public void SetSocketOptions()
    {
         initializeThread(); //re-initializes thread thus making it not alive
         while (true)
         {
             if (NetworkInterface.GetIsNetworkAvailable())
             {
                 bool sockOptnSet = false;
                 while (!sockOptnSet)
                 {
                     try
                     {
                         rxsock.SetSocketOption(SocketOptionLevel.IP, SocketOptionName.AddMembership, new MulticastOption(IPAddress.Parse("224.50.50.50")));
                         rxsock.SetSocketOption(SocketOptionLevel.IP, SocketOptionName.MulticastTimeToLive, 64);
                         sockOptnSet = true;
                     }
                     catch
                     {
                         //Catch exception here
                     }
                 }
             }
             break; // Break out from loop once socket options are set
         }
     }

当我的电脑没有连接到网络时，SetSocketOption 方法抛出异常，即使在网络连接后，我无法接收数据，因为未设置套接字选项。

为了避免这种情况，我使用了一个在后台检查中运行的线程用于网络可用性，一旦网络可用，它就会设置套接字选项。

它在某些 PC 上正常工作，但在其他一些 PC 上，NetworkInterface.GetIsNetworkAvailable()在网络连接之前返回 true(在识别网络时)。

因此，为了确保套接字选项已设置，我使用了一个 bool 变量 sockOptnSet设置为true 如果 try block 中的所有语句都按照方法内部所示执行 public void SetSocketOptions()这个程序在我试过的所有 PC 上都运行良好，但我怀疑我能在多大程度上依赖它来工作。

我的问题是:

1) 这是好的做法吗？

2) 如果不是，它可能导致的错误或问题是什么？我怎样才能更好地实现它？

Answer 1

Is this a good practice?

不，这不是一个好的做法。绝大多数异常，包括您的第一个异常，都属于令人烦恼的异常类别。软件应该可以运行，在您测试时运行良好，但在用户的机器上却不行。出了点问题，但您不知道是什么，您也无能为力。试图让你的程序继续运行是没有用的，它不能完成它应该做的工作。在您的情况下，当没有网络时，套接字永远不会接收数据。而且，正如您所发现的，试图解决这个问题只会招致更多问题。这很正常。

If this is bad practice, how can I implement it in a better way?

您需要人类的帮助。用户将必须设置机器以提供有效的网络连接。这需要一个用户界面，你必须有办法告诉人们他需要做什么来解决你的问题。您可以根据需要使它变得复杂或简单。只是一条错误消息，Exception.Message 的逐字副本就足够了。为 AppDomain.CurrentDomain.UnhandledException 事件编写事件处理程序是一个非常好的(也是必需的)策略。 Microsoft 花费了大量精力使异常消息尽可能清晰和有用，甚至将它们本地化为用户的母语，您希望利用这一点。即使异常消息令人费解，对消息文本的快速 Google 查询也会返回数百个匹配项。有了这个事件处理程序，您就不必做任何特别的事情了。您的程序会自动终止，您的用户知道该怎么做。

您当然可以让它变得更复杂，您发现 SetSocketOption() 可能会在网络可用后立即失败，但在您等待足够长的时间后会起作用。因此，这实际上是您可以解决的错误情况，只需等待足够长的时间即可。您是否应该编写代码来处理这个是您必须自己决定的事情。当您对程序的行为方式有足够的经验时，您就可以编写它，您永远不会预先编写它。通常是来自您的程序用户的反馈。