[PIC] Afficher une trame arrivant sur le port série

**lcoulon** · 14/04/2009, 16h35

Bonjour

J'aurai 2 questions svp :

1)

Je cherche à programmer un pic pour que lorsqu'il reçoit une trame définie sur son port série, il execute alors une tâche ...

Le code que j'ai écrit ici bloque systématiquement le déroulement du reste du programme, pouvez vous me dire ce qui est incorrect ?

Code :

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

1
2
 
char reception;

...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
case VAR_RX_UART_MESSAGE:
 
// Test si le mot "*Hello" est reçu sur le port série, si oui OUT1=1
 
ReadStringUART((BYTE *)reception, sizeof(reception));
if (strcmp(reception, (char *)"*Hello") == 0)
 
{
OUT1 = 1;
}

2)

Quand 1) fonctionnera je voudrai faire cette variante :

En fonction de l'entete que comportera le mot reçu je voudrais effectuer telle ou telle tâche :

exemple :

si le mot recu à pour entete '*' alors je voudrai que OUT1 passe à '1'
si le mot recu à pour entete '#' alors je voudrai que OUT2 passe à '1'
si le mot recu à pour entete '$' alors je voudrai que OUT3 passe à '1'

Je vous remercie beaucoup pour votre aide,

Invité(e) · 14/04/2009, 16h54

Bonjour

Envoyé par lcoulon

1)

Code :

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

1
2
3
4
5
6
7
 
char reception;
ReadStringUART((BYTE *)reception, sizeof(reception));
if (strcmp(reception, (char *)"*Hello") == 0)
{
    OUT1 = 1;
}

char et pointeur sur char sont deux choses différentes.

Code :

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

ReadStringUART((BYTE *)&reception, sizeof(reception));

Maintenant, en ce qui concerne la comparaison de 'reception' avec "*Hello", attention, reception est un caractère seul, "*Hello" une chaine de caractère terminée par \0..., or strcmp ne travaille qu'avec des chaines terminées par \0.

À la limite, tu peux écrire :

Code :

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

1
2
3
4
if (strncmp(&reception, "*Hello", sizeof reception) == 0)
{
    OUT1 = 1;
}

Envoyé par lcoulon

2)

Quand 1) fonctionnera je voudrai faire cette vairante :

En fonction de l'entete que comportera le mot reçu je voudrais effectuer telle ou telle tache :

exemple :

si le mot recu à pour entete '*' alors je voudrai que OUT1 passe à '1'
si le mot recu à pour entete '#' alors je voudrai que OUT2 passe à '1'
si le mot recu à pour entete '$' alors je voudrai que OUT3 passe à '1'

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
char premier_char_du_message;
switch(premier_char_du_message) {
  case '*':
    OUT1 = '1';
    break;
  case '#':
    OUT2 = '1';
    break;
  case '$':
    OUT3 = '1';
    break;
}

EDIT :
Cela dit, on ne sait pas à quoi correspond la fonction ReadStringUART... un peu plus d'info dessus permettrait peut-être de t'aider plus efficacement.

**lcoulon** · 14/04/2009, 17h10

merci Mabu, mais le fait d'ajouter &reception me permettra t-il de stocker l'intégralité de ce qui se présente sur le port série ?

mon but est capturer la trame entière et non pas qu'un seul caractère .
De plus, *Hello ne finit par /0 , est ce un pb dans mon cas ?

Ensuite concernant les entetes, les cases me permettront de faire tel ou tel choix mais comment faire en sorte d'identifier l'entete elle meme ?
quel code me permet "d'aller voir" la premier caractère d'une trame ?

Merci à vous,

**lcoulon** · 14/04/2009, 17h25

ci joint mon uart.c qui gère la fonction ReadStringUART

( comme je débute, ce n'est pas moi qui ai écrit ce pilote )

J'ai ajouté les &reception mais mon programme bloque toujours .

Invité(e) · 14/04/2009, 17h25

Envoyé par lcoulon

merci Mabu, mais le fait d'ajouter &reception me permettra t-il de stocker l'intégralité de ce qui se présente sur le port série ?

Non... mais comme tu as écrit "char reception;", je pensais que c'était voulu.

Si reception devient un tableau de caractères, les choses changent :

Code :

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

1
2
3
4
5
6
7
char reception[256];
/* on récupère la trame */
ReadStringUART((BYTE *)reception, sizeof(reception));
if (strncmp(reception, "*Hello", strlen("*Hello")) == 0)
{
OUT1 = 1;
}

Cela dit, merci de nous en dire plus sur ReadStringUART qu'on puisse t'aider... Que retourne cette fonction, ajoute-t-elle un \0 à la fin de la chaine...

Envoyé par lcoulon

mon but est capturer la trame entière et non pas qu'un seul caractère .
De plus, *Hello ne finit par /0 , est ce un pb dans mon cas ?

Et si, la chaine "*Hello" fini par '\0'.

Code :

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

1
2
3
4
5
6
7
8
9
10
 /* tableau de 6 caractères, mais chaine non-valide : ne pas utiliser avec 
   strlen, strcmp...*/
char test_1[6] = "*Hello";
 
/* tableau de 7 caractères, chaine valide */
char test_2[7] = "*Hello";
 
/* tableau de 7 caractères, chaine valide, mais on 
   laisse au compilateur le soin d'allouer la taille nécessaire*/
char test_3[] = "*Hello";

Envoyé par lcoulon

Ensuite concernant les entetes, les cases me permettront de faire tel ou tel choix mais comment faire en sorte d'identifier l'entete elle meme ?
quel code me permet "d'aller voir" la premier caractère d'une trame ?

Récupérer la premier caractère d'une trame, sans savoir à quoi ressemble la trame, impossible.

Récupérer le premier caractère d'une chaine (ou d'un tableau) de caractères, c'est facile : chaine[0] ou * chaine :

Code :

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

1
2
3
4
5
6
7
void premier_element(const char * chaine) {
    if(NULL != chaine) {
        printf("premier caractères de %s : \"%c\"\n", chaine, *chaine);
    } else {
        fprintf(stderr, "erreur : chaine nulle...");
    }
}

De la bonne lecture : Les pointeurs démythifiés !

**lcoulon** · 14/04/2009, 17h29

Nos messages se sont croisés, je viens de poster mon fichier uart.c qui gère ReadStringUART

Invité(e) · 14/04/2009, 17h56

Envoyé par lcoulon

Nos messages se sont croisés, je viens de poster mon fichier uart.c qui gère ReadStringUART

Ok, merci.

Donc, si ReadStringUART revoie la taille de ce qu'il a écrit, nous pouvons écrire :

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
 
/* ReadStringUART : ne travaille qu'en BYTE : les messages récupérés ne peuvent 
   faire plus de 127 ou 255 caractères (suivant si BYTE est signée ou non)
   Pour ne pas prendre de risque, je me limite à 64.*/
char reception[64];
BYTE size;
 
/* on récupère la trame */
size = ReadStringUART((BYTE *)reception, sizeof reception - 1);
 
/* on fait en sorte que reception devienne une chaine de caractères valide */
reception[size] = '\0'
 
/* et on peut utiliser strcmp sans risque */
if (strcmp(reception, "*Hello")) == 0)
{
    /* faire des choses */
}

**lcoulon** · 14/04/2009, 20h13

Je te remercie,

J'ai une erreur de syntaxe qui remonte sur cette ligne lorsque je compile ...
je ne vois pas pourquoi :

Code :

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

if (strcmp(reception, "*Hello")) == 0)

idem si je remplace par :

Code :

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

if (strcmp(reception, "*Hello") == 0)

Voici comment j'ai procédé :

j'ai déporté en début de programme :

Code :

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

1
2
3
 
char reception[64];
BYTE size;

et j'ai inclu dans une boucle infinie :

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
 
/* on récupère la trame */
size = ReadStringUART((BYTE *)reception, sizeof reception - 1);
 
/* on fait en sorte que reception devienne une chaine de caractères valide */
reception[size] = '\0'
 
/* et on peut utiliser strcmp sans risque */
if (strcmp(reception, "Hello") == 0)
 
{
    /* faire des choses */
OUT1 = 1;
last_OUT1_TickSample = TickGet(); 
}

Invité(e) · 15/04/2009, 10h02

Envoyé par lcoulon

J'ai une erreur de syntaxe qui remonte sur cette ligne lorsque je compile ...

Peux tu préciser ce que le compilateur te dit ?

**lcoulon** · 15/04/2009, 11h48

Le compilateur indique " syntax error" est pointe la ligne, sans me donner plus d'informations.

je me demande si ce n'est pas ces 2 lignes qui posent pb :

Code :

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

1
2
char reception[64];
BYTE size;

Je ne peux pas les mettre dans un case ou dans un if, n'est ce pas ?

Aussi,

Est ce que ReadStringUART revoie la taille de ce qui est reçu sur le port série ou le contenu de ce qui est reçu ?

( d'apres le pilote uart.c que j'utilise, je pense avoir utilisé la bonne fonction mais j'ai quand même un doute, il existe aussi getsUART et aussi ReadUART, j'ai du mal a savoir laquelle utiliser pour capturer ce qui se présente sur mon port série.

**jowo** · 15/04/2009, 12h05

Envoyé par lcoulon

( d'apres le pilote uart.c que j'utilise, je pense avoir utilisé la bonne fonction mais j'ai quand même un doute, il existe aussi getsUART et aussi ReadUART, j'ai du mal a savoir laquelle utiliser pour capturer ce qui se présente sur mon port série.

Tu as le code source et la documentation. Prends ton temps pour lire la doc et étudier le code source...

**pfeuh** · 15/04/2009, 12h09

Salut,

Envoyé par lcoulon

je me demande si ce n'est pas ces 2 lignes qui posent pb :

Code :

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

1
2
char reception[64];
BYTE size;

Est ce que BYTE est défini? Par quelque chose du genre:

Code :

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

typedef char BYTE;

A+

Pfeuh

**lcoulon** · 15/04/2009, 12h10

Je sais bien Jowo,

Mais justement, ça fait deja une bonne semaine que je suis sur ce pb sans avancer vraiment.

si quelqu'un pourrait m'aiguiller un peu vis a vis de uart.c et me donner un peu de code pour dégrossir, cela m'aiderait beaucoup.

Invité(e) · 15/04/2009, 13h16

Envoyé par lcoulon

Code :

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

1
2
char reception[64];
BYTE size;

Je ne peux pas les mettre dans un case ou dans un if, n'est ce pas ?

On peut toujours déclarer des variables en début de bloc.

Envoyé par lcoulon

Est ce que ReadStringUART revoie la taille de ce qui est reçu sur le port série ou le contenu de ce qui est reçu ?

Si on lit le code, on voit qu'il renvoie la taille.

Et pour rebondir sur la remarque de pfeuh, BYTE est il bien défini dans uart.h ?

**lcoulon** · 15/04/2009, 15h09

oui, BYTE est défini mais dans un autre fichier .h ( dans generic_typedefs.h ) :

Code :

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

typedef unsigned char        BYTE;    // 8-bit unsigned

**lcoulon** · 16/04/2009, 14h08

Auriez vous d'autres pistes qui pourraient expliquer pourquoi le code de mabu me remonte une erreur de syntaxe en pointant BYTE ?

j'ai passé pas mal de temps sans trouver comment faire pour que ce petit code fonctionne ...

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
/* ReadStringUART : ne travaille qu'en BYTE : les messages récupérés ne peuvent 
   faire plus de 127 ou 255 caractères (suivant si BYTE est signée ou non)
   Pour ne pas prendre de risque, je me limite à 64.*/
char reception[64];
BYTE size;
 
/* on récupère la trame */
size = ReadStringUART((BYTE *)reception, sizeof reception - 1);
 
/* on fait en sorte que reception devienne une chaine de caractères valide */
reception[size] = '\0'
 
/* et on peut utiliser strcmp sans risque */
if (strcmp(reception, "*Hello")) == 0)
{
    /* faire des choses */
}

Merci de votre aide,

**jowo** · 16/04/2009, 14h12

Hello

On n'est pas devin alors restons sérieux et calme....

Envoie-nous le message d'erreur et le code entier...

**diogene** · 16/04/2009, 15h07

.... erreur de syntaxe en pointant BYTE ?

Parce que tu as oublié d'inclure generic_typedefs.h ???

Invité(e) · 16/04/2009, 15h50

Parce que BYTE est défini plusieurs fois ?

**lcoulon** · 16/04/2009, 17h01

si je declare en tout début de programme :

Code :

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

1
2
char reception[64];
BYTE size;

et ensuite je place dans ma boucle infinie le code que vous m'avez écrit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
/* on récupère la trame */
size = ReadStringUART((BYTE *)reception, sizeof reception - 1);
 
/* on fait en sorte que reception devienne une chaine de caractères valide */
reception[size] = '\0'
 
/* et on peut utiliser strcmp sans risque */
if (strcmp(reception, "*Hello")) == 0)
{
    /* faire des choses */
				OUT1 = 1;
}

le compilateur m'indique main.c:2711:Error: syntax error

la ligne 2711 est un commentaire, c'est pour cela que je pensai a une mauvaise déclaration ( ce n'etait qu'une idée ... )

Code :

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

/* et on peut utiliser strcmp sans risque */

parfois lorsque le compilateur pointe une ligne donnée pour une erreur de syntaxe, le pb se situe dans la ligne précédente, ici il s'agirait de :

Code :

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

reception[size] = '\0'

[PIC] Afficher une trame arrivant sur le port série

C

Discussions similaires

Partager

Partager