Підключення перевантажених сигналів та слотів у Qt 5

У мене виникають проблеми з утисканням нового синтаксису сигнал / слот (використовуючи функцію вказівника на член) у Qt 5, як описано в Syntax New Signat Slot . Я спробував змінити це:

QObject::connect(spinBox, SIGNAL(valueChanged(int)),
                 slider, SLOT(setValue(int));

до цього:

QObject::connect(spinBox, &QSpinBox::valueChanged,
                 slider, &QSlider::setValue);

але я отримую помилку, коли намагаюся її скласти:

помилка: немає функції узгодження для дзвінка на QObject::connect(QSpinBox*&, <unresolved overloaded function type>, QSlider*&, void (QAbstractSlider::*)(int))

Я намагався з clang та gcc на Linux, як з -std=c++11.

Що я роблю неправильно, і як це можна виправити?

Проблема тут полягає в тому, що є два сигнали з таким ім'ям: QSpinBox::valueChanged(int)і QSpinBox::valueChanged(QString). З Qt 5.7 передбачені допоміжні функції для вибору потрібного перевантаження, щоб ви могли писати

connect(spinbox, qOverload<int>(&QSpinBox::valueChanged),
        slider, &QSlider::setValue);

Для Qt 5.6 і новіших версій вам потрібно сказати Qt, який ви бажаєте вибрати, переклавши його на потрібний тип:

connect(spinbox, static_cast<void (QSpinBox::*)(int)>(&QSpinBox::valueChanged),
        slider, &QSlider::setValue);

Я знаю, це некрасиво . Але обходу цього немає. Сьогоднішній урок такий: не перевантажуйте свої сигнали та слоти!

Додаток : те, що насправді дратує в ролях, це те

1. один повторює назву класу двічі
2. треба вказати повернене значення, навіть якщо воно зазвичай void(для сигналів).

Тож я іноді опиняюся за допомогою цього фрагмента C ++ 11:

template<typename... Args> struct SELECT { 
    template<typename C, typename R> 
    static constexpr auto OVERLOAD_OF( R (C::*pmf)(Args...) ) -> decltype(pmf) { 
        return pmf;
    } 
};

Використання:

connect(spinbox, SELECT<int>::OVERLOAD_OF(&QSpinBox::valueChanged), ...)

Я особисто вважаю це не дуже корисним. Я очікую, що ця проблема сама собою відірветься, коли Creator (або ваш IDE) автоматично вставить правильний склад при автоматичному завершенні операції з прийняття PMF. Але тим часом ...

Примітка: синтаксис з'єднання на основі PMF не вимагає C ++ 11 !

Додаток 2 : у Qt 5.7 для доповнення цього було додано допоміжні функції, зразки мого вирішення вище. Основний помічник - це qOverload(ви також отримали qConstOverloadі qNonConstOverload).

Приклад використання (з документів):

struct Foo {
    void overloadedFunction();
    void overloadedFunction(int, QString);
};

// requires C++14
qOverload<>(&Foo:overloadedFunction)
qOverload<int, QString>(&Foo:overloadedFunction)

// same, with C++11
QOverload<>::of(&Foo:overloadedFunction)
QOverload<int, QString>::of(&Foo:overloadedFunction)

Додаток 3 : якщо ви подивитесь на документацію будь-якого перевантаженого сигналу, тепер вирішення проблеми з перевантаженням чітко зазначено в самих документах. Наприклад, https://doc.qt.io/qt-5/qspinbox.html#valueChanged-1 каже

Примітка: Значення сигналу Змінено перевантажено в цьому класі. Для підключення до цього сигналу за допомогою синтаксису вказівника функції Qt надає зручний помічник для отримання вказівника функції, як показано в цьому прикладі:

   connect(spinBox, QOverload<const QString &>::of(&QSpinBox::valueChanged),
[=](const QString &text){ /* ... */ });

Повідомлення про помилку:

помилка: немає функції узгодження для дзвінка на QObject::connect(QSpinBox*&, <unresolved overloaded function type>, QSlider*&, void (QAbstractSlider::*)(int))

Важливою частиною цього є згадка про " невирішений перевантажений тип функції ". Компілятор не знає, ви маєте на увазі QSpinBox::valueChanged(int)чи QSpinBox::valueChanged(QString).

Існує кілька способів вирішити перевантаження:

• Надайте відповідний параметр шаблону для connect()

  QObject::connect<void(QSpinBox::*)(int)>(spinBox, &QSpinBox::valueChanged,
                                           slider,  &QSlider::setValue);

  Це змушує connect()вирішити &QSpinBox::valueChangedперевантаження, яке приймає int.

  Якщо у вас є невирішені перевантаження для аргументу слота, вам потрібно буде надати другий аргумент шаблону connect(). На жаль, не існує синтаксису, який би просив зробити висновок першого, тому вам потрібно буде подати обидва. Ось тоді другий підхід може допомогти:

• Використовуйте тимчасову змінну правильного типу

  void(QSpinBox::*signal)(int) = &QSpinBox::valueChanged;
  QObject::connect(spinBox, signal,
                   slider,  &QSlider::setValue);

  Завдання на signalвибір вибере бажану перевантаження, і тепер її можна успішно замінити в шаблон. Це однаково добре працює з аргументом "слот", і я вважаю його менш громіздким у такому випадку.

• Використовуйте конверсію

  Тут ми можемо уникати static_cast, оскільки це просто примус, а не зняття захистів мови. Я використовую щось на кшталт:

  // Also useful for making the second and
  // third arguments of ?: operator agree.
  template<typename T, typename U> T&& coerce(U&& u) { return u; }

  Це дозволяє нам писати

  QObject::connect(spinBox, coerce<void(QSpinBox::*)(int)>(&QSpinBox::valueChanged),
                   slider, &QSlider::setValue);

Насправді ви можете просто обернути свій слот лямбда, і це:

connect(spinbox, static_cast<void (QSpinBox::*)(int)>(&QSpinBox::valueChanged),
    slider, &QSlider::setValue);

буде виглядати краще. : \

Вирішення вище працює, але я вирішив це дещо по-іншому, використовуючи макрос, Тож про всяк випадок тут це:

#define CONNECTCAST(OBJECT,TYPE,FUNC) static_cast<void(OBJECT::*)(TYPE)>(&OBJECT::FUNC)

Додайте це у свій код.

Тоді, ваш приклад:

QObject::connect(spinBox, &QSpinBox::valueChanged,
             slider, &QSlider::setValue);

Стає:

QObject::connect(spinBox, CONNECTCAST(QSpinBox, double, valueChanged),
             slider, &QSlider::setValue);