Verificar se HttpStatusCode representa sucesso ou falha

Vamos supor que eu tenha a seguinte variável:

System.Net.HttpStatusCode status = System.Net.HttpStatusCode.OK;

Como posso verificar se este é um código de status de sucesso ou de falha?

Por exemplo, posso fazer o seguinte:

int code = (int)status;
if(code >= 200 && code < 300) {
    //Success
}

Também posso ter algum tipo de lista branca:

HttpStatusCode[] successStatus = new HttpStatusCode[] {
     HttpStatusCode.OK,
     HttpStatusCode.Created,
     HttpStatusCode.Accepted,
     HttpStatusCode.NonAuthoritativeInformation,
     HttpStatusCode.NoContent,
     HttpStatusCode.ResetContent,
     HttpStatusCode.PartialContent
};
if(successStatus.Contains(status)) //LINQ
{
    //Success
}

Nenhuma dessas alternativas me convence, e eu esperava por uma classe ou método .NET que pudesse fazer esse trabalho para mim, como:

bool isSuccess = HttpUtilities.IsSuccess(status);

Se você estiver usando a HttpClientclasse, terá uma HttpResponseMessagevolta.

Esta classe tem uma propriedade útil chamada IsSuccessStatusCodeque fará a verificação para você.

using (var client = new HttpClient())
{
    var response = await client.PostAsync(uri, content);
    if (response.IsSuccessStatusCode)
    {
        //...
    }
}

Caso você esteja curioso, esta propriedade é implementada como:

public bool IsSuccessStatusCode
{
    get { return ((int)statusCode >= 200) && ((int)statusCode <= 299); }
}

Então você pode simplesmente reutilizar este algoritmo se você não usar HttpClientdiretamente.

Você também pode usar EnsureSuccessStatusCodepara lançar uma exceção caso a resposta não seja bem-sucedida.

A classe HttpResponseMessage tem uma propriedade IsSuccessStatusCode, olhando para o código-fonte, é assim como usr já sugeriu 200-299 é provavelmente o melhor que você pode fazer.

public bool IsSuccessStatusCode
{
    get { return ((int)statusCode >= 200) && ((int)statusCode <= 299); }
}

A resposta aceita me incomoda um pouco, pois contém números mágicos, (embora estejam no padrão) em sua segunda parte. E a primeira parte não é genérica para códigos de status de inteiros simples, embora esteja perto da minha resposta.

Você poderia obter exatamente o mesmo resultado instanciando HttpResponseMessage com seu código de status e verificando o sucesso. Ele lança uma exceção de argumento se o valor for menor que zero ou maior que 999.

if (new HttpResponseMessage((HttpStatusCode)statusCode).IsSuccessStatusCode)
{
    // ...
}

Isso não é exatamente conciso, mas você pode torná-lo uma extensão.

Adicionando a resposta @TomDoesCode Se você estiver usando HttpWebResponse, você pode adicionar este método de extensão:

public static bool IsSuccessStatusCode(this HttpWebResponse httpWebResponse)
{
    return ((int)httpWebResponse.StatusCode >= 200) && ((int)httpWebResponse.StatusCode <= 299);
}

Sou parcial quanto à descoberta de métodos de extensão.

public static class HttpStatusCodeExtensions
{
    public static bool IsSuccessStatusCode(this HttpStatusCode statusCode)
    {
        var asInt = (int)statusCode;
        return asInt >= 200 && asInt <= 299;
    }
}

Enquanto o seu namespace estiver no escopo, o uso estará statusCode.IsSuccessStatusCode().

Depende de qual recurso HTTP você está chamando. Normalmente, o 2xxintervalo é definido como o intervalo de códigos de status de sucesso. Essa é claramente uma convenção que nem todo servidor HTTP seguirá.

Por exemplo, o envio de um formulário em um site geralmente retornará um redirecionamento 302.

Se você deseja desenvolver um método geral, a code >= 200 && code < 300ideia é provavelmente sua melhor chance.

Se você estiver chamando seu próprio servidor, provavelmente deve se certificar de padronizar 200.

Esta é uma extensão da resposta anterior, que evita a criação e posterior coleta de lixo de um novo objeto a cada invocação.

public static class StatusCodeExtensions
{
    private static readonly ConcurrentDictionary<HttpStatusCode, bool> IsSuccessStatusCode = new ConcurrentDictionary<HttpStatusCode, bool>();
    public static bool IsSuccess(this HttpStatusCode statusCode) => IsSuccessStatusCode.GetOrAdd(statusCode, c => new HttpResponseMessage(c).IsSuccessStatusCode);
}