Problème d'appels d'une variable entre fonctions

Version imprimable

25/04/2006, 11h46
julien.nasser

Problème d'appels d'une variable entre fonctions

Bonjour,

Voilà mon problème :
J'ai une classe CRequetes, j'ai déclaré
public : CString nom; dans Requetes.h
J'ai une fonction void CRequetes::affiche() dans laquelle je modifie nom :
nom="julien";
Je l'affiche AfxMessageBox(nom); L'affichage se passe bien

Mais dans une autre fonction void CRequetes::affiche2() je veux l'afficher en faisant AfxMessageBox(nom); et il m'affiche une chaine vide.

Mais pourquoi ça ne marche pas ?? :(
25/04/2006, 11h55
moulefrite

salut
declare ton CString nom dans le fichier global.h ;)
25/04/2006, 12h41
julien.nasser

Je n'ai pas de fichier global.h (c'est un projet MFC/MDI)
25/04/2006, 13h31
nico-pyright(c)

tu es sur de ne pas avoir instancié un autre objet de ta classe ?
25/04/2006, 13h35
julien.nasser

Ma classe CRequetes sert à manipuler une dialogbox
elle est dérivée de CDialog
class CRequetes : public CDialog
est-ce que le problème viendrait de là ?
25/04/2006, 15h00
hiko-seijuro

je suis dsl mais je crois qu'il nous manque des éléments pour te répondre est ce que tu pourrais montrer ou ces methodes sont appelées par exemple ?

Voilà les fichiers requetes.cpp et requetes.h
CString nom est déclaré en public dans requetes.h
je défini nom="julien" dans requetes.cpp
dans la fonction ComboGraphes, julien s'affiche
dans la fonction oninitdialog, c'est une chaine vide qui s'affiche

fichier requetes.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
 
// Requetes.cpp : implementation file
//
#include "stdafx.h"
#include "prog.h"
#include "Requetes.h"
#include "base.h"
#include "ADOLib.h"
#ifdef _DEBUG
#define new DEBUG_NEW
#undef THIS_FILE
static char THIS_FILE[] = __FILE__;
#endif
/////////////////////////////////////////////////////////////////////////////
// CRequetes dialog
 
CRequetes::CRequetes(CWnd* pParent /*=NULL*/)
 : CDialog(CRequetes::IDD, pParent)
{
 //{{AFX_DATA_INIT(CRequetes)
 m_Date_Recherche_Value = 0;
 m_RechercheType = _T("");
 m_RechercheCle = _T("");
 m_FiltrerType = _T("");
 m_FiltrerGraphe = _T("");
 m_FiltrerCle = _T("");
 //}}AFX_DATA_INIT
}
 
void CRequetes::DoDataExchange(CDataExchange* pDX)
{
 CDialog::DoDataExchange(pDX);
 //{{AFX_DATA_MAP(CRequetes)
 DDX_Control(pDX, IDC_COMBO2, m_RechercheGrapheControl);
 DDX_Control(pDX, IDC_DATETIMEPICKER1, m_Date_Recherche_Control);
 DDX_DateTimeCtrl(pDX, IDC_DATETIMEPICKER1, m_Date_Recherche_Value);
 DDX_CBString(pDX, IDC_COMBO1, m_RechercheType);
 DDX_CBString(pDX, IDC_COMBO3, m_RechercheCle);
 DDX_CBString(pDX, IDC_COMBO4, m_FiltrerType);
 DDX_CBString(pDX, IDC_COMBO5, m_FiltrerGraphe);
 DDX_CBString(pDX, IDC_COMBO6, m_FiltrerCle);
 //}}AFX_DATA_MAP
}
 
BEGIN_MESSAGE_MAP(CRequetes, CDialog)
 //{{AFX_MSG_MAP(CRequetes)
 ON_BN_CLICKED(IDOK2, OnOk2)
 ON_BN_CLICKED(IDOK3, OnOk3)
 ON_BN_CLICKED(IDCANCEL2, OnCancel2)
 ON_BN_CLICKED(IDCANCEL3, OnCancel3)
 ON_NOTIFY(DTN_DATETIMECHANGE, IDC_DATETIMEPICKER1, OnDatetimechangeDatetimepicker1)
 //}}AFX_MSG_MAP
END_MESSAGE_MAP()
/////////////////////////////////////////////////////////////////////////////
// CRequetes message handlers
void CRequetes::OnOK() 
{
  int nRetCode=IDOK;
  UpdateData();
  EndDialog( nRetCode );
}
void CRequetes::OnOk2() 
{
 int nRetCode=IDOK2;
 UpdateData();
 EndDialog( nRetCode );
}
void CRequetes::OnOk3() 
{
 int nRetCode=IDOK3;
 UpdateData();
 EndDialog( nRetCode );
}
void CRequetes::OnCancel() 
{
 CDialog::OnCancel();
}
void CRequetes::OnCancel2() 
{
 CDialog::OnCancel(); 
}
void CRequetes::OnCancel3() 
{
 CDialog::OnCancel(); 
}
void CRequetes::ComboGraphes(ADOConnect* myConn)
{
 nom="julien";
 AfxMessageBox(nom);
}
void CRequetes::OnDatetimechangeDatetimepicker1(NMHDR* pNMHDR, LRESULT* pResult) 
{
 NMDATETIMECHANGE *pDTChange= reinterpret_cast<NMDATETIMECHANGE *>(pNMHDR);
    if (pDTChange->dwFlags==GDT_NONE)
  //AfxMessageBox(" case décochée ! ");
  etat_case=false;
  else if (pDTChange->dwFlags==GDT_VALID)
  //AfxMessageBox(" case cochée ! ");
  etat_case=true;
    *pResult = 0;
}
BOOL CRequetes::OnInitDialog() 
{
 CDialog::OnInitDialog();
 AfxMessageBox(nom);
 return TRUE;  // return TRUE unless you set the focus to a control
               // EXCEPTION: OCX Property Pages should return FALSE
}

fichier requetes.h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
#if !defined(AFX_REQUETES_H__EBC09B53_AF2A_46AA_82B6_FC1B6C47233C__INCLUDED_)
#define AFX_REQUETES_H__EBC09B53_AF2A_46AA_82B6_FC1B6C47233C__INCLUDED_
#if _MSC_VER > 1000
#pragma once
#endif // _MSC_VER > 1000
// Requetes.h : header file
//
#include "ADOLib.h"
/////////////////////////////////////////////////////////////////////////////
// CRequetes dialog
class CRequetes : public CDialog
{
// Construction
public:
 CRequetes(CWnd* pParent = NULL);   // standard constructor
 void ComboGraphes(ADOConnect* myConn);
 bool etat_case;
 CStringArray remplir_combo;
// Dialog Data
 //{{AFX_DATA(CRequetes)
 enum { IDD = IDD_DIALOG1 };
 CComboBox m_RechercheGrapheControl;
 CDateTimeCtrl m_Date_Recherche_Control;
 CTime m_Date_Recherche_Value;
 CString m_RechercheType;
 CString m_RechercheCle;
 CString m_FiltrerType;
 CString m_FiltrerGraphe;
 CString m_FiltrerCle;
 //}}AFX_DATA
public:
 CString nom;
// Overrides
 // ClassWizard generated virtual function overrides
 //{{AFX_VIRTUAL(CRequetes)
 protected:
 virtual void DoDataExchange(CDataExchange* pDX);    // DDX/DDV support
 //}}AFX_VIRTUAL
// Implementation
protected:
 // Generated message map functions
 //{{AFX_MSG(CRequetes)
 virtual void OnOK();
 afx_msg void OnOk2();
 afx_msg void OnOk3();
 virtual void OnCancel();
 afx_msg void OnCancel2();
 afx_msg void OnCancel3();
 afx_msg void OnDatetimechangeDatetimepicker1(NMHDR* pNMHDR, LRESULT* pResult);
 virtual BOOL OnInitDialog();
 //}}AFX_MSG
 DECLARE_MESSAGE_MAP()
};
//{{AFX_INSERT_LOCATION}}
// Microsoft Visual C++ will insert additional declarations immediately before the previous line.
#endif // !defined(AFX_REQUETES_H__EBC09B53_AF2A_46AA_82B6_FC1B6C47233C__INCLUDED_)

25/04/2006, 16h17
Mayti4

OnInitDialog est appelée avant ComboGraphes, non ?

Donc il faut mettre nom="julien"; dans OnInitDialog.
25/04/2006, 16h20
julien.nasser

non justement, OnInitDialog est appelée APRES ComboGraphes
26/04/2006, 08h05
julien.nasser

J'arrive pas à voir d'où vient le problème :(
26/04/2006, 08h24
farscape

Par quel biais ComboGraphes est appelée avant OnInitDialog?
si dans le code tel que tu la posté ,AfxMessageBox n'affiche rien c'est que la variable n'a pas ete initialisée a sa valeur ou elle a été réinitialisée entre temps.

il n'y a pas d'autres solutions.
Pour s'assurer de l'ordonnancement des appels tu peux placer un point d'arrêt dans chaque fonction concernée et vérifier l'ordre d'appel.
:D
26/04/2006, 08h37
julien.nasser
En fait, c'est un petit peu compliqué, mais pour faire simple :

j'ouvre un fichier, pour cela, j'ai OnFileOpen dans CMainFrame qui fait appel à une fonction du document (fonction de traitement)
Code:

1 2 3 4 5 6 7 8 void CMainFrame::OnFileOpen() { CmonprogDoc* mondoc; mondoc = new CmonprogDoc; mondoc->affiche(); }
Ensuite, la fonction de traitement dans monprogDoc appelle automatiquement combographes de la manière suivante :
Code:

1 2 3 CRequetes req; req.ComboGraphes(myConn);
Tout ça a été fait en allant dans le menu fichier->ouvrir.
maintenant, en allant dans le menu fichier->requetes, j'ouvre une boite de dialogue de la classe CRequetes.

cela déclenche la fonction OnRequetes se trouvant dans CMainFrame qui elle-même appelle la classe CRequetes
Code:

1 2 3 4 5 6 void CMainFrame::OnReq() { CRequetes req; int res = req.DoModal(); ...
C'est ici que le OnInitDialog est appelé

PS : j'ai vérifié l'ordonnancement des fonctions en mettant
1) AfxMessageBox("combographes") dans ComboGraphes
2) AfxMessageBox("OnInitDialog") dans OnInitDialog
et combographes s'affiche bien avant OnInitDialog

Je veux bien que la variable "nom" se réinitialise entre combographes et OnInitDialog mais je ne vois pas comment et pourquoi ?
26/04/2006, 09h23
julien.nasser

Bon ben ça y est, je crois que j'ai enfin trouvé lol, mais bon c'était pas si évident.

J'ai fais plusieurs erreurs.

Je fais des appels à ComboGraphes et OnInitDialog à partir des classes CMainFrame et CmonprogDoc en déclarant les objets CRequetes req; Or apparemment, il vaut mieux déclarer des pointeurs sur ces objets : CRequetes* req.

Mais si je fais ça, il survient un nouveau problème : en effet, en déclarant les pointeurs, il faut les initialiser avec req = new CRequetes() et dans ce cas ma fameuse variable nom est aussi réinitialisée si je ne m'abuse.

Il faut donc que je place ma fonction OnRequetes non pas dans CMainFrame mais dans CmonprogDoc et que je déclare en public (dans monprogDoc.h) CRequetes* req; et que j'initialise tout au début une seule fois dans CmonprogDoc req = new CRequetes();

et là, apparemment ça marche, je remets en ordre tout ça et ça devrait être bon.

ouf !