Valore di ritorno massimo se query vuota

Ho questa domanda:

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

Cosa accadrà maxShoeSizese la compagnia 8 non ha alcun lavoratore?

AGGIORNAMENTO:
Come posso modificare la query per ottenere 0 e non un'eccezione?

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

Lo zero in DefaultIfEmptynon è necessario.

So che questa è una vecchia domanda e la risposta accettata funziona, ma questa domanda ha risposto alla mia domanda se un set così vuoto avrebbe comportato un'eccezione o un default(int)risultato.

La risposta accettata, tuttavia, mentre funziona, non è la soluzione ideale IMHO, che non viene fornita qui. Quindi lo sto fornendo nella mia risposta a beneficio di chiunque lo stia cercando.

Il codice originale del PO era:

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

Ecco come lo scriverei per evitare eccezioni e fornire un risultato predefinito:

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

Ciò causa il tipo restituito della Maxfunzione int?, che consente il nullrisultato e quindi ??sostituisce il nullrisultato con 0.

MODIFICA
Solo per chiarire qualcosa dai commenti, Entity Framework al momento non supporta la asparola chiave, quindi il modo di scriverla quando si lavora con EF sarebbe:

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

Dato che [TypeOfWorkers]potrebbe essere un nome di classe lungo ed è noioso da scrivere, ho aggiunto un metodo di estensione per dare una mano.

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

Questo solo maniglie int, ma lo stesso potrebbe essere fatto per long, doubleo qualsiasi altro tipo di valore è necessario. L'uso di questo metodo di estensione è molto semplice, basta passare la funzione di selezione e facoltativamente includere un valore da utilizzare per null, che per impostazione predefinita è 0. Quindi quanto sopra può essere riscritto in questo modo:

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

Spero che questo aiuti le persone ancora di più.

Max () non restituirà nulla in quel caso.

Si alzerà InvalidOperationException dal momento che la fonte non contiene elementi.

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

Se questo è Linq to SQL, non mi piace usarlo Any()perché provoca più query al server SQL.

Se ShoeSizenon è un campo nullable, quindi usare solo il .Max(..) ?? 0non funzionerà, ma quanto segue:

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

Non modifica assolutamente l'SQL emesso, ma restituisce 0 se la sequenza è vuota perché modifica il Max()per restituire un int?invece di un int.

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

(supponendo che ShoeSizesia di tipo int)

Se Workersè un DbSeto ObjectSetda Entity Framework la tua query iniziale genererebbe un InvalidOperationException, ma non si lamenta di una sequenza vuota ma si lamenta che il valore materializzato NULL non può essere convertito in un int.

Max genererà System.InvalidOperationException "La sequenza non contiene elementi"

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 query con DefaultIfEmpty()potrebbe essere significativamente più lenta . Nel mio caso è stata una semplice query con .DefaultIfEmpty(DateTime.Now.Date).

Ero troppo pigro per profilarlo, ma ovviamente EF ha cercato di ottenere tutte le righe e quindi prendere il Max()valore.

Conclusione: a volte la gestione InvalidOperationExceptionpotrebbe essere la scelta migliore.

Puoi usare un ternario all'interno .Max()per gestire il predicato e impostarne il valore;

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

Dovresti gestire la Workersraccolta come nulla / vuota se questa è una possibilità, ma dipenderà dalla tua implementazione.

Puoi provare questo:

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

Puoi verificare se ci sono lavoratori prima di eseguire il 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);
}