Refresh QLCDNumber

mrjj

Hi
So is
if(this->get_m_boxGGI() == 18703)
{
Window018703 *window018703 = new Window018703(); << the real instance ?
QObject::connect(ui->pushButtonOk, &QPushButton::clicked , window018703, &Window018703::windowView);

the only one ?
also then its the THIRD class that needs to talk to Window018703 ?

martial123

@mrjj

unless I'm wrong but yes it's the first instance
and

yes the third class talk to increment display ...

the first class is hide because it just serves in start to fill variables especially : m_boxGGI ...

mrjj

Hi
ok.
Where is the code ?
if(this->get_m_boxGGI() == 18703)
{...
is that in first window ?
So if you store
Window018703 *window018703 as a member, you can use it later to connect
when you open the third one?

martial123

@mrjj yes the code is in the first class ( first window)

how i can store in in membre ??

mrjj

@martial123
and from first window, you also open the third one ? correct?
To make it memeber.
simply put
Window018703 *window018703 = nullptr; in first window.h inside class
like

class MainWindow : public QMainWindow
{
    Q_OBJECT
......
private:
    Ui::MainWindow *ui;
  Window018703 *window018703 = nullptr;  // also include the #include "Window018703.H" in top of file
};

and then change code here

if(this->get_m_boxGGI() == 18703)
{
window018703 = new Window018703(); << use the one from .h
QObject::connect(ui->pushButtonOk, &QPushButton::clicked , window018703, &Window018703::windowView);

Then later the variable in .h (window018703 ) points to the instance.

martial123

@mrjj

in fact, it is the main who show the third window but its not importante,

i did what you told me , but i can not to catch my instance ( or member ...) like this with get methode to call him in the third classe :

in first window .cpp :

Window018703* get_window018703()
{
    return window018703;
    
}

.h :

        Window018703* get_window018703();

he tell me an error :
use of undeclared identifier 'window018703'

mrjj

Hi
Well then maybe you can connect in main ?
You really need access to the instance and there is no magical way to get it.
So if main know all 3 windows, you can connect there ?

martial123

@mrjj
the instance is created when i do that no ? :

if(this->get_m_boxGGI() == 18703)
       {
           window018703 = new Window018703();

           QObject::connect(ui->pushButtonOk, &QPushButton::clicked , window018703, &Window018703::windowView);

       }

mrjj

@martial123
Yes but you need that window018703 variable in third form.
And third form is created in main ?

martial123

@mrjj
yes my main show first and third window ... :

#include <QApplication>
#include <QtWidgets>
#include <QLabel>

#include "window1.h"
#include "window018703.h"
#include "window018704.h"


using namespace std;
int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
       window1 window1;
       window1.show();

       Window018704 window2;
       window2.show();
  
    return app.exec();
}

mrjj

@martial123

So connect it all up there?
Im getting a bit confused with the names, so not sure if 04 has 03 or reverse.


     window1 window1;
     window1.show(); << this has window018703 ?

     Window018704 window2;
     connect( window1.get_window018703(), xxxx )
       
       window2.show();

martial123

@mrjj

No ^^

i have 3 classes :

first : window1
second: Window018703
third : Window018704 ...

mrjj

@martial123
well you need one place where you have the instance you want to connect.
I not sure where Window018703 comes to life, but cant you call the get_window018703()
on it from main to get the instance.
It must be in window1 or window2 ? (variables)

martial123

@mrjj

I do not understand much anymore..

i have the first class (window1) wiche is call and show by main :

window1.cpp :

#include "window1.h"
#include "ui_window1.h"
#include <iostream>
#include "window018703.h"
#include "window018704.h"

//Constructor
window1::window1(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::window1)
{
    //lance la construction de la fenetre
    ui->setupUi(this);
    //Signal emis
    QObject::connect(ui->pushButtonCancel, &QPushButton::clicked, qApp, &QCoreApplication::quit);
    QObject::connect(ui->pushButtonOk, &QPushButton::clicked, this, &window1::processing);

}

//Destructor
window1::~window1()
{
    delete ui;
}

//Gestion des erreurs SLOT en fonction des signaux emis
// Range le contenu des champ dans un objet de type QString
//Si c'est champ sont vide, alors on emet une fenetre erreur
//Sinon, on ferme la fenetre.
//************* Maj : Au lieu de fermer la fenetre, elle doit emmetre un signal pour ouvrir la suivante.*************
//done  : test if on device choose by user , open the right window device.
void window1::processing()
{

    QString contents_m_fieldJob = ui->fieldJob->text();
    QString contents_m_fieldLotNumber = ui->fieldLotNumber->text();
    QString contents_m_fieldLogFile = ui->fieldLogFile->text();
    QString contents_m_fieldOperator = ui->fieldOperator->text();
    QString contents_m_boxNumberDevice = ui->boxNumberDevice->text();
    //Window018703 *obj1 = new Window018703();
    if(contents_m_fieldJob.isEmpty() || contents_m_fieldLotNumber.isEmpty() ||
      contents_m_fieldLogFile.isEmpty() || contents_m_fieldOperator.isEmpty() || contents_m_boxNumberDevice.toInt()== 0)
    {
        QMessageBox::information(this, "ERROR Message", "ERROR: Fill in all entries");
    }
    else
    {        
        QObject::connect(ui->pushButtonOk, &QPushButton::clicked , this, &QWidget::hide);
        if(this->get_m_boxGGI() == 18703)
        {
            //Window018703 *window018703 = new Window018703();
            window018703 = new Window018703();

            QObject::connect(ui->pushButtonOk, &QPushButton::clicked , window018703, &Window018703::windowView);
   


        }
        else if(this->get_m_boxGGI() == 18704)
        {
            Window018704 *window018704 = new Window018704();
            QObject::connect(ui->pushButtonOk, &QPushButton::clicked , window018704, &Window018704::windowView);

        }
    }

}


/***********************************GET***********************************/
// Permet de recuperer les
// utile pour le fichier de rapport
// numero device utile pour la configuration
// nombre de device utile pour la conf
int window1::get_m_fieldJob() const
{
    QString contents_m_fieldJob = ui->fieldJob->text();
    return contents_m_fieldJob.toInt();
}

int window1::get_m_fieldLotNumber() const
{
    QString contents_m_fieldLotNumber = ui->fieldLotNumber->text();
    return contents_m_fieldLotNumber.toInt();
}
int window1::get_m_fieldLogFile() const
{
    QString contents_m_fieldLogFile = ui->fieldLogFile->text();
    return contents_m_fieldLogFile.toInt();
}
int window1::get_m_fieldOperator() const
{
    QString contents_m_fieldOperator = ui->fieldOperator->text();
    return contents_m_fieldOperator.toInt();
}
int window1::get_m_boxNombreDevice() const
{
    QString contents_m_boxNombreDevice = ui->boxNumberDevice->text();
    return contents_m_boxNombreDevice.toInt();
}

int window1::get_m_boxGGI() const
{
    QString contents_m_boxGGI = ui->boxGGI->currentText();
    return contents_m_boxGGI.toInt();
}

Window018703* window1::get_window018703() const
{
    return window018703;

}
/**************************************************************************/

window1.h :

#ifndef WINDOW1_H
#define WINDOW1_H
#include <QWidget>

#include <QDialog>
#include <QMessageBox>
#include <iostream>
#include <QApplication>
#include "window018703.h"
#include "window018704.h"


namespace Ui {
class window1;
}

class window1 : public QDialog
{
    Q_OBJECT

    public:
        explicit window1(QWidget *parent = nullptr);
        ~window1();

        /*get*/
        int get_m_fieldJob() const;
        int get_m_fieldLotNumber() const;
        int get_m_fieldLogFile() const;
        int get_m_fieldOperator() const;
        int get_m_boxNombreDevice() const;
        int get_m_boxGGI() const;

        Window018703* get_window018703() const;

        /*set*/

    public slots:
        void processing();

    private:
        Ui::window1 *ui;
        Window018703 *window018703 = nullptr;
};

#endif // WINDOW1_H

main.cpp :

#include <QApplication>
#include <QtWidgets>
#include <QLabel>

#include "window1.h"
#include "window018703.h"
#include "window018704.h"


using namespace std;
int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
       window1 window1;
       window1.show();

      Window018704 window3;
      window3.show();
       //ui->lcdNumber_seatUp->QLCDNumber();
       //qApp->setStyleSheet("Window018703 { background-color: dark }");
       //pour rendre le bouton de tyoe QPushButton non utilisable:
       //QPushButton *bouton = new QPushButton("bouton");
       //bouton->setEnabled(false)
    return app.exec();
}

at the start of program, this main show window 1 and window3 ( window018704)

in the first class window1, we are a slot : void window1::processing()
in this slot if users are fill in all input, then we call slot with this connect :

window018703 = new Window018703();
QObject::connect(ui->pushButtonOk, &QPushButton::clicked , window018703, &Window018703::windowView);

this connect are receive by 2 class ( window018703) :
Window018703.cpp :

#include "window018703.h"
#include "ui_window018703.h"
#include <iostream>

//#define  B 1
using namespace std;
Window018703::Window018703(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::Window018703)
{
    ui->setupUi(this);
    //int a =5;
    cout << " nous somme avant l'appuis " <<  endl;

    QObject::connect(ui->pushButton, &QPushButton::clicked, this, &Window018703::set_lcdNumber_seatUp);
    cout << " nous somme apres l'appuis " <<  endl;

}

Window018703::~Window018703()
{
    delete ui;
}

void Window018703::windowView()

{
    this->show();
}

void Window018703::set_lcdNumber_seatUp()
{
    static int B = 0;
    cout << " önt rentre ici et a vaut = " << B << endl;
    //this->show();
    ui->lcdNumber_seatUp->display(B++);
}

slot void Window018703::windowView() is execute and the second window are show.

so now , if i the connect in second class is execute ( QObject::connect(ui->pushButton, &QPushButton::clicked, this, &Window018703::set_lcdNumber_seatUp);)

the display QLCDNumber is incremented because we are on the same instance (this)

but if i connect with the third class ( Window018704) , it doesn't work .

i have implemented the get but i can not :

Window018704.cpp :

#include "window018704.h"
#include "ui_window018704.h"
#include <iostream>
#include "window1.h"


using namespace std;

Window018704::Window018704(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::Window018704)
{
    ui->setupUi(this);
    cout << " classe 704 ... " <<  endl;

   window1 *instance1 = new window1(); ;
    Window018703 *instance3 = new Window018703();

   instance3 = instance1->get_window018703();

   QObject::connect(ui->pushButton_018704, &QPushButton::clicked , instance3, &Window018703::set_lcdNumber_seatUp);

}

Window018704::~Window018704()
{
    delete ui;
}

void Window018704::windowView()
{
    this->show();

}

Window018704.h :

#ifndef WINDOW018704_H
#define WINDOW018704_H

#include <QDialog>
#include "window018703.h"
#include <QApplication>
#include <QObject>

namespace Ui {
class Window018704;
}

class Window018704 : public QDialog
{
    Q_OBJECT

    public:
        explicit Window018704(QWidget *parent = nullptr);
        ~Window018704();

    public slots:
        void windowView();

private slots:
        //void on_pushButton_018704_clicked();


private:
        Ui::Window018704 *ui;
};

#endif // WINDOW018704_H

i have understand what u tell me , but i can not receve the good instance from first class ( window018703 = new Window018703(); ) to connect the good instance in the third class :

window1 *instance1 = new window1(); ;
Window018703 *instance3 = new Window018703();

instance3 = instance1->get_window018703();

QObject::connect(ui->pushButton_018704, &QPushButton::clicked , instance3, &Window018703::set_lcdNumber_seatUp);

Thank for your time ..

mrjj

@martial123
I fear, you make new ones on the go.

window1 *instance1 = new window1(); ;

is that not a new instance compared to the one in main.cpp?

Please tell from main.cpp

   window1 window1;<< this has window018703 ?
   Window018704 window2; << this has window018703 ?

we should be able to connect here.

martial123

@mrjj

Hi ,

You don't understand,
I have 3 graphic class:

Window 1
Window 018703
Window 018704

so Window1 is not Window 018703 ...
when i do that :

window1 window1;
       window1.show();

I created instance for the first class ( not by the second class 018703)
If I would like to created the instance of Window018703 i do :

Window018703 window2;
      window2.show();

ok ?
so the instance we want to recover is the instance Window018703 which is created in the slot in first class ( window1) like this ":

  if(this->get_m_boxGGI() == 18703)
        {
           **window018703 = new Window018703();

            QObject::connect(ui->pushButtonOk, &QPushButton::clicked , window018703, &Window018703::windowView);**
}

And we want to recover this instance(window018703) in the thrid class
so to do that, with your advices, i have created the function get_window018703() in the first class , who able to return the only one instance like this :

Window018703* window1::get_window018703() const
{
    return window018703;

}

for do that , u tell me to created the member in window1.h :

Window018703* window1::get_window018703() const
{
    return window018703;

}

so after that, in the third class ( window018704)
I try to recover this instance to call slot increment my display ... but i can not.

I don't want to created or recover this instance in my main because the connect are in the third class ... so I think it is necessary to recover instance ( window018703) in the third class .

you understand or it's not clear ??

Thank for your time, and sorry for that late response ..

martial123

no ideas ?

SGaist

Hi,

@martial123 said in Refresh QLCDNumber:

no ideas ?

Please show some patience and allow 24 hours to pass before bumping your own thread. People answering here do it on their own time and may not live in the same time zone as you.

As for your problem, currently your code is pretty convoluted.

Can you describe what your application is doing and the responsibility of the various widget ?
Also, please, give your class more meaningful names. Windows1, Window018703 and Window018704 doesn't really help at all understanding the relation between your widgets.

martial123

I have just write no idea lol
So I explain again

i have 3 classe .JUST 3.

Each class corresponds to a graphical interface.
So we have 3 Interface Graphical.

1 er interface
the first class is instantiated in the main.cpp
the first class correspond to a graphical interface. it serves to collect information write by user and store that in variables.
When all variable are full, this interface emit signal to connect with a slot in the second class

2 eme interface
The second interface are the main window, the main application
This second interface are called by the first. Is own slot allows to show this interface.
This second interface contain a lots of widgets, but in this topic, QLCDNumber are important.
This interface allows to incremented many QLCDNumber.
To do that, it has a slot who able to incremented QLCDNumber.

3 eme interface
The last class is composed with 1 widget (QPushButton) .
When user click on this button, the slot in interface 2 is called by connect to incremente value of QLCDNumber.

It's just that.

The main problem,
how to recover an instance in another class from the one that created it ?
Because if i use an other intance to call the slot in interface2 , My QLCDNumber doesn't refresh because it's not the same instance .. That is the problem.

Ty for time.I can not be more explicit ...

SGaist

So you are currently doing thing in the wrong sense and order.

The 3rd widget with its push button should not know anything about the second. From the looks of it, it should be instantiated and connected in the main function. Or alternatively be a member variable of your MainWindow class and be connect by the MainWindow itself.

As for your first widget, since it's a dialog, do you need to call it only once at the start of your application of multiple times ?

In any case, you can call it from within your MainWindow and handle its content there.