Verwendung von EnsureSuccessStatusCode und Behandlung der ausgelösten HttpRequestException

Was ist das Nutzungsmuster von HttpResponseMessage.EnsureSuccessStatusCode()? Es verfügt über den Inhalt der Nachricht und wirft HttpRequestException, aber ich sehe nicht ein, wie ich programmgesteuert anders damit umgehen soll als mit einem Generikum Exception. Zum Beispiel enthält es nicht das HttpStatusCode, was praktisch gewesen wäre.

Gibt es eine Möglichkeit, mehr Informationen daraus zu ziehen? Könnte jemand das relevante Verwendungsmuster von sowohl EnsureSuccessStatusCode()HttpRequestException als auch HttpRequestException anzeigen?

Die idiomatische Verwendung von EnsureSuccessStatusCodebesteht darin, den Erfolg einer Anforderung genau zu überprüfen, wenn Sie Fehlerfälle nicht auf eine bestimmte Weise behandeln möchten. Dies ist besonders nützlich, wenn Sie einen Client schnell prototypisieren möchten.

Führen Sie die folgenden Schritte nicht aus, wenn Sie sich entscheiden, Fehlerfälle auf bestimmte Weise zu behandeln .

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

Dies löst eine Ausnahme aus, um sie sofort zu erfassen, was keinen Sinn ergibt. Das IsSuccessStatusCodeEigentum von HttpResponseMessageist zu diesem Zweck vorhanden. Gehen Sie stattdessen wie folgt vor.

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

Ich mag EnsureSuccessStatusCode nicht, da es nichts Sinnvolles zurückgibt. Deshalb habe ich meine eigene Erweiterung erstellt:

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;
    }
}

Der Quellcode für den EnsureSuccessStatusCode von Microsoft ist zu finden hier

Synchrone Version basierend auf SO-Link :

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);
}

Was ich an IsSuccessStatusCode nicht mag, ist, dass es nicht "schön" wiederverwendbar ist. Beispielsweise können Sie eine Bibliothek wie polly verwenden , um eine Anforderung im Falle eines Netzwerkproblems zu wiederholen. In diesem Fall muss Ihr Code eine Ausnahme auslösen, damit polly oder eine andere Bibliothek damit umgehen kann ...

Ich verwende EnsureSuccessStatusCode, wenn ich die Ausnahme nicht mit derselben Methode behandeln möchte.

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();
        ...
    }
}

Die auf GetUserIdAsync ausgelöste Ausnahme wird auf DoSomethingAsync behandelt.