Как стать автором
Обновить

Комментарии 42

настоятельно рекомендую ознакомиться с литературой по паттернам и антипатеернам. что из них полезнее сказать сложно, но польза от ознакомления будет однозначно. главное относиться к ним без фанатизма ;)
В контексте веб-разработки самой полезной книгой будет "Разработка корпоративных приложений", Фаулера.
Она. Просто обязательна для ознакомления. Если нет в наличии GOF, рекомендую и ее приобрести.
Кстати, в ней есть и о Lazy Loading - то, что вы изобрели :)
Ну только не изобрёл, а нашел ;) И Фаулер этим термином пользуется, да и звучит не так по-пионерски)
Книгу купил, спасибо, изучаю)
и это... может стоит исправить тег на c.sharp ? а то "#" както некорректно себя ведет в контесте хабра
Так это легкий вариант синглтона, у меня таким способом куча-кучная вещей реализована (от редкопользуемый пропертей до хранения интерфейсов других модулей, которые тоже не пойми когда дергаются).
pietrovich прав - GOF выдал в своё время великую книгу по паттернам (http://www.ozon.ru/context/detail/id/245…) и её хотя-бы обзорно почитать стоит. А лично мне нравится как паттерны описываются здесь (и с примерами на C#).
Вот так всегда, в процессе реструктуризации обязательно велосипед получается))
Пульки уже кончились, потому просто спасибо!)
Заметка всё равно имеет право на жизнь. Может кого другого спасет от вело-строительства :)
Кстати да, тоже смотрю, вижу синглтон, и думаю, где подвох :) В рациональности же такого подхода сомневаться не приходится, до тех пор пока вы не натолкнетесь на лишнюю трату памяти из-за кэширования слишком большого, но не используемого в дальнейшем, объекта.
Согласен. GC от кривых рук не поможет и за памятью надо посматривать :)
имхо, в случае одноразового использования этого объекта его бы и создавли непосредственно перед использованем. раз уж дошло до ленивой инициализаци, то думаю в данном случае подход оправдан ;)
хотя замечание дельное, в дальнейшем кого-то может натолкнуть на "правильные" мысли :)
раз уж речь зашла про синглетоны, то в контексте "лени" мне более симпатична идея реализованнаяч в пятом варианте вот здесь: http://www.yoda.arachsys.com/csharp/sing…
Заметка про синглтон+поток очень полезна.
Правда меня в большинстве случаев устраивает первый вариант с атрибутом [ThreadStatic].
точно! именно этот вопрос и наводил на размышления))
ещё раз спасибо, посмотреть профиль pietrovich, посмотреть профиль Yustos
А в сишарпе таким образом объявленная переменная "коллектор" является static?
Нет, в данном примере нет. Для каждого объекта типа Лентяй будет свой коллектор.
Тогда это не singleton, а lazy load (что и отраженов названии класса). Singleton должен быть один на программу.
Вы правы.
Где вы здесь singleton увидели, подскажите пожалуйста. И что значит "легкий вариант синглтона"? :)
Да как где... В примере :)
Просто ни коллектор не статический, ни доступ к нему не статический. Вообще нихт статики (впрочем, можно поставить статик коллектору и будет уже другой эффект).
Вот отсутствием статики он отличается от синглтона, а проверкой на инициализанутость и создание по необходимости инстанса похож.
Не вникая в назначение синглтонов, но глядя в пример тут: http://www.dofactory.com/Patterns/Patter…
Видим:
...
    private static Singleton instance;
...
    public static Singleton Instance()
    {
        // Use 'Lazy initialization' <--- :)
        if (instance == null)
        {
            instance = new Singleton();
        }
        return instance;
    }

А "легкий", потому что реализован пропертей класса.
Да. Вот еще вспомнил.
В примере используется String, который допускает null. А ведь нужны и valueвые типы, вроде int. Его тоже можно долго вычислять. Например, проверить для объекта наличие в каких-то больших списках чего-либо.
Можно инициализировать каким-то значением, вроде 0, или -1, или что-то другое и надеяться, что такое значение туда кроме инициализации никогда не попадет. Но грабли есть грабли.
Небольшая модификация:
    private int? _val;
    public int val
    {
        get
        {
            if (!_val.HasValue)
                _val = ДолгаяФункцияПоВычислению();
            return _val.Value;
        }
    }

И спим спокойно :)
Для корректности:
private int? _val = null;
Это частный случай Lazy Loading, судя по коду.
потому и вспомнились паттерны :)
синглтона тут я не вижу.
в get/set вызывать обращение к DB (правильно я понял?) не совсем корректно на мой взгляд. если это сущность, то обрабатываться она должна как одно целое. entity должно загружаться полностью и за один раз (если в этом нет необходимости, см ниже)

(!) кроме того, подумайте про thread safe, в данном блоке необходима синхронизация (опять же, я не уверен на 100%, так как не знаю всего кода в целом)

я бы порекомендовал использовать паттерн "фабрика" в случае, если таких объектов, как вы указали не очень много. если же Вам нужно подгружать "легковесные" объекты, количество которых достаточно большое, то следует использовать DTO объекты, которые являются урезанными версиями entity и загружаются из базы через DAO слой.
1) Да, обращение именно в DB
2) Да, тут необходимо было продумать потоки, именно это мне и не давало покоя)
3) Взял на заметку. Спасибо)
- на Вашем месте я бы начал с профайлинга базы. нужно выяснить, почему получаются тормоза, какие конкретно запросы подвисают. в конце-концов это значение в любом случае придется извлечь, почему перестает тормозить, когда вы используете отложенную загрузку?
- метод fill() не совсем корректен. где он будет реализован и как возратит результат? в случае кластеров у вам обязательно будут будут проблемы с маршалингом. обычно загрузку объектов из DB выносят в отдельный слой, методы DAO возращают корректно сконструированные объекты DTO. DAO слой зовется из слоя бизнес логики. Business Layer возращает корректно сформированную модель. таким образом получается такой flow: DB > DAO > BUSINESS > ready-to-use model > WEB-LAYER. при такой организации очень просто реализовать кэш, если Вам это понядобится.
нужно выяснить, почему получаются тормоза, какие конкретно запросы подвисают

Начиная с логина и простейших SELECT'ов. Либо слишком мало ОЗУ, либо ещё что-то, не сильно относящееся к тебе разговора. Но свою роль в дисциплинировании сыграло) Конкретно данный участок перестаёт тормозить просто потому, что практически не обращается к БД.
Либо слишком мало ОЗУ, либо ещё что-то, не сильно относящееся к тебе разговора

проверьте количество подключений (кол-во выделяемых объектов Connection)
Конкретно данный участок перестаёт тормозить

1) значит, на самом деле нужно сделать кэш (только поищите стандартные реализации!!, нет смысла писать велосипед)
2) при нагрузке все равно возможны тормоза, нужно выяснить в чем проблема в базе, чтобы быть спокойным, что это потом не всплывет
Я не удивлюсь, если в обсуждаемой системе на каждый чих делается запрос к базе.
То бишь, в рамках одного запроса к странице из базы таскаются одни и те же данные в полном объёме.
+ поищите незакрытые Connection
метод fill() не совсем корректен. где он будет реализован и как возратит результат?
Ну, если начать просто перебирать возможные варианты, то существует как минимум два))
Fill(), который производит необходимые операции, назначает публичным переменным значения и возвращает retval
Fill(), который просто возвращает искомое значение и записывает его в переменную класса для дальнейшего использования.
Но мне это кажется таким моветоном (особенно учитывая вишеуказанный flow :) ), что привёл его исключительно для проформы.
При чём тут синхронизация? Если обращение к свойству многопоточное, синхронизация нужна, если однопоточное, то не нужна. Эдак можно к любой отложенной инициализации привязать синхронизацию =)
1) web приложение ВСЕГДА - многопоточное
2) я писал " (опять же, я не уверен на 100%, так как не знаю всего кода в целом)"
Да и пусть будет всегда многопоточное. Только толку с того? Почти все разработчики запросы обрабатывают синхронно, а это значит, что доступ к свойствам нестатических объектов однопоточный.

Учитывая, что писатель заметки открыл для себя отложенную инициализацию, я уверен, что про асинхронную обработку запросов он не знает.
Судя по тому, что Вы предлагаете синхронизацию, Вы про асинхронную обработку может быть слышали. Потому что имей Вы опыт, уже наверняка узнали бы стоимость синхронизации и не совали её куда ни попадя.
мдя, на личности Вы явно любите переходить. что-ж, пусть будет по Вашему. только у меня до сих пор нет уверенности в том, как работает кусок кода, который был приведен. и я не знаю, к своему горю, будет ли экземпляр этого класса использоваться локально или шариться. Вы же, безусловно, можете читать мысли на расстоянии. склоняюсь перед Вами.
Это был псевдокод. Для иллюстрации отложенной инициализации. Экземпляр этого класса никогда не будет создан.

Читать мысли людей с предложениями использования паттерна фабрика для частичной загрузки данных труда не составляет.
Абстрактная фабрика годится для случая, когда есть несколько разнообразных объектов с одинаковым поведением (интерфейсом). Например, есть набор данных для вывода в форме и стоит задача шаблонизировать вывод. Для этого случая можно сделать фабрику, чтобы абстрагировать получение рисовальщика.

class DataPainter
{
public abstact void Draw(Data data);
}

class PainterFabric
{
public abstract DataPainter GetDataPainter();
}

class DataForm
{
private Data data;

public void DrawData(PainterFabric painterFabric)
{
DataPainter dataPainter = painterFabric.GetDataPainter();
dataPainter.Draw(data);
}
}

Рисовальщики и фабрики наследуются от базовых. Обычно они создаются парно. После этого можете кормить в DrawData() любую фабрику, создающую экземпляры рисовальщиков по диагонали, ёлочкой и вообще, как заблагорассудится пользователю библиотекой.
А для того, чтобы загрузить часть полей сущности, использовать фабрику нельзя. Потому что разные свойства у полной и неполной сущности это разное поведение и абстрагироваться здесь некуда, потому что вызывать можно только виртуальные методы, а неполная сущность потому и неполная, что у неё нет каких-то свойств и вызвать то, чего нет, нельзя.

Есть какие-то аргументы по-поводу моего "чтения мыслей" или снова я перехожу на личности и Вы не знаете, где будет использоваться экземпляр этого класса? Мне лень было писать это всё в первый раз, поэтому я там написал только про синхронизацию.
Исходя из того, что отложенная инициализация это открытие и внутри проперти вызывается длинная операция, начинать надо не с записей в блогах, а с чтения хотя бы документации. Там чёрным по белому пишут в гайдлайнах, что длительные операции в свойствах надо декларировать в исключительных случаях, а в общем случае надо реализовывать такое через функцию.
Ну, в общем-то, благодаря этой записи в блоге я купил несколько книг и больше пока вопросов не возникает :)
Могу посоветовать ещё одну.
ASP.NET 2.0, Углубленное изучение. Дино Эспозито.
Зарегистрируйтесь на Хабре, чтобы оставить комментарий

Публикации

Изменить настройки темы

Истории