QPushbutton The slot function is triggered multiple times

Before the change

 --------------TODO Binding function --------------
      For screen / End the screen shot  
    connect(&this->attachToDevice, &QPushButton::clicked, this, &PhoneCard::onAttachToDevice);
    connect(&this->detachFromDevice, &QPushButton::clicked, this, &PhoneCard::onDetachFromDevice);

The signal slot will be bound every time , But not rebinding , That is, it becomes multiple signal slots

therefore Every time you click the refresh button Click again The refreshed key , The refreshed button will have an additional trigger mechanism ( For the time being )

Then when you click again After the refreshed button It will trigger multiple times . In the corresponding QMessageBox There will be many times

solve

void PhoneCard::drawButtons(int modType) {

    // to phone
    if(modType == 1){
        //  Put it on the phone 
        this->attachToDevice.setParent(this);
        this->attachToDevice.setGeometry(40,110,220,30);
        this->attachToDevice.setText(" Drop to this device ");
        this->attachToDevice.setFont(QFont("Times New Roman"));
        this->attachToDevice.setStyleSheet(style_but);
        this->attachToDevice.show();
        // disconnect
        this->detachFromDevice.setParent(this);
        this->detachFromDevice.setGeometry(40,150,220,30);
        this->detachFromDevice.setText(" Stop screen projection ");
        this->detachFromDevice.setFont(QFont("Times New Roman"));
        this->detachFromDevice.setStyleSheet(style_but);
        this->detachFromDevice.show();

         --------------TODO Binding function --------------
          For screen / End the screen shot   Because many times   So first   Unbundling   then   Again   binding 
        this->attachToDevice.disconnect();
        this->detachFromDevice.disconnect();
        connect(&this->attachToDevice, &QPushButton::clicked, this, &PhoneCard::onAttachToDevice);
        connect(&this->detachFromDevice, &QPushButton::clicked, this, &PhoneCard::onDetachFromDevice);

        this->usb.hide();
        this->wifi.hide();
    }
    else if(modType == 0){
        // to phone
        // wifi Key 
        this->wifi.setParent(this);
        this->wifi.setGeometry(40,110,220,30);
        this->wifi.setText("Wifi  Connect ");
        this->wifi.setFont(QFont("Times New Roman"));
        this->wifi.setStyleSheet(style_but);
        this->wifi.show();
        // USB
        this->usb.setParent(this);
        this->usb.setGeometry(40,150,220,30);
        this->usb.setText("USB  Connect ");
        this->usb.setFont(QFont("Times New Roman"));
        this->usb.setStyleSheet(style_but);
        this->usb.show();
         --------------TODO Binding function --------------
          Untie yourself WiFi USB
          Rebind 

    }
    else if(modType == -1){
        this->attachToDevice.hide();
        this->detachFromDevice.hide();
        this->usb.hide();
        this->wifi.hide();
    }
}

That is, before each binding disconnect Again connect

Of course This estimate only applies to Corresponding to a slot function , If it corresponds to multiple , You may have to rebind more than one at a time ~

other BUG

Repair After starting the screen projection Click refresh “ On screen ” The button is reset, but In a constant state bug

 fungcard.h  Add these built-in functions 
    QString getAttachStatus(){
        return this->attachToDevice.text();
        }
    QString getWifiStatus(){
        return this->wifi.text();
        }
    QString getUsbStatus(){
        return this->usb.text();
        }


  And then in mainmenu.cpp  When clicking refresh   Re judge    Make a stop 
void MainMenu::onRefreshClicked() {
    //  Determine the current status according to the text 
    int findType = -1;
    if(ToPhoneButton.text() == " Has run ")
        findType = 1;
    else if(ToPCButton.text() == " Has run ")
        findType = 0;
    else{
        QMessageBox::information(this,"JoyStick", " Please select the mode on the left first !",QMessageBox::Ok);
        return ;
    }

    for (PhoneCard &phone : phoneCards) {
        if(phone.getAttachStatus() == " On screen ..." || phone.getWifiStatus() == " On screen ..." || phone.getUsbStatus() == " On screen ...")
        {
            QMessageBox::information(this,"JoyStick", " Some screens have not stopped , Please stop and refresh !",QMessageBox::Ok);
            return;
        }
    }

    int number = 0;
    for (PhoneCard &phone : phoneCards) {
        phone.setParent(this);
        phone.drawOnRefresh(number++,findType);
    }
}

The lesson here is inline To get the state and judge