Discussion :

[scls19fr]

Bonjour,

j'ai un petit soucis avec la fonction strcmp qui me lance un "Bus error"

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
 
#include <stdio.h> 
#include <string.h>
 
int main(void)
{
  puts("Running");
 
  char* s;
 
  while(1) {
    puts("mydevice> ");
    fgets(s, 10, stdin);
 
    if ( strcmp(s,"IDN")==0 ) {
      puts("*IDN? identification \r\n");
    } else {
      puts("Error ! this firmware doesn't understand this command\r\n");
    }
  }
 
  puts("Stopping\r\n"); /* it should never happen */
 
  return 0;
}

Avez-vous une idée d'où peut venir mon problème (ça fait un petit moment que je tourne autour sans le voir... ?


tant que j'y suis
je compile avec gcc avec les flags -ansi -pedantic -Wall
et j'ai le warning suivant à la compilation :
warning: ISO C90 forbids mixed declarations and code
c'est à cause de char* s; mais je ne vois pas en quoi je mélange
code et déclaration...

... pour cette deuxième partie j'ai vu... il suffit que je place ma déclaration avant la fonction main


Merci d'avance

[diogene]

Code :

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

1
2
3
  char* s;
....
    fgets(s, 10, stdin);

s doit être un tableau de caractères, pas un simple pointeur.

Code :

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

1
2
3
  char s[10];
....
    fgets(s, 10, stdin);

c'est à cause de char* s; mais je ne vois pas en quoi je mélange

En C90, les définitions des variables doivent être faites en début de bloc, avant la première instruction du bloc. Ici, char *s est précédé de l'appel à puts

[scls19fr]

Merci j'avais pensé à ça

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
 
#include <stdio.h> 
#include <string.h>
 
#define STR_SIZE 10
char s[STR_SIZE];
 
int main(void)
{
  puts("Running");
 
  while(1) {
    puts("mydevice> ");
    fgets(s, STR_SIZE, stdin);
 
    if ( strcmp(s,"IDN")==0 ) {
      puts("*IDN? identification \r\n");
    } else {
      puts("Error ! this firmware doesn't understand this command\r\n");
    }
  }
 
  puts("Stopping\r\n"); /* it should never happen */
 
  return 0;
}

je n'ai effectivement plus le bus error par contre je ne valide jamais la condition !!!

Une idée ?

[Melem]

fgets lit aussi le '\n', c'est-à-dire que tu as dans s "IDN\n" et non "IDN". Il y a plusieurs entrées qui traitent de la bonne utilisation de fgets dans la FAQ ...

[scls19fr]

Merci beaucoup

Voici le code final, j'ai préféré éviter les puts et fgets....

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
 
#include <stdio.h> 
#include <string.h>
#include <ctype.h>
 
#define STR_SIZE 10
char s[STR_SIZE]; /* received string */
char s2[STR_SIZE]; /* string converted to upper case */
int i;
 
int main(void)
{
  printf("Running\n");
 
  while(1) {
    printf("mydevice> ");
    scanf("%s",s);
    for (i=0 ; i<STR_SIZE ; ++i)
    {
      s2[i]=toupper(s[i]);
    }
    if ( strcmp(s2, "*IDN?")==0 ) {
      printf("*IDN? = device identification\n");
    } else {
      fprintf(stderr,"Error ! this firmware doesn't understand this command\n");
    }
  }
 
  printf("Stopping\n"); /* it should never happen */
 
  return 0;
}