Discussion :

[GGelec]

Bonjour à tous,

Je travail actuellement sur un petit projet qui doit manipuler des tableaux. Je travail avec un pic18f8520.

Je suis à la recherche d'une solution pour identifier des commandes reçues. j'ai un doute sur la manière dont je vais m'y prendre.

Ces commandes arrives dans un

Code :

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

unsigned char buffer;[64]

et j'aimerai établir un tableau de commandes prédéfini comme ceci :

Code :

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

unsigned char cmd[] = {"CMD1","CMD2","CMD3"};

mon but est donc de décortiquer le buffer suivant les possibilités du tableau de commande.
J'ai réalisé quelques bout de code avec strcmp voire strncmp mais ça ne marche pas toujours...
je dois encore y travailler.

Quoiqu'il en soit, je suis preneur si quelqu'un voit une meilleur solution pour récupérer des commandes depuis un buffer.

Merci à tous

[Bktero]

Il faut que tu précises un peu ton problème. Donne un exemple, avec un contenu type de ton buffer et quelles opérations tu souhaites réaliser dessus.

[CGi]

Bonjour,

et j'aimerai établir un tableau de commandes prédéfini comme ceci :

Code :

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

1
2
 
unsigned char cmd[] = {"CMD1","CMD2","CMD3"};

Un tableau de pointeur sur char serait mieux adapté pour stocker des chaine de caractères.

[pfeuh]

Salut,

Un tableau de pointeur sur char serait mieux adapté pour stocker des chaine de caractères.

Je crois que c'est déjà un tableau de pointeurs vers des chaines définies en constantes. Par contre, je ne vois pas l'intérêt de changer les char*, qui sont faits pour travailler avec des chaines en unsigned char*.

Je propose un tableau de structures contenant le pointeur vers la commande et le pointeur vers la fonction à utiliser.

A+

Pfeuh

Je viens de comprendre qu'on disait la même chose, CGi!

[GGelec]

Il faut que tu précises un peu ton problème. Donne un exemple, avec un contenu type de ton buffer et quelles opérations tu souhaites réaliser dessus.

Merci d'abord pour vos réponses

Mon readbuffer sera rempli avec soit des messages soit des commandes.
Je dois interpréter au mieux le contenu du readbuffer.

Grâce à un tableau de commande hardcodé, j'aimerai juste comparer le contenu du readbuffer avec ce tableau de commande.

Si une commande correspond au message contenu dans le readbuffer alors établir une action suivant la commande. Si la commande n'est pas reconnu, l’interpréter comme un message simple.

[GGelec]

Il faut que tu précises un peu ton problème. Donne un exemple, avec un contenu type de ton buffer et quelles opérations tu souhaites réaliser dessus.

Bonjour à tous,

J'ai envisagé une solution, peut être trop compliqué pour ce que je souhaite faire mais elle devrait malgré tout fonctionner.

Je réalise trois étapes: strstr, strcat et enfin strcmp (voire strncmp).

Ce que j'aimerai faire c'est normaliser les commandes avec une longueur de 4 char.

J'ai également l'idée d'implémenter un entête dans la trame que mon système doit recevoir.

Un moyen donc de détecter si il s'agit d'une commande ou si il s'agit d'un message.

Grâce à strstr je détermine la présente de cet entête (de 4 char également) pour allé rechercher directement 4 char plus loin la possible commande.

J'ai écris un bout de code mais je ne suis pas certain au niveau du strcmp.
Je n'ai pas encore eu l'occasion de tester. Je le fais demain.

Quoiqu'il en soit voici le bout de code en question que j'aimerai implémenter :

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
 
 
char readBuffer[10];
char writeBuffer[10];
char checkCmd[10];
char tabCmd[10] = {"cmd1", "cmd2"};
char *ptr;
char detectCMD[10] = {"#CMD"}; //4 char
int res;
int i;
 
//Recherche le format des commandes
    ptr = strstr(readBuffer, detectCMD);
 
    if (ptr != NULL) {
        res = ptr - readBuffer + 1;
        printf("\r\n\"%s\" found at position %d\r\n",
                detectCMD, res);
        //définit la position de la commande reçue (4char)
        res += sizeOfCmd; //position de #CMD + 4 pour débuter la capture de la cmd
        //On place le tout dans un string pour ensuite le comparer au tableau de cmd
        for (i = res; i < res + 4; i++) {
            strcat(checkCmd, readBuffer[i]);
        }
        printf("\r\nCMD need to be check : \"%s\" \r\n", checkCmd);
        //Pour chaque possibilité de commande (2) on check si le string est comparable
        for (i = 0; i < 2; i++) {
            if (strcmp(tabCmd[i], checkCmd) == 0) {
                printf("\r\nCmd match %s\r\n", checkCmd);
            } else {
                printf("\r\nNot valide Cmd %s\r\n", checkCmd);
            }
        }
    } else {
        printf("\r\n\"%s\" not found\r\n", detectCMD);
        printf("\r\nProbably a incoming message\r\n");
    }

Donc si mon buffer contient "#CMDcmd1" = commande reçue, ensuite action suivant la commande.

j'espère être clair dans mon explication

D'avance un grand merci pour votre aide déjà

[gulain]

Avec ton système, tu ne va traiter que des commandes fixée, tu n'auras pas besoin de recevoir des arguments variables (un nombre, autre chose) ?


Essaye, tant que possible de poster du code correct (qui compile).

Code :

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

1
2
3
4
 
char tabCmd[10] = {"cmd1", "cmd2"}; //est faux :
char tabCmd[10] // est un tableau de 10 char.
{"cmd1", "cmd2"} // correspondrait à l'initialisation des deux premiers éléments d'un tableau de tableau de char.

Utilise

Code :

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

1
2
3
4
5
6
 
const char *tabCmd[] = {"cmd1", "cmd2"}; 
//ou 
const char *tabCmd[2] = {"cmd1", "cmd2"};
 
idem pour char *detectCMD = "#CMD"

Code :

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

1
2
3
4
5
6
 
for (i = res; i < res + 4; i++) {
            strcat(checkCmd, readBuffer[i]); //checkCmd n'est pas initialisé, ça ne va pas marcher
}
//doit être remplacé par 
memcpy(checkCmd,readBuffer[res],4)

Pour le reste, tu vas avoir une ligne sur deux de tes logs qui sera vide, je ne sais pas si c'est voulu, et des "Not valide Cmd" (enlève ce e !) sur des commandes valides. J'ai un petit doute sur les \r\n (\n ne suffirait-il pas ?), mais je ne connais pas bien Windows.

[GGelec]

Avec ton système, tu ne va traiter que des commandes fixée, tu n'auras pas besoin de recevoir des arguments variables (un nombre, autre chose) ?


Essaye, tant que possible de poster du code correct (qui compile).

Code :

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

1
2
3
4
 
char tabCmd[10] = {"cmd1", "cmd2"}; //est faux :
char tabCmd[10] // est un tableau de 10 char.
{"cmd1", "cmd2"} // correspondrait à l'initialisation des deux premiers éléments d'un tableau de tableau de char.

Utilise

Code :

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

1
2
3
4
5
6
 
const char *tabCmd[] = {"cmd1", "cmd2"}; 
//ou 
const char *tabCmd[2] = {"cmd1", "cmd2"};
 
idem pour char *detectCMD = "#CMD"

Code :

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

1
2
3
4
5
6
 
for (i = res; i < res + 4; i++) {
            strcat(checkCmd, readBuffer[i]); //checkCmd n'est pas initialisé, ça ne va pas marcher
}
//doit être remplacé par 
memcpy(checkCmd,readBuffer[res],4)

Pour le reste, tu vas avoir une ligne sur deux de tes logs qui sera vide, je ne sais pas si c'est voulu, et des "Not valide Cmd" (enlève ce e !) sur des commandes valides. J'ai un petit doute sur les \r\n (\n ne suffirait-il pas ?), mais je ne connais pas bien Windows.

Merci pour ta réponse.
Je vais tenir compte de ce que tu m'as expliqué ci dessus.
Je ne travail pas sous windows. Je fais de la programmation embarqué (pic).

Je réfléchi toujours sur la manière de procéder.

Les commandes à recevoir sont fixes dans le sens où la machine envoi vers mon système "1" ou "6", mais j'envisage de faire plus compliqué pour interpréter des commandes pour pouvoir traiter des commandes plus compliquée comme "TEST" même si la réception d'un "T" suffirait.

J'y travaille

désolé pour le "e"