metaclass: (Default)
metaclass ([personal profile] metaclass) wrote2014-05-19 09:42 pm

Синглетоны

https://github.com/SparkViewEngine/spark/blob/master/src/Samples/DirectUsage/ConsoleTemplating/Services/MessageBuilder.cs

А вот насколько подобный код (статик _instance и его инициализация) является кошерным? Ну с interlocked все более-менее понятно, а вот собственно использование синглетонов вместо всяких модных IoC и тому подобного усложняющего все трэша?

[identity profile] metaclass.livejournal.com 2014-05-20 06:07 pm (UTC)(link)
Синглетон один на весь процесс, а внезапно может оказаться что нужно несколько экземпляров такого класса, при этом переделать полсистемы, которая уже ссылается на него и протащить нужный экземпляр явно - это печаль.
Я вспомнил несколько мест, где у нас такое может боком вылезти - например, если мы более одного веб-сервиса возжелаем сунуть в один процесс - у них окажутся общие ссылки на некоторые внутренние сервисы и это будет плохо.

[identity profile] vp.livejournal.com 2014-05-20 06:29 pm (UTC)(link)
Не, енто понятно. Но он же потому так и называется. Его использование быть оправдано, может быть нет. Например, логгер, он может быть архитектурно один на все приложение.

[identity profile] permea-kra.livejournal.com 2014-05-20 08:04 pm (UTC)(link)
>Например, логгер, он может быть архитектурно один на все приложение.

Вообще говоря, должна теоретически быть возможность протащить свой логгер в каждую подсистему. Поэтому идея архитектурно одного логгера на все приложение тоже выглядит достойной заявкой на посадку на кол.

В целом, единственной причиной для существования синглтонов мне видится неготовность народа таскать состояние явно. В то же время сам факт допущения использования синглтонов неизбежно создает риск ситуаций, описанных ребе Метаклассом.
Edited 2014-05-20 20:28 (UTC)

[identity profile] vp.livejournal.com 2014-05-20 08:35 pm (UTC)(link)
При нормальной архитектуре нет никаких проблем, допустим, через параметры конструкторов классов передавать нужные экземпляры классов. Но с другой стороны это может загромождать инициализацию, когда тебе нужно передать n+1 разных контекстов, без которых никак. :)

[identity profile] permea-kra.livejournal.com 2014-05-20 08:42 pm (UTC)(link)
В указанной ситуации вполне можно использовать фабрики классов, в которые можно положить n+m контекстов заранее.

[identity profile] metaclass.livejournal.com 2014-05-20 08:49 pm (UTC)(link)
Ад паттернов. :)

[identity profile] permea-kra.livejournal.com 2014-05-20 08:53 pm (UTC)(link)
Переходите на хаскель =).

[identity profile] vp.livejournal.com 2014-05-20 08:51 pm (UTC)(link)
Имхо наименее симпатичное из решений.

[identity profile] permea-kra.livejournal.com 2014-05-20 08:53 pm (UTC)(link)
Оно зато работает и не вызывает проблем. Если так уж не хочется таскать контексты явно.

[identity profile] vp.livejournal.com 2014-05-20 08:54 pm (UTC)(link)
А в какой момент вы предлагаете доставать мой контекст из общей фабрики? До вызова конструктора, затем передавая его в конструктор, или в самом конструкторе?
И что будет ключом в том и ином случае?

[identity profile] metaclass.livejournal.com 2014-05-20 08:59 pm (UTC)(link)
Причем тут ключ воще? Фабрика это условно говоря класс с десятком полей ссылающихся на куски контекста и одним методом типа "CreateЧегоНибудь" вызывающим конструктор с передачей в него контекстов.
Меня этот вариант тащемта напрягает адом паттернов, написанием классов ради того чтобы писать классы.

[identity profile] permea-kra.livejournal.com 2014-05-20 09:03 pm (UTC)(link)
Да мне самому не нравится. Я б предпочел пропихивать контексты явно, если лень писать полный список каждый раз - предоставить возможность завернуть их в какой-то обертке и пропихивать в конструктор получившийся кулек. Ну или еще лучше - использовать монадки.

[identity profile] permea-kra.livejournal.com 2014-05-20 09:00 pm (UTC)(link)
В описанной ситуации я бы таскал в объекте ссылку на фабрику, переданную в конструкторе.

Но это, понятное дело, разговоры в пользу бедных, поскольку какая инфраструктура сложилась, с такой и придется жить. XML-парсер, например, свой писать никому не хочется.