循環內的異常處理。 為什么此代碼給出的不是所有代碼路徑都返回值錯誤？

Question

public static Boolean CanParseStringToInt(String s)
    {
        Boolean retry = false;
        int n = 0;

        do
        {
            try
            {
                n = int.Parse(s);
                return true;
            }
            catch (Exception ex)
            {
                if (!retry)
                {
                    retry = true;
                }
                else
                {
                    return false;
                }
            }
        }
        while (retry);
    }

我的目標是.Net 4.0並使用VS2012。 該代碼僅是示例。

Answer 1

編譯器無法絕對確定一段代碼是否最終將返回一個值。 查看暫停問題 ，以了解難度示例。

如果您以這種方式重組代碼，則其行為應相同，但將修復錯誤：

public static Boolean CanParseStringToInt(String s)
{
    Boolean retry = false;
    int n = 0;

    do
    {
        try
        {
            n = int.Parse(s);
            return true;
        }
        catch
        {
            if (!retry)
            {
                retry = true;
            }
            else
            {
                break;
            }
        }
    }
    while (retry);

    return false;
}

Answer 2

編譯器不夠聰明，無法知道您最終將在循環內返回。

我將在方法末尾添加以下行：

throw new InvalidOperationException("Should not have reached here.");

Answer 3

在while循環之后，您什么也不返回。

不知道此方法用於什么，但是我建議使用Int32.TryParse()代替此實現。

Answer 4

在catch塊內的if塊中沒有return語句。 因此，假定循環體已執行。 這意味着while條件也會執行，因此編譯器認為它可以返回false。 這意味着在while語句之后還必須有一個返回值

Answer 5

在您的捕獲中，else條件返回，但if不返回，而是分配。 這段時間之外也沒有任何回報。

       catch (Exception ex)
        {
            if (!retry)
            {
                retry = true;
            }
            else
            {
                return false;
            }
        }

通常，我喜歡編寫如下的重試邏輯；

 for (int i = 0; i < retries; i++)
 {
     try
     {
         //do stuff that could throw
     }
     catch (Exception e)
     {
         Thread.Sleep(500);
         if (i == retries - 1)
         {
             throw new Exception(e.Message);
         }
     }
  }

另外，再次查看代碼后，似乎您正在寫int.TryParse(string input, out int output)