信号没有被其他类捕获

Posted 2023-02-17

【中文标题】信号没有被其他类捕获

【英文标题】：Signals not getting caught in another class

【发布时间】：2019-06-25 07:21:29

【问题描述】：

我正在使用一个自定义类，该类在固定间隔后发出信号。该信号发射良好，因为我已将其与同一类中的插槽连接，并使用插槽中的 qdebug 语句进行了验证。问题是当我尝试在主窗口类中连接相同的信号时，没有调用插槽。 这是我的代码：

MainWindow.h

#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include <QMainWindow>
#include "counter.h"
#include <QTextEdit>

namespace Ui 
class MainWindow;


class MainWindow : public QMainWindow

    Q_OBJECT

public:
    explicit MainWindow(QWidget *parent = nullptr);
    ~MainWindow();

public slots:
    void on_pushButton_clicked();
    void updateText(int);
    void test(int);
    void anotherSlot();

private:
    Ui::MainWindow *ui;
    Counter *cobj;
    int v;
    QTextEdit *te;
;
#endif // MAINWINDOW_H

主窗口.cpp

#include "mainwindow.h"
#include "ui_mainwindow.h"
#include <QDebug>
#include <QThread>
#include "counter.h"

MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)

    ui->setupUi(this);
    cobj = new Counter(this);

    v = 0;
    te = ui->textEdit;
    bool success1 = connect(cobj, SIGNAL (mileStoneReached(int)), this, SLOT(updateText(int)),Qt::AutoConnection);
    bool success2 = connect(cobj,SIGNAL (mileStoneReached(int)), this,SLOT(test(int)),Qt::AutoConnection);
    bool success3 = connect(cobj,SIGNAL (anotherSignal()), this,SLOT (anotherSlot()),Qt::AutoConnection);
    qDebug() << success1 << "   " << success2 << "  " << success3;


MainWindow::~MainWindow()

    delete ui;


void MainWindow::on_pushButton_clicked()

    cobj = new Counter();

    te->setText("0");
    qDebug() <<QThread::currentThreadId();
    cobj->run();


void MainWindow::updateText(int x)

    qDebug() << Q_FUNC_INFO;
    v = (v+1) * 13;
    te->setText(QString("%1").arg(v));


void MainWindow::test( int x)

    qDebug() << Q_FUNC_INFO;
    qDebug() <<"___" ;


void MainWindow::anotherSlot()

    qDebug() << Q_FUNC_INFO;
    qDebug() <<"__######_" ;

计数器.h

#ifndef COUNTER_H
#define COUNTER_H
#include <QTimer>

class Counter : public QObject

    Q_OBJECT
public:
    Counter(QObject *parent= nullptr);

    void run();
    void reset();
    void init();

signals:
    void mileStoneReached(int x);
    void anotherSignal();

public slots:
    void increment();
    void test();

private:
    int cValue;
    QTimer *timer;
;
#endif // COUNTER_H

计数器.cpp

#include "counter.h"
#include<QDebug>

Counter::Counter(QObject*parent):QObject(parent)

    cValue= 0;
    init();


void Counter:: init()

    timer=new QTimer;
    connect(timer, SIGNAL(timeout()), this, SLOT(increment()),Qt::AutoConnection);
    connect(this,SIGNAL(mileStoneReached(int)), this,SLOT(test()),Qt::AutoConnection);


void Counter::reset()

    cValue=0;


void Counter::run()

    timer->start(100);


void Counter::increment()

    cValue++;
    if(cValue % 13 ==0)
    
        qDebug() << cValue;
        emit mileStoneReached( cValue);
    
    else
    
        emit (anotherSignal());
    


void Counter::test()

    qDebug() << "Signal caught";

输出：

true     true    true // all connections are fine
0x21ac // Main thread Id  irrelevant to problem
13 // counter value
Signal caught //qdebug  print
26
Signal caught
39
Signal caught
52
Signal caught
65
Signal caught
78
Signal caught
91
Signal caught
104
Signal caught
117
Signal caught

【参考方案1】：

在MainWindow 构造函数中，您实例化一个Counter* 并将其分配给cobj，但在MainWindow::on_pushButton_clicked 中您创建另一个Counter* 并再次将其分配给cobj，然后运行您的计数器，这将创建几个问题。

在MainWindow::on_pushButton_clicked 中，当您创建新的Counter* 时，您会发生内存泄漏，因为您从未在前一个实例上使用delete，您也可以在新实例上调用run，但已使用旧实例创建了连接它是在MainWindow 构造函数中创建的，因此永远不会调用连接的方法。

一个可能的解决方案是只使用 MainWindow 构造函数中的实例创建。

void MainWindow::on_pushButton_clicked()

    te->setText("0");
    qDebug() <<QThread::currentThreadId();

    cobj->run();

您也永远不会在MainWindow 中使用delete 和Counter*，这会导致另一个内存泄漏，因此请在您的MainWindow 析构函数中添加delete cobj。

我建议也通过使用函数指针来更改各种连接以使用新语法。

connect(cobj, SIGNAL (mileStoneReached(int)), this, SLOT(updateText(int)),Qt::AutoConnection);

变成：

connect(cobj, &Counter::mileStoneReached, this, &MainWindow::updateText,Qt::AutoConnection);

这更安全，因为一些检查是在编译时完成的，所以例如，如果你在函数名中打错字，它根本不会编译。

【讨论】：

哎呀。我是多么愚蠢！谢谢 SilvanoCerza，你有鹰的眼睛！