如何在 C# WCF 休息帖子中的 StreamReader.ReadToEnd() 之后抛出 WebFaultExce

Question

我有一个WCF REST项目，它以WebFaultExceptions的形式返回http状态代码。 这适用于GET调用，但是我在为POST调用返回WebFaultException时遇到了问题。请求正文中的数据使用内容类型为"application/x-www-form-urlencoded;charset=utf-8"。我认为问题是当上下文切换出 using 子句以抛出 WebFaultException 时，底层请求流被关闭。

如果我在"using"子句之前抛出一个WebFaultException，则异常会按预期返回。如果我从"using"子句中抛出WebFaultException，则异常不会返回给客户端。

有没有人对如何在使用流阅读器读取请求正文时成功抛出 WebFaultException 有任何建议？

这是我的服务器端代码的缩写版本。请注意，此示例的 httpstatuscode 对于我的实际实现是不现实的。

[WebInvoke(Method = "POST"
, UriTemplate = "urls/{id}"
, BodyStyle = WebMessageBodyStyle.WrappedRequest
)]
public string PostItem(string id, object streamdata)
{
    int _id = 0;
    if (int.TryParse(companyIdSr2, out _id))
    {
        using (System.IO.StreamReader reader = new System.IO.StreamReader(streamdata))
        {
            string body = reader.ReadToEnd();
            if(string.IsNullOrEmpty(body))
            {
                // this exception doesn't make it back to the client's request object
                ThrowError(HttpStatusCode.BadRequest, "empty body");
            }   
        }
    }
    else
    {
        // this exception is successfully returned to the client's request object
        ThrowError(HttpStatusCode.BadRequest, "invalid id");        
    }    
}
private static void ThrowError(HttpStatusCode status, string message)
{
    request_error error = new request_error
    {
        request_url = WebOperationContext.Current.IncomingRequest.UriTemplateMatch.RequestUri.OriginalString,
        error_status_code = status.ToString(),
        error_message = message,
    };
    throw new WebFaultException<request_error>(error, status);
}
public class request_error
{
    [XmlElement("request_url")]
    public string request_url { get; set; }
    [XmlElement("error_status_code")]
    public string error_status_code { get; set; }
    [XmlElement("error_message")]
    public string error_message { get; set; }
}

我已经看到了这个问题 - 使用流并关闭流时错误的 WebFaultException - 尽管它在一定程度上解决了这个问题，但未回答的是不处置或关闭流是否合理。

非常感谢，

泰 瑞

Answer 1

您链接到的问题现已收到已接受的答案，该答案解决了您的问题。

您声明：

尽管它在一定程度上解决了这个问题，但尚未回答的是不处置或关闭流是否合理。

对此，答案是：

但最好不要离开 StreamReader 而不释放它，因为它无法清理任何非托管资源。

为了支持这一点，我们在另一个 StackOverflow 线程上有以下答案，该线程论证了为什么调用 Dispose() 很重要：https://stackoverflow.com/a/2548694/700926

为了解决您在问题中提出的初始问题（WebFaultException不会发送回客户端），我最终按照此答案中的建议做了 - 对System.IO.StreamReader进行子类化并覆盖Close，以便它告诉 StreamReader 仅释放非托管资源而不关闭流。

我的 WcfFriendlyStreamReader 看起来像这样：

public class WcfFriendlyStreamReader : StreamReader
{
    public WcfFriendlyStreamReader(Stream s) : base(s) { }
    public override void Close()
    {
        base.Dispose(false);
    }
}

从 MSDN 文档中可以看出，调用Dispose(false)仅释放非托管资源。正如反编译器所揭示的那样，这也会导致Stream保持打开状态，这似乎可以解决问题：

protected override void Dispose(bool disposing)
{
  try
  {
    if (this.LeaveOpen || !disposing || this.stream == null)
      return;
    this.stream.Close();
  }
  finally
  {
    if (!this.LeaveOpen && this.stream != null)
    {
      this.stream = (Stream) null;
      this.encoding = (Encoding) null;
      this.decoder = (Decoder) null;
      this.byteBuffer = (byte[]) null;
      this.charBuffer = (char[]) null;
      this.charPos = 0;
      this.charLen = 0;
      base.Dispose(disposing);
    }
  }
}