Welcome to Software Development on Codidact!

Will you help us build our independent community of developers helping developers? We're small and trying to grow. We welcome questions about all aspects of software development, from design to code to QA and more. Got questions? Got answers? Got code you'd like someone to review? Please join us.

Post History

75%

+4 −0

Q&A connect with SLOT/SIGNAL: QPushButton clicked signal not received by main window

Qt's SLOT macro is very specific in what it accepts. When specifying a slot in the connect function, it's required to put parenthesis, so the correct code would be: connect(ui->pushButton, SIGNA...

posted 3y ago by jrh‭ · edited 3y ago by jrh‭

Answer

#2: Post edited by

jrh‭ · 2020-09-14T15:03:31Z (over 3 years ago)
Added an example of a connect statement that does not work

Copy Link

Raw

Markdown

## Qt's `SLOT` macro is very specific in what it accepts.
When specifying a slot in the `connect` function, it's required to put parenthesis, so the correct code would be:
```
connect(ui->pushButton, SIGNAL(clicked()), this, SLOT(pushButton_Clicked()));
```
With this syntax, the button `clicked()` signal will be received by `MainWindow`.
## Some background information
When you use the `SLOT()` macro, you aren't passing in a function pointer, it actually stringizes what you pass in behind the scenes.
So you could actually do something like this, and it would still be accepted.:
```
connect(ui->pushButton, SIGNAL(clicked()), this, SLOT(gibberish));
```
Unfortunately there are no errors or warnings to help find typos in usages of the `SLOT` macro, be very careful with this macro.
## Alternatives
It's possible to use function pointers instead of using the `SIGNAL`/`SLOT` macros.
```
connect(ui->pushButton, &QPushButton::clicked, this, &MainWindow::pushButton_Clicked);
```
For overloaded functions, it's necessary to explicitly specify the type of function pointer, (`auto` is no help).
```
void (QComboBox::*comboBox_IndexChanged)(int) = &QComboBox::currentIndexChanged;
QObject::connect(itemComboBox, comboBox_IndexChanged, this, &MainWindow::OnCellComboBoxChanged, Qt::AutoConnection);
```
I think there are some other methods too, but these are the best options I was able to find in Qt 5.7.0

## Qt's `SLOT` macro is very specific in what it accepts.
When specifying a slot in the `connect` function, it's required to put parenthesis, so the correct code would be:
```
connect(ui->pushButton, SIGNAL(clicked()), this, SLOT(pushButton_Clicked()));
```
With this syntax, the button `clicked()` signal will be received by `MainWindow`.
## Some background information
When you use the `SLOT()` macro, you aren't passing in a function pointer, it actually stringizes what you pass in behind the scenes.
So you could actually do something like this, and it would still be accepted.:
```
connect(ui->pushButton, SIGNAL(clicked()), this, SLOT(gibberish));
```
Unfortunately there are no errors or warnings to help find typos in usages of the `SLOT` macro, be very careful with this macro.
## Alternatives
It's possible to use function pointers instead of using the `SIGNAL`/`SLOT` macros.
```
connect(ui->pushButton, &QPushButton::clicked, this, &MainWindow::pushButton_Clicked);
```
For overloaded functions, it's necessary to explicitly specify the type of function pointer, (`auto` is no help).
```
void (QComboBox::*comboBox_IndexChanged)(int) = &QComboBox::currentIndexChanged;
QObject::connect(itemComboBox, comboBox_IndexChanged, this, &MainWindow::OnCellComboBoxChanged, Qt::AutoConnection);
// note:
// QObject::connect(itemComboBox, &QComboBox::currentIndexChanged(int), this, &MainWindow::OnCellComboBoxChanged(int));
// does NOT work. Unfortunately you cannot specify an overloaded method like this.
```
I think there are some other methods too, but these are the best options I was able to find in Qt 5.7.0

#1: Initial revision by

jrh‭ · 2020-09-02T21:07:42Z (over 3 years ago)

Copy Link

Raw

Markdown

## Qt's `SLOT` macro is very specific in what it accepts. 

When specifying a slot in the `connect` function, it's required to put parenthesis, so the correct code would be:

```
connect(ui->pushButton, SIGNAL(clicked()), this, SLOT(pushButton_Clicked()));
```

With this syntax, the button `clicked()` signal will be received by `MainWindow`.

## Some background information

When you use the `SLOT()` macro, you aren't passing in a function pointer, it actually stringizes what you pass in behind the scenes.

So you could actually do something like this, and it would still be accepted.:

```
connect(ui->pushButton, SIGNAL(clicked()), this, SLOT(gibberish));
```


Unfortunately there are no errors or warnings to help find typos in  usages of the `SLOT` macro, be very careful with this macro.

## Alternatives

It's possible to use function pointers instead of using the `SIGNAL`/`SLOT` macros.

```
connect(ui->pushButton, &QPushButton::clicked, this, &MainWindow::pushButton_Clicked);
```

For overloaded functions, it's necessary to explicitly specify the type of function pointer, (`auto` is no help).

```
void (QComboBox::*comboBox_IndexChanged)(int) = &QComboBox::currentIndexChanged;

QObject::connect(itemComboBox, comboBox_IndexChanged, this, &MainWindow::OnCellComboBoxChanged, Qt::AutoConnection);
```

I think there are some other methods too, but these are the best options I was able to find in Qt 5.7.0

Communities

Post History