Jak mogę się upewnić, że FirstOrDefault <KeyValuePair> zwrócił wartość

Oto uproszczona wersja tego, co próbuję zrobić:

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

Ponieważ „xyz” nie jest obecny w słowniku, metoda FirstOrDefault nie zwróci prawidłowej wartości. Chcę mieć możliwość sprawdzenia tej sytuacji, ale zdaję sobie sprawę, że nie mogę porównać wyniku z wartością „null”, ponieważ KeyValuePair jest strukturą. Poniższy kod jest nieprawidłowy:

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

Podczas próby skompilowania kodu program Visual Studio zgłasza następujący błąd:

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

Jak mogę sprawdzić, czy FirstOrDefault zwrócił prawidłową wartość?

c# linq

— desautelsj
źródło

Masz tam błąd, ale zakładam, że jest to kwestia kopiowania i wklejania: dni nie są listą i nie możesz użyć dodatku na KeyValuePair.

— Kobi

ooops ... masz rację Pisałem z pamięci i oczywiście popełniłem błąd. Dzięki za wskazanie tego.

— desautelsj

Prawdopodobnie było to: var days = new Dictionary <int, string> ();

— Nawet Mien

Odpowiedzi:

156

FirstOrDefaultnie zwraca null, zwraca default(T).
Powinieneś sprawdzić:

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

Z MSDN -Enumerable.FirstOrDefault<TSource> :

domyślnie ( TSource ), jeśli źródło jest puste; w przeciwnym razie pierwszy element w source .

Uwagi:

Jeśli twój kod jest ogólny, lepiej go użyć EqualityComparer<T>.Default.Equals(day, defaultDay), ponieważ .Equalsmoże zostać zastąpiony lub daymoże to być plik null.
W języku C # 7.1 będzie można użyć KeyValuePair<int, string> defaultDay = default;, zobacz literał „domyślny” z typem docelowym .
Zobacz także: Źródło odniesienia -FirstOrDefault

— Kobi
źródło

+1, KeyValuePair jest typem wartości (struct), a nie typem referencyjnym (klasa) ani typem wartości dopuszczającej wartość null, więc nie może mieć wartości null.

— Lucas

@ paper1337 - Dzięki, ale gdzie mi brakuje typeof? Ten kod kompiluje się i działa.

— Kobi

Przyszedłem tutaj, ponieważ nie było dla mnie jasne, co default(KeyValuePair<T1, T2>)z tego wyniknie. Ok, powinno być dość oczywiste, że spowoduje to pusty KVP. Ale ponieważ "bycie oczywistym" nie jest dobrym podejściem do pisania poprawnych aplikacji (a moja obecna implementacja jest zbyt złożona, aby jasno / czysto sprowokować ten przypadek), wypróbowałem ją z nowym projektem i - rzeczywiście - zwróciła KeyValuePairz właściwościami Keyi Valuebyciem jedno i drugie NULL... tylko po to, żeby uratować innym ludziom te 5 minut głupoty ;-)

— Nicolas.

@Nicolas - Żadnej głupoty tutaj. Zawsze dobrze jest sprawdzić samemu i upewnić się, że rozumiesz kod. Dodałem link do defaultsłowa kluczowego, którego tutaj wyraźnie brakuje. Dzięki!

— Kobi,

@JeffBridgman - To naprawdę dobra uwaga! Konkretnie tutaj nie jest to możliwe, ponieważ pracujemy z KeyValuePair. Gdybyś miał ogólny kod, day.Equalsnie jest nawet bezpieczny dla null i EqualityComparer<T>.Default.Equals(day, defaultDay)

— użyłbym

Moim zdaniem jest to najbardziej jasny i zwięzły sposób:

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

To całkowicie rozwiązuje problem przy użyciu dziwnych wartości domyślnych dla struktur.

— pokój na zewnątrz
źródło

Problem z tym polega na tym, że istnieje możliwość (w zależności od implementacji), że wyliczalne dni będą wyliczane dwukrotnie lub, co gorsza, zwracają różne wartości między wywołaniami Any () i First ()

— Ray Booysen

@RayBooysen Wywołanie ToArray lub ToList rozwiązuje problem i możesz użyć Count / Length i Indexer.

— Konsola

Zauważ, że odpowiedź @ Ray nie ma tutaj zastosowania, ponieważ daysjest Dictionary<int,string>. Więc będzie traktowane jako IEnumerable<KeyValuePair<int,string>>, a następnie zachowuje się zgodnie z oczekiwaniami, gdy Any()i First()są wywoływane. Wydaje mi się, że istnieją inne implementacje, które mogą zachowywać się inaczej niż IEnumerable<>. Nie wiem, czy czegoś mi brakuje.

— Emanuele Bellini

Zamiast tego możesz to zrobić:

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

i wtedy :

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

— Jocelyn Marcotte
źródło