Почему запрос к БД EF не верно работает?

Question

[Roman Zhidkov]

В данном коде, создаю новый лист типа стринг, подсчитываю количество голов матча у хозяев и гостей, но по какой-то не неведанной мне причине суммы голов всегда равны...

var calendar = _context.Matchs.Where(i => i.ChampionatId == id).Select(x => new List<string> {
                 x.Name,
                _context.StatisticPlayers.Where(i => i.MatchId == x.Id && i.CommandId == x.HomeId).Sum(n=>n.Goal).ToString(),
                _context.StatisticPlayers.Where(i => i.MatchId == x.Id && i.CommandId == x.GuestId).Sum(p=>p.Goal).ToString()
});

Если этот код я не много переделаю, то всё отлично работает,

var calendar = _context.Matchs.Where(i => i.ChampionatId == id).Select(x => new List<string> {
                 x.Name,
                _context.StatisticPlayers.Where(i => i.MatchId == x.Id && i.CommandId.ToString().Contains(x.HomeId.ToString())).Sum(n=>n.Goal).ToString(),
                _context.StatisticPlayers.Where(i => i.MatchId == x.Id && i.CommandId.ToString().Contains(x.GuestId.ToString())).Sum(p=>p.Goal).ToString()
});

Это глюк EF, или это мой глюк?

Answer 1

[Дмитрий Ковальский]

Вот вам маленькая загадка : если по каким-то причинам этот код выдаст исключение - вы будете знать в каком место оно возникло?
Последовательность LINQ-команд это конечно замечательно, но на мой взгляд этот код совершенно не читаем и с ходу понять что в нем не так на мой взгляд очень и очень трудно.
Если искать проблемы, то видимо у вас они кроятся в элементе "i.CommandId == x.HomeId" и "i.CommandId == x.GuestId". Какого типа эти поля? Что в них хранится и в каком виде?

UPD: Вы можете объяснить что у вас в конечном счете должно храниться внутри List< string > ? Что вы хотите сформировать ?

Answer 2

[Roman Zhidkov]

CommandId (int) - ид команды, в таблице "Статистика игроков", от сюда я вытаскиваю забитые голы игроками, которые должны быть в составе этой команды;
HomeId и GuestId (оба int) - ид команды, в таблице "Матчи" хозяева или гости соответственно.
Надеюсь более-менее внятно описал...

Даже больше, пробуем так - всё отлично:

var summ = _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 9).Sum(n=>n.Goal).ToString();
var summ1 = _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 1).Sum(n => n.Goal).ToString();

Так, плохо:

var calendar = _context.Matchs.Where(i => i.ChampionatId == id).Select(f => new List<string> {
                _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 9).Sum(n=>n.Goal).ToString(),
                _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 1).Sum(n => n.Goal).ToString()
            });

Comments

[Дмитрий Ковальский]

1) Это пишут или в комментариях к ответу(я совершенно случайно заметил вашу запись) Или в вопросе с меткой обновления.
2) Ваш второй код банально неверен. CommandId может быть 112, а HomeId = 1,2,11,12,112 и все пройдут проверку на Contains, хотя это неправильно.

[Roman Zhidkov]

Дмитрий Ковальский:
1. Спасибо! Замечание принято!
2. Да, согласен с Contains (определяет, содержит ли коллекция определенный элемент) - я ошибся, но тем не менее, мне не понятно, почему так происходит, пробую варианты:

//так работает
var summ = _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 9).Sum(n=>n.Goal).ToString();
var summ1 = _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 1).Sum(n => n.Goal).ToString();

// так нет
var bb = _context.Matchs.Select(h => new {
sum = _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 9).Sum(n => n.Goal),
sum1 = _context.StatisticPlayers.Where(i => i.MatchId == 143 && i.CommandId == 1).Sum(n => n.Goal)
});

[Дмитрий Ковальский]

VigVam: Вы написали большую длинную LINQ структуру которая развернется черт знает в какой скрипт на базу. Хотите просто, быстро и надежно - пишите хранимку которая на вход принимает ChampionatId и возвращает набор данных MatchId,Home,Guest. Не полагайтесь на EF. Если всеже вам хочется - напишите в несколько инструкций получение всех нужных вам данных - не в одну строку, а в несколько. А затем через отладку спокойно смотрите что в эти выборки попадает и как проходят проверки.