Prosta funkcja w Qt

pytanie zadane 14 lutego 2018 w C i C++ przez chucksqll Stary wyjadacz (12,930 p.)

Witam. Tworzę projekt Aplikacja Qt Widgets, dodaję kilka przycisków PushButton i teraz robię funkcje, której argumentem(chyba ze jakoś inaczej można to zrobić), będzie jakiś przycisk i będzie na nim wyświetlać tekst, coś takiego(choć błędnie napisanego)

void MainWindow::on_pushButton_4_clicked()
{
    if(isCross)
    {
        isCross=false;
        ui->pushButton_4->setText("X");
    }
    else ui->pushButton_4->setText("O");
}

i teraz chce rozdzielić treść i nagłówek na coś takiego

void MainWindow::Show(QWidget *PB)//ma przyjmowac jako argument przycisk
{
    if(PB->isCross)
    {
        isCross=false;
        ui->PB->setText("X");
    }
    else ui->PB->setText("O");
}
void MainWindow::on_pushButton_4_clicked()
{
 Show(this);//wyswietla napis na przycisku 4
}

Jestem tego świadomy, że ten drugi kod jaki napisałem może nie mieć większego sensu, natomiast myślę, że przekazałem zrozumiale o co mi chodzi.

komentarz 14 lutego 2018 przez Bondrusiek Maniak (61,370 p.)

W czym jest problem, zwraca Ci błąd?

2 odpowiedzi

odpowiedź 14 lutego 2018 przez j23 Mędrzec (194,920 p.)
wybrane 14 lutego 2018 przez chucksqll

Najlepsza

void MainWindow::Show(QPushButton *PB)
{
	if(!PB) return;

	if(isCross)
	{
		isCross = false;
		PB->setText("X");
	}
	else PB->setText("O");
}


void MainWindow::on_pushButton_4_clicked()
{
	Show(dynamic_cast<QPushButton*>(sender()));
}

O to chodzi?

PS. przy takim wywołaniu możesz podpiąć wszystkie przyciski pod jedno zdarzenie on_pushButton_clicked.

komentarz 14 lutego 2018 przez chucksqll Stary wyjadacz (12,930 p.)

Tak dokładnie o to chodziło, mógłbyś bardziej rozwinąć jak wszystkie przyciski podpiąć pod jedno zdarzenie?

komentarz 15 lutego 2018 przez j23 Mędrzec (194,920 p.)
edycja 15 lutego 2018 przez j23

W skrócie chodzi o to, żebyś nie robił czegoś w stylu:

void MainWindow::on_pushButton_1_clicked()
{
    Show(dynamic_cast<QPushButton*>(sender()));
}

void MainWindow::on_pushButton_2_clicked()
{
    Show(dynamic_cast<QPushButton*>(sender()));
}

...

void MainWindow::on_pushButton_9_clicked()
{
    Show(dynamic_cast<QPushButton*>(sender()));
}

Podepnij wszystkie przyciski, które mają wywołać Show(), do jednej obsługi zdarzenia. Coś w ten deseń:

connect(ui->pushButton_1, SIGNAL (clicked()), this, SLOT (on_pushButton_clicked()));
connect(ui->pushButton_2, SIGNAL (clicked()), this, SLOT (on_pushButton_clicked()));
...
connect(ui->pushButton_9, SIGNAL (clicked()), this, SLOT (on_pushButton_clicked()));