不要在using语句中调用WCF服务

不要在using语句中调用WCF服务

 

原文链接：http://www.cnblogs.com/dudu/archive/2011/01/18/1938144.html

 

如果你调用WCF服务时，像下面的代码这样在using语句中进行调用，需要注意一个问题。

using (CnblogsWcfClient client = new CnblogsWcfClient())
{
 client.Say("Hello, cnblogs.com!");
}

上面这段代码看上去没问题，CnblogsWcfClient是一个自动生成的WCF客户端代理，继承自System.ServiceModel.ClientBase。using语句结束时，会调用ClientBase实现的System.IDisposable.Dispose接口，实际就是调用ClientBase的Close()方法。用.NET Refector打开C:\Windows\Microsoft.NET\Framework\v4.0.30319\System.ServiceModel.dll，可以看到这样的代码，见下图：

不仅看上去没问题，似乎就是没问题。但是...问题就出在ClientBase.Close()上，Close()要关闭的是一个网络连接，如果这时网络连接出现问题，不能正常关闭会引发异常(ClientBase的Close方法就是这样设计的，引发异常，而不是强制关闭)，问题就来了。本来我们使用using的目的就是不管出现什么状况，即使天塌下来，也给我关闭掉;结果，关是关了，却没有闭，天还是塌下来了。

也许我们可以用“不可抗拒力”回避这个问题，但程序员的天性是解决问题。代码中任何一个小问题都不能忽视，因为我们很难预料这个小问题会不会带来大问题。

那如何解决这个问题呢?MSDN中有答案(去MSDN看看)，代码如下：

CnblogsWcfClient client = new CnblogsWcfClient();
try{
 client.Say("Hello, cnblogs.com!");
 client.Close();
}
catch (CommunicationException e)
{
...
 client.Abort();}catch (TimeoutException e)
{
 ...
 client.Abort();}catch (Exception e)
{
 ...
 client.Abort();
throw;
}

上面的代码显得有些哆嗦，如果你不关心是什么异常，只要出现异常就关闭的话，可以用下面的代码(代码来自TIP: Closing your WCF Connections properly)：

CnblogsWcfClient client = new CnblogsWcfClient();
client.Say("Hello, cnblogs.com!");
try{
if
(client.State != System.ServiceModel.CommunicationState.Faulted)
 {
 client.Close();
 }
}
catch
(Exception ex)
{
 client.Abort();
}

简化版代码：

CnblogsWcfClient client = new CnblogsWcfClient();
client.Say("Hello, cnblogs.com!");
try{
 client.Close();
}
catch{
 client.Abort();
}

好了，博客写好了，印象更深了，理解也更深了。在收获中分享，在分享中收获。