null値をチェックする適切な方法は何ですか？

null結合型の演算子にデフォルト値を簡単に割り当てることができるため、null結合演算子が大好きです。

 int y = x ?? -1;

それは素晴らしいことですが、で簡単なことをする必要がある場合は例外ですx。たとえば、を確認したい場合、Session通常はもっと冗長なものを書く必要があります。

私はこれを行うことができればいいのに：

string y = Session["key"].ToString() ?? "none";

しかし.ToString()、nullチェックの前にが呼び出されるため、Session["key"]nullの場合は失敗するため、これはできません。私はこれをやる：

string y = Session["key"] == null ? "none" : Session["key"].ToString();

それは機能し、私の意見では、3行の代替よりも優れています。

string y = "none";
if (Session["key"] != null)
    y = Session["key"].ToString();

それがうまくいくとしても、もっと良い方法があるかどうか私はまだ興味があります。私がいつも何Session["key"]回参照しなければならないかは関係ありません。1回はチェックのため、もう1回は割り当てのためです。何か案は？

どうですか

string y = (Session["key"] ?? "none").ToString();

特にToString()これを頻繁に行う場合は、拡張メソッドを作成できます。

public static string NullPreservingToString(this object input)
{
    return input == null ? null : input.ToString();
}

...

string y = Session["key"].NullPreservingToString() ?? "none";

または、もちろんデフォルトを取るメソッド：

public static string ToStringOrDefault(this object input, string defaultValue)
{
    return input == null ? defaultValue : input.ToString();
}

...

string y = Session["key"].ToStringOrDefault("none");

また、変換が失敗した場合asに生成さnullれるを使用することもできます。

Session["key"] as string ?? "none"

これは戻ってくる"none"誰かが詰めても、int中にSession["key"]。

常にであるstring場合は、キャストできます。

string y = (string)Session["key"] ?? "none";

これには、誰かがにint何かを詰め込んだ場合に間違いを隠す代わりに不平を言うという利点がありSession["key"]ます。;）

提案された解決策はすべて適切であり、質問に答えます。これは少し拡張するだけです。現在、回答の大部分はnull検証と文字列型のみを扱います。StateBagオブジェクトを拡張してジェネリックを含めることができますGetValueOrDefaultJon Skeetが投稿した回答と同様にメソッド。

文字列をキーとして受け入れ、セッションオブジェクトを型チェックする単純な汎用拡張メソッド。オブジェクトがnullまたは同じタイプでない場合はデフォルトが返され、それ以外の場合はセッション値が厳密に型指定されて返されます。

このようなもの

/// <summary>
/// Gets a value from the current session, if the type is correct and present
/// </summary>
/// <param name="key">The session key</param>
/// <param name="defaultValue">The default value</param>
/// <returns>Returns a strongly typed session object, or default value</returns>
public static T GetValueOrDefault<T>(this HttpSessionState source, string key, T defaultValue)
{
    // check if the session object exists, and is of the correct type
    object value = source[key]
    if (value == null || !(value is T))
    {
        return defaultValue;
    }

    // return the session object
    return (T)value;
}

と呼ばれるメソッドを使用します NullOr。

使用法

// Call ToString() if it’s not null, otherwise return null
var str = myObj.NullOr(obj => obj.ToString());

// Supply default value for when it’s null
var str = myObj.NullOr(obj => obj.ToString()) ?? "none";

// Works with nullable return values, too —
// this is properly typed as “int?” (nullable int)
// even if “Count” is just int
var count = myCollection.NullOr(coll => coll.Count);

// Works with nullable input types, too
int? unsure = 47;
var sure = unsure.NullOr(i => i.ToString());

ソース

/// <summary>Provides a function delegate that accepts only value types as return types.</summary>
/// <remarks>This type was introduced to make <see cref="ObjectExtensions.NullOr{TInput,TResult}(TInput,FuncStruct{TInput,TResult})"/>
/// work without clashing with <see cref="ObjectExtensions.NullOr{TInput,TResult}(TInput,FuncClass{TInput,TResult})"/>.</remarks>
public delegate TResult FuncStruct<in TInput, TResult>(TInput input) where TResult : struct;

/// <summary>Provides a function delegate that accepts only reference types as return types.</summary>
/// <remarks>This type was introduced to make <see cref="ObjectExtensions.NullOr{TInput,TResult}(TInput,FuncClass{TInput,TResult})"/>
/// work without clashing with <see cref="ObjectExtensions.NullOr{TInput,TResult}(TInput,FuncStruct{TInput,TResult})"/>.</remarks>
public delegate TResult FuncClass<in TInput, TResult>(TInput input) where TResult : class;

/// <summary>Provides extension methods that apply to all types.</summary>
public static class ObjectExtensions
{
    /// <summary>Returns null if the input is null, otherwise the result of the specified lambda when applied to the input.</summary>
    /// <typeparam name="TInput">Type of the input value.</typeparam>
    /// <typeparam name="TResult">Type of the result from the lambda.</typeparam>
    /// <param name="input">Input value to check for null.</param>
    /// <param name="lambda">Function to apply the input value to if it is not null.</param>
    public static TResult NullOr<TInput, TResult>(this TInput input, FuncClass<TInput, TResult> lambda) where TResult : class
    {
        return input == null ? null : lambda(input);
    }

    /// <summary>Returns null if the input is null, otherwise the result of the specified lambda when applied to the input.</summary>
    /// <typeparam name="TInput">Type of the input value.</typeparam>
    /// <typeparam name="TResult">Type of the result from the lambda.</typeparam>
    /// <param name="input">Input value to check for null.</param>
    /// <param name="lambda">Function to apply the input value to if it is not null.</param>
    public static TResult? NullOr<TInput, TResult>(this TInput input, Func<TInput, TResult?> lambda) where TResult : struct
    {
        return input == null ? null : lambda(input);
    }

    /// <summary>Returns null if the input is null, otherwise the result of the specified lambda when applied to the input.</summary>
    /// <typeparam name="TInput">Type of the input value.</typeparam>
    /// <typeparam name="TResult">Type of the result from the lambda.</typeparam>
    /// <param name="input">Input value to check for null.</param>
    /// <param name="lambda">Function to apply the input value to if it is not null.</param>
    public static TResult? NullOr<TInput, TResult>(this TInput input, FuncStruct<TInput, TResult> lambda) where TResult : struct
    {
        return input == null ? null : lambda(input).Nullable();
    }
}

私の好みは、キーが格納されているオブジェクトが1つでない場合に備えて、安全な文字列へのキャストを使用することです。を使用してToString()も、期待した結果が得られない場合があります。

var y = Session["key"] as string ?? "none";

@Jon Skeetが言うように、これをたくさん行う場合は、拡張メソッド、またはより適切には、強く型付けされたSessionWrapperクラスと組み合わせた拡張メソッドを使用します。拡張メソッドがなくても、強く型付けされたラッパーは良い考えかもしれません。

public class SessionWrapper
{
    private HttpSessionBase Session { get; set; }

    public SessionWrapper( HttpSessionBase session )
    {
        Session = session;
    }

    public SessionWrapper() : this( HttpContext.Current.Session ) { }

    public string Key
    {
         get { return Session["key"] as string ?? "none";
    }

    public int MaxAllowed
    {
         get { return Session["maxAllowed"] as int? ?? 10 }
    }
}

使用されます

 var session = new SessionWrapper(Session);

 string key = session.Key;
 int maxAllowed = session.maxAllowed;

補助関数を作成する

public static String GetValue( string key, string default )
{
    if ( Session[ key ] == null ) { return default; }
    return Session[ key ].toString();
}


string y = GetValue( 'key', 'none' );

スキートの答えは最高です-特に私は彼ToStringOrNull()が非常にエレガントであなたのニーズに最もよく合っていると思います。拡張メソッドのリストにオプションをもう1つ追加したいと思います。

元のオブジェクトまたはnullのデフォルトの文字列値を返します。

// Method:
public static object OrNullAsString(this object input, string defaultValue)
{
    if (defaultValue == null)
        throw new ArgumentNullException("defaultValue");
    return input == null ? defaultValue : input;
}

// Example:
var y = Session["key"].OrNullAsString("defaultValue");

var元の入力のタイプとして返​​されるため、戻り値に使用します。デフォルトの文字列としてのみnull

これは、サポートされていないバージョンの.NET用の小さなタイプセーフな「Elvis演算子」です。

public class IsNull
{
    public static O Substitute<I,O>(I obj, Func<I,O> fn, O nullValue=default(O))
    {
        if (obj == null)
            return nullValue;
        else
            return fn(obj);
    }
}

最初の引数はテストされるオブジェクトです。2つ目は関数です。3番目はnull値です。だからあなたの場合：

IsNull.Substitute(Session["key"],s=>s.ToString(),"none");

null許容型にも非常に役立ちます。例えば：

decimal? v;
...
IsNull.Substitute(v,v.Value,0);
....