Evgenii Legotckoi
26 квітня 2018 р. 16:39

Підручник Qt/C++ 078. Не змішуйте старий синтаксис сигналів і слотів у макросах SIGNAL SLOT з новим синтаксисом вказівників

Всі ми знаємо, що в Qt існує два синтаксиси сигналів та слотів:

Але також, як не варто заважати пиво з горілкою, з таким самим успіхом не варто змішувати два синтаксиси в рамках одного проекту.

Звичайно, існують проекти, в яких написано досить багато коду і зробити повний рефакторинг по зміні всього синтаксису є досить накладною роботою, але якщо ви поступово доповнюючи проект переходьте на новий синтаксис, постарайтеся в рамках одного сигналу слотового з'єднання оновити всі місця коду з даними з'єднанням новий синтаксис.

Справа в тому, що для обох випадків ( SINGAL SLOT макроси та синтаксис на покажчиках ) формується інший вміст moc файлів , що призводить до того, що змішане використання методів connect та disconnect не працює так, як очікувалося б. А якщо бути точним, то метод disconnect не працюватиме в тому випадку, якщо connect був викликаний з використанням макросів, а disconnect був викликаний з використанням покажчиків.


Приклад

Створимо пробний проект, у якому буде вікно та одна кнопка. Додамо у вікні слот. І в конструкторі класу вікна перевіримо чотири комбінації підключення сигналу кнопки до слота вікна:

  1. підключити SIGNAL SLOT - відключити SIGNAL SLOT
  2. connect SIGNAL SLOT - disconnect синтаксис на вказівниках
  3. connect на покажчиках - disconnect на покажчиках
  4. connect на покажчиках – disconnect SIGNAL SLOT

віджет.h

#ifndef WIDGET_H
#define WIDGET_H

#include <QWidget>

namespace Ui {
class Widget;
}

class Widget : public QWidget
{
    Q_OBJECT

public:
    explicit Widget(QWidget *parent = 0);
    ~Widget();

public slots:
    void checkSlot() {} // Slot for checking

private:
    Ui::Widget *ui;
};

#endif // WIDGET_H

widget.cpp

#include "widget.h"
#include "ui_widget.h"

#include <QDebug>

Widget::Widget(QWidget *parent) :
    QWidget(parent),
    ui(new Ui::Widget)
{
    ui->setupUi(this);

    qDebug() << "First test";
    qDebug() << connect(ui->pushButton, SIGNAL(clicked(bool)), this, SLOT(checkSlot()));
    qDebug() << disconnect(ui->pushButton, SIGNAL(clicked(bool)), this, SLOT(checkSlot()));

    qDebug() << "Second test";
    qDebug() << connect(ui->pushButton, SIGNAL(clicked(bool)), this, SLOT(checkSlot()));
    qDebug() << disconnect(ui->pushButton, &QPushButton::clicked, this, &Widget::checkSlot);

    qDebug() << "Third test";
    qDebug() << connect(ui->pushButton,&QPushButton::clicked, this, &Widget::checkSlot);
    qDebug() << disconnect(ui->pushButton, &QPushButton::clicked, this, &Widget::checkSlot);

    qDebug() << "Fourth test";
    qDebug() << connect(ui->pushButton,&QPushButton::clicked, this, &Widget::checkSlot);
    qDebug() << disconnect(ui->pushButton, SIGNAL(clicked(bool)), this, SLOT(checkSlot()));
}

Widget::~Widget()
{
    delete ui;
}

Висновок

First test
true
true
Second test
true
false
Third test
true
true
Fourth test
true
true

Таким чином, виходить, що пара connect SINGAL SLOT - disconnect на покажчиках дає не той результат, який очікувався. І за фактом слот залишається підключеним.

Тому новачкам рекомендую уважно ставитися до цього нюансу роботи з сигналами та слотами Qt.

Вам це подобається? Поділіться в соціальних мережах!

c
  • 03 травня 2018 р. 05:13

Who hasn’t mixed beer with vodka! 🤢🤮

Evgenii Legotckoi
  • 03 травня 2018 р. 13:38

vodka + beer = strong coctail, but it is bad ))

Юрий
  • 04 червня 2018 р. 22:41

disconnect - обязательно делать, во всех случаях?

Evgenii Legotckoi
  • 05 червня 2018 р. 02:03

Если удаляете объект, то все коннекты отключаются автоматически, здесь следить не требуется.

В остальных случаях disconnect нужно делать тогда, когда Вам нужно отключить сигнал-слотовое соединение в соответствии с логикой вашей программы, то есть на ваше усмотрение.

Коментарі

Only authorized users can post comments.
Please, Log in or Sign up