【问题标题】: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

【问题讨论】:

    标签: c++ qt


    【解决方案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 中使用deleteCounter*,这会导致另一个内存泄漏,因此请在您的MainWindow 析构函数中添加delete cobj

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

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

    变成:

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

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

    【讨论】:

    • 哎呀。我是多么愚蠢!谢谢 SilvanoCerza,你有鹰的眼睛!
    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2018-03-01
    • 1970-01-01
    • 1970-01-01
    • 2011-01-10
    • 2012-05-31
    相关资源
    最近更新 更多