Discussion :

[ABouras]

Bonjour

J'ai écris la fonction suivante pour lire les données envoyé par un module GPS en Xbee et reçue sur une clé USB XStick, le tout sur LINUX :

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
bool readMessage(int file, char *message, unsigned int nCountMax, unsigned long* pCountRead)
{
    int nbCharToRead;
    char data[] = "";
    int i;
 
    if (file != 0)
    {
	    nbCharToread = ?????
	    i = 0;
	    if (nbCharToRead > 30)
	    {
		    while (i<nCountMax && data != ".")
		    {
			    if (read(file,&data,1) == 0)
			    {
				    printf("reception error\n");
				    return false;
			    }
			    else
			    {   
				    message[i] = data;
				    i++;
			    }
		    }
	    }
	    message[i] = "\0";
	    *pCountRead = (unsigned long) i;
	    return true;
};

Mais il me manque une fonction pour remplir la variable

Code :

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

nbCharToread

pour connaître le nombre de caractère à lire ... Vous en connaissez une ?

Merci d'avance !

[Bktero]

Code :

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

char data[] = "";

Il y a un problème avec cette déclaration et cette initialisation : tu ne spécifie pas la taille de la chaine (entre crochets) donc la taille sera la taille minimale pour contenir la chaine d'initialisation (un seul caractère ici). Tu ne pourras donc pas stocker grand chose dans ce tableau. Pourquoi ne pas simplement utiliser un char ?

La variable message ne subit pas d'allocation dynamique dans ta fonction. Il faut être sûr de respecter la taille allouée par la fonction parente, ce qui ne me semble pas être le cas.

Pourquoi y a t-il un ; à la fin de la fonction ? D'ailleurs, il doit manquer un bout de la fonction (une accolade pour la fermer au moins).

Le nombre de caractère à lire ne devrait pas t-il donner par la doc de ton module ou alors ne devrais tu pas lire jusqu'à la fin ? A quoi sert cette condition : if (nbCharToRead > 30) ?

[DrDarko]

Bonjour,

if (read(file,&data,1) == 0)
{
printf("reception error\n");
return false;
}
else
{
message[i] = data;
i++;
}[/CODE]

Ton read n'est pas bon... En gros read renvoie -1 en cas d'erreur et non 0. S'il renvoie 0 c'est qu'il n'y a plus rien à lire donc ce n'est surement pas une erreur.

Après je sais pas trop ce que tu veux faire avec ton nbr de caractères à lire...???

[ABouras]

Bonjour et merci pour vos réponses. En fait je veux lire des trames de données envoyées par un GPS par protocole XBee. La clé USB reçoit les données suivante :

Code :

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

1
2
CHARS : 15931    SENTENCES = 0    CHECKSUM : 58
Heading : 55    Tilt: -46    Roll:2

et ainsi de suite ... J'arrive à les lire en tapant dans le terminal :

Code :

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

$ screen /dev/ttyUSB0

Moi j'aimerais pouvoir les lire avec un programme écrit en C. Voici ce que j'ai fais :

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
#include <stdio.h>
#include <stdlib.h>
#include <termios.h>
#include <sys/fcntl.h>
#include <unistd.h>
#include "serial_port.h"
 
void read_Serial_Port(const char* DEVICE_PORT)
{
	int file;
	struct termios options;
	char *message;
	unsigned int nCountMax = 60;
	unsigned long* pCountRead = 0;
	bool b;
 
	file = open(DEVICE_PORT, O_RDONLY | O_NOCTTY | O_NDELAY);
 
	if(file == -1){perror("Unable to open the serial port\n");}
	printf("Serial port open successful\n");
 
	tcgetattr(file, &options); 			
	cfsetispeed(&options, B9600); 					
	cfsetospeed(&options, B9600); 					
	options.c_cflag |= (CLOCAL | CREAD); 			  
	options.c_cflag |= PARENB; 						//No parity					
	options.c_cflag |= PARODD; 						
	options.c_cflag &= ~CSTOPB; 					
	options.c_cflag &= ~CSIZE; 						
	options.c_cflag |= CS8; 						//8 bits					
	options.c_iflag |= (INPCK | ISTRIP); 			
	tcsetattr(file, TCSANOW, &options); 	     
	fcntl(file, F_SETFL, FNDELAY);			
 
	printf("Reading serial port ...\n\n"); 
	b = readMessage(file, message, nCountMax, pCountRead);
	if (b == 0){printf("Error while reading serial port\n");}
	else printf("Serial port read successful\n");
	close(file);
	printf("Serial port closed\n");
};
 
bool readMessage(int file, char *message, unsigned int nCountMax, unsigned long* pCountRead)
{
    int nbCharToRead;
    char data[] = "";
    int i;
 
    if (file != 0)
    {
	    nbCharToRead = 100;
	    i = 0;
	    if (nbCharToRead > 30)
	    {
		    while (i<nCountMax && data != ".")
		    {
			    if (read(file,&data,1) == -1)
			    {
				    printf("reception error\n");
				    return false;
			    }
			    else
			    {   
				    message[i] = *data;
				    i++;
			    }
		    }
	    }
	    message[i] = 0;
	    *pCountRead = (unsigned long) i;
	    return true;
    }
};

Mais ça ne fonctionne pas ...

[diogene]

Il n'y a pas de place allouée pour stocker le message :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void read_Serial_Port(const char* DEVICE_PORT)
{
...
	char *message;
...
	b = readMessage(file, message, nCountMax, pCountRead);
...
bool readMessage(int file, char *message, unsigned int nCountMax, unsigned long* pCountRead)
{
...
	    message[i] = *data;
...

Allouer un pointeur n'alloue de la mémoire que pour le pointeur. Pour recevoir le message, il faut un tableau, de taille suffisante, de char.

[ABouras]

Je dois donc écrire :

Code :

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

char message[100];

?

[diogene]

Par exemple et à condition que la chaine à stocker ne dépasse pas 99 caractères