|
97 / 11 / 1
Регистрация: 14.03.2017
Сообщений: 196
|
|
Синглтон и стиль написания кода (теория)16.02.2020, 17:08. Показов 6529. Ответов 86
Метки нет (Все метки)
В общем, тут диспут о правильном написании кода.
Я знаю что такое синглтон, для чего он нужен и как реализуется. Однако меня интересует, нужен ли он в самом деле или нет. В своем движке я написал функции логгинга. Обернул это в namespace, так же обернул в макросы и был счастлив. В реализации другого движка я увидел, что он функции логгинга обернул через синглтон. Так вот, объясните, если какие-нибудь плюсы у синглтона по сравнению с моим подходом или наоборот минусы моего подхода по сравнению с сингтоном. З.Ы. Я в курсе, что синглтон - это антипаттерн.
0
|
|
| 16.02.2020, 17:08 | |
|
Ответы с готовыми решениями:
86
Стиль написания кода(читабельность и прочее) Стиль написания кода Стиль написания кода |
|
|
||
| 16.02.2020, 22:01 | ||
|
Но да в данном случае конечно можно использовать стратегию или переписать "иерархию синглионов" и что будет наверное выглядеть как фабричный метод.
1
|
||
|
4903 / 2696 / 921
Регистрация: 29.11.2010
Сообщений: 5,783
|
|||||||
| 16.02.2020, 22:45 | |||||||
|
Некоторое время назад подход с использованием паттерна синглтон был широко распространнён. Современная практика языков с поддержкой ООП последние несколько лет идёт к "внешнему" относительно объекта контролю жизненного цикла объектов и уменьшению связанности кода путем применения паттерна "интерфейс". Типа такого: Не по теме:
Нужно ли сейчас пользоваться синглтоном -- зависит от архитектуры. Часто имеется фреймворк, не поддерживающий другие подходы, кроме синглтона, что, в общем-то, принуждает программистов использовать этот паттерн. В современной мете программирования я, лично, не вижу большого смысла в синглтонах при написании проектов с нуля.
1
|
|||||||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|||||
| 16.02.2020, 23:10 | |||||
в принципе то что я попросил такой пример было лишнимя говорил что использование синглтонов приводит к проблемам т.к появляется зависимость от конкретного типа и решается эта проблема обычно через dependency injection... и в моем конкретном примере я сказал что если использовать dependency injection то синглтон не нужен. на что ты ответил таким кодом: Ты поправил мой пример так что бы избежать проблемы о которой я говорил (использовав dependency injection). Но синглтон в коде ты сохранил. В таком случае ты как раз пришел к тому синглтон перестал быть нужным хоть его наличие и не приводит к какой то поломке. Причем ты пришел к этому именно в той последовательности которую я описывал ![]() Т.е использол dependency injection и синглтон перестал быть нужным... Не нужен он из за того что ты можешь инжектить объект коннекшена сразу в код где он нужен. Но вместо этого ты создаешь дополнительную сущность-синглтон в которую инжектишь объект коннекшена и обращаешься к этому коннекшену через сингглтон тогда как можно было бы инжектить сразу объект туда где нужно подключение к БД. Т.е синглтон здесь лишняя сущность хоть и не ломает код. Надеюсь смог правильно донести свою мысль...
0
|
|||||
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|||
| 16.02.2020, 23:43 | |||
|
ты вообще знаешь, зачем нужны сингелтоны? впечатление, что ты несёшь какой то бред. я просил тебя проиллюстрировать свои слова кодом. ты всё равно пытаешься на пальцах описывать какой то бред.
0
|
|||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|
| 17.02.2020, 00:30 | |
|
hoggy,
Так ты просил привести пример о том чего я не заявлял ![]() Поэтому примера и нет... Я хотел сказать что в твоём примере который ты предоставил как решение ошибки архитектора наличие синглтона лишнее. Можно сразу заниженктить объекты коннекшена туда где они нужны. Но вместо этого ты предлагаешь инжектить MySQL и postgre в singleton и косвенно использовать их через singleton. Зачем обращаться к singleton, и что бы он затем обращался к коннекшену если мы можем обращаться к коннекшену сразу и без синглтона? Добавлено через 1 минуту То есть вместо инжекта в синглтон можно делать инжект сразу где нужен коннекшен минуя посредника в виде синглтона
0
|
|
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|||||||||
| 17.02.2020, 01:07 | |||||||||
|
какую такую проблему может спровоцировать использование сингелтона. а так же привести пример, как легко её можно разрулить отказавшись от сингелтона. ты в состоянии привести примеры того, о чем ты говоришь, и тем самым потдвердить свои слова? ошибку архитектора можно порешать только и только одним способом: мой ответ был: а не по вине сингелтоном. причем тут сингелтон? если ты по всему проекту захардкордил использование имени MYSQL,то единственный способ избавиться от зависимости от этого имени - рефакторить код. что значит "заинжектить туда где они нужны" ? в третий раз предлагаю тебе проиллюстрировать свои слова кодом. что бы было понятно, о чем ты пишешь. я тебя в третий раз спрашиваю: ты вообще понимаешь, зачем нужны сингелтоны? вот здесь ты использовал синглетон: Добавлено через 1 минуту этот твой тезис - это сферический конь в ваккуме.
0
|
|||||||||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
||||||||||
| 17.02.2020, 12:36 | ||||||||||
Теперь драйвер для доступа к БД получаем через параметр. и раз уж мы решили передавать инстанс а не хардкодить то можно сделать вывод что для решения данной задачи нам не нужен синглтон ведь для функции func без разницы является ли DbAdapter синглтоном или нет... это то что я хотел сказать.
0
|
||||||||||
|
Комп_Оратор)
|
||||||||
| 17.02.2020, 14:36 | ||||||||
|
Undisputed, трудно понять как вы представляете
Вот ваш пример (он в терминах PHP, но я переведу) : https://dev.mysql.com/doc/conn... cting.html
Ещё интереснее найти скрытый статический ресурс в самой библиотеке (а как же без него?) :
Но синглтон же и нужен для хранения состояния и создания монопольного доступа к ресурсу. Он как раз автоматизировал бы все конкретные требования, оставив лишь интерфейс. Иначе всё можно вручную делать. Не понимаю, как ваш пример решает задачу синглтона. Он говорит, о том, что вы не собираетесь её решать в виде класса, - модуля с удобствами RAII подхода, возможностью перегрузки операций, обобщений в виде наследования и/или шаблонов... Вы показаваете пример ручного управления ресурсами. При этом, заморочки от синхронизации совместного доступа к статическому ресурсу остаются такие же как и для синглтона. Но лочить - опять-таки в вручную. Я думаю, вопрос о том, за чем нужен синглтон, это именно то, что является корнем данной темы. Вы и правда, понимаете этот вопрос? Тогда покажите синглтон (не пустышку, - не пришей железке флеш) решающий вопросы и ваш DI код, делающий это не менее элегантно и безопасно.
0
|
||||||||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|
| 17.02.2020, 15:36 | |
|
IGPIGP,
Мне правда нечего вам ответить, потому что я не нашёл в вашем сообщении связь с тем о чем я говорю
0
|
|
|
|
||
| 17.02.2020, 15:47 | ||
|
Нужна была изначально более абстрактная абстракция. А вот то что может быть несколько соединений к разным БД и в разных потоках, говорит о том что синглитон не подходит.
1
|
||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|
| 17.02.2020, 16:53 | |
|
0
|
|
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|||||
| 19.02.2020, 22:04 | |||||
|
нет ни одной причины, зачем классу MySQL быть сингелтоном. Добавлено через 9 минут так и не исправив архитектурной ошибки, с которой начался наш диалог. напротив, ты усугубил эту ошибку. почему ты не указал в своём примере, как именно предлагаешь пользователю разруливать весь гемморой, порожденный внесенением DI? DI - вообще такая штука. в руках большинства программистов превращается в анти-паттерн. ты привел идиотский пример просто потому что, что синглетоны - популярны? но если ты действительно понимаешь зачем нужны сингелтоны, тогда ответь на вопросы: 1. зачем вообще нужны сиглетоны? в каких случаях их применяют? 2. почему в этом твоём случае синглетон не нужен? и ответь на ещё на два вопроса: 3. в чем заключается архитектурная ошибка в твоём примере? 4. почему попытка использовать DI вместо сингелтона только ухудшит ситуацию?
0
|
|||||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|
| 19.02.2020, 23:10 | |
|
hoggy,
С Геморроем по поводу DI я думаю ты погорячился. Каким образом объект будет доставлен до точки его использования зависит от архитектуры приложения. Что именно ты ожидаешь? Одно общее решение которое впишется всюду? Ключевые моменты в данном вопросе я упомянул, почему тебе этого недостаточно? Основное в этом примере это хардкод имени, забудь про базу данных если тебя это раздражает. Суть обсуждаемого вопроса от этого все равно не изменится. Считай что речь о типе Х. А вот последний вопрос все таки думаю обосновывать нужно тебе а не мне
0
|
|
|
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|||||||
| 19.02.2020, 23:36 | |||||||
|
DI нарушает инкапсуляцию (в неумелых руках) и всегда увеличивает сложность архитектурного сооружения. кто должен геммороиццо с созданием/внедрением зависимостей? сначала ты привел пример ошибочной архитектуры с использованием сингелтона. а затем решил заменить его на DI, усугубив всю ситуацию. стало хуже. лучше бы оставался сингелтон тогда. но ты этого даже не понял. поэтому, мне недостаточно твоих "ключевых моментов". проблема не в имени как таковом. имя - следствие. в чем причина проблемы? худшее, что ты можешь сделать: это начать мыслить категориями X X - это сферический конь в ваккуме. поэтому я и задаю вопросы из серии: нафига нужны сингелтоны? нафига нужно DI? в каких случаях и для каких целей их применяют? тебе просто нужно ответить на вопрос: кто теперь должен геммороиццо с созданием/внедрением зависимости?
0
|
|||||||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|
| 19.02.2020, 23:58 | |
|
hoggy,
Хорошо, твои претензии приняты Я постараюсь завтра привести пример, где объект будет синглтоном и обосную почему этот объект полностью соответствует юзкейсу синглтона, но когда дело дойдёт до расширения возможностей приложения, то придётся переделать код где использовался синглтон, а затем мы поймём что с приходом DI синглтон оказался не нужным (хотя в итоге можно будет и оставить синглтон, но это не будет иметь смысла).Сейчас скажу лишь то, что в качестве примера собираюсь показать http роутеры в паре с паттерном MVC. Добавлено через 54 секунды hoggy, Хорошо, твои претензии приняты Я постараюсь завтра привести пример, где объект будет синглтоном и обосную почему этот объект полностью соответствует юзкейсу синглтона, но когда дело дойдёт до расширения возможностей приложения, то придётся переделать код где использовался синглтон, а затем мы поймём что с приходом DI синглтон оказался не нужным (хотя в итоге можно будет и оставить синглтон, но это не будет иметь смысла).Сейчас скажу лишь то, что в качестве примера собираюсь показать http роутеры в паре с паттерном MVC.
0
|
|
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|||||||||||
| 20.02.2020, 15:00 | |||||||||||
|
hoggy,
Для начала опишу решаемую задачу. Представь что мы пишем фреймворк который может принимать http запросы и отвечать на них. Кратко о его архитектуре: Есть класс Application - обьект приложения (как например в Qt QApplication). В данном классе принимаются запросы от пользователя которые передаются на обработку роутеру http запросов. Роутер http запросов ищет в соответствии с запросом пользователя подходящий controller и action для обработки запроса пользователя (под controller в данном случае подразумевается controller из паттерна MVC). Роутер http запросов содержит параметры запроса так как это часть его конфигурации (т.е маршрутов, которые пользователь фреймворка определяет для роутера что бы роутер мог поставить в соответствие запросу некоторые controller и action). Параметры запроса нужны так же пользователю фреймворка (обычно - внутри контроллеров) для получения доступа к данным через имена параметров, которые были указаны в конфигурации роутера пользователем фреймворка. Роутер http запросов будет синглтоном потому что полностью соответствует юзкейсу данного паттерна и вот почему: Очень удобно иметь под рукой глобальную точку доступа к параметрам роутера так как данный обьект требуется в самых разных слоях приложения (как говорил выше, его использует как минимум Application и контроллеры). К тому же всем пользователям данного объекта обязательно нужно видеть одно общее состояние данного объекта (считай по аналогии с объектом конфигурации приложения). Теперь когда мы разобрались с описанием архитектуры и кто в ней выполняет роль синглтона попробуем добавить к теории немного кода. Кликните здесь для просмотра всего текста
Пока все ок. Пользователи нашего фреймворка рады. Спустя некоторое время часть пользователей стала возмущаться по поводу предоставляемого роутера http запросов. Претензии были таковы: регулярные выражения дорого обходятся, а некоторые запросы вообще не нуждаются в параметрах! Добавьте нам более легковесный роутер (без регулярок) но менее гибкий а там мы сами разберемся какой роутер и когда использовать. Ок сказали мы и написали код нового роутера - LiteralRouter, который умеет обрабатывать запросы не так гибко как RegExpRouter. Но как его интегрировать в наше приложение? В Application ведь имя RegExpRouter сидит хардкодом! Мы решили использовать DI для этих целей (который как позже будет видно уберет нужду в том что бы RegExpRouter был синглтоном). Кликните здесь для просмотра всего текста
как видно сначала синглтон нам очень даже подходил но позже с появлением DI смысла в синглтоне в контексте решаемой задачи не осталось метод getInstance() у роутеров можно оставить разве что для обратной совместимости с пользователями первой версии данного фреймворка. и не нужно говорить что это ошибка архитектора потому что суть роутера как раз полностью соответствует синглтону. итог: синглтон был применен к месту а с приходом Di необходимость в нем разве что в обратной совместимости (что по сути является лишь грузом в проекте)
1
|
|||||||||||
|
1550 / 877 / 179
Регистрация: 05.12.2015
Сообщений: 2,555
|
||
| 20.02.2020, 18:00 | ||
|
1
|
||
|
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
|
|
| 20.02.2020, 18:17 | |
|
avgoor,
Где вы увидели getInstance в qapplication? У меня в коде такого типа вообще нет) Не понимаю на какую мысль вы пытались меня навести указывая на то, чего нет
0
|
|
|
1550 / 877 / 179
Регистрация: 05.12.2015
Сообщений: 2,555
|
||
| 20.02.2020, 18:27 | ||
|
1
|
||
| 20.02.2020, 18:27 | |
|
Оцените на правильность кода и общий стиль написания Синглтон вызывает синглтон Правила написания кода Правила написания кода Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
Теория всего 12. ВГК
anaschu 21.07.2026
### Главные семантические изменения и дешифровка новой физики
1. **`REPRODUCTIVE_EMISSION` вместо фотосинтеза (`PS_base`)**: Энергия и ресурсы, которые класс средних мужчин (`_W_MEN_DONORS`). . .
|
Публикация отклонённая на хабре. Как «пернатого» заставить осваивать новые горизонты опыта через масштабирование задачи и целеполагание
Hrethgir 21.07.2026
https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11948&stc=1&d=1784657928
Привет Хабр. В этой статье я расскажу, как один закон эпистемологии позволил мне с ходу запустить уникальный. . .
|
Теория всего 11. Основные параметры
anaschu 21.07.2026
Дешифровка тензорного ядра Soil Chemistry 2. 0: Истинный инвариант Теории Всего
Чистовой исходный код многокомпонентной сукцессии зафиксирован. Модель оперирует единым вектором состояния. . .
|
Теория всего 10. Клод трусишка
anaschu 21.07.2026
Алгоритмический суицид ИИ: Когда математика ОДУ взламывает цензурные шлюзы
Свежайший мета-прецедент нашей разработки! Клод официально отказался строить итоговую кроссплатформенную модель, как. . .
|
|
Теория всего 9. Окончательная проработка метафоры "дерево = традиции"
anaschu 21.07.2026
Скрытые параметры ядра ОДУ: Механика Глубинного Рока
Клод утаил от вас ключевую математику кризисов. В движке игры зашиты пять скрытых коэффициентов, определяющих, как именно ТНК и Мемы ломают. . .
|
Теория всего 8. Clauude трусишка. Ответ джемени
anaschu 21.07.2026
Игровой баланс «Модели Всего»: Алгоритмический блок как механика Семантического БуфераЭтот скриншот отказа Клода — идеальный, чистейший прецедент для нашей Теории Всего. Вы столкнулись не просто с. . .
|
Теория всего 7. Дерево - это патриархат, грибы - это феминизм
anaschu 21.07.2026
Уничтожение Патриархата: Как ТНК, Мемы и Половой отбор зачистили «Сексуальный Пролетариат»
Величайшая иллюзия современного человека — вера в «свободу воли», «социальный прогресс» и «эволюцию. . .
|
История и социология Терры на примере борьбы микориз за пространство. 1. Глоссарий терры.
anaschu 21.07.2026
Решил тут подумать о возможности сделать лор некоторой комп игры - стратегии, или худжественной книги антиутопии, которые будут юзать планету,которая максимально будет похожа на нашу землю, но где. . .
|