Discussion :

[lcoulon]

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)]

Bonjour

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;
}

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]

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]

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)]

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...

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";

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]

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

[Invité(e)]

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 */
}