Qual é a maneira correta de verificar valores nulos?

Adoro o operador de coalescência nula porque facilita a atribuição de um valor padrão para tipos anuláveis.

 int y = x ?? -1;

Isso é ótimo, exceto se eu precisar fazer algo simples x. Por exemplo, se eu quiser verificar Session, geralmente acabo tendo que escrever algo mais detalhado.

Eu gostaria de poder fazer isso:

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

Mas você não pode, porque a .ToString()chamada é feita antes da verificação nula, portanto ela falha se Session["key"]for nula. Acabo fazendo isso:

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

Funciona e é melhor, na minha opinião, do que a alternativa de três linhas:

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

Mesmo que isso funcione, ainda estou curioso para saber se existe uma maneira melhor. Parece que não importa o que eu sempre precise fazer referência Session["key"]duas vezes; uma vez para o cheque e novamente para a tarefa. Alguma ideia?

A respeito

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

Se você costuma fazer isso especificamente comToString() isso, pode escrever um método de extensão:

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

...

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

Ou um método adotando um padrão, é claro:

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

...

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

Você também pode usar as, que gera nullse a conversão falhar:

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

Este voltaria "none"mesmo se alguém recheado um intno Session["key"].

Se sempre for um string, você pode transmitir:

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

Isso tem a vantagem de reclamar, em vez de esconder o erro, se alguém colocar intalgo ou algo Session["key"]. ;)

Todas as soluções sugeridas são boas e respondem à pergunta; então isso é apenas para estender um pouco. Atualmente, a maioria das respostas lida apenas com validação nula e tipos de sequência. Você pode estender o StateBagobjeto para incluir um GetValueOrDefaultmétodo genérico , semelhante à resposta postada por Jon Skeet.

Um método simples de extensão genérica que aceita uma cadeia como chave e, em seguida, o tipo verifica o objeto da sessão. Se o objeto for nulo ou não for o mesmo tipo, o padrão será retornado, caso contrário, o valor da sessão será retornado com forte digitação.

Algo assim

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

Nós usamos um método chamado NullOr.

Uso

// 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());

Fonte

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

Minha preferência, por um lado, seria usar uma conversão segura para string, caso o objeto armazenado com a chave não seja um. O uso ToString()pode não ter os resultados desejados.

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

Como o @Jon Skeet diz, se você se encontra fazendo muito isso, é um método de extensão ou, melhor ainda, talvez um método de extensão em conjunto com uma classe SessionWrapper fortemente tipada. Mesmo sem o método de extensão, o wrapper fortemente tipado pode ser uma boa ideia.

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

Usado como

 var session = new SessionWrapper(Session);

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

criar uma função auxiliar

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


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

A resposta de Skeet é a melhor - particularmente acho que ToStringOrNull() é bastante elegante e atende melhor às suas necessidades. Eu queria adicionar mais uma opção à lista de métodos de extensão:

Retorne o objeto original ou o valor padrão da cadeia para 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");

Use varpara o valor retornado, pois ele retornará como o tipo de entrada original, apenas como a sequência padrão quandonull

Este é o meu pequeno tipo "operador Elvis" seguro para versões do .NET que não suportam?

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

O primeiro argumento é o objeto testado. O segundo é a função. E terceiro é o valor nulo. Então, para o seu caso:

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

Também é muito útil para tipos anuláveis. Por exemplo:

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