Signal doubleClicked(QModelIndex) crashes application on next accepted() signal.

[anda_skoa]

Looks like you forgot to post the code of the slot connected to the signal.

Cheers,
_

[adutzu89]

Sorry , I hope your not sick of me anda_skoa, I try looking on the docs and over the internet for solutions and I post whenever I can't find a solution.

Here is the connect statement:

connect(table,SIGNAL(doubleClicked(QModelIndex)),this,SLOT(rSelect(QModelIndex)));

connect(table,SIGNAL(doubleClicked(QModelIndex)),this,SLOT(rSelect(QModelIndex)));

To copy to clipboard, switch view to plain text mode 

Here is the slot which in turns emits another signal so the QDialog know what data to extract:

void ListaAng::rSelect(QModelIndex index){
    int rand=index.row();
    QString id=index.sibling(rand,0).data().toString();
    emit semnalAngSel(id);
}

void ListaAng::rSelect(QModelIndex index){
    int rand=index.row();
    QString id=index.sibling(rand,0).data().toString();
    emit semnalAngSel(id);
}

To copy to clipboard, switch view to plain text mode 

Here I create the table and connect the semnalAngSel(QString) signal:

void widget::tabelSql(QString tab){
    curatare();     //this is a function which cleans the layout
    if(tab=="lAng"){
        ListaAng *lAng=new ListaAng();
        connect(lAng,SIGNAL(semnalAngSel(QString)),this,SLOT(ferVizAng(QString)));
        layoutCentral->addWidget(lAng);
        lAng->selectPrimR();
    }
}

void widget::tabelSql(QString tab){
    curatare();     //this is a function which cleans the layout
    if(tab=="lAng"){
        ListaAng *lAng=new ListaAng();
        connect(lAng,SIGNAL(semnalAngSel(QString)),this,SLOT(ferVizAng(QString)));
        layoutCentral->addWidget(lAng);
        lAng->selectPrimR();
    }
}

To copy to clipboard, switch view to plain text mode 

And here is where I call my Dialog:

void widget::ferVizAng(QString id){
    FerDialAng *fda=new FerDialAng(this,id);
    connect(fda,SIGNAL(semnalAct(QString)),this,SLOT(tabelSql(QString)));  //here I call the table to "refresh"
    fda->exec();
}

void widget::ferVizAng(QString id){
    FerDialAng *fda=new FerDialAng(this,id);
    connect(fda,SIGNAL(semnalAct(QString)),this,SLOT(tabelSql(QString)));  //here I call the table to "refresh"
    fda->exec();
}

To copy to clipboard, switch view to plain text mode 

[anda_skoa]

Hmm, hard to tell.

Looks very strange, but nothing stands out as obviously wrong.

Can you break it down to a compilable project that you can attach as a ZIP file?

Cheers,
_

[adutzu89]

I will try tommorow. Thank you.

[ChrisW67]

Put a breakpoint in each of the functions: ListaAng::rSelect(), void widget::tabelSql, widget::ferVizAng() and curatare().
Run your program in the debugger. Single step until it crashes then you will know what your program did to deserve it.

My money is on whatever you are hiding in curatare().
Really though, refreshing the content of the table should involve causing the QSqlQueryModel to requery the data, not stuffing around with new widgets and layouts.

// QSqlQueryModel *model;
 
model->setQuery(model->query());

// QSqlQueryModel *model;

model->setQuery(model->query());

To copy to clipboard, switch view to plain text mode 

[adutzu89]

Put a breakpoint in each of the functions: ListaAng::rSelect(), void widget::tabelSql, widget::ferVizAng() and curatare().
Run your program in the debugger. Single step until it crashes then you will know what your program did to deserve it.

My money is on whatever you are hiding in curatare().
Really though, refreshing the content of the table should involve causing the QSqlQueryModel to requery the data, not stuffing around with new widgets and layouts.

// QSqlQueryModel *model;
 
model->setQuery(model->query());

// QSqlQueryModel *model;

model->setQuery(model->query());

To copy to clipboard, switch view to plain text mode 

curatare() contains the following:

QLayoutItem *child;
    while ((child = layoutCentral->takeAt(0)) != 0) {
        delete child->widget();
        delete child;
    }

QLayoutItem *child;
    while ((child = layoutCentral->takeAt(0)) != 0) {
        delete child->widget();
        delete child;
    }

To copy to clipboard, switch view to plain text mode 

Which removes any widgets that are in my central widget, but this is only called when I trigger menu actions and not when I refresh the table, I will try first your solution as is it now I am remaking the tableview entirely.
And it only cause my app to crash after calling rSelect(QModelIndex) from doubleClick, if I call it from shortcut(QKeySequence) it does not crash.


Update:

Solved the issue, thank you both.
I restructured my code, and now I am just resetting the query as you,ChrisW67, said; also as you both seen I was calling both tabelSql() and ferVizAng() from widget and when semnalAct(QString) signal was emited the app would do tabelSql()(trying to clean the central widget and remaking the whole table) all over again which here is where my app was crashing.

Now I moved the ferVizAng() slot inside table file and just reseting the query when I press the accept button.

Again thank you both for the assistance