Discussion :

[gueulederack]

Bonjour,

Le projet se nomme Module_test1 c'est un application MFC smart device.

Je souhaite récupèrer l'adresse d'un objet nommé "serie" qui est déclaré dans le Module_test1.h et l'envoyer dans mon Module_test1View.cpp

Voici le code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
// Module_test1.h : fichier d'en-tête principal pour l'application Module_test1
 
class CModule_test1App : public CWinApp
{
public:
    CModule_test1App();
 
             ....
public:
    cl_Bluetooth serie;
 
    DECLARE_MESSAGE_MAP()
};

Code :

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

1
2
3
4
5
6
7
8
9
10
// Module_test1View.cpp : implémentation de la classe CModule_test1View

void CModule_test1View::OnBnClickedButton1()
{
char *recupTRAME=pApp->serie.Lecture();  //pApp est le pointeur sur le fichier Module_test1 mais il m'affiche une erreur lorsque je clique sur lecture gwes.exe
 LPCTSTR tmp=(LPCTSTR)recupTRAME;
     AfficheTrame.AddString(tmp);

}

Quelle fonction peut fonctionner ?

[farscape]

salut,
il y a erreur parce que ton pointeur pApp ne doit pas être valide ...
ton code ne permet pas de voir son initialisaton...
qui devrait être:

Code :

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

1
2
 
pApp=static_cast<CModule_test1App *>(AfxGetApp());

ou alors le probléme vient de ton utilisation:

Code :

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

1
2
 
char *recupTRAME=pApp->serie.Lecture()

vu que tu utilises un char * en retour, l'adresse retournée ne doit pas être située dans la pile sinon boum (plus loin)...
de toute façon plein de bug possible dans ce code pas terrible..
la chaine retournée se termine par un \0 ? sinon boum dans AddString .
en conclusion les char * c'est mal si on ne maitrise pas la chose..

[gueulederack]

Merci Farscape, ta fonction static_cast<CModule_test1App *>(AfxGetApp()) je l'utilise dans mon constucteur de Module_test1View car j'ai déclaré:

Code :

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

1
2
3
4
5
6
7
8
 
// Module_test1View.h : interface de la classe CModule_test1View
                    ...
class CModule_test1View : public CFormView
{
               ...
public:
	CModule_test1App *pApp;

Par contre je ne comprend pas pourquoi le type char* pose problème car c'est quand même un type de base du C++. Maintenant il me met plus l'erreur gwes.exe mais il m'affiche rien dans le listbox.

[farscape]

Par contre je ne comprend pas pourquoi le type char* pose problème car c'est quand même un type de base du C++.

l'adresse retournée ne doit pas être située dans la pile sinon boum

ce n'est pas une question de type mais d'utilisation, manipuler des char * est délicat si on est pas habitué à écrire du C, et les pièges sont nombreux ,j'en ai cité deux dans mon post précédent.
il faudrait voir l'implémentation de ta fonction Lecture().

[gueulederack]

Je te montre de suite ma classe Cl_Bluetooth

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
class cl_Bluetooth : public CWnd
{
	DECLARE_DYNAMIC(cl_Bluetooth)
 
public:
	cl_Bluetooth();
	~cl_Bluetooth();
    char* Lecture (void);
    void modifPARAM (char *Port,int Brate);
 
private:
   HANDLE hcomm;
   LPDCB serie;
   OVERLAPPED over;
 
protected:
	DECLARE_MESSAGE_MAP()
 
};

Code :

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

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
#include "stdafx.h"
#include "Module_test1.h"
#include "cl_Bluetooth.h"


// cl_Bluetooth

IMPLEMENT_DYNAMIC(cl_Bluetooth, CWnd)
//---------------------------------------------------------------------------
/*                               Partie PDA                                */
/*                                                                         */
/*                                                                         */
/*                        Définition des méthodes de Bluetooth             */
//---------------------------------------------------------------------------
cl_Bluetooth::cl_Bluetooth()
{char *Port=new char (5);
 serie=new DCB();
 int vitesse;
 strcpy(Port,"COM1");
 vitesse=4800;
 modifPARAM (Port,vitesse);
}
//-------------------------------------------------------------------------
cl_Bluetooth::~cl_Bluetooth()
{		/* Destructeur */
bool test;
 test=PurgeComm (hcomm,PURGE_RXCLEAR);
 if (test==true)
   {test=CloseHandle (hcomm); //fermeture du port
   }
 else
   {MessageBox(TEXT("Impossible de vider le port com"),TEXT("Erreur #003"),MB_ICONSTOP);
   }
}
//-----------------------------------------------------------------------------
char* cl_Bluetooth::Lecture ()
/* Lecture des données sur le port */
{bool test;
 unsigned long pt=0;
 char chaine[100];
 test=ReadFile(hcomm,&chaine,82,&pt,&over); //lecture du port série
 strcat (chaine,"\0");
 if (test==false)
   {MessageBox(TEXT("Impossible de lire sur le port com"),TEXT("Erreur #004"),MB_ICONSTOP);
   }
 PurgeComm (hcomm,PURGE_RXCLEAR); // vide le buffer du port
 return chaine; //on retourne la trame NMEA
}
//---------------------------------------------------------------------------
void cl_Bluetooth::modifPARAM (char *Port,int vitesse)
/* Modification des paramètres du port série */
{LPWSTR wch;
 strcat(Port,":");
 int Size = MultiByteToWideChar (CP_ACP, 0, Port, -1, NULL, 0);
 wch = new WCHAR[Size];
 MultiByteToWideChar (CP_ACP, 0, Port, -1, wch, Size); 

 bool test;
 hcomm=CreateFile(wch,GENERIC_READ,0,NULL,OPEN_EXISTING,FILE_FLAG_OVERLAPPED,NULL); //ouverture du port sélectionné
 if(hcomm==INVALID_HANDLE_VALUE) 
  {MessageBox(TEXT("Impossible d'ouvrire le port choisi"),TEXT("Erreur #001"),MB_ICONSTOP);
  }
 GetCommState (hcomm,serie); //lecture des paramètre de configuration du port
 serie->BaudRate=vitesse;
 serie->ByteSize=8;
 serie->Parity=NOPARITY;
 serie->StopBits=0;
 test=SetCommState (hcomm,serie); //fin de la configuration
 if (test==false)
   {MessageBox(TEXT("Impossible d'ouvrire le port choisi"),TEXT("Erreur #002"),MB_ICONSTOP);
   }
  PurgeComm (hcomm,PURGE_RXCLEAR);  //vider le buffer
}

De plus j'ai l'habitude d'utiliser les char* car j'ai débuter laz prog avec le llangage C avec devcpp. Mais c'est vrai que rien n'est automatique. ^^

[farscape]

oué c'est bien ce que je disais...
ta variable char chaine[100]; est bien dans la pile.
a la sortie d'une fonction les variables dans la pile sont nettoyées,
ce sont les base du C ça...
de toute façon ton code est truffé de bugs, comme le strcat (chaine,"\0");
et je ne suis pas allé plus loin dans la lecture..

[gueulederack]

Dslée farscape je viens de voir mon erreur. Je dois avouer que j'ai pas réfléchi lopngtemps sur la classe cl_Bluetooth car je me concentrais surtout sur la compréhension du fonctionnement d'un projet MFC. Je te montre ma nouvelle méthode de Lecture:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
void cl_Bluetooth::Lecture (char *chaine)
/* Lecture des données sur le port */
{bool test;
 unsigned long pt=0;
 test=ReadFile(hcomm,&chaine,82,&pt,&over); //lecture du port série
 if (test==false)
   {MessageBox(TEXT("Impossible de lire sur le port com"),TEXT("Erreur #004"),MB_ICONSTOP);
   }
 PurgeComm (hcomm,PURGE_RXCLEAR); // vide le buffer du port
}

Ainsi que son appel:

Code :

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

1
2
3
4
5
6
7
8
 
void CModule_test1View::OnBnClickedButton1()
{
 char *recupTRAME=new char (85);
 pApp->serie.Lecture(recupTRAME);
 LPCTSTR tmp=(LPCTSTR)recupTRAME;
 AfficheTrame.AddString(tmp);
}

[farscape]

tu as toujours le bug de l' \0 a la fin de la chaine...
AddString s'attend a une chaine se terminant par \0.
et tu as oublié de liberer la memoire allouée.

[gueulederack]

Exact j'ai rajouté ' \0 ' après l'appel du ReadFile, et j'ai désalloué mes chaines.

Bon faut quer j'aille en physique je testerais ce soir.

Encore merci

[Médinoc]

Code :

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

1
2
3
char *recupTRAME=new char (85);
 pApp->serie.Lecture(recupTRAME);
 LPCTSTR tmp=(LPCTSTR)recupTRAME

Très mauvais, ça:

1. Il faut des crochets et non des parenthèses (là, tu alloues UN char qui a la valeur 85)
2. On ne caste jamais char* en TCHAR* : Dans ce sens, passe par une CString, ça marche même pour les vieilles versions de MFC.