Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.88/26: Рейтинг темы: голосов - 26, средняя оценка - 4.88
97 / 11 / 1
Регистрация: 14.03.2017
Сообщений: 196

Синглтон и стиль написания кода (теория)

16.02.2020, 17:08. Показов 6320. Ответов 86
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
В общем, тут диспут о правильном написании кода.

Я знаю что такое синглтон, для чего он нужен и как реализуется. Однако меня интересует, нужен ли он в самом деле или нет.
В своем движке я написал функции логгинга. Обернул это в namespace, так же обернул в макросы и был счастлив. В реализации другого движка я увидел, что он функции логгинга обернул через синглтон.
Так вот, объясните, если какие-нибудь плюсы у синглтона по сравнению с моим подходом или наоборот минусы моего подхода по сравнению с сингтоном.

З.Ы. Я в курсе, что синглтон - это антипаттерн.
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
16.02.2020, 17:08
Ответы с готовыми решениями:

Стиль написания кода(читабельность и прочее)
Недавно делал тестовое задание для поиска работы, но мне сказали, что код не совсем корректно написан. Задание:Написать рекурсивный и не...

Стиль написания кода
Всем привет! Просматривал исходники Perl и обратил внимание на то, что стиль написания кода который представлен ниже, очень короткий по...

Стиль написания кода
Имеется внешний класс. Внутри этого внешнего класса объявлен private static класс. Клиенты внешнего класса не имеют представления о...

86
Эксперт С++
 Аватар для Avazart
8489 / 6156 / 615
Регистрация: 10.12.2010
Сообщений: 28,683
Записей в блоге: 30
16.02.2020, 22:01
Студворк — интернет-сервис помощи студентам
Цитата Сообщение от Undisputed Посмотреть сообщение
сначала все было ок и мы для работы с БД взяли синглтон (почему нет? удобно же...)
уже написано много кода с обращением к БД через тип MySQLSingleton
затем пришла задача сделать так, что бы поиск данных шел сразу по нескольким базам разного типа
придется много поработать что бы обеспечить подобное удовольствие
Это изначально неправильный выбор ибо в одном приложении может быть несколько соединений (к примеру в разных потоках).

Но да в данном случае конечно можно использовать стратегию или переписать "иерархию синглионов" и что будет наверное выглядеть как фабричный метод.
1
 Аватар для lemegeton
4903 / 2696 / 921
Регистрация: 29.11.2010
Сообщений: 5,783
16.02.2020, 22:45
Цитата Сообщение от Diochrome Посмотреть сообщение
Я знаю что такое синглтон, для чего он нужен и как реализуется. Однако меня интересует, нужен ли он в самом деле или нет.
Синглтон это паттерн, контролирующий время жизни объекта.

Некоторое время назад подход с использованием паттерна синглтон был широко распространнён.



Современная практика языков с поддержкой ООП последние несколько лет идёт к "внешнему" относительно объекта контролю жизненного цикла объектов и уменьшению связанности кода путем применения паттерна "интерфейс".

Типа такого:

Не по теме:

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
class User;
 
template<typename T>
class Repository {
public:
    virtual ~Repository() {}
};
 
template<typename T>
class SqlRepository : public Repository<T> {
};
 
template<typename T>
class RedisRepository : public Repository<T> {
};
 
class UserService {
public:
    UserService(Repository<User> *userRepository) : userRepository(userRepository) {}
private:
    Repository<User> *userRepository;
};
 
int main(int argc, char** argv) {
 
    Repository<User> *redisUserRepository = new RedisRepository<User>();
    Repository<User> *sqlUserRepository = new SqlRepository<User>();
 
    UserService *redisUserService = new UserService(redisUserRepository);
    UserService *sqlUserService = new UserService(sqlUserRepository);
 
    return 0;
}



Нужно ли сейчас пользоваться синглтоном -- зависит от архитектуры. Часто имеется фреймворк, не поддерживающий другие подходы, кроме синглтона, что, в общем-то, принуждает программистов использовать этот паттерн.
В современной мете программирования я, лично, не вижу большого смысла в синглтонах при написании проектов с нуля.
1
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
16.02.2020, 23:10
Цитата Сообщение от hoggy Посмотреть сообщение
какое это вообще имеет отношение к теме?
забудь, я подумал ты намекаеш на то что в зависимости от СУБД придется корректировать запросы
Цитата Сообщение от hoggy Посмотреть сообщение
я не понял, чего ты хочешь.
тоже забудь в принципе то что я попросил такой пример было лишним
Цитата Сообщение от hoggy Посмотреть сообщение
ты заявляешь, что якобы конкретно сингелтоны у тебя виноваты
а вот об этом стоит поговорить... не, я так не говорил...
я говорил что использование синглтонов приводит к проблемам т.к появляется зависимость от конкретного типа и решается эта проблема обычно через dependency injection... и в моем конкретном примере я сказал что если использовать dependency injection то синглтон не нужен.

на что ты ответил таким кодом:
Цитата Сообщение от hoggy Посмотреть сообщение
// инжект на сингелтонах
db::add<MySQL>();
db::add<PostrgeSQL>();
...
db::select("...");
и ты опять пришел к тому о чем я говорил, вот почему:
Ты поправил мой пример так что бы избежать проблемы о которой я говорил (использовав dependency injection). Но синглтон в коде ты сохранил. В таком случае ты как раз пришел к тому синглтон перестал быть нужным хоть его наличие и не приводит к какой то поломке.

Причем ты пришел к этому именно в той последовательности которую я описывал
Т.е использол dependency injection и синглтон перестал быть нужным...

Не нужен он из за того что ты можешь инжектить объект коннекшена сразу в код где он нужен.
Но вместо этого ты создаешь дополнительную сущность-синглтон в которую инжектишь объект коннекшена и обращаешься к этому коннекшену через сингглтон тогда как можно было бы инжектить сразу объект туда где нужно подключение к БД. Т.е синглтон здесь лишняя сущность хоть и не ломает код. Надеюсь смог правильно донести свою мысль...
0
Эксперт С++
 Аватар для hoggy
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
16.02.2020, 23:43
Цитата Сообщение от Undisputed Посмотреть сообщение
В таком случае ты как раз пришел к тому синглтон перестал быть нужным хоть его наличие и не приводит к какой то поломке.
это ещё почему это?

ты вообще знаешь, зачем нужны сингелтоны?

Цитата Сообщение от Undisputed Посмотреть сообщение
Не нужен он из за того что ты можешь инжектить объект коннекшена сразу в код где он нужен.
я не понимаю о чем ты говоришь.
впечатление, что ты несёшь какой то бред.

я просил тебя проиллюстрировать свои слова кодом.
ты всё равно пытаешься на пальцах описывать какой то бред.
0
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
17.02.2020, 00:30
hoggy,
Так ты просил привести пример о том чего я не заявлял
Поэтому примера и нет...

Я хотел сказать что в твоём примере который ты предоставил как решение ошибки архитектора наличие синглтона лишнее. Можно сразу заниженктить объекты коннекшена туда где они нужны. Но вместо этого ты предлагаешь инжектить MySQL и postgre в singleton и косвенно использовать их через singleton. Зачем обращаться к singleton, и что бы он затем обращался к коннекшену если мы можем обращаться к коннекшену сразу и без синглтона?

Добавлено через 1 минуту
То есть вместо инжекта в синглтон можно делать инжект сразу где нужен коннекшен минуя посредника в виде синглтона
0
Эксперт С++
 Аватар для hoggy
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
17.02.2020, 01:07
Цитата Сообщение от Undisputed Посмотреть сообщение
Так ты просил привести пример о том чего я не заявлял
я просил тебя привести пример, который иллюстриует
какую такую проблему может спровоцировать использование сингелтона.
а так же привести пример, как легко её можно разрулить отказавшись от сингелтона.

ты в состоянии привести примеры того, о чем ты говоришь,
и тем самым потдвердить свои слова?

Цитата Сообщение от Undisputed Посмотреть сообщение
Я хотел сказать что в твоём примере который ты предоставил как решение ошибки архитектора наличие синглтона лишнее.
во-первых, я не приводил примера, как можно порешать ошибку архитектора.
ошибку архитектора можно порешать только и только одним способом: увольнением архитектора рефакторингом.

мой ответ был:
Цитата Сообщение от hoggy Посмотреть сообщение
теперь приведи альтернативу.
и убедись, что стоимость твоего удовольствия не изменилась.
архитектурые ошибки случаются по вине недальновидных архитекторов,
а не по вине сингелтоном.

причем тут сингелтон?

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

Цитата Сообщение от Undisputed Посмотреть сообщение
Можно сразу заниженктить объекты коннекшена туда где они нужны.
я не понимаю этой фразы.
что значит "заинжектить туда где они нужны" ?

в третий раз предлагаю тебе проиллюстрировать свои слова кодом.
что бы было понятно, о чем ты пишешь.

Цитата Сообщение от Undisputed Посмотреть сообщение
Зачем обращаться к singleton, и что бы он затем обращался к коннекшену если мы можем обращаться к коннекшену сразу и без синглтона?
а зачем ты вообще решил использовать синглетон?
я тебя в третий раз спрашиваю: ты вообще понимаешь, зачем нужны сингелтоны?

вот здесь ты использовал синглетон:
Цитата Сообщение от Undisputed Посмотреть сообщение
MySQLSingleton::getInstance()->select("...");
// и так по всему коду где требуется база данных
Цитата Сообщение от Undisputed Посмотреть сообщение
сначала все было ок и мы для работы с БД взяли синглтон (почему нет? удобно же...)
вот зачем ты его здесь использовал?

Добавлено через 1 минуту
Цитата Сообщение от Undisputed Посмотреть сообщение
То есть вместо инжекта в синглтон можно делать инжект сразу где нужен коннекшен минуя посредника в виде синглтона
пока ты не проиллюстрируешь кодом, что имеешь ввиду,
этот твой тезис - это сферический конь в ваккуме.
0
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
17.02.2020, 12:36
Цитата Сообщение от hoggy Посмотреть сообщение
если ты по всему проекту захардкордил использование имени MYSQL,
то единственный способ избавиться от зависимости от этого имени - рефакторить код.
Вот об этом я и говорю... При использовании синглтона произошел хардкод (да, ты привел альтернативу где сохранил синглтон и частично избавился от хардкода, но дело в том что твоя альтернатива на самом деле не нуждается в синглтоне). Об этом чуть ниже.
Цитата Сообщение от hoggy Посмотреть сообщение
я не понимаю этой фразы.
что значит "заинжектить туда где они нужны" ?
это означает что вместо хардкода о котором шла речь выше я предложил использовать DI (ты сейчас скажешь что DI и синглтон можно использовать вместе и будешь частично прав, но дело в том что хоть такой код и можно написать но конкретно в данном случае наличие синглтона будет избыточным). Рассмотрим пример:
C++
1
2
3
4
5
6
7
8
9
10
void func(DbAdapter& adapter)
{
    adapter.query("...");
}
 
// против такого хардкода
void func()
{
    MySQL::getInstance()->query("...");
}
первая версия функции func более расширяема чем вторая (за счет того что убрали хардкод).
Теперь драйвер для доступа к БД получаем через параметр.

и раз уж мы решили передавать инстанс а не хардкодить то можно сделать вывод что для решения данной задачи нам не нужен синглтон ведь для функции func без разницы является ли DbAdapter синглтоном или нет... это то что я хотел сказать.
Цитата Сообщение от hoggy Посмотреть сообщение
я тебя в третий раз спрашиваю: ты вообще понимаешь, зачем нужны сингелтоны?
конечно да
Цитата Сообщение от hoggy Посмотреть сообщение
а зачем ты вообще решил использовать синглетон?
а я не использую синглтон в таких случаях... это популярный (но проблемный) подход поэтому я и привел такой пример...
0
Комп_Оратор)
Эксперт по математике/физике
 Аватар для IGPIGP
9007 / 4708 / 630
Регистрация: 04.12.2011
Сообщений: 14,003
Записей в блоге: 16
17.02.2020, 14:36
Undisputed, трудно понять как вы представляете заклинание общий термин DI. Я знаю о некоторой неповоротливости статических объектов и возможности минимизировать такую потребность статическим указателем, используя не статическую, а динамическую память для самого объекта. Но в принципе, без статического объекта (указателя) не обойтись. Рассыпать доступ к статическому ресурсу можно, но есть дискомфорт и снижение надёжности.
Вот ваш пример (он в терминах PHP, но я переведу) :
https://dev.mysql.com/doc/conn... cting.html
C++
1
2
3
4
5
6
7
sql::mysql::MySQL_Driver *driver;
sql::Connection *con;
 
driver = sql::mysql::get_mysql_driver_instance();
con = driver->connect("tcp://127.0.0.1:3306", "user", "password");
 
delete con;
а на словах велели передать:
Make sure that you free con, the sql::Connection object, as soon as you do not need it any more. But do not explicitly free driver, the connector object. Connector/C++ takes care of freeing that.
То есть надо бы не забыть удалить указатель con. Ещё веселее тот факт, что не дай бог вручную зарелизить указатель на объект для драйвера.
Ещё интереснее найти скрытый статический ресурс в самой библиотеке (а как же без него?) :
Note
get_mysql_driver_instance() calls get_driver_instance(), which is not thread-safe. Either avoid invoking these methods from within multiple threads at once, or surround the calls with a mutex to prevent simultaneous execution in multiple threads.
То есть нужна блокировка доступа к данному ресурсу. Это проблема не синглтона, а уникальности самого ресурса (статической семантики что-ли).
Но синглтон же и нужен для хранения состояния и создания монопольного доступа к ресурсу. Он как раз автоматизировал бы все конкретные требования, оставив лишь интерфейс.
Иначе всё можно вручную делать. Не понимаю, как ваш пример решает задачу синглтона. Он говорит, о том, что вы не собираетесь её решать в виде класса, - модуля с удобствами RAII подхода, возможностью перегрузки операций, обобщений в виде наследования и/или шаблонов... Вы показаваете пример ручного управления ресурсами. При этом, заморочки от синхронизации совместного доступа к статическому ресурсу остаются такие же как и для синглтона. Но лочить - опять-таки в вручную.
Я думаю, вопрос о том, за чем нужен синглтон, это именно то, что является корнем данной темы. Вы и правда, понимаете этот вопрос? Тогда покажите синглтон (не пустышку, - не пришей железке флеш) решающий вопросы и ваш DI код, делающий это не менее элегантно и безопасно.
0
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
17.02.2020, 15:36
IGPIGP,
Мне правда нечего вам ответить, потому что я не нашёл в вашем сообщении связь с тем о чем я говорю
0
Эксперт С++
 Аватар для Avazart
8489 / 6156 / 615
Регистрация: 10.12.2010
Сообщений: 28,683
Записей в блоге: 30
17.02.2020, 15:47
Цитата Сообщение от hoggy Посмотреть сообщение
если ты по всему проекту захардкордил использование имени MYSQL,
то единственный способ избавиться от зависимости от этого имени - рефакторить код.
В реальности SQL запросы в разных БД отличаются, поэтому как бы тут изначально что-то не так...
Нужна была изначально более абстрактная абстракция.
А вот то что может быть несколько соединений к разным БД и в разных потоках, говорит о том что синглитон не подходит.
1
Комп_Оратор)
Эксперт по математике/физике
 Аватар для IGPIGP
9007 / 4708 / 630
Регистрация: 04.12.2011
Сообщений: 14,003
Записей в блоге: 16
17.02.2020, 16:47
Цитата Сообщение от Undisputed Посмотреть сообщение
IGPIGP,
Мне правда нечего вам ответить, потому что я не нашёл в вашем сообщении связь с тем о чем я говорю
То есть, код приведенный мной и ваш пример, это не об одном и том же?
0
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
17.02.2020, 16:53
Цитата Сообщение от IGPIGP Посмотреть сообщение
То есть, код приведенный мной и ваш пример, это не об одном и том же?
Конечно нет. Я говорил о том что явное указание типа бьет по расширяемости кода
0
Эксперт С++
 Аватар для hoggy
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
19.02.2020, 22:04
Цитата Сообщение от Avazart Посмотреть сообщение
А вот то что может быть несколько соединений к разным БД и в разных потоках, говорит о том что синглитон не подходит.
совершенно верно.
нет ни одной причины, зачем классу MySQL быть сингелтоном.

Добавлено через 9 минут
Цитата Сообщение от Undisputed Посмотреть сообщение
первая версия функции func
ты сейчас в другую крайность кидаешься,
так и не исправив архитектурной ошибки,
с которой начался наш диалог.
напротив, ты усугубил эту ошибку.

почему ты не указал в своём примере,
как именно предлагаешь пользователю разруливать весь гемморой,
порожденный внесенением DI?

DI - вообще такая штука.
в руках большинства программистов превращается в анти-паттерн.

Цитата Сообщение от Undisputed Посмотреть сообщение
конечно да
Цитата Сообщение от Undisputed Посмотреть сообщение
а я не использую синглтон в таких случаях... это популярный (но проблемный) подход поэтому я и привел такой пример...
интересный сказ у тебя получается.
ты привел идиотский пример просто потому что,
что синглетоны - популярны?

но если ты действительно понимаешь зачем нужны сингелтоны,
тогда ответь на вопросы:

1.
зачем вообще нужны сиглетоны?
в каких случаях их применяют?

2.
почему в этом твоём случае синглетон не нужен?

и ответь на ещё на два вопроса:
3.
в чем заключается архитектурная ошибка в твоём примере?
4.
почему попытка использовать DI вместо сингелтона только ухудшит ситуацию?
0
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
19.02.2020, 23:10
hoggy,
С Геморроем по поводу DI я думаю ты погорячился.
Каким образом объект будет доставлен до точки его использования зависит от архитектуры приложения.
Что именно ты ожидаешь? Одно общее решение которое впишется всюду?
Ключевые моменты в данном вопросе я упомянул, почему тебе этого недостаточно?

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

А вот последний вопрос все таки думаю обосновывать нужно тебе а не мне
0
Эксперт С++
 Аватар для hoggy
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
19.02.2020, 23:36
Цитата Сообщение от Undisputed Посмотреть сообщение
С Геморроем по поводу DI я думаю ты погорячился.
я серьёзно.

DI нарушает инкапсуляцию (в неумелых руках)
и всегда увеличивает сложность архитектурного сооружения.

кто должен геммороиццо с созданием/внедрением зависимостей?

Цитата Сообщение от Undisputed Посмотреть сообщение
Что именно ты ожидаешь?
я ожидаю от тебя вминяемых примеров кода.

Цитата Сообщение от Undisputed Посмотреть сообщение
почему тебе этого недостаточно?
например, потому что ты променял шило на мыло.
сначала ты привел пример ошибочной архитектуры с использованием сингелтона.
а затем решил заменить его на DI,
усугубив всю ситуацию.

стало хуже.
лучше бы оставался сингелтон тогда.

но ты этого даже не понял.
поэтому, мне недостаточно твоих "ключевых моментов".

Цитата Сообщение от Undisputed Посмотреть сообщение
Основное в этом примере это хардкод имени
ты ошибаешься.
проблема не в имени как таковом.
имя - следствие.

в чем причина проблемы?

Цитата Сообщение от Undisputed Посмотреть сообщение
Суть обсуждаемого вопроса от этого все равно не изменится. Считай что речь о типе Х.
нет, я не буду считать, что речь о типе X.
худшее, что ты можешь сделать: это начать мыслить категориями X

X - это сферический конь в ваккуме.

поэтому я и задаю вопросы из серии:
нафига нужны сингелтоны? нафига нужно DI?
в каких случаях и для каких целей их применяют?

Цитата Сообщение от Undisputed Посмотреть сообщение
А вот последний вопрос все таки думаю обосновывать нужно тебе а не мне
обоснование лежит на поверности.

тебе просто нужно ответить на вопрос:
кто теперь должен геммороиццо с созданием/внедрением зависимости?
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 и контроллеры). К тому же всем пользователям данного объекта обязательно нужно видеть одно общее состояние данного объекта (считай по аналогии с объектом конфигурации приложения).

Теперь когда мы разобрались с описанием архитектуры и кто в ней выполняет роль синглтона попробуем добавить к теории немного кода.

Кликните здесь для просмотра всего текста
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
#include <iostream>
#include <string>
#include <functional>
 
struct BaseController{};
struct RegExpRouter
{
    static RegExpRouter& getInstance() 
    {
        static RegExpRouter router;
        return router;
    }
 
    bool hasMatch(const std::string& request_uri)
    {
        // проходимся по конфигурации и ищем подходящий маршрут для заданной строки запроса
        // возвращаем true если маршрут найден и false в противном случае
        return true;
    }
    
    std::string getParam(const std::string& name)
    {
        return {};
    }
    
    // Возвращает контроллер который подошел к запросу пользователя
    BaseController* controller(){return nullptr;}
    
    // std::function нацелен на подходящий для обработки action контроллера
    std::function<void()> action(){return {};}
    
private:
    RegExpRouter() = default;
};
 
// таких контроллеров у нас будет много
// один отвечает за видео контент как этот, другой за фото и т.д
struct VideoController : BaseController
{
    // action откуда пользователь тянет данные для просмотра видео
    void actionShow()
    {
        // но какого именно видео?
        // правильно, того который запросил пользователь
        // а как мы узнаем какое именно видео запросил пользователь?
        // конечно через параметры роутера! он же ведь хранит эти данные
        // и как "хорошо" что он синглтон
        // тогда мы можем сделать так:
        RegExpRouter& router = RegExpRouter::getInstance();
        auto id = router.getParam("video_id");
        // отдаем пользователю данные которые он запросил
    }
};
 
struct Application
{
    void run()
    {
        //while (true)
        {
            std::string request_uri;
            std::cin >> request_uri;
 
            RegExpRouter& router = RegExpRouter::getInstance();
            if (router.hasMatch(request_uri)) {
                dispatch(router.controller(), router.action());
            }
            
            // не забываем о создании условий для завершения приложения
        }
    }
    
private:
    void dispatch(BaseController* controller, std::function<void()> action)
    {
        // выполняем необходимые операции перед запуском контроллера и его экшена
        
        action(); // запустили action
    }
};
 
int main()
{
    Application app;
    app.run();
}


Пока все ок. Пользователи нашего фреймворка рады. Спустя некоторое время часть пользователей стала возмущаться по поводу предоставляемого роутера http запросов. Претензии были таковы: регулярные выражения дорого обходятся, а некоторые запросы вообще не нуждаются в параметрах! Добавьте нам более легковесный роутер (без регулярок) но менее гибкий а там мы сами разберемся какой роутер и когда использовать. Ок сказали мы и написали код нового роутера - LiteralRouter, который умеет обрабатывать запросы не так гибко как RegExpRouter.

Но как его интегрировать в наше приложение? В Application ведь имя RegExpRouter сидит хардкодом! Мы решили использовать DI для этих целей (который как позже будет видно уберет нужду в том что бы RegExpRouter был синглтоном).

Кликните здесь для просмотра всего текста
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
struct LiteralRouter
{
    bool hasMatch(const std::string& request_uri)
    {
        // проходимся по конфигурации и ищем подходящий маршрут для заданной строки запроса
        // возвращаем true если маршрут найден и false в противном случае
        return true;
    }
    
    // Возвращает контроллер который подошел к запросу пользователя
    BaseController* controller(){return nullptr;}
    
    // std::function нацелен на подходящий для обработки action контроллера
    std::function<void()> action(){return {};}
};
 
// Application
void run()
{
    //while (true)
    {
        std::string request_uri;
        std::cin >> request_uri;
 
        // теперь роутер это поле класса Application
        // и пользователь может внедрять любой роутер в класс Application
        if (router.hasMatch(request_uri)) {
            // перед вызовом контроллера внедрим роутер в базовый контроллер
            // ОДНИМ МАХОМ МЫ ВНЕДРИЛИ РОУТЕР ВО ВСЕ ВОЗМОЖНЫЕ КОНТРОЛЛЕРЫ ПОЛЬЗОВАТЕЛЯ ФРЕЙМВОРКА!!!
            // и заметь, без особых проблем. Поменяв несколько строчек.
            router.controller()->setRouter(router);
            dispatch(router.controller(), router.action());
        }
        
        // не забываем о создании условий для завершения приложения
    }
} 
 
// базовй контроллер в соответствии с правками выше теперь выглядит вот так
// так как все контроллеры унаследованы от него - они автоматически получают доступ к TRouter
// который может быть самым разным
// теперь нет необходимости роутерам быть синглтонами - контроллеры получат доступ к роутеру через поле базового класса
// которое хранит объект текущего роутера выбранного пользователем фреймворка
// иначе говоря, объект роутера во все контроллеры мы доставили поменяв несколько строк кода
// а раз объект роутера уже внедрен в контроллер, то зачем ему быть синглтоном?
struct BaseController
{
    template<typename TRouter>
    void setRouter(const TRouter&){}
};

как видно сначала синглтон нам очень даже подходил
но позже с появлением DI смысла в синглтоне в контексте решаемой задачи не осталось
метод getInstance() у роутеров можно оставить разве что для обратной совместимости с пользователями первой версии данного фреймворка. и не нужно говорить что это ошибка архитектора потому что суть роутера как раз полностью соответствует синглтону.

итог: синглтон был применен к месту
а с приходом Di необходимость в нем разве что в обратной совместимости (что по сути является лишь грузом в проекте)
1
 Аватар для avgoor
1550 / 877 / 179
Регистрация: 05.12.2015
Сообщений: 2,555
20.02.2020, 18:00
Цитата Сообщение от Undisputed Посмотреть сообщение
как например в Qt QApplication
А QApplication - это что? И зачем ему статический метод Instance()? Может это вас наведет на какую-то мысль.
1
901 / 478 / 93
Регистрация: 10.06.2014
Сообщений: 2,700
20.02.2020, 18:17
avgoor,
Где вы увидели getInstance в qapplication? У меня в коде такого типа вообще нет)
Не понимаю на какую мысль вы пытались меня навести указывая на то, чего нет
0
 Аватар для avgoor
1550 / 877 / 179
Регистрация: 05.12.2015
Сообщений: 2,555
20.02.2020, 18:27
Цитата Сообщение от Undisputed Посмотреть сообщение
getInstance в qapplication?
getInstance не увидел. А instance() он наследует от QCoreApplication. В общем, как и писал hoggy, вы не понимаете, что такое синглтон и когда его использовать.
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
20.02.2020, 18:27
Помогаю со студенческими работами здесь

Стиль написания кода. Отступы
Что лучше: проблелы или табуляция? Ибо поначалу я слышал и был уверен что лучше использвать табуляцию. Но сейчас используя sublime text c...

Оцените на правильность кода и общий стиль написания
Приветствую всех :) В кои-то веки решил начать изучать Java, наткнулся на некий курс, в котором подробно разбирается MVC, а параллельно...

Синглтон вызывает синглтон
Привет. Что-то запутался. Есть Синглтон(надеюсь я его правильно сделал): public class Table extends JTable { private static...

Правила написания кода
Всем привет!Читала о правильном написании кода, отступы, что был &quot;елочкой&quot;, но до меня так и не дошло, как это на практике делается. Буду...

Правила написания кода
Я вот до сих пор не понимаю, как происходит такая вещь. Допустим: у меня есть Главная страница сайта &lt;b&gt;index.php&lt;/b&gt; и есть...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
40
Ответ Создать тему
Новые блоги и статьи
Асинхронный приём данных из COM-порта
Argus19 01.05.2026
Асинхронный приём данных из COM-порта Купил на aliexpress термопринтер QR701. Он оказался странным. Поключил к Arduino Nano. Был очень удивлён. Наотрез отказывается печатать русские буквы. Чтобы. . .
попытка написать игровой сервер на C++
pyirrlicht 29.04.2026
попытка написать игровой сервер на плюсах с открытым бесконечным миром. возможно получится прикрутить интерпретатор питон для кастомизации игровой логики. что есть на текущий момент:. . .
Контроль уникальности выбранного документа-основания при изменении реквизита
Maks 28.04.2026
Алгоритм из решения ниже разработан на примере нетипового документа "ЗаявкаНаРемонтСпецтехники", разработанного в КА2. Задача: уведомлять пользователя, если указанная заявка (документ-основание). . .
Благородство как наказание
Maks 24.04.2026
У хорошего человека отношения с женщинами всегда складываются трудно. А я человек хороший. Заявляю без тени смущения, потому что гордиться тут нечем. От хорошего человека ждут соответствующего. . .
Валидация и контроль данных табличной части документа перед записью
Maks 22.04.2026
Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в КА2. Задача: контроль и валидация данных табличной части документа перед записью с учетом регламента компании. . .
Отчёт о затраченных материалах за определенный период с макетом печатной формы
Maks 21.04.2026
Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .
Отчёт о спецтехнике находящейся в ремонте
Maks 20.04.2026
Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .
Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)"
Hrethgir 19.04.2026
Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru