0

Есть функция, которая отдает объкты, которых нет в БД по пришедшему списку. Как видно, с помощью Read я сначала считываю все данные, а потом отбираю. Как я могу избавиться от этого Read() чтобы ускорить сам запрос???

public List<PermDTO> GetPermsNotIn(List<PermDTO> perms)
{
    var res = this.db.PermRepository.Read().ToList();
    var res2 =  res.GroupJoin(perms, r => new { r.kgroup_name, r.perm_name }, p => new { p.kgroup_name, p.perm_name }, (r, p) => new
    {
        id = r.id,
        description = r.description,
        isactive = r.isactive,
        kgroup_name = r.kgroup_name,
        perm_name = r.perm_name,
        count = p.Count()
    });
    var res3 = res2.Where(w => w.count == 0);
    var res4 = res3.Select(s => new PermDTO()
    {
        id = s.id,
        description = s.description,
        isactive = s.isactive,
        kgroup_name = s.kgroup_name,
        perm_name = s.perm_name,
    });
    var res5 = res4.ToList();

    return res5;
}


    public virtual IQueryable<T> Read()
    {            
        return context.Set<T>().AsNoTracking();
    }
5
  • Достаточно убрать ToList() после Read(). 9 авг 2019 в 9:43
  • @AlexanderPetrov не достаточно. Если убрать, то при group by будет ошибка: Sql = "((System.Data.Entity.Infrastructure.DbQuery<Loyalty.DAL.Models.Backoffice.Perm>)m1).Sql" выдал исключение типа "System.NotSupportedException" 9 авг 2019 в 9:52
  • ToList реально не нужен и абсолютно лишняя операция, которая лишний раз материализует данные. (Для начала можно заменить на .AsEnumerable) Не совсем понятно, почему нужно сливать всю таблицу на клиента, чтобы сравнивать есть объект/нет объекта, почему бы не переложить эту логику на базу данных? Нельзя ли просто базе скормить набор IDшников и использовать Not In? (not in - не самая быстрая операция на сервере, но явно лучше группировок на клиенте) Альтернатива какая-то извращенная придумывается: перегнать perms обратно из PermDTO в Perm чтобы серверная сторона понимала, что это её класс.
    – A K
    9 авг 2019 в 9:53
  • @AK я бы с радостью кормил список id-шников, как я это сделал с другими 4мя таблицами, но в этой таблицы ключ по двум стобцам, и я не могу применить инструкцию Contains 9 авг 2019 в 10:06
  • Виталий, а как вам направление которое рассматривается в такой статье - построить expression и передать на сервер? Это первое направление, по которому возможно двинуться (я правда не очень понял что из чего вычитаем. Правильно ли: вернуть perms минус то, что есть в базе?). Второе направление: а зачем вам нужны эти элементы? Чтобы отобразить пользователю или чтобы вставить в базу? Какая более общая задача, которую решаем?
    – A K
    9 авг 2019 в 16:51

1 ответ 1

0
var res = this.db.PermRepository.Read().Where(w => w.count == 0).ToArray();

тут ты не вытаскиваешь все данные, а только те которые по условию совпадают.

4
  • 3
    добро пожаловать на stack overflow на русском! пожалуйста, постарайтесь оставлять чуть более развёрнутые ответы. дополнить ответ можно, нажав править 9 авг 2019 в 12:00
  • как это применимо к моему вопросу? 9 авг 2019 в 12:42
  • ну вы хотели ускорить процесс, вот метод ускорения
    – anuar
    9 авг 2019 в 12:44
  • как его применить к моей задаче? 9 авг 2019 в 20:07

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.