espionnage port serie

Version imprimable

31/01/2011, 15h10
chris83160

espionnage port serie

Bonjour, je dois scrupter un port série en permanence pour pouvoir arrêter un pc des qu'il reçoit une info sur le port série. Cette info est la suivante : POWER OFF! qui se repete indefiniement pendant une tempo de 30s. J'ai fait un programme en c++ qui marche juste lorsque j'envoie POWER OFF! via hyperterminal. Mais des que je lui envoie deux "POWER OFF!POWER OFF!" la comparaison des deux chaines ne marche pas. Si quelqu'un peut m'aider car je suis pas un pro dans le domaine de la programmation en C++. D'avance merci

Citation:

Envoyé par chris83160

Bonjour, je dois scrupter un port série en permanence pour pouvoir arrêter un pc des qu'il reçoit une info sur le port série. Cette info est la suivante : POWER OFF! qui se repete indefiniement pendant une tempo de 30s. J'ai fait un programme en c++ qui marche juste lorsque j'envoie POWER OFF! via hyperterminal. Mais des que je lui envoie deux "POWER OFF!POWER OFF!" la comparaison des deux chaines ne marche pas. Si quelqu'un peut m'aider car je suis pas un pro dans le domaine de la programmation en C++. D'avance merci

Code:

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
int main () 
{     
         char ChaineRecueTmp[32] ;
         char ChaineRecue[32] ;
         int nbMaxCar = 10 ;
         int nbCarRecus ;
         short MessageRecu = FALSE ;
 
         memset (ChaineRecueTmp, '\0',32);
         memset (ChaineRecue, '\0',32);
 
         if ( OuvrirCom("COM1", 19200, 8, "Aucune", 1 ) ){
            printf("\n Ouverture du port effectuee.\n") ;
         }
         else{
            printf("\n Erreur ouverture port.\n") ;
         }
 
         while (!MessageRecu){
               sleep (5000);
               if ( RecevoirCom(ChaineRecueTmp, nbMaxCar, &nbCarRecus ) ) {
                  if ( nbCarRecus != 0 ){
                     printf (" Trame recue correctement.\n") ;
                     //strncat (ChaineRecue, ChaineRecueTmp, 10);
                     if (strcmp (ChaineRecue, "POWER OFF!") == 0 )
                        { 
                        system("shutdown -s -f -t 30");
                        return EXIT_SUCCESS;
                        }
                     else {
                        printf (" Erreur comparaison avec POWER OFF!: %s\n",ChaineRecue) ;             
                        }
                     }
                  else {   
                     printf (" nbCarRecus : %d\n",&nbCarRecus) ;             
                     }
                  }
         } //Fin du While   
} //Fin du main

31/01/2011, 17h21
Lharuun

Bonjour,

le comportement est normal a mon sens

quand tu envoies POWER_OFF!POWER_OFF!

c'est la chaine qui va etre comparée avec strcmp

et forcément POWER_OFF!POWER_OFF! != POWER_OFF!

Essais plutot avec strstr qui va vérifier qu'une chaine se trouve dans l'autre.

http://www.cplusplus.com/reference/c...string/strstr/

Cordialement
31/01/2011, 18h17
uknow

Citation:

Envoyé par Lharuun

Bonjour,

le comportement est normal a mon sens

quand tu envoies POWER_OFF!POWER_OFF!

c'est la chaine qui va etre comparée avec strcmp

et forcément POWER_OFF!POWER_OFF! != POWER_OFF!

Essais plutot avec strstr qui va vérifier qu'une chaine se trouve dans l'autre.

[Petite anecdote] Sous réserve que la chaine soit délimitée par un 0; ce qui est rarement le cas quand on traite des données issues d'un buffer binaire (le port série en l'occurrence).
Sauf si l'expéditeur pense à envoyer le délimiteur, il faut le mettre à la main pour pouvoir manipuler les données sous forme d'une chaine de caractères. [/anecdote]
01/02/2011, 10h14
chris83160

Merci je ne connaissais pas cette commande. Mais elle fait reference a des pointeurs et moi je veux juste comparer un "serie" de caracteres. Je ne vois pas comment modifier mon programme ou mes variables pour arriver a ca, merci de votre aide..
01/02/2011, 12h28
Lharuun
Bonjour,

Mais tu sais que quand tu passes ta chaine
Code:

1 2 char ChaineRecue[32] ;
a strcmp, il y a plus que de "très fortes" chances que le prototype de la fonction soit plutot du genre :
Code:

1 2 3 4 5 6 int strcmp(char *ch1,char *ch2) et non pas int strcmp(char ch1[32],char ch2[32])
D'ailleru quand tu fais strcmp(ChaineRecue,"POWER_OFF!")

ChaineRecue est en fait un pointeur dans le cas des tableau.
ce qui explique que tu puisses accéder a un caractère de ton tableau des façons suivante :

Citation:

ChaineRecue[0] == *ChaineRecue
ChaineRecue[1] == *(ChaineRecue+1)
et ainsi de suite

ChaineRecue est en fait un pointeur sur le premier élément de ton tableau.

Par contre, pour reprendre l'anecdote, il faut bien faire attention a avoir le '\0' apres le dernier caractere de tes chaines de caractere, sinon risque de SEGFAULT

Si tu fais du C assez souvent, je ne saurais que te conseiller fortement de lire un peu de littérature a propos des pointeurs, c'est assez utile.

cherche un peut du côté de strstr : RTFM man strstr
C'est pas bien compliqué.
Cordialement

cordialement

ok, donc si je te suis bien dans " l'anecdote" comme je recois non stop" POWER OFF! POWER OFF! POWER OFF!......"il n'y a jamais ce \0.
J'ai mis ds mon code ceci

Code:

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
int main () 
{     
         char ChaineRecueTmp[32] ;
         char ChaineRecue[32] ;
         int nbMaxCar = 10 ;
         int nbCarRecus ;
         short MessageRecu = FALSE ;
 
         memset (ChaineRecueTmp, '\0',32);
         memset (ChaineRecue, '\0',32);
 
         if ( OuvrirCom("COM1", 19200, 8, "Aucune", 1 ) )
         {
            printf("\n Ouverture du port effectuee.\n") ;
         }
         else{
            printf("\n Erreur ouverture port.\n") ;
         }
 
         while (!MessageRecu)
         {
               sleep (5000);
               if ( RecevoirCom(ChaineRecueTmp, nbMaxCar, &nbCarRecus ) ) 
               {
                  if ( nbCarRecus != 0 )
                  {
                     printf (" Trame recue correctement.\n") ;
                     //if (strncmp (ChaineRecue, "POWER OFF!" ,10) == 0 )
                      if (strstr(ChaineRecue, "POWER OFF!")!=NULL)
                       {
                        system("shutdown -s -f -t 30");
                        return EXIT_SUCCESS;

Mais helas pas plus de succes j'ai toujours une difference de comparaison...je comprends pas pourquoi!!!

01/02/2011, 14h15
Lharuun

Bonjour,

Quand tu as reçu ta sequence, est ce que tu peux faire un printf histoire de voir
ce que tu as dans ChaineRecue juste comme ca ?
08/02/2011, 13h56
uknow
Pour info :
Code:

1 2 3 4 5 6 int foo(char *ch1,char *ch2) //et int foo(char ch1[32],char ch2[32]) //et int foo(char ch1[],char ch2[])
Sont strictement équivalents (dans le sens absolu du terme équivalents).
Les [32] et [ ] sont trompeurs, ils laisseraient croire qu'on a déclaré deux arguments de types char[32] ou char[ ]; mais en fait il s'agit de deux char*.

De même :
Code:

1 2 3 ChaineRecue[n] //et *(ChaineRecue+n)
Sont garantis être équivalents (par la norme).

Pour revenir à \0, c'est bien la fonction "RecevoirCom" qui devrait s'en charger. Elle devrait le placer manuellement à la fin de la réception, et éventuellement vérifier que le buffer de réception est assez large pour contenir les nouvelles données reçues.

Il faudrait s'assurer que les configurations du port COM sont identiques au niveau de la machine qui envoie et celle qui reçoit (baude rate, parité, nb de bits, bits de stop, éventuellement le contrôle de flux). Tu peux dans un premier temps t'aider d'un terminal RS232 (hyper terminal pour windows) pour vérifier que la machine reçoit bien ce qui a été envoyé, et sous sa bonne forme.

Ensuite, peux-tu si possible, afficher la valeur de "nbCarRecus" après une réception ?
08/02/2011, 14h19
dragonjoker59

Et ça marche, ou pas avec le strncmp que tu as commenté ?
10/02/2011, 13h22
salseropom

en C++, pour comparer des strings, on utilise l'opérateur ==

De plus, la fonction membre find() te permets de trouver l'occurence d'une string dans une autre.

Je pense qu'en C++ il vaut mieux utiliser les strings que les char* (qui sont plus pour du C)

Ca ne répond certes pas à ta question, mais cela peut t'éviter des erreurs du genre '\0', ou '\n' etc...