powered by simpleCommunicator - 2.0.50     © 2025 Programmizd 02
Форумы / WinForms, .Net Framework [игнор отключен] [закрыт для гостей] / generic+Include
13 сообщений из 13, страница 1 из 1
generic+Include
    #39834189
vb_sub
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Всем привет, при реализации паттерна UnitOfWork+Repository у меня возникла проблема.

Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
58.
59.
60.
61.
62.
63.
64.
65.
66.
67.
68.
69.
70.
71.
72.
73.
74.
75.
76.
77.
78.
79.
80.
81.
82.
83.
84.
85.
86.
87.
88.
89.
90.
91.
92.
93.
94.
95.
96.
97.
98.
99.
100.
101.
102.
103.
104.
105.
106.
107.
108.
109.
110.
111.
112.
113.
114.
115.
116.
117.
118.
119.
120.
121.
122.
123.
124.
125.
126.
127.
128.
129.
130.
131.
132.
133.
134.
135.
136.
137.
138.
139.
140.
141.
142.
143.
144.
145.
146.
147.
148.
149.
150.
151.
152.
153.
154.
155.
156.
157.
158.
159.
160.
161.
162.
163.
164.
165.
166.
167.
168.
169.
170.
171.
172.
173.
174.
175.
176.
177.
178.
179.
180.
181.
182.
183.
184.
185.
186.
187.
188.
189.
190.
191.
192.
193.
194.
195.
196.
197.
198.
199.
200.
201.
202.
203.
204.
205.
206.
207.
208.
209.
	
  public class ApplicationContext : IdentityDbContext<ApplicationUser>
    {
        /////////////////
        ///Initialization
        /////////////////

        /// <summary>
        /// Запасные части
        /// </summary>
        public DbSet<SparePart> SpareParts { get; set; }
    }


    /// <summary>
    /// Контракт репозитория
    /// </summary>
    public interface IRepository<T>
            where T : class
    {
        /// <summary>
        /// Подгружаем связанные объекты сущностей
        /// </summary>
        /// <param name="includeProperties">Список из подгружаемых навигационных свойств,
        /// <returns></returns>
        IEnumerable<T> GetWithDeptInclude(IList<Expression<Func<T, object>>> includeProperties);
    }


    public class Repository<T>
            : IRepository<T>
            where T : class
    {
        #region readOnly

        private readonly ApplicationContext applicationContext;

        #endregion


        public Repository(ApplicationContext applicationContext)
        {
            this.applicationContext = applicationContext;
        }


        private IQueryable<T> Include(IList<Expression<Func<T, object>>> includeProperties)
        {
            IQueryable<T> query = applicationContext.Set<T>().AsNoTracking();

            foreach (var include in includeProperties)
            {
                query = query.Include(include);
            }
            return query;
        }


        /// <summary>
        /// Вытягивает из ApplicationDbContext сущность и подтягиваем навигационное свойство
        /// </summary>
        public IEnumerable<T> GetWithDeptInclude(IList<Expression<Func<T, object>>> includeProperties)
        {
            return Include(includeProperties).ToList();
        }
    }


    ///Контроллер
    public class SparePartController : ControllerBase
    {
        #region SpareParts

        private readonly IRepository<SparePart> repository;

        private readonly UnitOfWork unitOfWork;
        #endregion

        /// <summary>
        /// Получить запчасти
        /// </summary>
        /// <returns>Принтеры</returns>
        [HttpGet]
        public async Task<IActionResult> GetSpareParts([FromServices] ApplicationContext applicationContext)
        {

            //Сейчас могу вытащить только топовый уровень для Include
            var PropertyList = new List<Expression<Func<SparePart, object>>>();

            PropertyList.Add(x => x.PrinterSpareParts);

            var SparPart = repository
                .GetWithDeptInclude(PropertyList);


            //хотелось бы что бы можно было посылать такой запрос, но в рамках паттерна UOW, без внедрения 
            //ApplicationContext в контроллер, причем чтобы можно было запрашивать произвольное количество уровне вложенностей 
            //для ThenInclude

            var dg = applicationContext
                .SpareParts
                .Include(f => f.PrinterSpareParts)
                .ThenInclude(f => f.Printer);

            return new JsonResult(SparPart);

        }


        public SparePartController(UnitOfWork unitOfWork)
        {
            repository = unitOfWork.Repository<SparePart>();
            this.unitOfWork = unitOfWork;
        }

    }

    ///модели


    /// <summary>
    /// Принтер
    /// </summary>
    public class Printer
    {
        #region Props

        /// <summary>
        /// Идентификатор принтера
        /// </summary>
        public int ID { get; set; }

        /// <summary>
        /// Название принтера
        /// </summary>
        public string Title { get; set; }


        /// <summary>
        /// Идентификатор производителя
        /// </summary>
        public int PrinterProducerId { get; set; }
        /// <summary>
        /// Производитель принтера
        /// </summary>
        public virtual PrinterProducer PrinterProducer { get; set; }


        public IList<PrinterSparePart> PrinterSpareParts { get; set; }
        #endregion


        public Printer()
        {
        }
    }


    /// <summary>
    /// Производитель принтеров
    /// </summary>
    public class PrinterProducer
    {

        #region props

        /// <summary>
        /// Идентификатор производителя
        /// </summary>
        public int Id { get; set; }


        /// <summary>
        /// Название производителя
        /// </summary>
        public string Title { get; set; }



        /// <summary>
        /// Принтеры
        /// </summary>
        public virtual ICollection<Printer> Printers { get; set; }

        #endregion

    }

    /// <summary>
    /// Вспомогательный класс для осуществления связи многие-ко-многим между  Printer и SparePart
    /// </summary>
    public class PrinterSparePart
    {
        public int PrinterId { get; set; }
        public Printer Printer { get; set; }
        public int SparePartId { get; set; }
        public SparePart SparePart { get; set; }
    }


    public class ApplicationContext : IdentityDbContext<ApplicationUser>
    {
        ///Initialization ApplicationContext 

        /// <summary>
        /// Запасные части
        /// </summary>
        public DbSet<SparePart> SpareParts { get; set; }
    }   



Собственно вопрос, можно ли расширить функционал GetWithDeptInclude, так, чтобы можно было задавать произвольную комбинацию Incllude c произвольным уровнем вложенности (ThenInclude)?
Что-то типа

Код: c#
1.
PropertyList.Add(x => x.TopProperty).ThenInclude(y=>y.NestedProperty).ThenInclude(z=>z.NestedNestedProperty)....; 



Можете посоветовать в какую сторону смотреть чтобы реализовать такую функциональность только на дженериках, без рефлексии? Спасибо
...
Рейтинг: 0 / 0
generic+Include
    #39834302
Roman Mejtes
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
...
Рейтинг: 0 / 0
generic+Include
    #39834342
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
...
Рейтинг: 0 / 0
generic+Include
    #39834345
vb_sub
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthat,
что лучше инжектить контексты в контроллеры?
...
Рейтинг: 0 / 0
generic+Include
    #39834374
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
vb_subfkthat,
что лучше инжектить контексты в контроллеры?
Нет. Репо - нормальный паттерн. Антипаттерн - это обобщенный репо на все случаи жизни.
...
Рейтинг: 0 / 0
generic+Include
    #39834381
Фотография ЕвгенийВ
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatvb_subfkthat,
что лучше инжектить контексты в контроллеры?
Нет. Репо - нормальный паттерн. Антипаттерн - это обобщенный репо на все случаи жизни.
Так нормальный или все же нет?
...
Рейтинг: 0 / 0
generic+Include
    #39834382
vb_sub
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
ЕвгенийВ,
холивар же.
...
Рейтинг: 0 / 0
generic+Include
    #39834383
vb_sub
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthat,
Вы в контроллерах держите namespace от ORM, или же через пробрасываете абстракцию?
...
Рейтинг: 0 / 0
generic+Include
    #39834515
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
vb_subМожете посоветовать в какую сторону смотреть чтобы реализовать такую функциональность только на дженериках, без рефлексии? Спасибо

Советую возвращать IQueryable и делать Include по месту требования.
Выбросьте все статьи на помойку, где вам рекомендуют возвращать IEnumerable -- их писали абсолютные бездарности и неучи, без шуток.
...
Рейтинг: 0 / 0
generic+Include
    #39834518
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
По-хорошему, конечно, Include зло и порождает очень плохо поддерживаемый код, в котором вы только и будете ловить ошибки забытого Include, в конце плюнете и будете инклюдить всё и всегда, даже там, где это не нужно. А потом откроете для себя Lazy, но столкнётесь с проблемой SQL+1. Затем решите как-то грамотно комбинировать Include + Lazy. И поимеете две проблемы сразу.

Потом разочаруетесь в EF, жизни и вообще, решите, что лучше пойти работать плотником.

Естественно, решение, это работать с проекциями, а сущности EF это просто декларативное описание структуры данных. Нормально работать с DDD можно только в CQRS, рано или поздно понимание придёт :)
...
Рейтинг: 0 / 0
generic+Include
    #39834525
ViPRos
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
hVostt,

а еще дальше плюнешь на ддд, ббб, ооо,..., и другую фигню и будешь решать поставленную задачу оптимальным образом - простую просто, сложную сложно :)
...
Рейтинг: 0 / 0
generic+Include
    #39834529
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
ViPRoshVostt,

а еще дальше плюнешь на ддд, ббб, ооо,..., и другую фигню и будешь решать поставленную задачу оптимальным образом - простую просто, сложную сложно :)

Это уже последняя миля )
...
Рейтинг: 0 / 0
generic+Include
    #39834883
vb_sub
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
hVosttПо-хорошему, конечно, Include зло и порождает очень плохо поддерживаемый код, в котором вы только и будете ловить ошибки забытого Include, в конце плюнете и будете инклюдить всё и всегда, даже там, где это не нужно. А потом откроете для себя Lazy, но столкнётесь с проблемой SQL+1. Затем решите как-то грамотно комбинировать Include + Lazy. И поимеете две проблемы сразу.

Потом разочаруетесь в EF, жизни и вообще, решите, что лучше пойти работать плотником.

Естественно, решение, это работать с проекциями, а сущности EF это просто декларативное описание структуры данных. Нормально работать с DDD можно только в CQRS, рано или поздно понимание придёт :)

Че-то грустно это как-то, я надеялся на свет в конце тоннеля.
...
Рейтинг: 0 / 0
13 сообщений из 13, страница 1 из 1
Форумы / WinForms, .Net Framework [игнор отключен] [закрыт для гостей] / generic+Include
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Найденые пользователи ...
Разблокировать пользователей ...
Читали форум (0):
Пользователи онлайн (0):
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]