connect action to slot new syntax

[hobbyProgrammer]

Hi,

I am trying to connect an action that's created in the designer to a slot.

The action is actionLogout and can be called via ui->actionLog_out where ui is from MainWindow.
The slot is logout().

I already tried this:

    logoutAct = new QAction(tr("logout"));
    connect(this, logoutAct, this, &MainWindow::logout);

it says: "no matching member function for call to connect"

[mrjj]

Hi
You seem slightly confused about the syntax
It is
Who has the signal, the signal , Who has the Slot, the Slot

logoutAct = new QAction(tr("logout"));
connect(logoutAct, &QAction::Triggered, this, &MainWindow::logout);

so we have
logoutAct (who) , &QAction::Triggered (the signal ) , this (who), &MainWindow::logout (slot)

[JonB]

@hobbyProgrammer
One thing to note: as per @mrjj's reply, you should (actually must) use either the old SIGNAL/SLOT() syntax (deprecated) or the new non-macro ones (recommended) like @mrjj's. Don't even try mixing them as you have! Stop using either SIGNAL() or SLOT() from now onward!

[hobbyProgrammer]

@mrjj thank you, at least it doesn't come with errors now, but it doesn't respond to me clicking 'logout' in the menubar.

I tried:

    connect(ui->actionLog_out, &QAction::triggered, this, &MainWindow::logout);

and

    QAction *logoutAct = ui->actionLog_out;
    connect(logoutAct, &QAction::triggered, this, &MainWindow::logout);

[JonB]

@hobbyProgrammer
If you have just @mrjj's code it won't. That creates an "anonymous"/"standalone" QAction. You have to connect the QAction to the appropriate item on the QMenu, else they don't have anything to do with each other. I don't know if or how the Designer does that. And make sure you put a breakpoint or output something in whatever your MainWindow::logout is, so you are sure whether it is called or not.

[J.Hilk]

I updated my example..
https://github.com/DeiVadder/LoginExample

[JonB]

@J-Hilk Looks like you've got a full-time job there :)

[J.Hilk]

@JonB If it's only taking a couple of seconds/minutes 🤷‍♂️
;)