Перевірка, чи HttpStatusCode представляє успіх чи невдачу

Припустимо, у мене є така змінна:

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

Як я можу перевірити, це код стану успіху чи помилки?

Наприклад, я можу зробити наступне:

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

Я також можу отримати якийсь білий список:

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

Жодна з цих альтернатив мене не переконує, і я сподівався на клас або метод .NET, які можуть зробити цю роботу за мене, наприклад:

bool isSuccess = HttpUtilities.IsSuccess(status);

Якщо ви використовуєте HttpClientклас, то отримаєте HttpResponseMessageназад.

Цей клас має корисну властивість під назвою, IsSuccessStatusCodeяка зробить перевірку за вас.

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

Якщо вам цікаво, ця властивість реалізована як:

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

Таким чином, ви можете просто використовувати цей алгоритм, якщо не використовуєте HttpClientбезпосередньо.

Ви також можете використати EnsureSuccessStatusCodeвикид, якщо відповідь не була успішною.

Клас HttpResponseMessage має властивість IsSuccessStatusCode, дивлячись на вихідний код, він такий, так як usr вже припустив, що 200-299 - це, мабуть, найкраще, що ви можете зробити.

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

Прийнята відповідь мене трохи турбує, оскільки вона містить магічні числа (хоча вони є стандартними) у своїй другій частині. І перша частина не є загальною для простих цілочисельних кодів стану, хоча вона близька до моєї відповіді.

Ви могли б досягти точно такого ж результату, створивши інстанцію HttpResponseMessage зі своїм кодом стану та перевіривши успіх. Він викидає виняток аргументу, якщо значення менше нуля або більше 999.

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

Це не зовсім лаконічно, але ви можете зробити це продовженням.

Додавання до відповіді @TomDoesCode Якщо ви використовуєте HttpWebResponse, ви можете додати цей метод розширення:

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

Я прихильний до відкритості методів розширення.

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

Поки ваш простір імен знаходиться в області дії, використання буде таким statusCode.IsSuccessStatusCode().

Це залежить від того, який ресурс HTTP ви викликаєте. Зазвичай,2xx діапазон визначається як діапазон кодів стану успіху. Це явно домовленість, якої дотримуватиметься не кожен HTTP-сервер.

Наприклад, надсилання форми на веб-сайті часто повертає переадресацію 302.

Якщо ви хочете розробити загальний метод, тоді code >= 200 && code < 300ідея, мабуть, найкращий вистріл.

Якщо ви телефонуєте на власний сервер, то вам, мабуть, слід переконатись, що ви стандартизуєтесь 200.

Це розширення попередньої відповіді, яке дозволяє уникнути створення та подальшого збору сміття нового об’єкта для кожного виклику.

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