Mine field Layout problem (solved).

Mangusta89

Hi everyone, I've recently started to develope a simple game (mine field) and I already have a problem.
I create a QPushButton "newGame", which purpose is to create a grid of buttons.
I post some lines of the code to help the explanation:

@
mine_field::mine_field(QWidget *parent) : QWidget(parent) {

QHBoxLayout *menuLayout = new QHBoxLayout();
QPushButton *newGame = new QPushButton(tr("New Game"));
menuLayout->addWidget(newGame);

QVBoxLayout *mainLayout = new QVBoxLayout();
mainLayout->addLayout(menuLayout);
setLayout(mainLayout);

connect(newGame, SIGNAL(clicked()), this, SLOT(create()));
}

void mine_field::create()
{
QGridLayout *buttonLayout = new QGridLayout();
for (int i=0;i<10;++i)
for (int j=0; j<10; ++j){
QPushButton *button = new QPushButton();
buttonLayout->addWidget(button, i, j);
}
mainLayout -> addLayout(buttonLayout);
}
@

I get a segmentation fault, I think because mainLayout is not defined in function create().
Anyone can help me?
Thanks.

SGaist

Hi and welcome to devnet,

You are shadowing mainLayout in your constructor that's why it crashes in create()

Mangusta89

Thank for the answer, how can I set mainLayout as global, visible both for constructor and for create() ?

SGaist

Seems you already do have it as a class member. Otherwise you wouldn't be able to compile

Mangusta89

Yes, I declared it in header file:
@ private:
QHBoxLayout *menuLayout; @

So instead of write in constructor:
@ QHBoxLayout *menuLayout = new QHBoxLayout (); @

Can I use it as?
@ this->menuLayout @

Mangusta89

Ok, I resolved the problem just declaring in constructor:
@menuLayout = new QHBoxLayout ();@

Thank you for your patience :)

SGaist

You're welcome !

Since you have it working now, please update the thread title by prepending [solved] so the other forum users may know a solution has been found :)