java - 处理未使用的 return 语句

Question

我有以下代码:

public static QHttpResponse SafeExecute(QHttpRequest request, int retries, int sleep)
{
    if (retries < 1)
        throw new Exception("retries must be at least 1");

    for (int index = 0; index < retries; index++)
    {
        try
        {
            return QWebClient.Execute(request);
        }
        catch (Exception)
        {
            if (index == retries - 1)
                throw;

            Thread.Sleep(sleep);
        }
    }

    return null;
}

此函数中的第二个 return 从未执行任何操作。使用 return null 或 throw new Exception("Shouldn't be Here") 会更好吗？

或者我应该通过将结果存储在变量中并返回它来完全避免这种情况吗？ (这似乎没有必要)

Answer 1

是的，对于设计上应该无法访问的路径，抛出异常是有意义的。也作为一种明确的方式来记录您的方法的预期操作。

“InvalidLogicException”会很好，但不存在。

在 C# 中，最“合适”的可用异常类型可能是以下类型之一:

throw new InvalidProgramException("BUG: This code should be unreachable by design");
throw new InvalidOperationException("BUG: This code should be unreachable by design");
throw new NotSupportedException("BUG: This code should be unreachable by design");