visual studio无法检测代码返回路径

Question

我在重构一些代码时经历过，也许我是盲目的但是我没有看到为什么下面的代码不起作用。

public bool Foo()
{
    try
    {
        return SomeFunctionMightThrow();
    }
    catch (Exception ex)
    {
        Logger.log(ex);
        throw ex;
    }
}

到目前为止，如果我重构代码那么好

public bool Foo()
{
  try
  {
    return SomeFunctionMightThrow();
  }
  catch (Exception ex)
  {
    LogAndThrow(ex);
  }
}

private void LogAndThrow(Exception ex)
{
  Logger.log(ex);
  throw ex;
}

代码现在不编译。

Answer 1

编译器没有考虑到你的LogAndThrow()方法总是会抛出异常 - 在这种情况下， Foo()所有代码路径都必须返回一个布尔值，所以你可以这样做：

public bool Foo()
{
  try
  {
    return SomeFunctionMightThrow();
  }
  catch (Exception ex)
  {
    LogAndThrow(ex);
    return false; //will never get here
  }
}

一般来说，无论如何，我会建议你在应用程序的一个中心位置使用try/catch块进行日志记录，即在最高级别， 除非你能真正处理异常 - 它看起来不像你在这里。

Answer 2

您可以将其更改为...

public bool Foo()
{
  try
  {
    return SomeFunctionMightThrow();
  }
  catch (Exception ex)
  {
    // note: this will reset the exception stack trace
    throw LogAndReturnException(ex);
  }
}

private Exception LogAndReturnException(Exception ex)
{
  Logger.log(ex);
  return ex;
}

这基本上完全符合您的要求，但为VS提供了代码返回路径。

或者对于保留堆栈跟踪的更简单的方法，只需将catch更改为：

catch (Exception ex)
{
  Log(ex);
  throw; // retains stack trace
}