QT 信号槽不工作

Question

为什么这段代码没有调用所需的槽？

我试图让它在过去 3 小时内正常工作！

void TForm::on_clicked( bool checked )
{
    QMessageBox *messageBox = new QMessageBox(this);
    QPushButton *buttonAccept0 = new QPushButton("OK", messageBox);
    QPushButton *buttonReject = new QPushButton("Cancel", messageBox);
    messageBox->addButton(buttonAccept0, QMessageBox::ButtonRole::AcceptRole);
    messageBox->addButton(buttonReject, QMessageBox::ButtonRole::NoRole);
    messageBox->setWindowTitle("Confirmation");
    messageBox->setText("Remove?");
    messageBox->setModal(true);
    messageBox->show(); 
    QObject::connect(buttonAccept0, SIGNAL(accepted()), this, SLOT(a1()));
}
void TForm::a1()
{
    std::string ff = "";
}

Answer 1

您的问题有三点值得注意：

---

首先，您的错误是由于您使用了不正确的信号（正如其他成员已经提到的那样）。

要更正您的错误，请对您的代码进行以下更改：

QObject::connect(buttonAccept0, SIGNAL(clicked()), this, SLOT(at()));

---

其次，与他/她的回答中提到的@M74 不同，完全允许在on_clicked 函数内声明消息框和信号。 @M74 担心的原因是因为潜在的内存泄漏，但 Qt 有一种机制可以以父母的形式克服这一点，当您在函数开头定义消息框时，您正确地做到了这一点。

不幸的是，@M74 的担忧并非完全没有根据，因为消息框只会在父级被销毁时才会被销毁。

如果您的TForm 实例在整个应用程序期间都保持活动状态，并且您多次执行on_clicked，您将创建许多messageBox 实例并在后台等待被删除，这是一种内存泄漏本身。

避免这种情况的更好方法是修改代码以使messageBox 成为函数本地的堆栈变量。这两个QPushButton 实例仍然可以在堆内存中创建，因为它们会在消息框被销毁时自动销毁，这也是由于父级关系。

void TForm::on_clicked(bool checked)
{
    QMessageBox messageBox;
    QPushButton *buttonAccept0 = new QPushButton("Ok", &messageBox);
    QPushButton *buttonReject = new QPushButton("Cancel", &messageBox);
    messageBox.addButton(buttonAccept0, QMessageBox::AcceptRole);
    messageBox.addButton(buttonReject, MessageBox::NoRole);
    messageBox.setWindowTitle("Confirmation");
    messageBox.setText("Remove?");
    messageBox.setModal(true);
    connect(buttonAccept0, SIGNAL(clicked()), this, SLOT(a1()));
    messageBox.show();
}

注意：

1. 通常最好在显示对话框之前设置所有连接，以防在建立连接之前触发信号。这就是我将show 语句移到最后的原因。

2. 你可以替换这两个语句：

   messageBox.setModal(true);

   messageBox.show();

与

    messageBox.exec();

结果是一样的。

---

最后，

QMessageBox 带有许多用于通用目的的静态函数。问题中显示的代码完全适合标准用例之一，因此可以使用以下代码重写所有内容：

if (QMessageBox::question(nullptr, "Confirmation", "Remove?", 
    QMessageBox::Ok | QMessageBox::Cancel, 
    QMessageBox::Cancel) == QMessageBox::Ok)
{
    a1();
}