Макс возвращается значение, если пустой запрос

Question

У меня есть этот запрос:

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

Что будет в maxShoeSize, если компания 8 не имеет рабочих вообще?

UPDATE:
Как я могу изменить запрос, чтобы получить 0, а не исключение?

Answer 1

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

Нуль в DefaultIfEmpty не является необходимым.

Answer 2

Max() ничего не вернет в этом случае.

Он поднимет InvalidOperationException, так как источник не содержит элементов.

Answer 3

Макс выбросит System.InvalidOperationException "Последовательность не содержит элементов"

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

Answer 4

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

(при условии, что ShoeSize имеет тип int)

If Workers является DbSet или ObjectSet от Entity Framework ваш первоначальный запрос будет генерировать InvalidOperationException, но не жалуется на пустую последовательность, но жалуется, что материализованное значение NULL не может быть преобразовано int o a int.

Answer 5

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

Answer 6

NB: запрос с DefaultIfEmpty() может быть значительно более медленным. В моем случае это был простой запрос с .DefaultIfEmpty(DateTime.Now.Date).

Я слишком ленив, чтобы профиль. Но, очевидно, EF попытался получить все строки, а затем взять значение Max().

Заключение: иногда обработка InvalidOperationException может быть лучшим выбором.

Answer 7

Я знаю, что это старый вопрос, и принятый ответ работает, но этот вопрос ответил на мой вопрос о том, приведет ли такой пустой набор к исключению или к результату default(int).

Однако принятый ответ, хотя он и работает, не является идеальным решением IMHO, которое здесь не приводится. Таким образом, я предоставляю его в своем собственном ответе на благо любого, кто его ищет. Исходный код

Ор был:

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

Это, как я хотел бы написать его, чтобы предотвратить исключения и предоставить результат по умолчанию:

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

Это приводит к тому, тип возвращаемого значения функции Max в int?, что позволяет получить результат null, а затем ?? заменяет результат null на 0.

Answer 8

Вы можете использовать тернар в пределах .Max() для обработки предиката и установить его значение;

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

Вам необходимо будет обрабатывать Workers коллекция является пустой/пустой, если это возможно, но это будет зависеть от реализации.

Answer 9

Вы можете попробовать это:

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

Answer 10

Вы можете проверить, есть ли какие-либо рабочие, прежде чем делать Макс().

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

Answer 11

Если это Linq к SQL, я не хотел бы использовать любой(), так как это приводит к несколько запросов к серверу SQL.

Если ShoeSize не обнуляемым поле, а затем, используя только .Max(..) ?? 0 не будет работать, но следующий будет:

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

Это абсолютно не меняет излучаемого SQL, но она возвращает 0, если последовательность пуст, потому что он меняет Max(), чтобы вернуть int? вместо int.