Dynamically created buttons.

[Tomasz]

Hello!

I've got a problem with buttons and pointers. I need to create dynamically something like "array" of buttons on one of my application windows (each time the window opens). Now I've got some code:

QPushButton *btn[HOW_MANY_BUTTONS];
 
    for(int j=0; j<HOW_MANY_BUTTONS; j++)
    {
        btn[j] = new QPushButton;
        btn[j]->setGeometry(0,0,40,40);
 
        ui->verticalLayout->addWidget(btn[j],Qt::AlignVCenter);
    }

QPushButton *btn[HOW_MANY_BUTTONS];

    for(int j=0; j<HOW_MANY_BUTTONS; j++)
    {
        btn[j] = new QPushButton;
        btn[j]->setGeometry(0,0,40,40);

        ui->verticalLayout->addWidget(btn[j],Qt::AlignVCenter);
    }

To copy to clipboard, switch view to plain text mode 

It works just fine. But number of buttons is changing, and I don't want to create array of pointers which is bigger or smaller than number of buttons. Any suggestions how should I write function which is creating as many buttons as I need in a moment?

thanks in advance
best regards
Tomasz

[SixDegrees]

Use a std::vector instead of a fixed-size array. Or, simply let the layout do the bookkeeping, including destruction.

[Tomasz]

Use a std::vector instead of a fixed-size array. Or, simply let the layout do the bookkeeping, including destruction.

I'll read about std::vector. But what about that layout? How should it work?

thanks in advance
best regards
Tomasz

[Tomasz]

I'm trying to use std::vector but, can I have pointers as elements of vector? How can add buttons to vector?

thanks in advance
best regards
Tomasz

[wysota]

QVector<QPushButton*> btn; // or std::vector
 
for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
        QPushButton *b = new QPushButton;
        b->setGeometry(0,0,40,40);
        ui->verticalLayout->addWidget(b,Qt::AlignVCenter);
        btn << b;
}

QVector<QPushButton*> btn; // or std::vector

for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
        QPushButton *b = new QPushButton;
        b->setGeometry(0,0,40,40);
        ui->verticalLayout->addWidget(b,Qt::AlignVCenter);
        btn << b;
}

To copy to clipboard, switch view to plain text mode 

[BalaQT]

hi ,
u can use QHash too.
i will use like this ,

QHash<int,QPushButton*> btn;
 
int HOW_MANY_BUTTONS=200; //u can change this value dynamically as u wish;
for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
        QPushButton *b=new QPushButton;
        btn[j]=b;
        b->setGeometry(0,0,40,40);
        ui->verticalLayout->addWidget(b,Qt::AlignVCenter);        
}

QHash<int,QPushButton*> btn;

int HOW_MANY_BUTTONS=200; //u can change this value dynamically as u wish;
for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
        QPushButton *b=new QPushButton;
        btn[j]=b;
        b->setGeometry(0,0,40,40);
        ui->verticalLayout->addWidget(b,Qt::AlignVCenter);        
}

To copy to clipboard, switch view to plain text mode 

hope it helps

Bala

[SixDegrees]

Note also, in addition to the above, that a QLayout itself acts much like a vector: it has count(), addItem(), removeItem() and itemAt() functions that might serve just as well as using an external vector to keep track of the items managed by the layout. It isn't really possible to say whether this approach would work without knowing more about your application, but sometimes it allows you to use a single object - the layout - rather than two - the layout plus an external container.

[Tomasz]

QVector<QPushButton*> btn; // or std::vector
 
for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
        QPushButton *b = new QPushButton;
        b->setGeometry(0,0,40,40);
        ui->verticalLayout->addWidget(b,Qt::AlignVCenter);
        btn << b;
}

QVector<QPushButton*> btn; // or std::vector

for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
        QPushButton *b = new QPushButton;
        b->setGeometry(0,0,40,40);
        ui->verticalLayout->addWidget(b,Qt::AlignVCenter);
        btn << b;
}

To copy to clipboard, switch view to plain text mode 

So, if I want to clear that vector I can use:

btn.clear();

btn.clear();

To copy to clipboard, switch view to plain text mode 

It will clear also verticalLayout?

thanks in advance
best regards
Tomasz

[wysota]

It will clear also verticalLayout?

No, it won't.

[BalaQT]

hi,

void QVector::clear ()
Removes all the elements from the vector and releases the memory used by the vector.

u need to remove the widgets from the layout.
use

void QLayout::removeWidget ( QWidget * widget )

void QLayout::removeWidget ( QWidget * widget )

To copy to clipboard, switch view to plain text mode 

hope it helps
Thnks
Bala

[Tomasz]

So, it is as I thought. But what if I do something like that:

QVector<QPushButton*> btn;
 
btn.clear();       
 
for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
      QPushButton *b = new QPushButton;
      b->setGeometry(0,0,40,40);
      btn << b;
      ui->verticalLayout->addWidget(btn[j],Qt::AlignVCenter);
}

QVector<QPushButton*> btn;

btn.clear();       

for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
      QPushButton *b = new QPushButton;
      b->setGeometry(0,0,40,40);
      btn << b;
      ui->verticalLayout->addWidget(btn[j],Qt::AlignVCenter);
}

To copy to clipboard, switch view to plain text mode 

Can I then remove item from layout using:

ui->verticalLayout->removeWidget(btn[i]);

ui->verticalLayout->removeWidget(btn[i]);

To copy to clipboard, switch view to plain text mode 

Or maybe different pointer points on that widget?

thanks in advance
best regards
Tomasz

[BalaQT]

Can I then remove item from layout using ui->verticalLayout->removeWidget(btn[i]);

yes surely u can

Or maybe different pointer points on that widget?

no its the same pointer

hope it helps
Bala

[Tomasz]

But if I remove that widget from layout, pointer stays in vector? And I can use it again or just remove it from vector too?

thanks in advance
best regards
Tomasz

[BalaQT]

But if I remove that widget from layout, pointer stays in vector?

yes pointer stays in vector, since u didnt remove it from Vector.

And I can use it again

yes, surely u can use it again.

or just remove it from vector too?

if u dont want that widget then remove it from vector too..

layout and vector are different...

for ur understanding ... vector is holding the widget pointer.
that widget will be added to the layout by addWidget
and removed by removeWidget

hope it helps
Bala

[Tomasz]

One more question to end that thread. I've got my buttons, now I'm connecting them to QSignalMapper:

QVector<QPushButton*> btn;
 
btn.clear();       
 
for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
      QPushButton *b = new QPushButton;
      b->setGeometry(0,0,40,40);
      btn << b;
      ui->verticalLayout->addWidget(btn[j],Qt::AlignVCenter);
      connect(btn[j], SIGNAL(clicked()), sigMap, SLOT(map()));
      sigMap->setMapping(btn[j], j);
}
 
connect(sigMap, SIGNAL(mapped(int)), this, SLOT(func(int)));

QVector<QPushButton*> btn;

btn.clear();       

for(int j=0; j<HOW_MANY_BUTTONS; j++)
{
      QPushButton *b = new QPushButton;
      b->setGeometry(0,0,40,40);
      btn << b;
      ui->verticalLayout->addWidget(btn[j],Qt::AlignVCenter);
      connect(btn[j], SIGNAL(clicked()), sigMap, SLOT(map()));
      sigMap->setMapping(btn[j], j);
}

connect(sigMap, SIGNAL(mapped(int)), this, SLOT(func(int)));

To copy to clipboard, switch view to plain text mode 

What will now happen with QSignalMapper if I remove buttons from layout and vector? Should I clean somehow signal mapper? What happen with all connections?

thanks in advance
best regards
Tomasz

[wysota]

If you "remove" buttons from the layout and vector nothing will happen and connections to the mapper will still be maintained. But if you delete the buttons, all their connections will be broken.

From a technical point of view your vector doesn't hold buttons, it holds pointers to buttons. So you are not removing buttons from the vector (or the layout as a matter of fact) but pointers to buttons.

[Tomasz]

So the safest way to do the cleaning is to: remove widgets from layout, then DELETE all created buttons with:

delete(btn[x]); // btn is vector with pointers

delete(btn[x]); // btn is vector with pointers

To copy to clipboard, switch view to plain text mode 

and at the end - i can safely clear vector, and then connections will be broken, and QSignalMapper will be 'clean'? Yes? And then I can create everything again?

thanks in advance
best regards
Tomasz

[wysota]

It's enough to delete the buttons. They will be taken out of layouts.

qDeleteAll(btn); 
btn.clear();

qDeleteAll(btn); 
btn.clear();

To copy to clipboard, switch view to plain text mode 

[Tomasz]

One more question - should I use disconnect() to disconnect all signals (from created buttons) from slots.

thanks in advance
best regards
Tomasz

[wysota]

One more question - should I use disconnect() to disconnect all signals (from created buttons) from slots.

Why don't you try it first with and without disconnecting and then answer your own question?