Comment attendre une trame sur le port série ?

**petitclem** · 15/07/2008, 14h39

Bonjour à tous,

j'ai écris un programme qui attend et réceptionne bien la trame que je veux, mais pour l'enregistrement de ces données (octets) dans un fichier j'ai un gros problème : les données ne sont pas lisibles dans un fichier texte.

Je vous poste 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
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
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
 
 
#include <vcl.h>
 
#include <stdio.h>
#include <io.h>
#include <fcntl.h>
#include <sys\stat.h>
 
#include <ostream.h>
 
 
#pragma hdrstop
 
//---------------------------------------------------------------------------
 
#pragma argsused
 
void EcrireOctetsEnChaineDecimaleVersFichierOuvert(std::ostream & fich, void const *data, size_t taille);
 
int main(int argc, char* argv[])
{
	HANDLE hdl;
	DCB dcb;
	bool etat;
	COMMTIMEOUTS timeouts={0};
	unsigned char buf [20];
	unsigned long nblu;
 
	int handle;
 
 
	hdl=CreateFile("COM5",GENERIC_READ | GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, NULL);
 
	if (hdl==INVALID_HANDLE_VALUE)
	{
		printf("erreur %d\n",GetLastError());
		Sleep (2000);
		return (1);
	}
	else
	{
    	SetupComm(hdl,5000,5000);
		printf ("ouverture reussie !\n\n");
		etat=GetCommState(hdl, &dcb);
 
		if (etat==false)
		{
			printf ("GetCommState erreur %d\n", GetLastError());
			return (2);
		}
		else
		{
			printf ("GetCommState reussie\n\n");
        }
 
 
 
		dcb.BaudRate = CBR_38400;
		dcb.ByteSize = 8;
		dcb.Parity = NOPARITY;
		dcb.StopBits = ONESTOPBIT;
 
 
		etat=SetCommState(hdl,&dcb);
 
		if (etat==false)
		{
			printf ("SetCommState erreur %d\n", GetLastError());
			Sleep (2000);
			return (3);
		}
		else
		{
			printf ("Vitesse : %d baud\n", dcb.BaudRate);
			printf ("Bits de donnees : %d bits\n", dcb.ByteSize);
			printf ("Parite : %s\n", dcb.Parity);
			printf ("Bits de stop : 1 bit\n\n");
 
			printf ("Com serie configuree !\n\n");
			Sleep (2000);
		}
 
		int TimeoutRead=500;
		timeouts.ReadIntervalTimeout=MAXWORD;
		timeouts.ReadTotalTimeoutMultiplier=0;
		timeouts.ReadTotalTimeoutConstant=TimeoutRead;
		timeouts.WriteTotalTimeoutMultiplier=0;
		timeouts.WriteTotalTimeoutConstant=0;
 
		etat=SetCommTimeouts(hdl,&timeouts);
 
		if (etat==false)
		{
			printf ("TimeOuts erreur %d\n", GetLastError());
			return (4);
		}
		else
		{
			printf ("TimeOuts initialises\n\n");
 
		}
 
 
 
 
		handle = open("fichier.txt", O_APPEND | O_CREAT | O_TEXT | O_RDWR,
							S_IREAD | S_IWRITE );
		if (handle==-1)
		{
			printf ("Erreur creation fichier : erreur %d\n\n", GetLastError());
 
		}
		else
		{
			printf ("Creation du fichier reussie\n\n");
 
		}
 
 
		printf ("Attente reception\n\n");
 
		while (1)
		{
 
			ReadFile(hdl,&buf,1,&nblu,NULL);
			if (etat==-1)
			{
				printf ("Erreur lecture port : erreur %d\n\n", GetLastError());
			}
 
 
 
			int i=0;
			if (nblu!=0)
			{
 
				for (buf [i]; i < 18; i++)
				{
					printf ("Octets lus : %X\n\n", buf[i]);
 
					break;
 
				}
 
 
				//unsigned int conv = atoi(&buf[i]);
				//write(  handle, &conv   , sizeof(conv));
				write(  handle, &buf[i]  , 1);
 
 
				printf ("Stockage \n");
			}
 
		}
 
 
		close (handle);
		CloseHandle(hdl);
 
 
 
 
 
	}
 
 
 
 
}

Si vous pouviez m'aider svp !!!
Merci et à bientôt !

**wiztricks** · 15/07/2008, 14h45

j'ai écris un programme qui attend et réceptionne bien la trame que je veux, mais pour l'enregistrement de ces données (octets) dans un fichier j'ai un gros problème : les données ne sont pas lisibles dans un fichier texte.

Qu'est ce qui te permet d'être sûr que tes données sont des caractères ASCII alphanumériques? (imprimables lorsqu'on fait "cat" du fichier).
- W

**petitclem** · 15/07/2008, 14h49

Heu !!!

Et bien, je voudrai pouvoir lire ces données avec le bloc notes par exemple.
J'utilise hexedit pour savoir si les données sont correctes !!!

**wiztricks** · 15/07/2008, 15h08

OK, regardons ca:

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
		while (1)
		{
			
			ReadFile(hdl,&buf,1,&nblu,NULL);
			if (etat==-1)
			{
				printf ("Erreur lecture port : erreur %d\n\n", GetLastError());
			}
// Si la taille du buffer est 1, pourquoi s'inquiéter du nombre de caractère lus?		

			int i=0;
			if (nblu!=0)
			{
				
				for (buf [i]; i < 18; i++)
				{
					printf ("Octets lus : %X\n\n", buf[i]);
					break;
				}
// Ca devrait afficher le caractère lu suivi de 17 x0
// à la sortie i = 18...
 
				//unsigned int conv = atoi(&buf[i]);
				//write(  handle, &conv   , sizeof(conv));
				write(  handle, &buf[i]  , 1);
// le caractère lu est dans buf[0] alors qu'on écrit buf[18] qui avec
// un peu de chance sera toujours 0
 
				printf ("Stockage \n");
			}
 
		}

Hmm, tu as voulu faire quoi?
- W

**petitclem** · 15/07/2008, 15h13

Oui bon d'accord je me suis emmêlé

car en fait je reçois toujours une trame de 18 octets, et j'aimerai pouvoir écrire octet par octet dans le fichier texte pouvant être lu par bloc notes par exemple.

**wiztricks** · 15/07/2008, 15h29

Il serait sage que ton code lise des trames de 18 bytes (ou de n'importe quelle longueur) et non seulement des bytes.
Comme tu le disais pus haut, la couche RS232 va lire des bytes... Super...
Ou est la couche "au dessus" supposée les assembler en trames?

Sinon pour écrire dans un fichier les bytes lus, cela devrait suffire:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
		while (1)
		{
			
			etat = ReadFile(hdl,&buf,1,NULL,NULL);
			if (etat==-1)
			{
				printf ("Erreur lecture port : erreur %d\n\n", GetLastError());
			}

		        printf ("Octet lu : %X\n\n", buf[0]);
			write(  handle, &buf[0]  , 1);
			printf ("Stockage \n");
                 }

mais cela ne lit pas encore des trames.
- W

**petitclem** · 15/07/2008, 16h18

OK, d'accord ça marche pour lire et stocker le premier octet.
Mais, ce n'est toujours pas lisible par le bloc notes

**Médinoc** · 15/07/2008, 16h31

Est-ce qu'au moins, c'est lisible par hexedit ?
Si c'est lisible par hexedit et correct, ça veut dire qu'on enregistre correctement les données brutes, et qu'il n'y a plus qu'à les convertir de "données brutes" vers le format d'affichage que tu veux.

**wiztricks** · 15/07/2008, 16h33

Tel que écrite, je ne vois pas trop comment on sort du while (1) sans interrompre l'exécution du programme.
Il n'est pas certain dans ce cas que les données "cachées" du fichier en cours d'écriture aillent sur le disque.
- W

**petitclem** · 15/07/2008, 17h21

Est-ce qu'au moins, c'est lisible par hexedit ?
Si c'est lisible par hexedit et correct, ça veut dire qu'on enregistre correctement les données brutes, et qu'il n'y a plus qu'à les convertir de "données brutes" vers le format d'affichage que tu veux.

Oui, c'est lisible par hexedit !!!

Et je n'arrive pas à convertir ces "données brutes" en format d'affichage que je veux !!!

**Médinoc** · 15/07/2008, 17h24

c'est ton write qu'il faut changer.
Et c'est à ce genre de moment que tu regrettes d'avoir utiliser open() au lieu de fopen(), car une boucle de fprintf() aurait si facilement résolu l'affaire...

**petitclem** · 15/07/2008, 17h34

Merci bien Médinoc, mais là tu me rassure pas !

Comment aurais je dû faire alors ?

**Médinoc** · 15/07/2008, 17h44

Eh bien comme je l'ai dit, deux étapes:

Commencer par remplacer open() par fopen(), tes descripteurs par des FILE*, write() par fwrite(), puis vérifie que le fichier obtenu est toujours le même.
Remplace ton fwrite() par une boucle de fprintf() correspondant au format désiré.

**wiztricks** · 15/07/2008, 17h47

Envoyé par Médinoc

Eh bien comme je l'ai dit, deux étapes:

Commencer par remplacer open() par fopen(), tes descripteurs par des FILE*, write() par fwrite(), puis vérifie que le fichier obtenu est toujours le même.
Remplace ton fwrite() par une boucle de fprintf() correspondant au format désiré.

Pour autant qu'on souhaiterait stocker les données en hexadécimal, ce serait une solution...
Si tel est le cas, convertir le buffer lu dans un autre buffer avec un sprintf sans avoir à remplacer open/fopen, etc... et écrire le résultat dans le fichier fait aussi l'affaire avec beaucoup moins de modifications.

- W

**Médinoc** · 15/07/2008, 17h49

Tu peux. Mais c'est du boulot en plus, je trouve.
Pourquoi t'obstiner à utiliser open()/write() au lieu de fopen()/fwrite(), alors qu'avec fwrite() tu peux changer quand tu veux (et même faire deux versions dont tu choisis la bonne à l'exécution du prog) ?

**Médinoc** · 15/07/2008, 17h51

PS: Ou mieux que fopen()/fwrite(), utiliser les streams...

**wiztricks** · 15/07/2008, 18h00

Envoyé par Médinoc

Tu peux. Mais c'est du boulot en plus, je trouve.
Pourquoi t'obstiner à utiliser open()/write() au lieu de fopen()/fwrite(), alors qu'avec fwrite() tu peux changer quand tu veux (et même faire deux versions dont tu choisis la bonne à l'exécution du prog) ?

Pour l'instant, j'essaie de comprendre pourquoi c_boireau n'arrive pas à formater le fichier qu'il a créé.
La solution peut être de l'écrire dans un autre format (par exemple en hexa).
Mais cela ne lui est pas complètement inconnu puisqu'il affiche déjà les caractères lus dans ce format...

Note: S'il s'agit d'enregistrer les données échangées par le protocole en guise d'injection ou de trace, je ne vois pas l'intérêt de convertir quoi que ce soit.
Par contre, il serait peut être utile de faire un programme qui lise le fichier de trace et en transforme le contenu en quelque chose de manipulable sous editeur.

- W

**petitclem** · 16/07/2008, 10h05

Bonjour,
j'ai essayé d'utiliser fopen() et fwrite() à la place de open() et write() mais mon fwrite() n'est pas correct car il ne stocke pas la même chose que j'avais avec mon write()

Comment dois je utiliser fwrite() ?

Voila la fonctions que j'avais écrite write() (qui marche) et la fwrite() :

Code :

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

1
2
 
write(  handle, &buf[i]  , 1);

Code :

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

1
2
 
fwrite (&buf[i], 1 , sizeof (buf[i]), fichier);

Eh bien comme je l'ai dit, deux étapes:

1. Commencer par remplacer open() par fopen(), tes descripteurs par des FILE*, write() par fwrite(), puis vérifie que le fichier obtenu est toujours le même.
2. Remplace ton fwrite() par une boucle de fprintf() correspondant au format désiré.

Bizarre, je remplace write() par fwrite(), puis je remplace fwrite() par fprintf()

Donc toujours ce problème de formatage !!!!

PLEASE !!!

**Médinoc** · 16/07/2008, 10h31

Je ne comprends pas pourquoi ça ne marche pas avec fwrite().
Tu ne dois pas passer à la seconde étape (écriture "texte" au lieu de données brutes) tant que l'écriture brute par fwrite() ne marche pas.

PS: Quels paramètres passes-tu à fopen() ?
Et as-tu pensé à aussi remplacer close() par fclose() ?

**petitclem** · 16/07/2008, 10h38

Moi non plus je ne comprend pas pourquoi ça ne marche pas car je pense avoir fait ce qu'il fallait !!

Je poste 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
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
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
 
//---------------------------------------------------------------------------
 
#include <vcl.h>
 
#include <stdio.h>
#include <io.h>
#include <fcntl.h>
#include <sys\stat.h>
 
#pragma hdrstop
 
//---------------------------------------------------------------------------
 
#pragma argsused
int main(int argc, char* argv[])
{
 
	HANDLE hdl;
	DCB dcb;
	bool etat;
	COMMTIMEOUTS timeouts={0};
	unsigned char buf [20];
	unsigned long nblu;
 
 
	FILE *fichier;
 
	hdl=CreateFile("COM5",GENERIC_READ | GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, NULL);
 
	if (hdl==INVALID_HANDLE_VALUE)
	{
		printf("erreur %d\n",GetLastError());
		Sleep (2000);
		return (1);
	}
	else
	{
    	SetupComm(hdl,5000,5000);
		printf ("ouverture reussie !\n\n");
		etat=GetCommState(hdl, &dcb);
 
		if (etat==false)
		{
			printf ("GetCommState erreur %d\n", GetLastError());
			return (2);
		}
		else
		{
			printf ("GetCommState reussie\n\n");
        }
 
 
 
		dcb.BaudRate = CBR_38400;
		dcb.ByteSize = 8;
		dcb.Parity = NOPARITY;
		dcb.StopBits = ONESTOPBIT;
 
 
		etat=SetCommState(hdl,&dcb);
 
		if (etat==false)
		{
			printf ("SetCommState erreur %d\n", GetLastError());
			Sleep (2000);
			return (3);
		}
		else
		{
			printf ("Vitesse : %d baud\n", dcb.BaudRate);
			printf ("Bits de donnees : %d bits\n", dcb.ByteSize);
			printf ("Parite : %s\n", dcb.Parity);
			printf ("Bits de stop : 1 bit\n\n");
 
			printf ("Com serie configuree !\n\n");
			Sleep (2000);
		}
 
		int TimeoutRead=500;
		timeouts.ReadIntervalTimeout=MAXWORD;
		timeouts.ReadTotalTimeoutMultiplier=0;
		timeouts.ReadTotalTimeoutConstant=TimeoutRead;
		timeouts.WriteTotalTimeoutMultiplier=0;
		timeouts.WriteTotalTimeoutConstant=0;
 
		etat=SetCommTimeouts(hdl,&timeouts);
 
		if (etat==false)
		{
			printf ("TimeOuts erreur %d\n", GetLastError());
			return (4);
		}
		else
		{
			printf ("TimeOuts initialises\n\n");
 
		}
 
 
		fichier = fopen ("fichier.txt","at");
		if (fichier==NULL)
		{
			printf ("Erreur creation fichier : erreur %d\n\n", GetLastError());
 
		}
		else
		{
			printf ("Creation fichier reussie\n\n");
		}
 
 
		while (1)
		{
			int i=0;
 
			ReadFile(hdl,&buf,1,&nblu,NULL);
			if (etat==-1)
			{
				printf ("Erreur lecture port : erreur %d\n\n", GetLastError());
			}
 
 
			if (nblu!=0)
			{
 
				for (buf [i]; i < 18; i++)
				{
					printf ("Octets lus : %X\n\n", buf[i]);
 
					break;
 
				}
 
				fwrite (&buf[i], 1 , sizeof (buf[i]), fichier);
 
 
 
 
				printf ("Stockage \n");
			 }
 
		}
 
 
        fclose (fichier);
		CloseHandle(hdl);	
 
 
 
	}
 
	return 0;
}
//---------------------------------------------------------------------------

Comment attendre une trame sur le port série ?

C++

Discussions similaires

Partager

Partager