ecriture fonction strcpy

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:

---

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:

---

1 2	c[i]=c1[i]

---

merci.

Code:

---

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:

---

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:

---

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 ?

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 :) :P

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 ?

Citation:

---

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

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.

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

Citation:

---

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

---

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:

---

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

---

Citation:

---

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

---

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

bon désolé, no comprendo :roll:

voilà donc :

1- comment remplir des chaînes, pour lesquelles on a alloué de l'espace avec malloc(), avec des caractères ?

2- comment faire alors pour ne pas écraser les pointeurs, faut-il faire encore un pointeur sur le pointeur de début de chaîne et le déplacer avec une boucle ?

ps : comme c'est dans un but d'apprentissage et de comprendre comment marche la bête, je veux éviter les fonctions toutes faites.

merci.

Pour remplir une chaîne, t'as plusieurs possibilités :

- Utiliser ne fonction de saisie (fgets idéalement, sinon scanf, gets, etc.).
- strcpy
- Remplir lors de la déclaration...

ok,

donc *(c+i) n'est ps une lvalue alors, c a d qu'on ne peut pas lui affecter qq une fois que l'on a déjà "mis" une chaîne dedans alors car

ex :

Code:

---

1 2 3 4

char *c, *pc; pc=c; c="bonjour";

---

Code:

---

1 2	*(c+2)='x' ou *(pc+2) ne marchent pas

---

donc voilà ma dernière question.

Je vote pour celui qui m'explique clairement les pointeurs et pour qu'un grand débat national soit organisé sur ce sujet trop souvent ignoré :D

Citation:

---

Envoyé par gauguin

bon désolé, no comprendo :roll:

voilà donc :

1- comment remplir des chaînes, pour lesquelles on a alloué de l'espace avec malloc(), avec des caractères ?

---

L'espace qui a été alloué, et dont l'adresse du premier élément a été stockée dans le pointeur, est un tableau de char. Pour y mettre une chaine, on peut utiliser plusieurs solutions.

• l'accès direct caractère par caractère, dans une boucle, par exemple.
• strcpy()
• memcpy()
• un tas d'autres moyens plus complexes, sprintf(), fgets() etc.

Ne jamais perdre de vue qu'une chaine est un tableau de char initialisé terminé par un 0. Sinon, le comportement est indéfini.

Citation:

---

2- comment faire alors pour ne pas écraser les pointeurs,

---

On s'abstient d'écrire dans le pointeur ou de le modifier...

Citation:

---

faut-il faire encore un pointeur sur le pointeur de début de chaîne et le déplacer avec une boucle ?

---

En général, un indice suffit. On a affaire à un tableau dont on ne connait pas nom, mais dont on connait l'adresse du premier élément et le type des éléments. On a donc le droit d'utiliser la notation indicée ([i]) avec le pointeur pour atteindre n'importe quel élément du tableau, du moment qu'on est dans les limites.
.