web-dev-qa-db-ja.com

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

Null-coalescing演算子は、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"]_を2回参照しなければならないようです。 1回はチェックのため、もう1回は割り当てのためです。何か案は？

c#.netsessionnullnull-coalescing-operator
122
Chev

どう？

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

これを頻繁に行う場合具体的には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");
130
Jon Skeet

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

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

誰かが"none"にintを詰め込んでも、これはSession["key"]を返します。

21
Andomar

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

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

これには、誰かがintまたは何かをSession["key"]。 ;）

14
Ry-

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

キーとして文字列を受け取り、セッションオブジェクトをタイプチェックする単純な汎用拡張メソッド。オブジェクトが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;
}
10
Richard

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

私の好みは、1回限り、キーとともに格納されたオブジェクトが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;
6
tvanfosson

補助関数を作成する

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


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

スキートの答えは最高です。特に、彼の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の場合のデフォルト文字列としてのみ、元の入力の型として返されるためです。

2
one.beat.consumer

これは、？をサポートしない.NETのバージョン用の私の小さなタイプセーフな「エルビス演算子」です。

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番目はヌル値です。あなたの場合：

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

Null許容型に対しても非常に便利です。例えば：

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