В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

supapro.cxx

2959 membersпожаловаться на группу
2020 August 19

L

Lee Armstrong in supapro.cxx
Побитый Кирпич
сервис локатор гавно, потому что он скрывает зависимости класса
Говорю же, вы просто не умеете его готовить.

Я его использую вот так.
Есть фарбика, которая тащит зависимости из сервис локатора и конструирует класс. И тестируемость класса осталась, и никаких неявных зависимостей.
На фрагменте кода макрос, который пишется где-нибудь в cpp файле, который регистриует фабрику (лямбду) в синглтоне и потом в другом месте можно по typeid сконструировать.
https://pastebin.com/3gU7LXj8
Pastebin
CORE_DEFINE_SERVICE(SpriteManager,                    "service::SpriteManager" - Pastebin.com
Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.
источник
18:53пожаловаться#1

D

Danya in supapro.cxx
Побитый Кирпич
Если это так то это тупо. Если функция принимает концепт Foo, но зовёт внутри метод bar, то на самом деле она принимает концепт FooAndBar
https://godbolt.org/z/4Es1cW
godbolt.org
Compiler Explorer - C++ (x86-64 gcc 10.2)
struct FooAndBar {
   void foo() {}
   void bar() {}
};

template <typename T>
concept foo = requires(T o) {
           { o.foo() };
};

template <foo T>
void takes_foo_calls_bar(T o) {
   o.bar();
}

int main() {
   FooAndBar o;
   takes_foo_calls_bar(o);
}
источник
18:54пожаловаться#2

ПК

Побитый Кирпич... in supapro.cxx
Danya
https://godbolt.org/z/4Es1cW
godbolt.org
Compiler Explorer - C++ (x86-64 gcc 10.2)
struct FooAndBar {
   void foo() {}
   void bar() {}
};

template <typename T>
concept foo = requires(T o) {
           { o.foo() };
};

template <foo T>
void takes_foo_calls_bar(T o) {
   o.bar();
}

int main() {
   FooAndBar o;
   takes_foo_calls_bar(o);
}
ну всё, концепты гавно значит
источник
18:55пожаловаться#3

D

Danya in supapro.cxx
Lee Armstrong
Говорю же, вы просто не умеете его готовить.

Я его использую вот так.
Есть фарбика, которая тащит зависимости из сервис локатора и конструирует класс. И тестируемость класса осталась, и никаких неявных зависимостей.
На фрагменте кода макрос, который пишется где-нибудь в cpp файле, который регистриует фабрику (лямбду) в синглтоне и потом в другом месте можно по typeid сконструировать.
https://pastebin.com/3gU7LXj8
Pastebin
CORE_DEFINE_SERVICE(SpriteManager,                    "service::SpriteManager" - Pastebin.com
Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.
Ну это какая-то смесь СервисЛокатора и DI контейнера
источник
18:55пожаловаться#4

ПК

Побитый Кирпич... in supapro.cxx
Мб конечно тут отмазка что это хрен провалидируешь в С++
источник
18:55пожаловаться#5

L

Lee Armstrong in supapro.cxx
Danya
Ну это какая-то смесь СервисЛокатора и DI контейнера
Не важно что это. Это разбивает ваш главный аргумент почему локатор говно.
источник
18:56пожаловаться#6

L

Lee Armstrong in supapro.cxx
Выходит что локатор все таки хорош?
источник
18:57пожаловаться#7

D

Danya in supapro.cxx
Lee Armstrong
Выходит что локатор все таки хорош?
Так это не локатор, а смесь DI контейнера и СервисЛокатора :)
источник
18:57пожаловаться#8

D

Danya in supapro.cxx
Так конечно лучше, чем сервис локатор чистый
источник
18:58пожаловаться#9

ПК

Побитый Кирпич... in supapro.cxx
Lee Armstrong
Выходит что локатор все таки хорош?
Нет, потому что у тебя всё ещё неясный интерфейс класса
источник
18:58пожаловаться#10

L

Lee Armstrong in supapro.cxx
Побитый Кирпич
Нет, потому что у тебя всё ещё неясный интерфейс класса
Почему?
источник
18:59пожаловаться#11

D

Danya in supapro.cxx
Всё равно непонятно почему надо делать Impl класс и не сделать просто SpriteManager
источник
18:59пожаловаться#12

ПК

Побитый Кирпич... in supapro.cxx
Lee Armstrong
Почему?
Класс, который принимает в конструкторе ServiceLocator это то же самое как класс принимающий в конструкторе std::any, но при этом если передашь ему не тот тип в std::any, то будет ошибка
источник
18:59пожаловаться#13

L

Lee Armstrong in supapro.cxx
Побитый Кирпич
Класс, который принимает в конструкторе ServiceLocator это то же самое как класс принимающий в конструкторе std::any, но при этом если передашь ему не тот тип в std::any, то будет ошибка
Мне кажется ты прохо прочитал что я скинул. У меня мои классы не принимают сервис локатор. Принимает фабрика. А если нужной зависимости не окажется. То этот случай обработается и до конструирования класса дело вообще не дойдет.
источник
19:01пожаловаться#14

L

Lee Armstrong in supapro.cxx
И вот Конструктор класса. Никакой неявности. 
SpriteManagerImpl(std::shared_ptr<core::Logger>,
                    std::shared_ptr<service::Time>);
источник
19:02пожаловаться#15

L

Lee Armstrong in supapro.cxx
Так что сервис локатор хорош.
источник
19:03пожаловаться#16

D

Danya in supapro.cxx
Так зачем нужен Impl класс?
источник
19:03пожаловаться#17

D

Danya in supapro.cxx
Это DI контейнер, названный СервисЛокатором
источник
19:03пожаловаться#18

D

Danya in supapro.cxx
Надо называть вещи своими именами
источник
19:03пожаловаться#19

L

Lee Armstrong in supapro.cxx
Danya
Так зачем нужен Impl класс?
Влом исправлять, не более
источник
19:04пожаловаться#20