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

受け入れられた回答は、その2番目の部分にマジックナンバーが含まれているため（標準ではありますが）、少し気になります。そして、最初の部分は単純な整数ステータスコードに一般的ではありませんが、私の答えに近いです。

ステータスコードを使用して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);
}