Pointeur sur ma classe issue de CDialog

**Nixeus** · 14/03/2010, 18h07

Bonjour à tous,

Une petite question de newbie.

J'ai développé une petite application graphique avec visual studio 2008 .
Je possède deux forms

CDialog1
CDialog2

Sont les deux classes gérant mes boites de dialogues.

Dans CDialog1.h je créer une donnée membre (pointeur).

Je souhaiterai récupérer cette donnée dans mon autre classe.

Pour se faire, il faut bien entendu que je puisse récupérer l'instance de ma Dialog1 dans ma Dialog2.

Le soucis est que....je ne sais pas comment faire

Je sais que cela se fait avec le constructeur ( si j'ai bien compris), mais je ne vois pas comment faire.

Quelqu'un pourrait m'éclairer à ce sujet ?

Merci beaucoup,

Cordialement,

Nixeus

**LittleWhite** · 14/03/2010, 18h28

Bonjour,

Comme il a été dis dans le chat, il se peut que ce que vous cherchez à faire vient d'un problème de design de l'application.

Sinon pour expliquer l'histoire du constructeur:

Dialog1.h

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
class Dialog1
{
    private:
        int* m_MyData;
    public:
        Dialog1();
        int* getMyData(void)const { return m_MyData; };
};

( Le code du Dialog1.cpp n'est pas très important. )

Dialog2.h

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class Dialog2
{
    private:
 
        Dialog1* d1;
 
    public:
        Dialog2(const Dialog1& d1):d1(d1) {};
        void foo(void)
        {
            d1->getMyData(); // Par exemple
        }
};

Voilà, dans l'idée, le code qu'il faut faire.

Sinon, il peut y avoir l'utilisation d'un Singleton ( même si on le recommande pas )

**Nixeus** · 14/03/2010, 18h37

Merci pour ta réponse LittleWhite.

POur mon costructeur, celui de base est le suivant :

CDialog1Dlg(CWnd* pParent = NULL)

Je dois donc le transformer en :

CDialog1Dlg(CWnd* pParent = NULL, const Dialog1& d1 ):d1(d1){};

**LittleWhite** · 14/03/2010, 18h57

Ceci est faux:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

CDialog1Dlg(CWnd* pParent = NULL, const Dialog1& d1 ):d1(d1){};

Pardon, c'est de ma faute.
Le prototype sera le suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
CDialog1Dlg(const Dialog1& d1, CWnd* pParent = NULL )

Car les valeurs par défault, ne peuvent être qu'à la fin du prototype.
De plus, la liste d'initialisation

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
:d1(d1)

Je pense qu'il faudrait que vous regardiez la FAQ, ou un tutoriel, disponible sur le site, pour ce sujet.

Puis votre code ( corps de la fonction ), dans le .cpp, devra copier d1 dans le d1 de la classe.

**Nixeus** · 14/03/2010, 19h02

Merci beaucoup, voici donc mon constructeur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
CDialog1Dlg(const CTestPDADlg& accueilDlg, CWnd* pParent = NULL ):accueilDlg(accueilDlg){};

Cependant le compilateur me dit :

Erreur 1 error C2614: 'CDialog1Dlg'*: initialisation de membre non conforme*: 'accueilDlg' n'est ni une base ni un membre c:\users\crashoverhead\desktop\testpda\testpda\testpda\Dialog1Dlg.h 16 TestPDA

Comment dois-je interpréter cette erreur ?

Encore merci

**3DArchi** · 14/03/2010, 19h25

Salut,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
CDialog1Dlg(CTestPDADlg *accueilDlg_, CWnd* pParent = NULL )
   :CDialog(pParent),m_accueilDlg(accueilDlg_){};
// ou
CDialog1Dlg(CTestPDADlg &accueilDlg_, CWnd* pParent = NULL )
   :CDialog(pParent),m_accueilDlg(&accueilDlg_){};
 
// en supposant :
class CDialog1Dlg : public CDialog
{
 /* [... ]*/
CTestPDADlg *m_acceuilDlg
};

Ta variable membre étant un pointeur, c'est bien sûr un pointeur à passer en paramètre. Ou une référence non constante (puisque le pointeur n'est pas constant) dont tu récupères l'adresse.

Mais comme l'a dit LittleWight, c'est un mauvais choix de design qui couple trop fortement tes deux classes et va rendre les évolutions et les corrections de bugs laborieuses.

**Nixeus** · 14/03/2010, 19h45

Merci de m'avoir répondu !

Honnetement, je suis complètement perdu entre mon header et mon cpp....

J'avais oublié que c++ demandait autant de rigueur.

Penses tu que si j'adapte mon programme avec une classe "fabricateur" ( mon prof de BTS appelais ca comme cela), ça puisse faciliter les choses ?