Problem with the connect method in Qt5

[jsulm]

@Sewing Interesting, here the code compiles without any modifications (without &).
What compiler do you use?

[JonB]

@jsulm
Assuming his code is indeed using ViewerImage::open instead of MainWindow::open, don't we need to his declaration of the former?

[Sewing]

@ jsulm: gcc 5 and Qt-5.5

btw: same problem seems to appear for

  QMenu* fileMenu = menuBar()->addMenu(tr("&File"));

  QAction* openAct = fileMenu->addAction(tr("&Open..."), this, &ImageViewer::open);

which does not match the only viable addAction overload in qmenu.h

   QAction *addAction(const QString &text, const QObject *receiver, const char* member, const QKeySequence &shortcut = 0);

although this example is taken from the official QT Webpage. Am I missing something here, does my Qt Version not match the code or what seems to be the issue?

[JonB]

@Sewing
Is your ImageViewer::open a char? Like I said before, don't we need to know what your definition of ImageViewer::open is before we start figuring how it matches declaration prototypes? OK, maybe I'm being a bit pedantic/dumb, can I assume it's a function?

That overload doesn't look right as the "only" one available.... What about:

QAction *addAction(const QString &text, const QObject *receiver, PointerToMemberFunction method, const QKeySequence &shortcut = 0)

http://doc.qt.io/qt-5/qmenu.html#addAction-4
?

For PointerToMemberFunction you might want to view https://stackoverflow.com/questions/29218092/where-is-qt-s-pointertomemberfunction-defined

I think you may also need to read https://isocpp.org/wiki/faq/pointers-to-members#fnptr-vs-memfnptr-types to describe Is the type of “pointer-to-member-function” different from “pointer-to-function”?

All of this may be why we need to know the exact definition of your ImageViewer::open versus the example's MainWindow::open?

[Sewing]

thank you for your help guys =)

open is a function and its signature looks like

void ImageViewer::open()

in qmenu.h the overloads I have are

  QAction *addAction(const QString &text);
    QAction *addAction(const QIcon &icon, const QString &text);
    QAction *addAction(const QString &text, const QObject *receiver, const char* member, const QKeySequence &shortcut = 0);
    QAction *addAction(const QIcon &icon, const QString &text, const QObject *receiver, const char* member, const QKeySequence &shortcut = 0);

I dont get it...

Also, what is the difference between addAction and connect?

[JonB]

@Sewing
I don't do C++/header files, but are you sure there aren't some additional overloads, perhaps somewhere away from the ones you show, for addAction()? They don't all have to be next to each other....

For addAction vs connect:

A menu consists of a list of action items. Actions are added with the addAction(), addActions() and insertAction() functions.
the QMenu class contains convenience functions for creating actions suitable for use as menu items.
A QAction may contain an icon, menu text, a shortcut, status text, "What's This?" text, and a tooltip

QAction *QMenu::addAction(const QIcon &icon, const QString &text, const QObject *receiver, const char *member, const QKeySequence &shortcut = 0)

This convenience function creates a new action with an icon and some text and an optional shortcut shortcut. The action's triggered() signal is connected to the member slot of the receiver object. The function adds the newly created action to the menu's list of actions, and returns it.

An addAction() wraps up the addition of an item on a menu and the association of the slot with clicking (triggered() signal) that item.

[kshegunov]

@jsulm said in Problem with the connect method in Qt5:

What compiler do you use?

g++ requires you to write the ampersand, even though it strictly shouldn't be necessary. This:

class A
{
public:
    void open(int) {}
};

int main(int argc, char *argv[])
{
    void (A::*ptr)(int) = A::open;
    return 0;
}

generates (g++ 7.2):

error: invalid use of non-static member function 'void A::open(int)'
    void (A::*ptr)(int) = A::open;
                             ^~~~

and has been like this for as long as I can remember.

[Sewing]

@Sewing said in Problem with the connect method in Qt5:

thank you for your help guys =)

open is a function and its signature looks like

void ImageViewer::open()

in qmenu.h the overloads I have are

  QAction *addAction(const QString &text);
    QAction *addAction(const QIcon &icon, const QString &text);
    QAction *addAction(const QString &text, const QObject *receiver, const char* member, const QKeySequence &shortcut = 0);
    QAction *addAction(const QIcon &icon, const QString &text, const QObject *receiver, const char* member, const QKeySequence &shortcut = 0);

I dont get it...

Also, what is the difference between addAction and connect?

Well I guess I found the issue. I am using Qt 5.5.1 via the system Packages on Xubuntu 16.04. However, the necessary overloaded veersion of addAction were introduced in Qt 5.6 according to the API.

How can I get Qt 5.6 without having to install it myself from Source?
Is there an Update on the system Packages yet which includes this more recent Qt Version?

[jsulm]

@Sewing Just download official Qt Online Installer and install: http://download.qt.io/official_releases/online_installers/qt-unified-linux-x64-online.run

[Sewing]

All right, not sure how to link against this install in my Cmake Project though =/

with Qt5.5 I just used find_package command...

[jsulm]

@Sewing Should be the same: just select the Kit for this new Qt version and then run cmake.