Problème d'initialisation

Bonjour,

j'ai un problème concernant ceci.

Code:

---

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:

---

*sep = '\0';

---

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

Code:

---

char *sep = NULL

---

et j'ai remplace par

Code:

---

char *sep

---

mais j'ai toujours l'erreur

Salut,

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

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

Citation:

---

Envoyé par vince3320

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.

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.

Citation:

---

Envoyé par 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 :roll:

---

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

Citation:

---

Envoyé par 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.

---

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

Citation:

---

Envoyé par vince3320

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

---

Pourquoi faire ?

Citation:

---

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

---

Heu... Joker !

Citation:

---

Envoyé par Man_Utd

Code:

---

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:

---

*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).

Citation:

---

Envoyé par Médinoc

hegros: pareil.

---

Gné ?

Citation:

---

Envoyé par Man_Utd

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 !

Citation:

---

Envoyé par Man_Utd

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:

---

1 2 3 4 5 6 7

char *p = strchr( line, '\n');   if (p != NULL) { *p = 0; }

---

Citation:

---

Envoyé par Emmanuel Delahaye

Citation:

---

Envoyé par Man_Utd

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:

---

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

---

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; } <...>

---

Citation:

---

Envoyé par Man_Utd

Code:

---

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