我有一个客户端类,它通过 WebRequest
连接到服务器。现在我正在为如何最好地实现 WebException
处理而苦恼。对于良好的 UI 反馈,我只需要(到目前为止)异常消息和状态。
我想出了三种不同的方法。第一个简单地重新抛出异常,我在调用方法中进行处理。这可能是最通用的解决方案:
public class ClientSimpleRethrow
{
public bool IsConnected { get; set; }
public void Connect()
{
try
{
// do connection
}
catch (WebException)
{
IsConnected = false;
throw;
}
IsConnected = true;
}
}
第二个版本抛出一个自定义异常,将 WebException
作为内部异常。这给了我只捕捉和处理我所期望的好处。例如,可能存在另一个错误,它也会抛出一个我想区别对待的 WebException
。到目前为止,这是我最喜欢的解决方案:
public class ClientThrowCustom
{
public bool IsConnected { get; set; }
public void Connect()
{
try
{
// do connection
}
catch (WebException ex)
{
IsConnected = false;
throw new ClientConnectionException(ex.Message, ex);
}
IsConnected = true;
}
}
我还可以考虑第三种可能性。异常被捕获,我只设置了我真正需要的属性来处理 UI 中的异常。我已经通过 IsConnected
属性获得了出错的信息。该解决方案确实具有不会捕获两次的好处。
public class ClientProperties
{
public bool IsConnected { get; set; }
public string ConnectionErrorMessage { get; set; }
public WebExceptionStatus? ConnectionErrorStatus { get; set; }
public void Connect()
{
try
{
// do connection
}
catch (WebException ex)
{
IsConnected = false;
ConnectionErrorMessage = ex.Message;
ConnectionErrorStatus = ex.Status;
return;
}
IsConnected = true;
ConnectionErrorMessage = null;
ConnectionErrorStatus = null;
}
}
我需要一些最佳实践建议,我应该选择哪一个以及为什么。我希望这个问题不要太主观,因为我真的认为,为什么选择一种设计而不是其他设计应该有一些确凿的事实。
预先感谢您的帮助!
最佳答案
对于初学者来说,我认为第三种方法不好。异常的好处是它们不会强制调用上下文在继续之前检查错误代码。在 try block 中,您可以编写代码,就好像每个函数都成功一样。第三种方法取消了这种精确性并使调用代码看起来更复杂。
至于前两个,我更喜欢第二个。在我看来,它有两个好处:
您可以创建一个异常类,它在您的应用程序的上下文中具有某种意义,而不是在底层 API 的上下文中工作。
WebException
对 API 有好处,但如果您在尝试登录某个系统时在方法中获取它,LogInException
在应用程序的上层更有帮助.自定义异常允许您从基础
MyApplicationException
类开始并从那里构建。从那里,您可以捕获 API 异常并尽快将它们转换为MyApplicationException
子异常。这样,当 API 异常波及到您的应用程序时,您会立即知道存在一些您没有考虑到并且需要解决的极端情况。未捕获的应用程序异常仅意味着您没有正确使用自己的代码。
关于c# - 我应该为客户端连接选择哪种异常处理,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/4735230/