รูปแบบการใช้งานHttpResponseMessage.EnsureSuccessStatusCode()คืออะไร? มันพ้นเนื้อหาของข้อความและพ่นHttpRequestExceptionแต่ฉันไม่เห็นวิธีการที่จะจัดการกับมันโปรแกรมใด ๆ Exceptionที่แตกต่างกว่าทั่วไป ตัวอย่างเช่นไม่รวมถึงสิ่งHttpStatusCodeที่น่าจะเป็นประโยชน์

มีวิธีใดบ้างในการดึงข้อมูลเพิ่มเติมออกมา ใครสามารถแสดงรูปแบบการใช้งานที่เกี่ยวข้องของทั้งสองEnsureSuccessStatusCode()และ HttpRequestException

การใช้สำนวนEnsureSuccessStatusCodeคือการตรวจสอบความสำเร็จของคำขออย่างรัดกุมเมื่อคุณไม่ต้องการจัดการกรณีความล้มเหลวในลักษณะใดวิธีหนึ่งโดยเฉพาะ สิ่งนี้มีประโยชน์อย่างยิ่งเมื่อคุณต้องการสร้างต้นแบบไคลเอ็นต์อย่างรวดเร็ว

เมื่อคุณตัดสินใจว่าต้องการจัดการกรณีความล้มเหลวในลักษณะเฉพาะอย่าทำสิ่งต่อไปนี้

var response = await client.GetAsync(...);
try
{
    response.EnsureSuccessStatusCode();
    // Handle success
}
catch (HttpRequestException)
{
    // Handle failure
}

สิ่งนี้ทำให้เกิดข้อยกเว้นเพียงเพื่อจับมันทันทีซึ่งไม่สมเหตุสมผล IsSuccessStatusCodeทรัพย์สินของHttpResponseMessageมีเพื่อวัตถุประสงค์นี้ ทำสิ่งต่อไปนี้แทน

var response = await client.GetAsync(...);
if (response.IsSuccessStatusCode)
{
    // Handle success
}
else
{
    // Handle failure
}

ฉันไม่ชอบ SureSuccessStatusCode เพราะมันไม่ส่งคืนอะไรที่มีความหมาย นั่นคือเหตุผลที่ฉันสร้างส่วนขยายของตัวเอง:

public static class HttpResponseMessageExtensions
{
    public static async Task EnsureSuccessStatusCodeAsync(this HttpResponseMessage response)
    {
        if (response.IsSuccessStatusCode)
        {
            return;
        }

        var content = await response.Content.ReadAsStringAsync();

        if (response.Content != null)
            response.Content.Dispose();

        throw new SimpleHttpResponseException(response.StatusCode, content);
    }
}

public class SimpleHttpResponseException : Exception
{
    public HttpStatusCode StatusCode { get; private set; }

    public SimpleHttpResponseException(HttpStatusCode statusCode, string content) : base(content)
    {
        StatusCode = statusCode;
    }
}

ซอร์สโค้ดสำหรับรหัส SureSuccessStatusCode ของ Microsoft สามารถพบได้ที่นี่

เวอร์ชันซิงโครนัสตามลิงค์ SO :

public static void EnsureSuccessStatusCode(this HttpResponseMessage response)
{
    if (response.IsSuccessStatusCode)
    {
        return;
    }

    var content = response.Content.ReadAsStringAsync().GetAwaiter().GetResult();

    if (response.Content != null)
        response.Content.Dispose();

    throw new SimpleHttpResponseException(response.StatusCode, content);
}

สิ่งที่ฉันไม่ชอบเกี่ยวกับ IsSuccessStatusCode ก็คือมันไม่สามารถใช้ซ้ำได้ "อย่างดี" ตัวอย่างเช่นคุณสามารถใช้ไลบรารีเช่นpollyเพื่อทำคำขอซ้ำในกรณีที่มีปัญหาเครือข่าย ในกรณีนี้คุณต้องใช้รหัสของคุณเพื่อเพิ่มข้อยกเว้นเพื่อให้พอลลี่หรือไลบรารีอื่น ๆ สามารถจัดการได้ ...

ฉันใช้ SureSuccessStatusCode เมื่อฉันไม่ต้องการจัดการกับ Exception ด้วยวิธีการเดียวกัน

public async Task DoSomethingAsync(User user)
{
    try
    {
        ...
        var userId = await GetUserIdAsync(user)
        ...
    }
    catch(Exception e)
    {
        throw;
    }
}

public async Task GetUserIdAsync(User user)
{
    using(var client = new HttpClient())
    {
        ...
        response = await client.PostAsync(_url, context);

        response.EnsureSuccesStatusCode();
        ...
    }
}

ข้อยกเว้นที่เกิดขึ้นใน GetUserIdAsync จะได้รับการจัดการบน DoSomethingAsync

ด้านล่างนี้คือโซลูชันที่ฉันเสนอ ข้อบกพร่องเพียงอย่างเดียวคือเนื่องจากตัวจัดการทรัพยากรกรอบงาน ASP.NET Core เป็นแบบภายในของกรอบงานฉันจึงไม่สามารถใช้สตริงข้อความสากลของ Microsoft ซ้ำได้โดยตรงดังนั้นฉันจึงใช้ข้อความภาษาอังกฤษแบบคำต่อคำที่นี่

ข้อดี

• บันทึกเนื้อหาสำหรับข้อผิดพลาดของเซิร์ฟเวอร์ 5xx
  • บางครั้งข้อผิดพลาดของเซิร์ฟเวอร์เป็นข้อผิดพลาดของไคลเอ็นต์ในการปลอมตัวเช่นไคลเอนต์ที่ใช้ปลายทางที่เลิกใช้แล้วซึ่งในที่สุดก็ถูกปิด
• ทำให้ง่ายต่อการเปิดเผยข้อผิดพลาดเมื่อเขียนการทดสอบการรวมโดยใช้ ConfigureTestContainer<T>

จุดด้อย

• ไม่มีประสิทธิภาพ
  • หากคุณอ่านเนื้อหาตอบกลับและเนื้อหายาวมากคุณจะทำให้ลูกค้าทำงานช้าลง สำหรับลูกค้าบางรายที่มีข้อกำหนดในการตอบสนองแบบเรียลไทม์ที่นุ่มนวลการกระวนกระวายใจนี้อาจไม่สามารถยอมรับได้
• ความรับผิดชอบที่ไม่ถูกต้องสำหรับการบันทึกและการตรวจสอบข้อผิดพลาด
  • หากนี่เป็นข้อผิดพลาดของเซิร์ฟเวอร์ 5xx เหตุใดไคลเอ็นต์จึงสนใจเนื่องจากไคลเอ็นต์ไม่ได้ทำอะไรผิด เพียงแค่โทรresponse.EnsureSuccessStatusCode();และปล่อยให้เซิร์ฟเวอร์จัดการกับมัน
  • ทำไมไม่ตรวจสอบบันทึกข้อผิดพลาดของเซิร์ฟเวอร์เมื่อมี Internal Server Error?
• ต้องอ่านContentคุณสมบัติก่อนตรวจสอบสถานะ อาจมีสถานการณ์ที่ไม่พึงปรารถนาซึ่งหนึ่งในนั้นคือการขาดประสิทธิภาพ

การใช้งาน

using (var requestMessage = new HttpRequestMessage(HttpMethod.Post, "controller/action"))
{
  using (var response = await HttpClient.SendAsync(requestMessage))
  {
    var content = await response.Content.ReadAsStringAsync();
    response.EnsureSuccessStatusCode2(content);
    var result = JsonConvert.DeserializeObject<ResponseClass>(content);
  }
}

API

    public static class HttpResponseMessageExtensions
    {
        public static void EnsureSuccessStatusCode2(this HttpResponseMessage message, string content = null)
        {
            if (message.IsSuccessStatusCode)
                return;
            var contentMessage = string.IsNullOrWhiteSpace(content) ? string.Empty : $"Content: {content}";
            throw new HttpRequestException(string.Format(
                System.Globalization.CultureInfo.InvariantCulture,
                "Response status code does not indicate success: {0} ({1}).{2}",
                (int)message.StatusCode,
                message.ReasonPhrase,
                contentMessage)
                );
        }
    }