Discussion :

[gauguin]

Salut,

J'essaie d'ecrire un petit bout de code qui copie une chaîne de caractères dans une autre. Ce ne sont pas des tableaux mais des pointeurs.
C'est une sorte de fonction strcpy.

Cependant ça ne fonctionne pas et le programme se ferme.
But why ?

Code :

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

1
2
3
4
5
6
7
8
 
char *c, *c1;
  c=malloc(49+1);  
  c1=malloc(49+1);   
  c="bonjour";
  c1="au revoir";
  int i=0;
  for(i=0; i<50; i++)  *(c+i)=*(c1+i);

ET pareil avec

Code :

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

1
2
 
c[i]=c1[i]

merci.

[gl]

Code :

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

1
2
3
 
  c=malloc(49+1);  
  c1=malloc(49+1);

La tu alloues la memoire pour contenir tes chaines, jusque la ca va.

Code :

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

1
2
3
 
  c="bonjour";
  c1="au revoir";

Boum, tu ecrase les adresses fournies par malloc par celle de chaines constantes.
Tu as donc une belle fuite de memoire. La copie de chaine ne se fait pas avec = mais avec strcpy

Code :

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

1
2
3
 
  int i=0;
  for(i=0; i<50; i++)  *(c+i)=*(c1+i);

c et c1 contenant l'adresse de chaines constantes, la modification de c est un comportement indefini.

. En outre, tu ne verifies pas si l'allocation a correctement fonctionnee (c != NULL et c1 != NULL)
. tu ne liberes pas la memoire allouee (meme si ici c'est impossible puisque tu n'as plus de pointeur sur la zone allouee)
. De plus, pourquoi utilises tu une allocation dynamique pour des tableaux de taille fixe ?

[gauguin]

Pour répondre à la question, je me mélange un peu les pinceaux à vrai dire.

L'allocation dynamique je l'utilise juste parce que j'ai appris ça recemment

sinon, le fait de rentrer une chaine ( c="bonjour"; ) à partir d'un pointeur auquel on a alloué de la mémoire écrase les adresses fournies par malloc() ?

comment faire alors pour remplir une chaîne après avoir alloué de la mémoire ?

[crocodilex]

Pour répondre à la question, je me mélange un peu les pinceaux à vrai dire.

L'allocation dynamique je l'utilise juste parce que j'ai appris ça recemment

sinon, le fait de rentrer une chaine ( c="bonjour"; ) à partir d'un pointeur auquel on a alloué de la mémoire écrase les adresses fournies par malloc() ?

comment faire alors pour remplir une chaîne après avoir alloué de la mémoire ?

La réponse t'a été donnée plus haut. Il faut utliser strcpy().
Mais vu le titre de ton post, ce n'est donc pas la réponse que tu attends.
Tu peux éventuellement utiliser la fonction memcpy ou bien copier caractère par caractère dans une boucle.

[gauguin]

voilà, précisément, je veux copier caractères par caractères.

Et je ne comprends pas pourquoi ma boucle dans le 1er message ne marche pas.
(même si j'enlève le malloc()).

[gl]

voilà, précisément, je veux copier caractères par caractères.

Et je ne comprends pas pourquoi ma boucle dans le 1er message ne marche pas.
(même si j'enlève le malloc()).

Car c contient l'adresse d'une chaine constante qui peut ne pas etre modifiable et de surcroit pas assez longue.

Tant que les chaines initiales ne sont pas correctes, le reste ne fonctionnera pas.
Si tu souhaites avoir deux chaines valide contenant "bonjour" et "au revoir" pouvant faire 49 caracteres de long (hors \0 final), tu peux les creer avec:

Code :

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

1
2
char c[49+1] = "bonjour";
char c1[49+1] = "au revoir";

[crocodilex]

voilà, précisément, je veux copier caractères par caractères.

Et je ne comprends pas pourquoi ma boucle dans le 1er message ne marche pas.
(même si j'enlève le malloc()).

La copie dans la boucle se passait mal parceque tu avais écrasé tes pointeurs.