[Introduction to EF] - Problem {5} - Departments with more than 5 employees - Въпрос относно авторското решение.

msmilkoff 338 Точки

[Introduction to EF] - Problem {5} - Departments with more than 5 employees - Въпрос относно авторското решение.

В авторското решение на задачата забелязах две неща, които (според това което аз съм чел) са супер грешни.

Заявката се присвоява на променлива от тип IEnumerable.
По правило не трябва ли винаги да се работи с IQueryable? Така заявката, която се изпраща към базата е доста по оптимизирана, защото се възползваме от Linq-to-Entities. И по-важно, Extension методите изглеждат същите, но не са. Методите върху IEnumerable, очакват като параметър Func<нещо си>, докато при IQueryable очакват Expression<Func<нещо си>>. Няма нужда да се обяснява, че разликата е огромна.

Вътрешният Foreach итерира по department.Employees.
Няма ли опасност да се стигне до т. нар. "Select N + 1" проблем? Накратко - ще се генерира една заявка за департамента и след това по една за всеки служител в него, което не е готино от гледна точка на performance.
Не е ли по добър вариантът:

var departments = db.Departments
                .Where(d => d.Employees.Count > 5)
                .OrderBy(d => d.Employees.Count)
                .Select(d => new
                {
                    Name = d.Name,
                    Manager = d.Employee.FirstName,
                    DepEmployees = d.Employees
                });

            foreach (var department in departments)
            {
                Console.WriteLine($"{department.Name} {department.Manager}");

                var employees = department.DepEmployees;
                foreach (var employee in employees)
                 // ......

Ако някой с по-дълбоки познания по EF разясни дали наистина е така или пък аз се бъркам - ще е супер.

Тагове:

31/10/2016 15:07:42 Databases Advanced - Entity Framework 31/10/2016 15:08:16

RFilipov 136 Точки

Не се бъркаш.

Единият вариант е проекция със Select.

Другият е така нареченият eager-loading със Include:

var departments = db.Departments
.				.Include("Employees")
                .Where(d => d.Employees.Count > 5)
                .OrderBy(d => d.Employees.Count)
                .Select(d => new
                {
                    Name = d.Name,
                    Manager = d.Employee.FirstName,
                    DepEmployees = d.Employees
                });

Авторското решение е N + 1 това можеш да го видиш във SQL Profiler.

31/10/2016 15:55:35

val4o89 240 Точки

Като гледам авторското решение и на база знанията, които се очаква да имаме до момента, а не на програмист с 10г. опит

След някой друг урок ще правим нещата, както сте ги чели.

-1

31/10/2016 16:23:19

stambi4a 126 Точки

Най- добрият начин е native. В това видео се прави измерване и се вижда,че native заявките са много по- бързи от другите. Select e за предпочитане пред include, защото в една таблица може да имаш 20, 30 и повече колони,при положение, че често ти трябват само няколко което генерира излишен трафик по мрежата и в паметта. Тази статия разглежда доста подробно performance-a в EF.

31/10/2016 17:23:13