@gridem17 окт 2017 в 11:06

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

9 мин

13K

C++ * Ненормальное программирование *

+23

Комментарии 10

@mayorovp 17 окт 2017 в 11:31

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

@gridem 17 окт 2017 в 12:45

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

@eao197 17 окт 2017 в 16:14

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

Lock _{lock_};

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

@gridem 17 окт 2017 в 16:24

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

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

@eao197 17 окт 2017 в 16:35

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

@gridem 17 окт 2017 в 18:19

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

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

@Antervis 18 окт 2017 в 06:23

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

@gridem 18 окт 2017 в 07:26

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

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

НЛО прилетело и опубликовало эту надпись здесь

@gridem 23 окт 2017 в 07:49

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

Зарегистрируйтесь на Хабре, чтобы оставить комментарий