Problème de segmentation.

**Gryzzly** · 27/12/2005, 09h41

Bonjour,

dans le code suivant:

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
#include<float.h>
 
 
int Analyse(char **ligne)
{
  int res;
  char *mot;
  char *endptr;
 
  while(**ligne == ' ') (*ligne)++;
 
  /* on est en fin de chaine */
  if(**ligne == '\0')
    {
      fprintf(stderr,"erreur : rien n'a ete lu\n");
      return 1;
    }
 
  while(**ligne != '\0')
    {
      /* c'est un chiffre */
      if(isdigit(**ligne))
	{
	 res =  sscanf(*ligne,"%[0-9.]s ",mot);
	 if(res != 1)
	   return 1;
	  strtod(mot,&endptr);
	  if(*endptr != '\0')
	    return 1;
 
	  while(isdigit(**ligne) || **ligne == '.') (*ligne)++;
	}
 
 
      /* c'est une lettre */
      if(isalpha(**ligne))
	{
	  res = sscanf(*ligne,"%[a-z]s ",mot);
	  if(res != 1)
	    return 1;
	  while(isalpha(**ligne)) (*ligne)++;
	}
    }
 
  if(**ligne == '\0')
    return 0;
  return 1;
}
 
 
 
int main(int argc, char*argv[])
{
  int i;
  for(i=1;i<argc;i++)
    {
      if(Analyse(argv[i]))
	{
	  fprintf(stderr,"erreur\n");
	  return 1;
	}
      printf ("'%s'\n",argv[i]);
    }
  return 0;
}

j'obtient un seg fault.
Quand j'utilises gdb,j'ai ce message:

Code :

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

1
2
 
Program exited normally.

Est ce que vous voyez l'erreur?

Merci

**PRomu@ld** · 27/12/2005, 09h49

Est ce que vous voyez l'erreur?

Non j'ai pas mes lunettes.

En fait rassure moi d'un truc : sous gdb, t'a bien lancé ton programme avec des paramètres ? Parce que si c'est pas le cas, c'est normal qu'il ne te trouve pas d'erreur. (en fait il ne rentre pas dans la boucle for de ton main)

Exécute le avec des paramètres et là tu auras un beau segfault.

[EDIT]
Après un yeutage très furtif, il y a un truc qui me parait bizarre :
ta fonction analyse prend un pointeur sur une chaine de caractère (char **) et tu lui passe une chaine de caractère (argv[i] est un char *)

**Amokrane** · 27/12/2005, 10h00

Après un yeutage très furtif, il y a un truc qui me parait bizarre :
ta fonction analyse prend un pointeur sur une chaine de caractère (char **) et tu lui passe une chaine de caractère (argv[i] est un char *)

Ben c'est normal ça , non ? vu que le nom d'un tableau est un pointeur vers son premier élément ? plus l'* qui a avant dans son int main ça en fait bien un pointeur vers une chaine ?

**PRomu@ld** · 27/12/2005, 10h09

argv est un char ** et argv[i] est un char * et la fonction analyse attend un char ** en paramètre.

**Gryzzly** · 27/12/2005, 10h18

J'ai changé le char ** en char * dans la fonction Analyse mais j'ai toujours le seg fault

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
#include<float.h>
 
 
int Analyse(char *ligne)
{
  int res;
  char *mot;
  char *endptr;
 
  while(*ligne == ' ') ligne++;
 
  /* on est en fin de chaine */
  if(*ligne == '\0')
    {
      fprintf(stderr,"erreur : rien n'a ete lu\n");
      return 1;
    }
 
  while(*ligne != '\0')
    {
      /* c'est un chiffre */
      if(isdigit(*ligne))
	{
	  res =  sscanf(ligne,"%[0-9.]s ",mot);
	  if(res != 1)
	   return 1;
	  strtod(mot,&endptr);
	  if(*endptr != '\0')
	    return 1;
 
	  while(isdigit(*ligne) || *ligne == '.') ligne++;
	}
 
 
      /* c'est une lettre */
      if(isalpha(*ligne))
	{
	  res = sscanf(ligne,"%[a-z]s ",mot);
	  if(res != 1)
	    return 1;
	  while(isalpha(*ligne)) ligne++;
	}
    }
 
  if(*ligne == '\0')
    return 0;
  return 1;
}
 
 
 
int main(int argc, char*argv[])
{
  int i;
  for(i=1;i<argc;i++)
    {
      if(Analyse(argv[i]))
	{
	  fprintf(stderr,"erreur\n");
	  return 1;
	}
      printf ("'%s'\n",argv[i]);
    }
  return 0;
}

**vince3320** · 27/12/2005, 10h25

Salut!

Juste deux choses :

Si tu ne sais pas utiliser gdb, utilise printf!

exemple :

Code :

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

1
2
3
4
 
printf ("ca passe\n"); fflush (stdout);
res =  sscanf(ligne,"%[0-9.]s ",mot); 
printf ("coucou?\n"); fflush (stdout);

Ensuite, tu regarde ce qui s'est affiche ou non.
Comme cela, tu remonte a ton erreur.

Le fflush te permet de vider le buffer : il est imperatif de l'utiliser dans ces cas la.

Une erreur vient du fait que tu utilise des char *

Code :

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

1
2
3
 
  char *mot;
  char *endptr;

sans les avoir alloue dynamiquement...
Du coup, ton programme a un comportement indefini...

**Trap D** · 27/12/2005, 10h32

Il faut faire par exemple char mot[128], c'est indispensable car sinon tu écros dans la nature avec ton sscanf (qui d'ailleurs ne sert strictement à rien, tu peux t'en passer en cherchant un peu). pour le pointeur char *end, ce n'est pas grave puisque tu passes l'adresse de end dans le strtod.

**PRomu@ld** · 27/12/2005, 10h33

J'ai entendu dire que flush demandait le vidage du buffer mais que le système d'exploitation ne réalisait pas toujours ce vidage (quel égoïste celui là !!!)

Par conséquent, j'ai pris l'habitude d'utiliser fprintf sur l'erreur standard. Et comme ceci je suis sur que le message s'affichera bien :

Code :

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

1
2
 
fprintf(stderr," ... ");

**Gryzzly** · 27/12/2005, 10h33

Envoyé par vince3320

Salut!

Juste deux choses :

Si tu ne sais pas utiliser gdb, utilise printf!

exemple :

Code :

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

1
2
3
4
 
printf ("ca passe\n"); fflush (stdout);
res =  sscanf(ligne,"%[0-9.]s ",mot); 
printf ("coucou?\n"); fflush (stdout);

Ensuite, tu regarde ce qui s'est affiche ou non.
Comme cela, tu remonte a ton erreur.

Le fflush te permet de vider le buffer : il est imperatif de l'utiliser dans ces cas la.

Une erreur vient du fait que tu utilise des char *

Code :

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

1
2
3
 
  char *mot;
  char *endptr;

sans les avoir alloue dynamiquement...
Du coup, ton programme a un comportement indefini...

Bonjour,
Merci de tes conseils,j'ai effectivement des resultats correctes.

Je voudrais savoir :
Outre le fait d'allouer ,comment il fallait que je m'y prenne pour argv dans la boucle si je garde char ** ligne dans ma fonction Analyse?

**Gryzzly** · 27/12/2005, 10h35

Envoyé par Trap D

Il faut faire par exemple char mot[128], c'est indispensable car sinon tu écros dans la nature avec ton sscanf (qui d'ailleurs ne sert strictement à rien, tu peux t'en passer en cherchant un peu). pour le pointeur char *end, ce n'est pas grave puisque tu passes l'adresse de end dans le strtod.

J'ai utilisé sscanf car il me permet d'etre sur d'avoir un double ou une chaine de caractère bien écrit .
Edit:en fait je ne comprends pas pourquoi, le programme reconnait
"1222gfgfgg" et "dffdfdfd4447"

**Trap D** · 27/12/2005, 10h39

Comment reconnais-tu qu'une chaîne de caractères est bien écrite ?
D'accord pour les entiers et les floats/double à la rigueur, bien que strtod et strtol soient plus précis car ils donnent l'endroit où la conversion s'arrête dans la chaîne passée en paramètre.

**Gryzzly** · 27/12/2005, 10h42

Envoyé par Trap D

Comment reconnais-tu qu'une chaîne de caractères est bien écrite ?
D'accord pour les entiers et les floats/double à la rigueur, bien que strtod et strtol soient plus précis car ils donnent l'endroit où la conversion s'arrête dans la chaîne passée en paramètre.

En testant ,avec des parametres "non valides" pour mon exo ,le programme reconnait "gfgfg1112" et "1222hfdghfg" alors qu'il ne devrait pas.
Existe t-il une manière de procéder pour etre sur du résultat?

**Trap D** · 27/12/2005, 11h02

Le principe est que tu définisses exactement ce que tu entends par nombre, nombre flottant, chaine de caractères, séparateurs (caractères à sauter).
Tu peux ensuite faire un automate qui étudie la chaîne.
Il te faut une variable indiquant le type de données que tu es en train de lire,
une autre indiquant le début de la donnée que tu lis.
Si tu n'y arrives pas, je te montrerai comment faire.

Problème de segmentation.

C

Discussions similaires

Partager

Partager