Winapi - Rs232

**rin01** · 10/07/2006, 17h57

Bonjour,
Je suis entrain de développer une interface (avec les API/C) me permettant de recevoir des données en continue à partir du port série.
Je souhaite afficher les octets recus en 'temps réel' dans une status bar.
Le problème est que je ne sais pas quel évènement sous Windows scruter...
J'ai tenté de faire un truc du genre: "lorsqu'on appuie sur un bouton, on ouvre le port COM, on le lit et on affiche les données dans une zone d'une status bar" malheureusement sans succès car ce qui est affiché dans ma status bar ne change pas!

(cf screen)
Je sollicite votre aide svp et je précise que je ne suis pas informaticien.
mon code:

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
/* Message de la barre d'outils : Bouton 1 - Connecter*/
		case ID_TOOLBAR_BTN_1:
			{
				char cMsg[50];
				
				/* variables locales */ 
                char buffer[256]; 
                int nBytesRead=0,PORT=3; 
                
                /*tentative d'ouverture du PORT COM*/
                if(!OpenCOM(3)) {
                /*on affiche dans la zone de status le message*/
                sprintf(cMsg, "Impossible d'ouvrir COM%d",PORT);
                SendMessage(hStatusbar,
                SB_SETTEXT,
                ID_STATUS_ZONE_1,
                (LPARAM)(LPSTR)cMsg);
                }
                else {
                sprintf(cMsg, "COM%d ouvert",PORT);
                SendMessage(hStatusbar,
                SB_SETTEXT,
                ID_STATUS_ZONE_1,
                (LPARAM)(LPSTR)cMsg);   
                
                    if(ReadCOM(buffer, sizeof(buffer)-1, &nBytesRead)) 
                    { 
                      buffer[nBytesRead] = '\0';
                      
                      /*on affiche les octets recus dans la zone de status*/
                      sprintf(cMsg, "%d reçus: %d",nBytesRead, buffer);
                      SendMessage(hStatusbar,
                      SB_SETTEXT,
                      ID_STATUS_ZONE_3,
                      (LPARAM)(LPSTR)cMsg);
                                           
                    } 
                    else 
                      sprintf(cMsg, "erreur lors de la réception!",nBytesRead, buffer);
                      SendMessage(hStatusbar,
                      SB_SETTEXT,
                      ID_STATUS_ZONE_2,
                      (LPARAM)(LPSTR)cMsg);      
                     
                     }
                

				/* On affiche le dernier bouton clique dans la barre d'etat */
				SendMessage(hStatusbar,
					SB_SETTEXT,
					ID_STATUS_ZONE_3,
					(LPARAM)(LPSTR)"Connect");
			}
			break;

Invité · 11/07/2006, 08h58

il faudrai peut être rafraichir la statusbar par un refresh.

**joellel** · 11/07/2006, 09h41

As tu vérifié si ta lecture du port série fonctionne correctement, par exemple en regardant le contenu de buffer après le read()?
En plus, je ne vois nulle part de fermeture du port...

**rin01** · 11/07/2006, 11h17

Envoyé par gcorbineau

il faudrai peut être rafraichir la statusbar par un refresh.

Ok, et comment rafraîchir la status bar car je n'ai pas trouvé cette fonction dans la doc WIN32?
Faut-il créer une fonction spécifique et comment procéder stp?

**rin01** · 11/07/2006, 11h25

Envoyé par joellel

As tu vérifié si ta lecture du port série fonctionne correctement, par exemple en regardant le contenu de buffer après le read()?

Je reçois des caractères je pense car j'obtiens les mêmes caractères avec l'executable fourni par l'exemple...

Envoyé par joellel

En plus, je ne vois nulle part de fermeture du port...

Il me semble appeler la fonction CloseCOM() dans mon code?

**joellel** · 11/07/2006, 12h09

Envoyé par rin01

Je reçois des caractères je pense car j'obtiens les mêmes caractères avec l'executable fourni par l'exemple...

Il me semble appeler la fonction CloseCOM() dans mon code?

Vérifie quand même, comme ça, tu seras sûr et tu pourras te concentrer sur le VRAI problème

**Mayti4** · 11/07/2006, 14h13

C'est l'adresse du buffer que tu affiches.

char cMsg[50];

C'est un peu petit si tu places un buffer de 255 octets dedans..

**rin01** · 11/07/2006, 16h35

J'ai modifié le code:

/* Message de la barre d'outils : Bouton 1 - Connecter*/
case ID_TOOLBAR_BTN_1:
{
char cMsg[512],buffer[255];
int nBytesRead;

SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_1,
(LPARAM)(LPSTR)"COM");

/*si l'ouverture du port échoue, message d'erreur*/
if (!OpenCOM(3)){
SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_2,
(LPARAM)(LPSTR)"not connected");
return -1;//erreur

}
/*ouverture réussie, on affiche les données dans la status bar*/
while(ReadCOM(buffer, sizeof(buffer)-1, &nBytesRead)) {

SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_2,
(LPARAM)(LPSTR)"connected");

sprintf(cMsg,"reçus: %s\n",buffer);
SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_3,
(LPARAM)(LPSTR)buffer);

}
/*On ferme le PORT*/
CloseCOM();

}
break;

/* Message de la barre d'outils : Bouton 2 - Déconnecter*/
case ID_TOOLBAR_BTN_2:
{
char cMsg[50];
/*On ferme le PORT*/
CloseCOM();
/* On affiche le dernier bouton clique dans la barre d'etat */
SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_1,
(LPARAM)(LPSTR)"");
SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_2,
(LPARAM)(LPSTR)"Disconnected");
SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_3,
(LPARAM)(LPSTR)"");

}
break;

Dans ma status, j'obtiens enfin des caractères de manière continue mais j'ignore comment les afficher en hexa. J'ai essayé %x, %d,%o, %s sans succès!
De plus, un clic sur mon bouton 2 ou un déplacement de la fenêtre ne coupe plus la communication et fait planter le programme...
Si quelqu'un voit ou se trouve l'erreur car je suis un peu perdu...

**joellel** · 11/07/2006, 17h08

Attention au sprintf()! Il va te mettre dans cMsg tout ce qu'il y a en mémoire à pertir de l'adresse buffer et ce jusqu'au premier \0 rencontré.
Je te suggère d'écrire:

Code :

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

1
2
3
strncpy(cMsg, buffer, nbBytesRead);
cMsg[nbBytesRead]='\0';

ça peut éviter les plantages.

Pour l'affichage en hexa, je ne sais pas.

**Médinoc** · 11/07/2006, 17h53

Tu ne peux afficher en hexa qu'octet par octet avec *printf().

Le mieux est de faire une fonction de conversion perso.

Code C++ non-testé :

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
//Taille minimale du buffer: (nSize*2)+1 TCHARs.
void DataToHexa(LPTSTR szBuffer, void const * pcData, size_t nSize)
{
	BYTE const * const pcBytes = static_cast< BYTE const * >(pcData);
	static LPCTSTR const sczHexa=TEXT("0123456789ABCDEF");

	for(size_t i=0 ; i<nSize ; i++)
	{
		BYTE const b = pcBytes[ i ];
		*szBuffer = sczHexa[ (b >> 4) & 0x0F ];
		szBuffer++;
		*szBuffer = sczHexa[ b & 0x0F ];
		szBuffer++;
	}
	*szBuffer = TEXT('\0');
}

PS: Evite de caster ainsi tes constantes chaînes: Non-seulement tu forces un char * en TCHAR * sans faire gaffe, mais en plus tu perds la constance (même si ici tu la perdras de toute façon).

**rin01** · 12/07/2006, 09h40

Envoyé par Médinoc

Tu ne peux afficher en hexa qu'octet par octet avec *printf().

Le mieux est de faire une fonction de conversion perso.

Code C++ non-testé :

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
//Taille minimale du buffer: (nSize*2)+1 TCHARs.
void DataToHexa(LPTSTR szBuffer, void const * pcData, size_t nSize)
{
	BYTE const * const pcBytes = static_cast< BYTE const * >(pcData);
	static LPCTSTR const sczHexa=TEXT("0123456789ABCDEF");

	for(size_t i=0 ; i<nSize ; i++)
	{
		BYTE const b = pcData[i];
		*szBuffer = sczHexa[ (b >> 4) & 0x0F ];
		szBuffer++;
		*szBuffer = sczHexa[ b & 0x0F ];
		szBuffer++;
	}
	*szBuffer = TEXT('\0');
}

PS: Evite de caster ainsi tes constantes chaînes: Non-seulement tu forces un char * en TCHAR * sans faire gaffe, mais en plus tu perds la constance (même si ici tu la perdras de toute façon).

Bonjour, je n'ai pas tres bien compris comment utiliser la fonction DataToHexa et qu'est ce qu'il faut mettre en paramètre

**Médinoc** · 12/07/2006, 09h54

C'est pourtant simple :

Premier paramètre : Un buffer de caractères (un tableau de TCHARs, quoi). La taille doit être suffisante, comme dit dans le code.
Second paramètre : Pointeur vers tes données à afficher en hexa.
Troisième paramètre : Taille en octets des données.

**rin01** · 12/07/2006, 10h23

Envoyé par Médinoc

C'est pourtant simple :

Premier paramètre : Un buffer de caractères (un tableau de TCHARs, quoi). La taille doit être suffisante, comme dit dans le code.
Second paramètre : Pointeur vers tes données à afficher en hexa.
Troisième paramètre : Taille en octets des données.

Je m'excuse de mon ignorance mais comme je disais, je suis électronicien et non informaticien.
Pour en revenir à la fonction, je l'ai juste copié-collé dans mon programme sans l'utiliser et le compilateur me génère des erreurs...
y a t-il des fichiers d'entêtes spécifiques à inclure car lors de la compilation j'ai des erreurs ?
Est-ce que tu pourrais mettre un exemple d'utilisation de cette fonction stp ?

main.c: In function `DataToHexa':
main.c:58: error: `static_cast' undeclared (first use in this function)
main.c:58: error: (Each undeclared identifier is reported only once
main.c:58: error: for each function it appears in.)
main.c:58: error: syntax error before "BYTE"
main.c:61: error: 'for' loop initial declaration used outside C99 mode
main.c:63: warning: dereferencing `void *' pointer
main.c:63: error: void value not ignored as it ought to be

**Médinoc** · 12/07/2006, 10h26

Ah, je n'avais pas lu que tu étais en C.
Comme indiqué, mon code était en C++...
(Et puis, j'avais fait une petite erreur)

Note: Comme header, les headers Windows devraient suffire (pour les types et la macro TEXT())

Code C non-testé aussi :

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
//Taille minimale du buffer: (nSize*2)+1 TCHARs.
void DataToHexa(LPTSTR szBuffer, void const * pcData, size_t nSize)
{
	BYTE const * const pcBytes = pcData;
	static LPCTSTR const sczHexa=TEXT("0123456789ABCDEF");

	size_t i;
	for(i=0 ; i<nSize ; i++)
	{
		BYTE const b = pcBytes[ i ];
		*szBuffer = sczHexa[ (b >> 4) & 0x0F ];
		szBuffer++;
		*szBuffer = sczHexa[ b & 0x0F ];
		szBuffer++;
	}
	*szBuffer = TEXT('\0');
}

**rin01** · 12/07/2006, 11h10

Envoyé par Médinoc

Ah, je n'avais pas lu que tu étais en C.
Comme indiqué, mon code était en C++...
(Et puis, j'avais fait une petite erreur)

Note: Comme header, les headers Windows devraient suffire (pour les types et la macro TEXT())

Code C non-testé aussi :

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
//Taille minimale du buffer: (nSize*2)+1 TCHARs.
void DataToHexa(LPTSTR szBuffer, void const * pcData, size_t nSize)
{
	BYTE const * const pcBytes = pcData;
	static LPCTSTR const sczHexa=TEXT("0123456789ABCDEF");

	size_t i;
	for(i=0 ; i<nSize ; i++)
	{
		BYTE const b = pcBytes[ i ];
		*szBuffer = sczHexa[ (b >> 4) & 0x0F ];
		szBuffer++;
		*szBuffer = sczHexa[ b & 0x0F ];
		szBuffer++;
	}
	*szBuffer = TEXT('\0');
}

D'accord, mais je ne sais pas quand appeler cette fonction...En fait lors d'un clic sur un bouton, j'ouvre le port série et j'y lit les données:

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
/* Message de la barre d'outils : Bouton 1 - Connecter*/
case ID_TOOLBAR_BTN_1:
{
char cMsg[512],buffer[255];
int nBytesRead;
BOOL test;
        
SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_1,
(LPARAM)(LPSTR)"COM");
		
/*si l'ouverture du port échoue, message d'erreur*/
if (!OpenCOM(3)){

SendMessage(hStatusbar,
SB_SETTEXT,		     
ID_STATUS_ZONE_2,		      
(LPARAM)(LPSTR)"not connected");

return -1;//erreur
}
		
/*ouverture réussie, on affiche les données dans la status bar*/
test = ReadCOM(buffer, sizeof(buffer)-1, &nBytesRead);
while (test) {

SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_2,
(LPARAM)(LPSTR)"connected");

}
 /*On ferme le PORT*/
 CloseCOM(); 
            
}
break;

Donc comment j'appelle la fonction DataToHexa, et ou je l'appelle stp ?

**Médinoc** · 12/07/2006, 11h22

Tu déclares cMsg en tableau de TCHARs (255*2+1 = 511, donc ça tient dans ton buffer de 512), et tu appelles:

Code :

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

DataToHexa(cMsg, buffer, sizeof(buffer));

(ou sizeof(buffer)-1, puisque c'est tout ce que tu as lu)
... entre la lecture (read) et l'affichage (SendMessage), tout simplement.
Et tu fais un SendMessage de cMsg, pas de buffer.

**rin01** · 12/07/2006, 15h41

Envoyé par Médinoc

Tu déclares cMsg en tableau de TCHARs (255*2+1 = 511, donc ça tient dans ton buffer de 512), et tu appelles:

Code :

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

DataToHexa(cMsg, buffer, sizeof(buffer));

(ou sizeof(buffer)-1, puisque c'est tout ce que tu as lu)
... entre la lecture (read) et l'affichage (SendMessage), tout simplement.
Et tu fais un SendMessage de cMsg, pas de buffer.

Ok, je crois que la conversion fonctionne mais les autres boutons ne sont plus fonctionnels et si je tente de déplacer la fenêtre, le programme plante...

/*ouverture réussie, on affiche les données dans la status bar*/
test = ReadCOM(buffer, sizeof(buffer)-1, &nBytesRead);
while (test) {
SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_2,
(LPARAM)(LPSTR)"connected");
buffer[nBytesRead]='\0';
DataToHexa(mes, buffer, sizeof(buffer)-1);

SendMessage(hStatusbar,
SB_SETTEXT,
ID_STATUS_ZONE_3,
(LPARAM)(LPCSTR)mes);
}
/*On ferme le PORT*/
CloseCOM();

**Médinoc** · 12/07/2006, 15h49

Je peux voir la déclaration de mes ?
(LPCTSTR) dans le cast, pas (LPCSTR)... (bien que ça n'ait pas d'influence normalement)

**rin01** · 12/07/2006, 16h10

Envoyé par Médinoc

Je peux voir la déclaration de mes ?
(LPCTSTR) dans le cast, pas (LPCSTR)... (bien que ça n'ait pas d'influence normalement)

Voici la routine pour le bouton Connecter

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
/* Message de la barre d'outils : Bouton 1 - Connecter*/
case ID_TOOLBAR_BTN_1:
		{
        char cMsg[512],buffer[255];
        int nBytesRead,i;
        BOOL test;
        TCHAR mes[512];
        
        SendMessage(hStatusbar,
		SB_SETTEXT,
		ID_STATUS_ZONE_1,
		(LPARAM)(LPSTR)"COM");
		
		/*si l'ouverture du port échoue, message d'erreur*/
		if (!OpenCOM(3)){
              SendMessage(hStatusbar,
		      SB_SETTEXT,
		      ID_STATUS_ZONE_2,
		      (LPARAM)(LPSTR)"not connected");
		      return -1;//erreur

        }
		/*ouverture réussie, on affiche les données dans la status bar*/
		test = ReadCOM(buffer, sizeof(buffer)-1, &nBytesRead);

		while (test) {
              SendMessage(hStatusbar,
		      SB_SETTEXT,
		      ID_STATUS_ZONE_2,
		      (LPARAM)(LPSTR)"connected");
              buffer[nBytesRead]='\0';

              /*Conversion en hexa*/
              DataToHexa(mes, buffer, sizeof(buffer)-1);

              /*Affichage du résultat*/                   
              SendMessage(hStatusbar,
		      SB_SETTEXT,
		      ID_STATUS_ZONE_3,
		      (LPARAM)(LPCSTR)mes);
              test = ReadCOM(buffer, sizeof(buffer)-1, &nBytesRead);

              }
        /*On ferme le PORT*/
        CloseCOM();             
        }
            break;

**Médinoc** · 12/07/2006, 22h46

Eh bien ici, je ne vois pas ce qui peut poser problème sur les autres boutons de ta toolbar...
Ni gêner le déplacement de la fenêtre...

C'est bien le déplacement en plus qui foire, pas le redimensionnement ?

PS: Dans la doc de SB_SETTEXT, ils parlent de limitation du texte à 127 caractères ou un truc du genre...
Peut-être devrais-tu lire (ou afficher) moins à la fois ?

Ou bien, une tout autre voie: Je pense que ton programme se fige dans une boucle infinie après réception (ta boucle while(test)), mais tu ne vois qu'il est planté qu'une fois que tu tentes de déplacer la fenêtre...

Winapi - Rs232

Windows

Discussions similaires

Partager

Partager