Valeur de retour maximale si requête vide

J'ai cette question:

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

Que se passera-t-il maxShoeSizesi l'entreprise 8 n'a pas du tout de travailleurs?

MISE À JOUR:
Comment puis-je changer la requête pour obtenir 0 et pas une exception?

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

Le zéro DefaultIfEmptyn'est pas nécessaire.

Je sais que c'est une vieille question et que la réponse acceptée fonctionne, mais cette question a répondu à ma question de savoir si un tel ensemble vide entraînerait une exception ou un default(int)résultat.

La réponse acceptée cependant, bien que cela fonctionne, n'est pas la solution idéale à mon humble avis, qui n'est pas donnée ici. Ainsi je le fournis dans ma propre réponse pour le bénéfice de tous ceux qui le recherchent.

Le code original de l'OP était:

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

Voici comment je l'écrirais pour éviter les exceptions et fournir un résultat par défaut:

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

Cela provoque le type de retour de la Maxfonction int?, ce qui autorise le nullrésultat, puis ??remplace le nullrésultat par 0.

EDIT
Juste pour clarifier quelque chose à partir des commentaires, Entity Framework ne prend actuellement pas en charge le asmot - clé, donc la façon de l'écrire lorsque vous travaillez avec EF serait:

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

Comme le [TypeOfWorkers]peut être un nom de classe long et qu'il est fastidieux à écrire, j'ai ajouté une méthode d'extension pour vous aider.

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

Cela ne les poignées int, mais la même chose pourrait être fait pour long, doubleou tout autre type de valeur que vous avez besoin. L'utilisation de cette méthode d'extension est très simple, il vous suffit de passer votre fonction de sélection et d'inclure éventuellement une valeur à utiliser pour null, qui vaut par défaut 0. Donc, ce qui précède pourrait être réécrit comme suit:

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

Espérons que cela aide encore plus les gens.

Max () ne retournera rien dans ce cas.

Cela lèvera InvalidOperationException car la source ne contient aucun élément.

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

S'il s'agit de Linq to SQL, je n'aime pas l'utiliser Any()car cela entraîne plusieurs requêtes sur le serveur SQL.

Si ce ShoeSizen'est pas un champ Nullable, utiliser uniquement le .Max(..) ?? 0ne fonctionnera pas, mais ce qui suit:

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

Cela ne change absolument pas le SQL émis, mais il renvoie 0 si la séquence est vide car il change le Max()pour retourner un int?au lieu d'un int.

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

(en supposant que ShoeSizec'est de type int)

Si Workersest un DbSetou ObjectSetdepuis Entity Framework, votre requête initiale lèverait un InvalidOperationException, mais ne se plaignant pas d'une séquence vide, mais se plaignant que la valeur matérialisée NULL ne peut pas être convertie en un int.

Max lancera System.InvalidOperationException "La séquence ne contient aucun élément"

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

NB: la requête avec DefaultIfEmpty()peut être significativement plus lente . Dans mon cas, c'était une simple requête avec .DefaultIfEmpty(DateTime.Now.Date).

J'étais trop paresseux pour le profiler, mais évidemment EF a essayé d'obtenir toutes les lignes, puis de prendre la Max()valeur.

Conclusion: la manipulation InvalidOperationExceptionpeut parfois être le meilleur choix.

Vous pouvez utiliser un ternaire à l'intérieur .Max()pour gérer le prédicat et définir sa valeur;

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

Vous devrez gérer la Workerscollection comme étant nulle / vide si c'est une possibilité, mais cela dépendra de votre implémentation.

Vous pouvez essayer ceci:

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

Vous pouvez vérifier s'il y a des ouvriers avant de faire 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);
}