.NET Core Web Api EF Как сделать группировку значений в запросе?

Question

[Андрей]

Добрый всем день!

Подскажите пожалуйста, я не так давно начал изучать asp.net core + EF подтянулся... не всё ещё понимаю, я пытаюсь сделать выборку из БД с группировкой результата. Мне необходимо выбрать все строки с определённым кодом и сгруппировать их по названию товара (Nomenklatura).

Модель такая:

public class Orders
    {
        public int Id { get; set; }
        public int Kod { get; set; }
        public string Nomenklatura { get; set; }
        public int Kolich { get; set; }
}

Вот метод, который ругается на меня, что я List привожу к IEnumerable

[HttpGet]
        public IEnumerable<Orders> Get([FromQuery(Name = "kod")]int kod)
        {
            var orders = db.Orders
                .Where(x => x.Kod == kod)
                .GroupBy(x => x.Nomenklatura).ToList();
           
            return orders;
        }

Вроде Where возвращает IEnumerable...

Буду рад любому совету!
Заранее спасибо!

Comments

[Владислав Килин]

Where в данной ситуации (работа с EF) возвращает `IQueryable`, но затем вы вызываете метод GroupBy, который возвращает `IQueryable>`, который уже к `IEnumerable` не приведется.
Если вы хотите возвращать группированные значения, то тогда у вас должен быть корректный контракт вашего API, например `IDictionary>`, тогда ваш код может выглядеть так:

[HttpGet]
public IDictionary<string, IEnumerable<Order>> Get(int kod)
{
    return db.Orders.Where(x => x.Kod == kod).GroupBy(x => x.Nomenklatura).ToDictionary(x => x.Key, x => x.ToList());
}

Или если контракт метода корректный и группировка вам не нужна, то тогда достаточно `return db.Orders.Where(x => x.Kod == kod).ToList();`

Также обратите внимание на два пункта:
1. Обращение к базе данных прямо из контроллера считается дурным тоном. Вы огребете массу проблем, дублирования кода и прочего в будущем. Попробуйте воспользоваться паттерном Repository.
2. Раз это dotnet core, то вы можете очень удобно пользоваться асинхронными операциями, а то кому нужны дедлоки? Почитайте про async контроллеры.

[Андрей]

Спасибо, что откликнулись, Владислав!

Без группировки - всё работает без проблем. Но она нужна :(

Я попробовал Ваш пример, и студия снова ругается. Опять на приведение типов.

Error	CS0266	Cannot implicitly convert type 'System.Collections.Generic.Dictionary<string, System.Collections.Generic.List<WebApi_MKAR_Test_01.Models.Orders>>' to 'System.Collections.Generic.IDictionary<string, System.Collections.Generic.IEnumerable<WebApi_MKAR_Test_01.Models.Orders>>'. An explicit conversion exists (are you missing a cast?)

:(

За советы - отдельное спасибо. Я в самом начале пути, и хочется писать грамотно и правильно, да бы не переучиваться. Почитаю про паттерн и про асинхронные контроллеры.

Answer 1

[Андрей]

Вот код, который мне подсказали, который даёт необходимый результат:
GET localhost:65171/api/orders/966

[
    "Перчатки",
    "Варежки"
]

public IEnumerable<string> GetNomenklatura(int kod)
        {
            var orders = db.Orders
                .Where(x => x.Kod == kod)
                .GroupBy(x => x.Nomenklatura)
                .Select(x => x.Key)
                .ToList();
            return orders;
        }

Большое спасибо, всем кто помогал мне в поиске решения!

Answer 2

[eRKa]

GroupBy по сути Вам вернет массив массивов, к List привести Вы его не сможете, можете привести к Dictionary, например

var orders = db.Orders
                .Where(x => x.Kod == kod)
                .GroupBy(x => x.Nomenklatura)
                .ToDictionary(x => x.FirstOrDefault().Nomenklatura, x => x.ToList() )

тогда получить сгруппированное можно по ключу
orders["anyNomenklatura"]
и там будет список

Comments

[Андрей]

Спасибо, что откликнулись!
Всё не просто...
Попробовал ваш совет, ошибка теперь другая :(

Error	CS1061	'IGrouping<string, Orders>' does not contain a definition for 'Nomenklatura' and no accessible extension method 'Nomenklatura' accepting a first argument of type 'IGrouping<string, Orders>' could be found (are you missing a using directive or an assembly reference?)

Вроде EF придумали для облегчения труда разработчика... А получилось не однозначно. :(

[eRKa]

Андрей, покажите Ваши модели Orders и Nomenklatura

[Андрей]

eRKa, А модели Nomenklatura у меня нет.

Только Orders

public class Orders
    {
        public int Id { get; set; }
        public int Kod { get; set; }
        public string Nomenklatura { get; set; }
        public int Kolich { get; set; }
}

[eRKa]

Андрей, писал сразу сюда и я ошибся. Дело в том, что внутри
ToDictionary(x => x.Nomenklatura, x => x.ToList())
x - это список (List) Ваших orders и у этого списка нет поля Nomenklatura, поэтому он ругается. Попробуйте так

ToDictionary(x => x.FirstOrDefault().Nomenklatura, x => x.ToList())

[Vladimir S]

так надо

.GroupBy(x => x.Nomenklatura)
                .ToDictionary(x => x.Key, x => x.ToList() )

[Андрей]

Vladimir S, Спасибо, за отклик!

Попробовал, ругается, что не может к нужному типу привести.

Решение, которое я пытался найти - мне подсказали. Опубликую его отдельно.