pass variable from tab to dialog

[TheCipo76]

@mrjj sorry.. yes is only one table in the same database

i'm already using QSqlTableModel in my app

but if i use QSqlTableModel for the tableview the data was stored immediately on the database..

instead i want only populate it with comboboxes (to generate a preview)
and then store the data only with "Registra" click (after that i have checked input by code)..

all to prevent duplication, error and other problem that can be created by users input.

[mrjj]

@TheCipo76
Ah, i assumed user could only select valid data from the combo boxes.
But you could change
QSqlTableModel::setEditStrategy() to QSqlTableModel::OnManualSubmit
check the model from code and do
QSqlTableModel::submitAll() to save to db.
?

[TheCipo76]

@mrjj how can i access to tabs comboboxes data from mainwindow??

i've declarated in .h file like "public" all the comboboxes (in all the tab forms)

when i click "Aggiorna" i want to read all comboboxes of all tabs (with 2 nidificated loop: 1 for all tab, and inside the first, 1 for all comboboxes of each tab) and populate the qsqltablemodel (tableview) in mainwindow form

i'm trying to store all data in some QString before to insert in model

i've tried with this code:

Lavorazione = tabcomplessivo::array_Lavorazione[j]->currentText();

but i have this error:

invalid use of non-static data member 'array_Lavorazione'

[mrjj]

@TheCipo76

Hi
The syntax you are using would mean a static variable. ( hence the error)
You need a pointer to the dialog. ( i assume the combobox are in other dialog ? )
dialogptr->tabcomplessivo->array_Lavorazione[j]->currentText();

Often you would do it same place where you show dialog
like
TheDialog dia; // we dont need to NEW it as we use exec() which blocks.
if(dia.exec() == QDialog::Accepted)
{
// grab the data
dia.tabcomplessivoPtr.array_Lavorazione[j]->currentText();
...
}

[SGaist]

Are you only preparing the data to be stored in the database or will you also be showing entries from the database in that table view ?

[TheCipo76]

@mrjj
as you can see in the first post isn't a dialog but
tabcomplessivo and tabcomponente were
simple forms pass like tabs to tabwidget

[mrjj]

@TheCipo76
So they are actual inside the mainwindow ?

[TheCipo76]

@mrjj Yes

[TheCipo76]

@SGaist only prepare a preview
then (on "Registra" click) it will be stored in db

otherwise the tableview need filter in order to don't show (and modify) other data (item) in the table

both case is good however

[SGaist]

In that case, wouldn't it be easier to use a QTableWidget to setup the rows and then QSqlQuery to update your database when you click on Registra

[TheCipo76]

@SGaist yes but i can't access to data in tabcomplessivo and tabcomponente
as i've already wrote up they were in the mainwindow and declared pulic

[SGaist]

Just to be sure, do you have several different widgets that you built with designer and that you are putting together in that dialog ?

[TheCipo76]

@SGaist i've two different forms (but similar 90%)
n.1 tabcomplessivo
some tabcomponente
all in the mainwindow

i've created with designer and added to tablewidget with code (first post) and populated dinamically with different table (from database) with different query

i have always only one tabcomplessivo and some (depends by item in database)
tabcomponente

complessivo = componente + componente +... + componente
i hope that's clear (my english is not very good..)

i have to describe all operations on every componente to build one complessivo

[SGaist]

Then you should implement proper APIs for these different forms to pass values around so each will be an independent entity.

You might have to rethink a bit what element of your UI is responsible of what. For example, does it really make sens that Aggiorna is part of the left widget ? Shouldn't it be outside so that the slot connected to it can grab the data generated and then put them in the QTableWidget.

[TheCipo76]

Really there's no way to read data from Tabs??

TabWidget (and then all the Tabs) were inside the dialog as you see in the picture..

"Aggiorna" is already in the Dialog (not in TabWidget).

Including tabcomplessivo (form) and tabcomponente (form) in the TabWidget do not have any sense if i can't access to it's data from the Dialog.

Create a single Connect/signal from each comboBox/LineEdit do not allow to insert complete row of Model (TableView).

My idea was:

• firts select/insert in Tabs
• in any moments i can view the actual input with Aggiorna click() that put all selections/input data in the Model (TableView).

I've already implemented QSqlTableModel (TableView) and Aggiorna void to get data from Dialog and insert to Model (Codice, Complessivo..) the only missing part is get data from Tabs (with 2 loop as i've already wrote).

I have to delete all and do with another method??

[mrjj]

@TheCipo76
Hi
The reason I are confused is that is not clear if you simply added
the tabs in Designer. ( with combo boxes and all)
In that case the combobox are directly readable to the dialog via UI

OR

You made a new form ui object with the both .h and .cpp
and you make a new instance of your form and use
as Tab widget.
In that case UI is private and you need an access method to get data and the pointer of
the widget as you create it to be used as TabWidget.

[TheCipo76]

@mrjj it's the second case..

now i've two possibility

delete forms (tabcomplessivo and tabcomponente) and insert directly ComboBoxes and LineEdits data in the Tabs (Simplest way)

or

implement a method to get data from froms

The second way is my favourite but i don't know to do it

my initial question is precisely what
how to get data from Tabs (forms uses as TabWidget)

Probably i've explain not very well.. Sorry

[mrjj]

@TheCipo76
Hi
Well you just keep the pointer around from
when you new the Form class and insert as tabwidget.

And then in Form class , you add access method that will return
the data from the combo boxes since UI is private.

[TheCipo76]

@mrjj this is tabcomponente .h

#ifndef TABCOMPONENTE_H
#define TABCOMPONENTE_H

#include <QWidget>
#include <QComboBox>
#include <QSqlDatabase>

namespace Ui {
class tabcomponente;
}

class tabcomponente : public QWidget
{
    Q_OBJECT

public slots:

    void MACCHINE();

public:
    explicit tabcomponente(QWidget *parent = nullptr, QString nome = "");
    ~tabcomponente();

private:
    Ui::tabcomponente *ui;

    QSqlDatabase aDatabase;

    //Array Lavorazioni
    QComboBox *array_Lavorazione[5];
    QComboBox *array_Macchine[5];
    QLineEdit *array_PZOra[5];

};

#endif // TABCOMPONENTE_H

and this is how i've added to TabWidget

ui->tabWidget->addTab(new tabcomponente(this, comp.value(0).toString()), comp.value(0).toString());

comp.value(0) get name of Tab from Database

where the pointer will be added??

[mrjj]

Hi
I mean keep the pointer around. like
tabcomp = new tabcomponente(this, comp.value(0).toString());
ui->tabWidget->addTab(tabcomp);

in .h you have
tabcomponente *tabcomp=0;

so later u can use tabcomp to get access .

Right now, you are not keeping the returned pointer from new.

Alternatively, you can get it from tabwidget.
via the http://doc.qt.io/qt-5/qtabwidget.html#widget
You have to use qobject_cast to cast to the right type as it returns the widget
as base class, not your type.

tabcomponente* thetab = qobject_cast<tabcomponente*>(  ui->tabWidname->widget( INDEX )  );
  if( thetab ) { // MUST perform check. it can fail if not that type.
    thetab->xxxx
  }