Signal doesn't work properly

[VRonin]

What's the link between Review and MainWindow?

[gabor53]

@VRonin
In Review the data added to the database and I want to update the table in MainWindow using the record just added to the db.

[VRonin]

@gabor53 Sorry, I did not explained myself. Review is a Dialog, where do you make this dialog appear?

[gabor53]

@VRonin
In MainWindow line 75

void MainWindow::on_actionAdd_a_Friend_triggered()
{
    Additem *mAddItem = new Additem;
    mAddItem->exec ();
}

[VRonin]

Now you lost me completely.
typedef Review Additem; ?? I guess not because

void Review::on_pushButton_Fix_clicked()
{
    Additem *mAdditem = new Additem(this);

[gabor53]

@VRonin
Originally, after main.cpp MainWindow.cpp loads. MainWindow has the tableview that displays 4 columns from the database. The user can call Additem.cpp from MainWindow to add more items to the db. After data entry Review.cpp is called where the user can review the data he/she entered. Here there are two choices: either click PushButton_fix to correct errors (this takes back to Additem.cpp) or if all correct you can click Add_to_Db and the item is added to the db. After adding RecAdded signal emitted to add the new record to the tableview in MainWindow. I hope this helps.

[VRonin]

Ok, your design is "borderline". Could you please post additem.h and additem.cpp too please?

[gabor53]

@VRonin
additem.h
additem.cpp

Thank you for your help.

[VRonin]

Unrelated to main topic but I just took a glimpse at your code:

QString date = QDate::currentDate().toString ();
    list = date.split (QRegExp("\\s+"));
    int size = list.size ();
    QString sYear = list.at (size-1);

    yeark = sYear.toInt ();

do you mean?

yeark = QDate::currentDate().year();

[gabor53]

Yes. I was complicated.

[Venkatesh V]

Hai....
you just try by this line of code instead of your connect statement
connect(this,&Review::RecAdded,this,&Review::updateTable,Qt::UniqueConnection);
It may help you..

[gabor53]

Hi @Venkatesh-V
It didnt do the trick.

[VRonin]

No, clearly not. I'm working on the solution, I'll post it shortly. I'm trying to find the time and avoid to correct too much

[gabor53]

@VRonin
Thank you. I appreciate it.

[gabor53]

@gabor53
When you have time please let me know how can I improve the design.

[Venkatesh V]

Hai..@gabor53
Can you share your whole code? so that i can try to resolve.

[gabor53]

@Venkatesh-V See above

[gabor53]

@Venkatesh-V Thank you

[gabor53]

So far whatever I've come up with it opens a second MainWindow.