Wiążące argumenty do sygnałów/gniazd

Question

Generalnie mam wiele sygnałów zdarzeń, które chcę podłączyć do tego samego gniazda. Chcę wiedzieć, w jaki sposób mogę przekazywać parametry oparte na łańcuchach do tego samego gniazda, aby gniazdo wiedziało, z którego źródła pochodzi ten sygnał. Jedną z możliwości jest utworzenie tak wielu gniazd, jak sygnały, a następnie połączenie ich w sposób 1: 1, ale jest to efektywne, biorąc pod uwagę, że kod dla całego przetwarzania jest bardzo podobny. Próbowałem to zrobić, ale ja dostaję jakieś błędy:

connect(selecter1,SIGNAL(selected(QString)),this,SLOT(backgroundTypeChoiceMade(QString))); 
connect(button1,SIGNAL(clicked()),this,SLOT(backgroundTypeChoiceMade("button1"))); 
connect(button2,SIGNAL(clicked()),this,SLOT(backgroundTypeChoiceMade("button2"))); 

Błąd jest związany z parametrami Ja przejeżdżających w ciągu ostatnich 2 komend .. I backgroundTypeChoiceMade jest zadeklarowane tak:

void backgroundTypeChoiceMade(QString); 

Czy ktoś może mi powiedzieć, jaki jest błąd w powyższym kodzie?

Answer 1

Nie można przekazać stałych do connect(), ponieważ efektywne parametry są wydedukowane w czasie wykonywania, a nie w czasie kompilacji.

Mimo że jest to sprzeczne z zasadą OO, można użyć QObject::sender(), która podaje wskaźnik do emitera QObject.

Przykład poniżej:

void YourClass::YourClass() : 
    m_button1(new QPushButton()), 
    m_button2(new QPushButton()) 
{ 
    connect(m_button1, SIGNAL(clicked()), this, SLOT(yourSlot())); 
    connect(m_button2, SIGNAL(clicked()), this, SLOT(yourSlot())); 
} 

void YourClass::yourSlot() 
{ 
    if ((QPushButton* button = dynamic_cast<QPushButton*>(sender())) 
    { 
    // Now button points to a QPushButton* that you can compare with the pointers you already have 

    if (button == m_button1) 
    { 
     // Whatever 
    } else 
    if (button == m_button2) 
    { 
     // Whatever 
    } 
    } 
} 

Jeśli masz wiele przycisków, można również użyć QSignalMapper dostarczając identyfikator dla każdego przycisku.

Answer 2

Można użyć QSignalMapper. Chociaż QSignalMapper jest odpowiedzią na twoje pytanie, myślę, że odpowiedź Jona Hansona jest drogą, którą powinieneś podjąć. W ten sposób dostajesz o wiele bardziej czysty kod.

Answer 3

Co jest nieefektywne w korzystaniu z oddzielnych gniazd? Jeśli w procedurach obsługi slotów występuje wspólność, przenieś to do funkcji, np. rozszerzenie przykład ereOn za:

void YourClass::YourClass() : 
    m_button1(new QPushButton()), 
    m_button2(new QPushButton()) 
{ 
    connect(m_button1, SIGNAL(clicked()), this, SLOT(yourSlot1())); 
    connect(m_button2, SIGNAL(clicked()), this, SLOT(yourSlot2())); 
} 

void YourClass::common(int n) 
{ 
} 

void YourClass::yourSlot1() 
{ 
    common (1); 
} 

void YourClass::yourSlot2() 
{ 
    common (2); 
} 

Answer 4

można użyć coś jak napisałem na moim blogu: http://uint32t.blogspot.com/2008/11/using-boost-bind-and-boost-function.html

Myślę, że jednym z komentarzy ma dość dobre rozwiązanie, jak również.

Answer 5

Cztery metody. Jeden nie jest do dupy.

1. QSignalMapper. Działa, ale tworzy niechlujny kod.
2. Nazwane gniazda. Brudny dla znacznej liczby nadawców i nie działa dla dynamicznie generowanych nadawców (np. Przycisków na liście).
3. sender() -porównanie. Potrafi obsługiwać dynamiczne nadajniki, ale nadal jest dość brzydka.
4. Podpis Nie ssać. Daje ci to, czego naprawdę chciałeś przez cały czas: sparametryzowane sygnały.

Szczególnie jeśli używasz niewielkiej liczby sygnałów i typów nadajników oraz kiedy nadawcy są generowani dynamicznie, podklasy nadawcy są najczystszym sposobem. Pozwala to na przeciążenie istniejących sygnałów, aby zawierały potrzebne parametry.

i teraz, okablowania się sygnały i sloty prostu działa:

Keypad::Keypad(QWidget *parent) : QWidget(parent) 
{ 
    for (int i = 0; i < 10; ++i) 
    { 
     // KeypadButton keeps track of the identifier you give it 
     buttons[i] = new KeypadButton(i, this); 
     // And passes it as a signal parameter. Booyah. 
     connect(buttons[i], SIGNAL(clicked(int)), this, SIGNAL(digitClicked(int))); 
    } 
    createLayout(); 
} 

void Keypad::digitClicked(int digit) 
{ 
    // The slot can find the clicked button with ease: 
    dial(button[i]); // or whatever 
    //... 
} 

i dodatkowy kod jest out-of-sight w podklasie nigdy nie będziesz musiał ponownie dotknąć.

Zobacz przykładową implementację podklasy do emisji sygnałów QPushButton w celu emisji sygnałów . Omówiono również wszystkie cztery metody - nazwane gniazda ("rozwiązanie trywialne"), nadawcę(), podklasę i program odwzorowujący sygnał.

Zastrzeżenie: Oczywiście działa najlepiej w przypadku niewielkiej liczby typów nadajników. Ale tak się zwykle dzieje. I w tym przypadku warto.

Answer 6

Jeśli naprawdę nie chcesz używać QSignalMapper, można zrobić coś takiego:

class SignalForwarderWithString: public QObject 
{ 
    Q_OBJECT 
public: 
    SignalForwarderWithString(QString data = "", QObject *parent = 0) : QObject(parent), _data(data) {} 
    QString _data; 
signals: 
    void forward(QString); 
public slots: 
    void receive() { emit forward(_data); } 
}; 

... 
connect(selecter1,SIGNAL(selected(QString)),this,SLOT(backgroundTypeChoiceMade(QString))); 

SignalForwarderWithString *sfws; 
sfws = new SignalForwarderWithString("button1", this); 
connect(button1,SIGNAL(clicked()), sfws, SLOT(receive(QString))); 
connect(sfws, SIGNAL(forward(QString)), this,SLOT(backgroundTypeChoiceMade(QString))); 

sfws = new SignalForwarderWithString("button2", this); 
connect(button2,SIGNAL(clicked()), sfws, SLOT(receive(QString))); 
connect(sfws, SIGNAL(forward(QString)), this,SLOT(backgroundTypeChoiceMade(QString))); 

ale QSignalMapper jest tak proste ...

QSignalMapper *mapper = new QSignalMapper(this); 
connect(button1, SIGNAL(clicked()), mapper, SLOT(map())); 
mapper->setMapping(button1, "button 1"); 
connect(button2, SIGNAL(clicked()), mapper, SLOT(map())); 
mapper->setMapping(button2, "button 2"); 
// you might have to tweak the argument type for your slot... 
connect(mapper, SIGNAL(mapped(const QString &), this, SLOT(backgroundTypeChoiceMade(QString))); 

Answer 7

można teraz naprawdę wiążą wartość podczas łączenia. Qt5 dodał wsparcie dla tego.

przykład:

connect(sender, &Sender::valueChanged, 
    tr1::bind(receiver, &Receiver::updateValue, "senderValue", tr1::placeholder::_1)); 

Patrz more info.

NB: można oczywiście użyć std :: bind lub boost :: bind zamiast tr1 :: bind.