ค่าส่งคืนสูงสุดหากแบบสอบถามว่างเปล่า

ฉันมีคำถามนี้:

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

จะเกิดอะไรขึ้นmaxShoeSizeถ้า บริษัท 8 ไม่มีพนักงานเลย?

อัปเดต:
ฉันจะเปลี่ยนคิวรีได้อย่างไรเพื่อให้ได้ 0 และไม่ใช่ข้อยกเว้น

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

DefaultIfEmptyไม่จำเป็นต้องเป็นศูนย์

ฉันรู้ว่านี่เป็นคำถามเก่าและคำตอบที่ได้รับการยอมรับ แต่คำถามนี้ตอบคำถามของฉันเกี่ยวกับว่าชุดว่างดังกล่าวจะส่งผลให้เกิดข้อยกเว้นหรือdefault(int)ผล

อย่างไรก็ตามคำตอบที่ได้รับการยอมรับในขณะที่ใช้งานได้นั้นไม่ใช่วิธีที่สมบูรณ์แบบสำหรับ IMHO ซึ่งไม่ได้ให้ที่นี่ ดังนั้นฉันจะให้มันในคำตอบของฉันเองเพื่อประโยชน์ของทุกคนที่กำลังมองหามัน

รหัสดั้งเดิมของ OP คือ:

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ผลแล้ว??แทนที่ผลกับ null0

แก้ไข
เพื่อชี้แจงบางสิ่งจากความคิดเห็น Entity Framework ไม่สนับสนุนasคำหลักในปัจจุบันดังนั้นวิธีเขียนเมื่อทำงานกับ EF จะเป็น:

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

เนื่องจาก[TypeOfWorkers]อาจเป็นชื่อชั้นยาวและน่าเบื่อในการเขียนฉันจึงได้เพิ่มวิธีการขยายเพื่อช่วยออก

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

นี้จะจับintแต่เดียวกันสามารถทำได้สำหรับlong, doubleหรือประเภทค่าอื่น ๆ ที่คุณจำเป็นต้อง การใช้วิธีการขยายนี้เป็นเรื่องง่ายมากคุณเพียงแค่ส่งผ่านฟังก์ชั่นตัวเลือกของคุณและเลือกที่จะรวมค่าที่จะใช้เป็นค่าว่างซึ่งเริ่มต้นที่ 0 ดังนั้นด้านบนสามารถเขียนใหม่ได้เช่น:

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

หวังว่าจะช่วยให้ผู้คนออกไปมากยิ่งขึ้น

Max ()จะไม่ส่งคืนสิ่งใดในกรณีนั้น

มันจะยกInvalidOperationExceptionเนื่องจากแหล่งที่มาไม่มีองค์ประกอบ

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

ถ้านี่คือ Linq ไปยัง SQL ฉันไม่ชอบที่จะใช้Any()เพราะมันส่งผลให้หลายแบบสอบถามไปยังเซิร์ฟเวอร์ SQL

ถ้าShoeSizeไม่ใช่เขตข้อมูล null ให้ใช้เพียง.Max(..) ?? 0จะไม่ทำงาน แต่จะต่อไปนี้:

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

มันอย่างไม่เปลี่ยนแปลง SQL ที่ปล่อยออกมา แต่มันก็ไม่กลับมา 0 ถ้าลำดับที่ว่างเปล่าเพราะมันเปลี่ยนแปลงMax()ไปกลับแทนint?int

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

(สมมติว่าShoeSizeเป็นประเภทint)

หากWorkersเป็นDbSetหรือObjectSetจาก Entity Framework แบบสอบถามเริ่มต้นของคุณจะโยนInvalidOperationExceptionแต่ไม่บ่นเกี่ยวกับลำดับที่ว่างเปล่า แต่บ่นว่า materialized ค่า NULL intไม่สามารถแปลงเป็น

Max จะโยน System.InvalidOperationException "Sequence ไม่มีองค์ประกอบ"

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

หมายเหตุ: แบบสอบถามที่มีDefaultIfEmpty()อาจจะมีนัยสำคัญช้าลง .DefaultIfEmpty(DateTime.Now.Date)ในกรณีของฉันที่เป็นแบบสอบถามที่เรียบง่ายด้วย

ฉันขี้เกียจเกินกว่าจะโพรไฟล์ได้ แต่เห็นได้ชัดว่า EF พยายามที่จะรับทุกแถวจากนั้นรับMax()ค่า

สรุป: บางครั้งการจัดการInvalidOperationExceptionอาจเป็นตัวเลือกที่ดีกว่า

คุณสามารถใช้ ternary ภายใน.Max()เพื่อจัดการเพรดิเคตและตั้งค่า

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

คุณจะต้องจัดการกับWorkersคอลเลกชันที่เป็นโมฆะ / ว่างเปล่าถ้าเป็นไปได้ แต่มันจะขึ้นอยู่กับการใช้งานของคุณ

คุณสามารถลองสิ่งนี้:

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

คุณสามารถตรวจสอบว่ามีพนักงานคนใดก่อนที่จะทำ 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);
}