Комментарии / Профиль azrielfuzz

29.10.2018 09:00:55

Mon, 29 Oct 2018 09:00:55 GMT

Признаю)

05.10.2018 09:55:45

Fri, 05 Oct 2018 09:55:45 GMT

Все объекты сравниваются по значению, если это возможно; в противном случае происходит сравнение по адресу. Конкретно лямбды сравниваются по адресу.

03.10.2018 08:14:48

Wed, 03 Oct 2018 08:14:48 GMT

Наоборот, сможете. И объект-функтор, и лямбда-выражение, и экземпляр std::function, и отдельную функцию (ну, и метод класса/структуры, конечно).

02.10.2018 10:04:45

Tue, 02 Oct 2018 10:04:45 GMT

Этому был посвящён целый пункт.
Но если вкратце: функторы сравниваются по адресу объекта, методы (функции-члены) — по адресу экземпляра класса (которому они принадлежат) и своему адресу.
Естественно, сравниваются только однотипные объекты: как функторы, так и методы классов. Разнотипные обработчики считаются неравными.

01.10.2018 09:01:54

Mon, 01 Oct 2018 09:01:54 GMT

Большое спасибо за разъяснение.
Явно есть, куда двигаться дальше.

28.09.2018 15:41:43

Fri, 28 Sep 2018 15:41:43 GMT

Это действительно решило проблему сборку (хоть предупреждение сборщика и осталось).
Спасибо; недостаток убираю.

28.09.2018 15:33:41

Fri, 28 Sep 2018 15:33:41 GMT

Такие вещи лучше прятать в cpp файлы.

А как это сделать в случае с шаблонами? Вы можете привести небольшой пример?

28.09.2018 15:28:47

Fri, 28 Sep 2018 15:28:47 GMT

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

Ну и нет нет никаких гарантий с точки зрения исключений в обработчиках

А вот тут не совсем понятно, почему событие должно обрабатывать исключения в клиентском коде (если можно так назвать обработчики).

28.09.2018 09:32:51

Fri, 28 Sep 2018 09:32:51 GMT

Не очень понятно, при чём здесь C# (которым я, кстати, во многом вдохновлялся), если в нём, в отличие от C++, данный механизм встроен в язык. Насчёт «ради Qt» тоже не очень понятно, потому что данная реализация позволяет как раз не использовать сторонние библиотеки и фреймворки (по крайней мере, только ради сигналов).

28.09.2018 09:27:15

Fri, 28 Sep 2018 09:27:15 GMT

А какая у Вас версия Qt? Перепроверил — всё подтвердилось. Я сейчас использую Qt5.6.1-1 (надо бы, конечно, обновиться...). Компилятор — VS++14 (соответственно, и MSBuild в качестве сборщика).
Ошибка на этапе сборке.
При описанном в cpp-файле классе с директивой Q_OBJECT выдаётся предупреждение

Warning MSB8017 A circular dependency has been detected while executing custom build commands for item "GeneratedFiles\Debug\filename.moc". This may cause incremental build to work incorrectly.

и все методы, которые должны быть сгенерированы moc'ом (metaObject, qt_metacast и т.д.), становятся unresolved, вызывая соответствующие ошибки. Кроме того в Generated Files, где под каждую конфигурацию должен присутствовать сгенерированный файлик moc_filename.cpp, находится только несуществующий filename.moc.
Возможно, дело именно в MSBuild'е или его взаимодействии с Qt. Если это и вправду так, нужно будет действительно убрать этот пункт из недостатков.

28.09.2018 08:57:15

Fri, 28 Sep 2018 08:57:15 GMT

Спасибо за поддержку совместимости с GCC!

27.09.2018 14:59:33

Thu, 27 Sep 2018 14:59:33 GMT

C++11 это вы преуменьшаете :) std::shared_mutex ----> C++17.

Действительно, размахнулся что-то.

Не собралось под GCC из-за некоторого несоответствия стандарту

Это всё моя привычка разработки под VC. Надо будет поисправлять.

У меня впечатление, что shared_ptr для Holder не обязателен и я бы попытался заменить его на unique_ptr в списке обработчиков + сырой readonly указатель во всех остальных местах.

Немного не понял. В списке обработчиков как раз не Holder, а EventHandler, в котором уже Holder. Вы всё-таки при EventHandler?

лучше использовать std::make_shared

там, где применяется не он, используется private конструктор (в Holder'ах, например, это сделано для правильной инициализации поля m_me).

Некоторые внутренние типы лучше спрятать из публичного интерфейса

так, вроде убраны же в анонимные namespace'ы.
А вообще, за замечания спасибо)

27.09.2018 14:48:39

Thu, 27 Sep 2018 14:48:39 GMT

А у Вас класс, описанный в .cpp, имеет директиву Q_OBJECT (и соответственно, её применение в виде, например, сигналов)?

27.09.2018 10:21:56

Thu, 27 Sep 2018 10:21:56 GMT

В контексте GUI ожидается граф сцены, в ивенте содержатся координаты мыши, а ивент проходится по графу сцены.

Как я уже сказал, я не считаю этот подход равнозначным. Как Вы верно заметили, даже в Qt реализованы они оба (и The Event System, и Signals & Slots). Кроме того, use case'ов много (не только лишь GUI), не везде удобно одно и то же.

Тогда очевидный вопрос зачем? Когда есть множество реализаций тех же сингалов?

Согласен. Как минимум, когда не хочется/можется подключать библиотеку с очередной реализаций сигналов. Возможно, кому-то будет удобно и полезно. Плюс, небольшая демонстрация разных возможностей С++ (но это тоже скорее доп).

27.09.2018 09:52:50

Thu, 27 Sep 2018 09:52:50 GMT

Так, вроде бы, там написано, зачем. Вы с чем-то из этого не согласны?

27.09.2018 09:47:32

Thu, 27 Sep 2018 09:47:32 GMT

В Qt есть The Event System и Signals & Slots.

Данная реализация как раз представляет собой нечто близкое к 'Signals & Slots', а не к 'The Event System'; последнее ближе к очереди (циклу) событий, там объекты взаимодействуют через цикл событий, а не напрямую. На мой взгляд, это всё же другой подход.

Почему для сравнения берутся сигналы из Qt, а не сигналы из Boost?

Вообще не было цели сравнивать хоть с чем-то. Это, скорее, просто бонус, вызванный тем, что с сигналами/слотами Qt мне приходилось работать (а с сигналами из boost нет).

17.01.2017 13:53:24

Tue, 17 Jan 2017 13:53:24 GMT

если честно, не понял, зачем там void().
разве для составления unused (который по сути можно объявить как bool unused[] =) не достаточно будет только параметра true?

17.01.2017 11:37:10

Tue, 17 Jan 2017 11:37:10 GMT

Ну и, я думаю, стоило бы упомянуть C++ 17 fold expressions

Спасибо за ликбез, я как-то пропустил сию замечательную вещь.

17.01.2017 11:36:20

Tue, 17 Jan 2017 11:36:20 GMT

И лучше всё-таки использовать универсальные ссылки и forward, а не const &

Соглашусь. Весьма небрежно с моей стороны.