gridem Oct 17 2017 at 14:06

Универсальный адаптер

9 min

13K

C++*Abnormal programming*

+26

Comments 10

mayorovp Oct 17 2017 at 14:31

Метод call лучше вызывать не как this->call а как T_base::call. Иначе получится сюрприз при попытке сделать адаптер для типа, который содержит другой метод call.

gridem Oct 17 2017 at 15:45

Действительно, так лучше. Поправил.

eao197 Oct 17 2017 at 19:14

В разделе №4.2 в коде примера:

Lock _{lock_};

откуда берется имя Lock?

gridem Oct 17 2017 at 19:24

Там выше есть определение:

using Lock = std::unique_lock<std::mutex>;

eao197 Oct 17 2017 at 19:35

Но тогда Lock получается заточен только под std::mutex в качестве типа T_locker.

gridem Oct 17 2017 at 21:19

Спасибо, поправил в тексте. В оригинальном коде все ок:

https://github.com/gridem/GodAdapter/blob/master/include/god_adapter/shared.h#L32

Antervis Oct 18 2017 at 09:23

а зачем вообще делать type alias на Lock? Можно же положиться на c++17 deduction guide и просто писать std::lock_guard _{_mutex};?

gridem Oct 18 2017 at 10:26

Я это писал еще в 2015 году. Тогда еще не наступил с++17. К тому же, не все до сих пор перешли на с++17.

Но так можно сделать, да.

UFO landed and left these words here

gridem Oct 23 2017 at 10:49

Обычно обобщенная базовая функциональность покрывается изрядным количеством тестов, чтобы потом не было мучительно больно.