Maximaler Rückgabewert bei leerer Abfrage

Ich habe diese Frage:

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

Was wird passieren, maxShoeSizewenn Unternehmen 8 überhaupt keine Mitarbeiter hat?

UPDATE:
Wie kann ich die Abfrage ändern, um 0 und keine Ausnahme zu erhalten?

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

Die Nullstellung DefaultIfEmptyist nicht erforderlich.

Ich weiß, dass dies eine alte Frage ist und die akzeptierte Antwort funktioniert, aber diese Frage beantwortete meine Frage, ob eine solche leere Menge zu einer Ausnahme oder einem default(int)Ergebnis führen würde.

Die akzeptierte Antwort ist jedoch, obwohl sie funktioniert, meiner Meinung nach nicht die ideale Lösung, die hier nicht angegeben wird. Daher gebe ich es in meiner eigenen Antwort zum Nutzen aller, die danach suchen.

Der ursprüngliche Code des OP war:

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

So würde ich es schreiben, um Ausnahmen zu vermeiden und ein Standardergebnis bereitzustellen:

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

Dies bewirkt den Rückgabetyp des Max Funktion lautet int?, wodurch das nullErgebnis zugelassen und ??das nullErgebnis durch ersetzt wird 0.

BEARBEITEN
Um etwas aus den Kommentaren zu verdeutlichen, unterstützt Entity Framework das asSchlüsselwort derzeit nicht. Die Vorgehensweise bei der Arbeit mit EF lautet also wie folgt :

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

Da [TypeOfWorkers]dies ein langer Klassenname sein könnte und das Schreiben mühsam ist, habe ich eine Erweiterungsmethode hinzugefügt, um zu helfen.

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

Dies funktioniert nur int , aber das gleiche könnte getan werden long, doubleoder jeder anderen Werttyp Sie benötigen. Die Verwendung dieser Erweiterungsmethode ist sehr einfach. Sie übergeben einfach Ihre Auswahlfunktion und geben optional einen Wert für null an, der standardmäßig 0 ist. Das oben Gesagte könnte also wie folgt umgeschrieben werden:

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

Hoffentlich hilft das den Leuten noch mehr.

Max () gibt in diesem Fall nichts zurück.

Es wird eine InvalidOperationException ausgelöst, da die Quelle keine Elemente enthält.

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

Wenn dies Linq to SQL ist, verwende ich es nicht gerne Any() da es zu mehreren Abfragen an SQL Server führt.

Wenn ShoeSizees sich nicht um ein nullbares Feld handelt, .Max(..) ?? 0funktioniert die Verwendung nur des Felds nicht, aber das Folgende funktioniert:

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

Das ausgegebene SQL wird absolut nicht geändert, aber es gibt 0 zurück, wenn die Sequenz leer ist, weil es das ändert Max() , um ein int?anstelle von einem zurückzugeben int.

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

(vorausgesetzt, das ShoeSizeist vom Typint )

Wenn Workerses sich um ein DbSetoder ObjectSetaus Entity Framework handelt, würde Ihre anfängliche Abfrage eine auslösen InvalidOperationException, sich jedoch nicht über eine leere Sequenz beschweren, sondern sich darüber beschweren, dass der materialisierte Wert NULL nicht in eine konvertiert werden kann int.

Max löst die System.InvalidOperationException aus. "Sequenz enthält keine Elemente."

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

Hinweis: Die Abfrage mit ist DefaultIfEmpty()möglicherweise erheblich langsamer . In meinem Fall war das eine einfache Abfrage mit .DefaultIfEmpty(DateTime.Now.Date).

Ich war zu faul, um es zu profilieren, aber offensichtlich hat EF versucht, alle Zeilen abzurufen und dann den Max()Wert zu übernehmen.

Fazit: Manchmal ist die Handhabung InvalidOperationExceptiondie bessere Wahl.

Sie können ein ternäres Element verwenden .Max(), um das Prädikat zu behandeln und seinen Wert festzulegen.

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

Sie müssten damit umgehen, dass die WorkersSammlung null / leer ist, wenn dies möglich ist, dies hängt jedoch von Ihrer Implementierung ab.

Sie können dies versuchen:

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

Sie können überprüfen, ob Mitarbeiter vorhanden sind, bevor Sie Max () ausführen.

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