Maksymalna wartość zwracana w przypadku pustego zapytania

Mam to zapytanie:

int maxShoeSize = Workers
    .Where(x => x.CompanyId == 8)
    .Max(x => x.ShoeSize);

Co się stanie, maxShoeSizejeśli firma 8 w ogóle nie będzie miała pracowników?

AKTUALIZACJA:
Jak mogę zmienić zapytanie, aby uzyskać 0, a nie wyjątek?

int maxShoeSize = Workers.Where(x => x.CompanyId == 8)
                         .Select(x => x.ShoeSize)
                         .DefaultIfEmpty(0)
                         .Max();

Zerowanie DefaultIfEmptynie jest konieczne.

Wiem, że to stare pytanie i przyjęta odpowiedź działa, ale to pytanie odpowiadało na moje pytanie, czy taki pusty zestaw skutkowałby wyjątkiem, czy default(int)wynikiem.

Przyjęta odpowiedź jednak, choć działa, nie jest idealnym rozwiązaniem IMHO, którego nie podano tutaj. W ten sposób udzielam go w mojej własnej odpowiedzi dla dobra każdego, kto tego szuka.

Oryginalny kod OP był następujący:

int maxShoeSize = Workers.Where(x => x.CompanyId == 8).Max(x => x.ShoeSize);

Oto jak napisałbym to, aby zapobiec wyjątkom i podać domyślny wynik:

int maxShoeSize = Workers.Where(x => x.CompanyId == 8).Max(x => x.ShoeSize as int?) ?? 0;

Powoduje to, że zwracany typ Maxfunkcji jest int?, który pozwala na nullwynik, a następnie ??zastępuje nullwynik przez 0.

EDYTUJ
Aby wyjaśnić coś z komentarzy, Entity Framework obecnie nie obsługuje assłowa kluczowego, więc sposób zapisania go podczas pracy z EF byłby następujący:

int maxShoeSize = Workers.Where(x => x.CompanyId == 8).Max<[TypeOfWorkers], int?>(x => x.ShoeSize) ?? 0;

Ponieważ [TypeOfWorkers]może to być długa nazwa klasy i jest żmudna w pisaniu, dodałem metodę rozszerzenia, aby pomóc.

public static int MaxOrDefault<T>(this IQueryable<T> source, Expression<Func<T, int?>> selector, int nullValue = 0)
{
    return source.Max(selector) ?? nullValue;
}

To tylko uchwyty int, ale to samo można zrobić dla long, doublelub innego rodzaju wartości, czego potrzebujesz. Korzystanie z tej metody rozszerzenia jest bardzo proste, po prostu przekazujesz swoją funkcję selektora i opcjonalnie dołączasz wartość, która ma być użyta dla null, która domyślnie wynosi 0. Więc powyższe można przepisać w następujący sposób:

int maxShoeSize = Workers.Where(x => x.CompanyId == 8).MaxOrDefault(x => x.ShoeSize);

Miejmy nadzieję, że pomoże to ludziom jeszcze bardziej.

Max () w takim przypadku nic nie zwróci.

Podniesie InvalidOperationException, ponieważ źródło nie zawiera żadnych elementów.

int maxShoeSize = Workers.Where(x => x.CompanyId == 8)
                     .Select(x => x.ShoeSize)
                     .DefaultIfEmpty()
                     .Max();

Jeśli jest to Linq to SQL, nie lubię go używać, Any()ponieważ powoduje to wiele zapytań do serwera SQL.

Jeśli ShoeSizenie jest polem dopuszczającym wartość null, wówczas użycie samego polecenia .Max(..) ?? 0nie zadziała, ale następujące działania:

int maxShoeSize = Workers.Where(x = >x.CompanyId == 8).Max(x => (int?)x.ShoeSize) ?? 0;

Absolutnie nie zmienia wyemitowanego kodu SQL, ale zwraca 0, jeśli sekwencja jest pusta, ponieważ zmienia znak na Max()zwracający int?zamiast int.

int maxShoeSize=Workers.Where(x=>x.CompanyId==8)
    .Max(x=>(int?)x.ShoeSize).GetValueOrDefault();

(zakładając, że ShoeSizejest to typoweint )

Jeśli Workersjest DbSetlub ObjectSetz Entity Framework, Twoje początkowe zapytanie wyrzuciłoby InvalidOperationException, ale nie narzekało na pustą sekwencję, ale narzekało, że zmaterializowanej wartości NULL nie można przekonwertować na plik int.

Max wyrzuci System.InvalidOperationException "Sekwencja nie zawiera żadnych elementów"

class Program
{
    static void Main(string[] args)
    {
        List<MyClass> list = new List<MyClass>();

        list.Add(new MyClass() { Value = 2 });

        IEnumerable<MyClass> iterator = list.Where(x => x.Value == 3); // empty iterator.

        int max = iterator.Max(x => x.Value); // throws System.InvalidOperationException
    }
}

class MyClass
{
    public int Value;
}

Uwaga: zapytanie z DefaultIfEmpty()może być znacznie wolniejsze . W moim przypadku było to proste zapytanie z .DefaultIfEmpty(DateTime.Now.Date).

Byłem zbyt leniwy, aby go profilować, ale oczywiście EF próbował uzyskać wszystkie wiersze, a następnie przyjąć Max()wartość.

Wniosek: czasami obsługa InvalidOperationExceptionmoże być lepszym wyborem.

Możesz użyć trójargumentu wewnątrz .Max()do obsługi predykatu i ustawiania jego wartości;

// assumes Workers != null && Workers.Count() > 0
int maxShoeSize = Workers.Max(x => (x.CompanyId == 8) ? x.ShoeSize : 0);

Jeśli jest taka możliwość, musisz obsłużyć Workerszbiór jako pusty / pusty, ale będzie to zależeć od Twojej implementacji.

Możesz spróbować tego:

int maxShoeSize = Workers.Where(x=>x.CompanyId == 8).Max(x => x.ShoeSize) ?? 0;

Możesz sprawdzić, czy są jacyś pracownicy, zanim wykonasz Max ().

private int FindMaxShoeSize(IList<MyClass> workers) {
   var workersInCompany = workers.Where(x => x.CompanyId == 8);
   if(!workersInCompany.Any()) { return 0; }
   return workersInCompany.Max(x => x.ShoeSize);
}