Wie man einen QString an einen Qt-Slot von einem QMenu über QSignalMapper oder sonstwie weitergibt

Question

Ich habe ein QMenu mit vielen Untermenüs. Diese werden dynamisch erzeugt, d. H. Die Namensmenüs kommen von einer db und werden in einer Schleife erzeugt. Nun wollte ich den gleichen Slot auslösen() oder ähnliches, wenn ein Menü angeklickt wurde, aber ich brauchte den QString Menünamen, der an Slot übergeben wurde, damit ich menüspezifische Aktionen ausführen konnte. Ich habe versucht this, d. H. Eine QAction * an das ausgelöste Ereignis übergeben und setData verwendet, aber ich bekomme den Laufzeitfehler.

Objekt :: connect: Kein QAction solches Signal :: ausgelöst (QAction *)

for(int j=0; j<channelTypes[i].getNumChannels() ; j++){ 
     QAction *subMenuAct = subMenu->addAction(tr(c_name)); // c_name the menu name 
     subMenuAct->setData(ch_name); 
     connect(subMenuAct, SIGNAL(triggered(QAction *)), this, SLOT(playChannel(QAction *))); // playChannel is the slot 
} 

void <ClassName>::playChannel(QAction *channelAction) 
{ 
    QString str = channelAction->data().toString(); 
    qDebug() << "Selected - " << str; 
} 

Alternativ habe ich auch versucht, QSignalMapper wo signalMapper ein Datenelement im Konstruktor initialisiert ist

signalMapper = new QSignalMapper(this); 

und

for(int j=0; j<channelTypes[i].getNumChannels() ; j++){ 
     QAction *subMenuAct = subMenu->addAction(tr(c_name)); 

     connect(subMenuAct, SIGNAL(triggered()), signalMapper, SLOT(map())); 
     signalMapper->setMapping(subMenu, ch_name); 
     connect(signalMapper, SIGNAL(mapped(QString)), this, SLOT(playChannel(QString))); 
} 

Im zweiten Fall bekomme ich keinen Fehler, jedoch wird die Slot-Funktion playChannel nicht aufgerufen. Würde mich wirklich freuen, wenn jemand helfen könnte, es zu lösen.

Update 1: Der einzige Unterschied, den ich von anderen Beispielen sehe ich gesehen habe, ist, dass Menschen in der Regel Verbindungssignale von mehreren Widgets zu einem einzigen Slot (sagen verschiedene Tasten) sind. In meinem Fall verbinde ich mehrere Untermenüs (mit unterschiedlichen Namen) zu einem einzelnen Steckplatz. Sollte dies einen Unterschied machen?

Update 2: Es funktionierte nach der vorgeschlagenen Korrektur in der Lösung below für QSignalMapper. Auch die Tatsache, dass ich SubMenu als Argument für setMapping verwendete, wo stattdessen das MenuAction-Element verwendet werden sollte. Aber jetzt erhalte ich ein Ereignis, das mehrmals ausgelöst wird, d.h. so oft wie es Einträge im Hauptmenü für die ausgewählte Untermenü-Kategorie gibt. Wenn der Kanaltyp Englisch (Hauptmenü) mit vier Einträgen ist, HBO, Sternfilme usw. (Untermenü) und ich HBO wähle, wird das Ereignis viermal mit der Zeichenfolge HBO ausgelöst. Es funktioniert gut, wenn ich einen separaten Signal-Mapper für jedes Untermenü erstellen. Aber ich hatte gehofft, dass ein einzelner Mapper verwendet werden sollte und ich hier etwas falsch mache. Einige weitere Details in den Kommentaren zur Antwort.

Answer 1

Nachdem Sie das Menü QAction hinzugefügt haben, müssen Sie nur QMenu an den Steckplatz anschließen. Sie verbinden nicht jede Aktion einzeln an den Schlitz:

for(int j=0; j<channelTypes[i].getNumChannels() ; j++){ 
    ch_name = <name from the database for the channel j>; 
    QAction *subMenuAct = subMenu->addAction(tr(ch_name)); 
    subMenuAct->setData(ch_name); 
} 

connect(subMenu, SIGNAL(triggered(QAction *)), 
     this, SLOT(playChannel(QAction *)), Qt::UniqueConnection); 

Wie ich weiß nicht, wie Sie, wenn Sie subMenu jedes Mal das dynamische Menü gefüllt löschen ist, die Qt::UniqueConnection sicherzustellen, dass der Schlitz nicht wieder angeschlossen werden mehrmals.

---

Für die Signal-Mapper-Version sollten Sie die Aktionen nur mit dem Mapper in der Schleife verbinden. Die Verbindung vom Mapper zum Slot sollte nur einmal durchgeführt werden.

for(int j=0; j<channelTypes[i].getNumChannels() ; j++){ 
    ch_name = <name from the database for the channel j>; 
    QAction *subMenuAct = subMenu->addAction(tr(ch_name)); 
    connect(subMenuAct, SIGNAL(triggered()), signalMapper, SLOT(map())); 
    signalMapper->setMapping(subMenuAct, ch_name); 
} 
connect(signalMapper, SIGNAL(mapped(QString)), this, SLOT(playChannel(QString))); 

Und für diesen Fall sollte der Schlitz playChannel eine QString anstelle eines QAction* akzeptieren.