Como posso ter certeza de que FirstOrDefault <KeyValuePair> retornou um valor

Esta é uma versão simplificada do que estou tentando fazer:

var days = new Dictionary<int, string>();
days.Add(1, "Monday");
days.Add(2, "Tuesday");
...
days.Add(7, "Sunday");

var sampleText = "My favorite day of the week is 'xyz'";
var day = days.FirstOrDefault(x => sampleText.Contains(x.Value));

Como 'xyz' não está presente no dicionário, o método FirstOrDefault não retornará um valor válido. Quero ser capaz de verificar essa situação, mas percebo que não posso comparar o resultado com "null" porque KeyValuePair é uma estrutura. O seguinte código é inválido:

if (day == null) {
    System.Diagnotics.Debug.Write("Couldn't find day of week");
}

Quando você tenta compilar o código, o Visual Studio gera o seguinte erro:

Operator '==' cannot be applied to operands of type 'System.Collections.Generic.KeyValuePair<int,string>' and '<null>'

Como posso verificar se FirstOrDefault retornou um valor válido?

c# linq

— desautelsj
fonte

Você tem um bug aí, mas presumo que seja uma coisa de copiar e colar: dias não é uma lista e você não pode usar o add em KeyValuePair.

— Kobi

opa ... você está correto, eu estava digitando de memória e obviamente cometi um erro. Obrigado por apontar isso.

— desautelsj

Provavelmente foi: var days = new Dictionary <int, string> ();

— Even Mien

Respostas:

155

FirstOrDefaultnão retorna nulo, ele retorna default(T).
Você deve verificar:

var defaultDay = default(KeyValuePair<int, string>);
bool b = day.Equals(defaultDay);

Do MSDN -Enumerable.FirstOrDefault<TSource> :

padrão ( TSource ) se a fonte estiver vazia; caso contrário, o primeiro elemento na origem .

Notas:

Se o seu código for genérico, é melhor usar EqualityComparer<T>.Default.Equals(day, defaultDay), porque .Equalspode ser substituído ou daypode ser um null.
No C # 7.1 você poderá usar KeyValuePair<int, string> defaultDay = default;, consulte literal "padrão" digitado no destino .
Veja também: Fonte de Referência -FirstOrDefault

— Kobi
fonte

+1, KeyValuePair é um tipo de valor (struct), não um tipo de referência (classe) ou um tipo de valor anulável, portanto, não pode ser nulo.

— Lucas

@ paper1337 - Obrigado, mas onde estou perdendo typeof? Este código compila e funciona.

— Kobi,

Vim aqui porque não estava claro para mim o default(KeyValuePair<T1, T2>)que resultaria. Ok, deveria ter sido bastante óbvio, que resultaria em um KVP vazio. Mas como "ser óbvio" não é uma boa abordagem para escrever aplicativos adequados (e minha implementação atual é muito complexa para provocar de forma clara / limpa este caso), eu tentei com um novo projeto e - de fato - ele retornou um KeyValuePaircom propriedades Keye Valuesendo ambos NULL.... apenas para proteger outras pessoas nestes 5 minutos de estupidez ;-)

— Nicolas

@Nicolas - Sem estupidez aqui. É sempre uma boa ideia verificar por si mesmo e certificar-se de que entendeu seu código. Eu adicionei um link para a defaultpalavra-chave, ele está claramente faltando aqui. Obrigado!

— Kobi

@JeffBridgman - Esse é realmente um bom ponto! Especificamente aqui não é possível, porque estamos trabalhando com KeyValuePair. Se você tivesse um código genérico, day.Equalsnem mesmo é seguro para nulos, e eu teria usadoEqualityComparer<T>.Default.Equals(day, defaultDay)

— Kobi

Esta é a forma mais clara e concisa em minha opinião:

var matchedDays = days.Where(x => sampleText.Contains(x.Value));
if (!matchedDays.Any())
{
    // Nothing matched
}
else
{
    // Get the first match
    var day = matchedDays.First();
}

Isso contorna completamente o uso de coisas estranhas de valor padrão para estruturas.

— lado da paz
fonte

O problema com isso é que existe a possibilidade (dependendo da implementação) de que os dias enumeráveis sejam enumerados duas vezes ou, pior ainda, retornem valores diferentes entre as chamadas Any () e First ()

— Ray Booysen

@RayBooysen Uma chamada de ToArray ou ToList resolve o problema e você pode usar Count / Length e um Indexer.

— Console de

Observe que a resposta de @Ray não se aplica aqui, porque daysé a Dictionary<int,string>. Portanto, ele será considerado como um IEnumerable<KeyValuePair<int,string>>, comportando-se conforme o esperado quando Any()e First()for chamado. Eu acho que existem outras implementações que podem se comportar de forma diferente como IEnumerable<>. Não sei se estou perdendo alguma coisa.

— Emanuele Bellini,

Você pode fazer isso:

var days = new Dictionary<int?, string>();   // replace int by int?
days.Add(1, "Monday");
days.Add(2, "Tuesday");
...
days.Add(7, "Sunday");

var sampleText = "My favorite day of the week is 'xyz'";
var day = days.FirstOrDefault(x => sampleText.Contains(x.Value));

e depois :

if (day.Key == null) {
    System.Diagnotics.Debug.Write("Couldn't find day of week");
}

— Jocelyn Marcotte
fonte