Discussion :

[barnoufal]

Bonjour,

J'essaie de modifier un caractère d'une chaîne de caractères dans une structure manipulée par un pointeur.

Si on prend par exemple une structure qui inclut une chaîne de caractère dans sa définition :

Code :

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

1
2
3
4
5
 
typedef struct{
  int annee;
  char *marque;
}Voiture;

Si plus loin je créé un pointeur de Voiture v. Je peux initialiser la Voiture pointée par v de plusieurs façons :

Code :

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

1
2
3
4
 
Voiture *v = (Voiture *)malloc(sizeof(Voiture));
v->marque = "Skoda";
(*v).marque = "BMV";

Mais si je veux modifier le 3ème caractère du champ marque, j'obtiens systématiquement un segfault ou une erreur avec l'une ou l'autre de ces solutions :

Code :

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

1
2
3
4
5
 
(*v).marque[2] = 'W'; //segmentation fault
v->marque[2] = 'W'; //segmentation fault
v->(marque + 2) = 'W'; //erreur
// etc...

Je pourrais bien sûr réinitialiser le champ en entier mais j'aimerais vraiment comprendre comment faire. Quelqu'un aurait une idée?

Merci.

[fregolo52]

salut,

ce n'est pas bon, tu dois faire un malloc pour la structure puis un malloc pour le char *.

Code :

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

1
2
Voiture *v = (Voiture *)malloc(sizeof(Voiture));
v->marque = (char *)malloc(5*sizeof(char));

et après tu fais un strcpy !!! pas de v->marque = "Skoda";

cette ligne sera bonne après :
v->marque[2] = 'W'; //segmentation fault

[Invité(e)]

Bonjour,

Attention, lors de l'allocation de la chaine de caractère, il faut penser à stocker le '\0' final :

Code :

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

1
2
3
4
5
const char modele[] = "Skoda";
 
Voiture *v = malloc(sizeof(Voiture));
v->marque = malloc(strlen(modele) + 1);
strcpy(v->marque, modele);

Ou mieux, si la fonction est strdup disponible :

Code :

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

1
2
3
4
const char modele[] = "Skoda";
 
Voiture *v = malloc(sizeof(Voiture));
v->marque = stdrup(modele);

Et dans les deux cas, une fois qu'on a fini d'utiliser les structures :

Code :

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

1
2
free(v->marque);
free(v);

[barnoufal]

Merci pour vos réponses ça fonctionne mieux!

Mais je ne comprends pas vraiment pourquoi ça ne pouvait pas marcher.
J'ai déjà vu des exemples où des chaines de caractères sont initialisées directement sans malloc?

Dans mon exemple je me disais que j'avais la structure dans le tas et la chaîne de caractère marque dans la pile. Bon ça me paraissait un peu bizarre mais ça passait à l'exécution... Où est-ce que je me trompe??

[fregolo52]

J'ai déjà vu des exemples où des chaines de caractères sont initialisées directement sans malloc?

es-tu capable de retrouver un exemple ?
A mon avis, ce n'était pas un char *.

char* est un pointeur, donc une adresse.
je ne sais pas quelle est ta formation. Pour ma part, à l'école, les profs nous faisaient de belles représentations de l'espace mémoire pour bien comprendre les pointeurs.

[Invité(e)]

J'ai déjà vu des exemples où des chaines de caractères sont initialisées directement sans malloc?

On peut utiliser des chaines de caractères statiques (des tableaux) :

Code :

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

1
2
3
 
/* création d'une chaine de 14 caractères */
char chaine[] = "toto est beau";

ou

Code :

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

1
2
3
/* création d'une chaine de 6 caractères */
char chaine[6];
strcpy(chaine, "Hello");

Après, quand tu parles de chaines sans utiliser de malloc[1], tu penses peut-être aux objets string qui appartiennent au c++.

[1] ni realloc, ni calloc, ni strdup

EDIT :

En bonus, un peu de lecture sur les pointeurs : http://emmanuel-delahaye.developpez....age=Page5#LXXI

[barnoufal]

J'ai vu des choses comme ce qu'a écrit Mabu et des trucs comme ci dessous. Cet exemple vient du site qu'il m'a conseillé (merci!). Il n'y a pas de malloc avant.

Code :

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

1
2
 
char const *pf = "hello";