Discussion :

[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

[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 :

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

[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

[uknow]

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]

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

[Lharuun]

Bonjour,

Mais tu sais que quand tu passes ta chaine

Code :

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

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 :

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

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 :

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