Давно не кодил и такое увидел / ADO.NET, LINQ, Entity Framework, NHibernate, DAL, ORM

Мобильная версия Контакт Правила FAQ Помощь

Гость

Войти | Регистрация | Профиль | Очистить

Форумы | Пользователи | Статистика | Мод. лог | Поиск

Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр

Форумы / ADO.NET, LINQ, Entity Framework, NHibernate, DAL, ORM [игнор отключен] [закрыт для гостей] / Давно не кодил и такое увидел

12 сообщений из 12, страница 1 из 1

Давно не кодил и такое увидел

#40037686

lans

Участник

Сообщения: 1 046

Рейтинг: 0 / 0

Всем доброго дня. Давно не работал разработчиком точнее года так 4. Подвернулся не большой проект WIn Forms + C# + ADO.NET и увидел я такую интересную штуку что Ienumerable<> пропертю заполняют прям из самой проперти. Это нормально по всем канонам современной разработки? Извиняюсь заранее если вопрос тупой

Код: c#

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.

        private IEnumerable<Doc> _Docs;
        public IEnumerable<Doc> Docs
        {
            get
            {
                if (_Docs == null)
                {
                    _Docs = DBManager.MGDocs.GetDocsByID(this.ID);

                }
 

                return _Docs;
            }
        
        }

...

Рейтинг:

0 / 0

21.01.2021, 14:17

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037687

Shocker.Pro

Участник

Откуда: ->|<- :адуктО

Сообщения: 23 134

Рейтинг: 0 / 0

lans

Код: c#

DBManager.MGDocs.GetDocsByID(this.ID);

Это ненормально.
Свойство не должно работать долго и обращаться к инфраструктуре.

Выносить в метод, к тому же асинхронный.

...

Рейтинг:

0 / 0

21.01.2021, 14:19

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037688

Shocker.Pro

Участник

Откуда: ->|<- :адуктО

Сообщения: 23 134

Рейтинг: 0 / 0

lans

пропертю заполняют прям из самой проперти

сам по себе шаблон в принципе нормальный, свойство кешируется при первом обращении.
В современном шарпе записывается короче:

Код: c#

get => _prop ??= CountProp()

...

Рейтинг:

0 / 0

21.01.2021, 14:22

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037689

Shocker.Pro

Участник

Откуда: ->|<- :адуктО

Сообщения: 23 134

Рейтинг: 0 / 0

Shocker.Pro

lans

Код: c#

DBManager.MGDocs.GetDocsByID(this.ID);

Хотя тут я поторопился, возможно, если GetDocsByID возвращает Expression в чистом виде, то обращения не будет. Но тогда не очень понятен смысл кеширования, зачем тут вообще _Doc. В общем, так себе код....

...

Рейтинг:

0 / 0

21.01.2021, 14:29

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037692

IPmen

Гость

Shocker.Pro,

У нас тоже так на некоторых проектах. Я предложил такие проперти сразу заполнять когда и объект заполняется сам но не всегда пропертя эта может использоваться и нет смысла тянуть заранее данные. А как все же верно?

...

Рейтинг:

0 / 0

21.01.2021, 14:47

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037704

Shocker.Pro

Участник

Откуда: ->|<- :адуктО

Сообщения: 23 134

Рейтинг: 0 / 0

IPmen

Делай метод с кешированием или Lazy, в чем проблема? Нафига проперти, которая смотрит во внешний сервис.

...

Рейтинг:

0 / 0

21.01.2021, 15:27

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037708

Shocker.Pro

Участник

Откуда: ->|<- :адуктО

Сообщения: 23 134

Рейтинг: 0 / 0

Потребитель класса предполагает, что при обращении к проперти он получает какой-то фрагмент состояния этого класса. А не запускает, длительные блокирующие операции, которые могут и поток подвесить и исключение вернуть.
И отладчик VS предполагает то же самое. И потом при отладке этого п...ца ты хватаешься за голову, когда у тебя студия хочет показать значения свойств и запускает сотню обращений к недоступной в контексте отладки базы.

(смайлик *wall*)

ЗЫ: К конструкторам это тоже относится. А то есть любители в конструктор обращение в БД запихать!

...

Рейтинг:

0 / 0

21.01.2021, 15:39

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037744

love_bach

Участник

Сообщения: 687

Рейтинг: 0 / 0

Это не нормально. Но, если это так везде в проекте, - страдать тебе придется

PS. Скажи что они все делают неправильно, что надо все переписать с нуля, перепиши 15%, убей окончательно проект и увольнялся :)

...

Рейтинг:

0 / 0

21.01.2021, 18:46

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037747

17-77

Участник

Сообщения: 362

Рейтинг: 0 / 0

не нормально, хотя бы потому, что непонятно с одного взгляда, как будет работать
* то ли это аналог рид онли коллекции
* то ли предподготовленный запрос
* то ли внутри GetDocsByID есть ToList/ToArray (тогда вообще жопа)

а вторая причина - куча сайд эффектов и непоняток:
* что если для каждого экземпляра корневого объекта из 1000 будет вызван Docs
* что если внутри GetDocsByID миллион записей
* что если DBManager или MGDocs null

IPmen

У нас тоже так на некоторых проектах. Я предложил такие проперти сразу заполнять когда и объект заполняется сам но не всегда пропертя эта может использоваться и нет смысла тянуть заранее данные. А как все же верно?

* сначала отдельно идешь в базу, достаешь все что надо
* а потом из этого строишь и заполняешь объект
* а вот потом можешь обращаться к нему и к его пропертям
все это три разных места

обозначенным способом можно делать вычисляемые свойства, но которые используют уже заполненные значения из других свойств этого же объекта, ну например FullName, который компонуется из FirstName и LastName

...

Рейтинг:

0 / 0

21.01.2021, 19:08

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037769

skyANA

Участник

Откуда: Зеленоград

Сообщения: 22 795

Рейтинг: 0 / 0

lans

Что тут современного-то? Простая реализация шаблона ActiveRecord.

...

Рейтинг:

0 / 0

21.01.2021, 21:37

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037770

skyANA

Участник

Откуда: Зеленоград

Сообщения: 22 795

Рейтинг: 0 / 0

Ну или возьмём какой-нибудь NHibernate c lazy property в маппинге в виде коллекции...

Развели тут консилиум

...

Рейтинг:

0 / 0

21.01.2021, 21:41

| Ответить | Цитировать | Написать

Давно не кодил и такое увидел

#40037902

Roman Mejtes

Участник

Откуда: г. Пермь

Сообщения: 4 264

Рейтинг: 0 / 0

а что, если я получаю значение поля, но последовательность перебирать не буду, а потом получу "кэшированную версию" перечисления и начну её итерировать. когда произойдет материализация объектов? можно легко выстрелить себе в ногу, как мне кажется.
в целом вызов методов в свойстве нормальное явление, если это свойство отражает внутренние состояние объекта, вычисляет что то небольшое или выдает значение. От внешних ссылок или каких то сервисов это не должно зависеть.

...

Рейтинг:

0 / 0

22.01.2021, 12:07

| Ответить | Цитировать | Написать

12 сообщений из 12, страница 1 из 1

Форумы / ADO.NET, LINQ, Entity Framework, NHibernate, DAL, ORM [игнор отключен] [закрыт для гостей] / Давно не кодил и такое увидел

Цитировать

Написать

Автор*:

Ввести пароль для входа

Тема*:

Сообщение

Данное сообщение тематическое

Сообщение содержит картинки или видео 18+

Автор:

ВНИМАНИЕ! На данном подфоруме действуют строгие правила. Удостоверьтесь, что ваше сообщение соответствует им!

Форум или тема закрыты для гостей. Необходима авторизация!

Загрузить последнюю сохраненную версию

Вложение:

Вставить как галерею

Максимальный размер вложений: 4,0 МБ, аудио/видео: 8,0 МБ. Картинки большего размера ужимаются, если возможно.

Введите код, изображенный на картинке. Если код нечитаемый, кликните картинку, чтобы загрузить другой вариант.

Отправляя сообщение, я выражаю свое согласие с правилами форума и принимаю пользовательское соглашение.

Читали тему (0):

Читали форум (0):

Пользователи онлайн (0):

start [/forum/topic.php?fid=17&tid=1349040]:	0ms
get settings:	12ms
get forum list:	15ms
check forum access:	5ms
check topic access:	5ms
track hit:	400ms
get topic data:	13ms
get forum data:	3ms
get page messages:	51ms
get tp. blocked users:	2ms
others:	234ms

total:	740ms