Discussion :

[Premium]

Bonjour,

j'ai un problème concernant ceci.

Code :

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

1
2
3
4
5
6
char * sep = NULL;
sep = strchr(ligne,' ');
if(sep == NULL)
/*message erreur*/
*sep = '\0';
<...>

J'ai un seg fault qui est provoque par :

Code :

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

*sep = '\0';

je ne comprends pas le problème.
J'ai retire l'initialisation de

Code :

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

char *sep = NULL

et j'ai remplace par

Code :

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

char *sep

mais j'ai toujours l'erreur

[vince3320]

Salut,

A mon avis, il manque l'allocation dynamique... (j'espere ne pas dire de bétise!)

[hegros]

Tu ne peux pas affecter une valeur à un objet pointé dont le pointeur est NULL ( car aucun objet pointé).

J'espere ne pas dire de bêtise ce matin moi aussi

[Premium]

Salut,

A mon avis, il manque l'allocation dynamique... (j'espere ne pas dire de bétise!)

J'ai oublié de préciser que ligne est un parametre de fonction.

[Médinoc]

vince3320: Hélas, si, tu dis une erreur: Son pointeur n'est pas affecté par une allocation dynamique, mais par une recherche dans un bloc mémoire existant.
hegros: pareil.

Man_Utd:
1) Vérifie que ligne ne pointe pas vers des données constantes (comme une constante chaîne, par exemple. La version C de strchr() renvoit toujours un pointeur non-const même si le pointeur d'origine est const, hélas...)
2) vérifie qu'on ne passe pas par le " *sep='\0'; " après le message d'erreur.

[Premium]

Tu ne peux pas affecter une valeur à un objet pointé dont le pointeur est NULL ( car aucun objet pointé).

J'espere ne pas dire de bêtise ce matin moi aussi

C'est ce que je pensait aussi mais en retirant l'initialisation comme je l'ai dit,le seg fault persiste à cette ligne

[Premium]

vince3320: Hélas, si, tu dis une erreur: Son pointeur n'est pas affecté par une allocation dynamique, mais par une recherche dans un bloc mémoire existant.
hegros: pareil.

Man_Utd:
1) Vérifie que ligne ne pointe pas vers des données constantes (comme une constante chaîne, par exemple. La version C de strchr() renvoit toujours un pointeur non-const même si le pointeur d'origine est const, hélas...)
2) vérifie qu'on ne passe pas par le " *sep='\0'; " après le message d'erreur.

Je ne comprends pas pour faire *sep = '\0',il faut d'abord que je vérifies que cela est possible,or tu sembles suggérer le contraire

[Emmanuel Delahaye]

A mon avis, il manque l'allocation dynamique...

Pourquoi faire ?

(j'espere ne pas dire de bétise!)

Heu... Joker !

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
char * sep = NULL;
sep = strchr(ligne,' ');
if(sep == NULL)
/*message erreur*/
*sep = '\0';
<...>

J'ai un seg fault qui est provoque par :

Code :

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

*sep = '\0';

C'est sûr que si sep vaut NULL et que tu fais *sep = 0; le comportement est indéfini. Si sep vaut NULL, surtout ne rien faire d'autre que d'indiquer l'erreur. (En tout cas, ne pas déréférencer un pointeur NULL).

[Emmanuel Delahaye]

hegros: pareil.

Gné ?

[Emmanuel Delahaye]

mais en retirant l'initialisation comme je l'ai dit,le seg fault persiste à cette ligne

Tu ne peux déréférencer un pointeur qui si il est valide.

char *p = NULL; : p est explicitement invalide
char *p; p est non initialisé donc implicitement invalide.

Pourquoi tu veux mettre ce 0. Laisse le tranquille, ne fait rien !

[Emmanuel Delahaye]

Je ne comprends pas pour faire *sep = '\0',il faut d'abord que je vérifies que cela est possible,or tu sembles suggérer le contraire

C'est possible si p est != NULL.

Utilisation classique : supprimer le '\n' d'une ligne saisie avec fgets() :

Code :

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

1
2
3
4
5
6
7
 
   char *p = strchr( line, '\n');
 
   if (p != NULL)
   {
      *p = 0;
   }

[Premium]

mais en retirant l'initialisation comme je l'ai dit,le seg fault persiste à cette ligne

Tu ne peux déréférencer un pointeur qui si il est valide.

char *p = NULL; : p est explicitement invalide
char *p; p est non initialisé donc implicitement invalide.

Pourquoi tu veux mettre ce 0. Laisse le tranquille, ne fait rien !

Je souhaites mettre '\0' pour recuperer les elements.
par exemple si ma chaine est :

Code :

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

"125 1254 788 bonjour bonsoir"

je place '\0' juste apres le 5 de 125,puis je fais de meme...
Lorsqu' il ne restera qu'un element à analyser "bonsoir" ma boucle s'arretera car in n'y aura plus d'espace.
Je mets un bout de code (plus complet):
PS:le code est pour les chiffres mais c'est le meme principe pour les mots

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
 
Arbre CreerArbreFonction(char **ligne)
{
  Environnement Env;
  double val;
  char *chiffre;
  char *sep = NULL;
  int n;
  /*ne sert pas pour resoudre le seg fault*/
  Arbre a = malloc(sizeof(Noeud));
  if(a == NULL)
    {
      fprintf(stderr,"erreur allocation de l'arbre\n"); 
      return NULL;
    }
 
  while(**ligne != '\0')
    {
      /* c'est un chiffre */
      if(isdigit(**ligne) || **ligne == '.')
	{
	  chiffre = *ligne;
 
	  sep = strchr(*ligne,' ');
 
	  /* il n'y a pas d'espace donc chaine se termine par '\0' */
	  if(sep == NULL)
	    {
	      **ligne = '\0';
	    }
 
	  else
	    {
	      *sep = '\0';
	      *ligne = sep;
	      (*ligne)++;
	      espace(ligne);
	    }
 
 
	  if(EstDouble(chiffre,&val))
	    {	    
	      fprintf(stderr,"le chiffre : '%s' n'est pas correctement ecrit\n",chiffre);
	      return NULL;
	    }
<...>

[Emmanuel Delahaye]

Code :

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

"125 1254 788 bonjour bonsoir"

je place '\0' juste apres le 5 de 125,puis je fais de meme...

Pour extraire des valeurs numériques séparées par espaces,
- soit le nombre est fixe et sscanf() convient,
- soit il est variable, et une boucle avec strtol() (par exemple) suffit. Bien gérer le 2 ème paramètre et le début de la chaine à analyser...