Redéfinir la taille d'un char*

Bonsoir,

J'ai un problème pour redéfinir la taille d'un char*. Voici mon code :

Code:

---

1 2 3 4 5 6 7

void String::setSize(int size){ * char *tmp = new char[size+1]; * strcpy(tmp, this->s); * delete[] this->s; * this->s = new char[size+1]; * strcpy(this->s, tmp); }

---

Avec this->s un attribut de ma classe...

Le code ci-dessus ne fonctionne pas...

Est-ce que qqun aurait une idée ?

Merci

Déjà, pourquoi faire 2 copies ?
Et que se passe-t-il si la nouvelle taille est plus petite que l'ancienne ?

PS : J'espère qu'il s'agit d'un exercice. Pour le vrai travail, des classes de chaîne de caractère existent déjà...

Oui oui il s'agit bien d'un exercice :)

Mon problème c'est que je suis très habitué au Java, j'ai donc de gros problèmes pour bien comprendre les pointeurs et les références... -_-

Concernant la première copie, c'est pour sauvegarder la valeur qui se trouve dans la string à redimensionner...

Ensuite je delete la string puis je la recrée à la nouvelle taille, puis je copy la valeur de la string temporaire dans this->s...

Logiquement, cela devrait fonctionner, non ?

Merci :)

Que veux tu dire par 'ça ne fonctionne pas' ? Ca plante ? Ca ne compile pas ?

Bonjour,

On ne connait pas le reste du programme, mais à mon avis, ça serait bien de vérifier que this->s a bien été instancié avant de l'utiliser.

Ensuite, le char* tmp n'est jamais détruit.
Au fur et à mesure que le programme entrera dans cette méthode il consommera de plus en plus de mémoire qu'il ne rendra jamais.

Heu .... je sais que ça peut paraitre stupide comme remarque, mais les étoiles au début de chaque ligne vont faire échouer la compilation.

Citation:

---

Envoyé par dragonjoker59

Que veux tu dire par 'ça ne fonctionne pas' ? Ca plante ? Ca ne compile pas ?

---

Enfaite ca compile. J'instancie ma String à "abcd" puis j'appelle setSize(10), mais ma String mesure toujours 5 (avec le caractère \0).

Citation:

---

Bonjour,

On ne connait pas le reste du programme, mais à mon avis, ça serait bien de vérifier que this->s a bien été instancié avant de l'utiliser.

Ensuite, le char* tmp n'est jamais détruit.
Au fur et à mesure que le programme entrera dans cette méthode il consommera de plus en plus de mémoire qu'il ne rendra jamais.

---

Le this->s est bien instancié... Par contre en effet j'ai oublié de deleter tmp.

Citation:

---

Heu .... je sais que ça peut paraitre stupide comme remarque, mais les étoiles au début de chaque ligne vont faire échouer la compilation.

---

C'est le copier-coller de DevC++ qui m'ajoute les étoiles... Elles n'y sont pas dans le programme ^^

Merci de votre aide...

Comment mesures tu la taille de la chaine ??

Désolé pour les étoiles .... mais j'aurais vraiment trouvé ça drôle que ce soit l'origine de ton problème.

Bon, en fait ça marche ^^.
Tu as une chaîne de caractères, 'abcd\0' qui a une taille de 5 (en tenant compte du '\0').
Quand tu fais ton setSize, tu alloues à ton buffer une nouvelle taille, mais tu copies 'abcd\0' avec strcpy. du coup quand tu affiches le contenu de ta nouvelle chaîne, ça t'affiche 'abcd' (le caractère '\0' servant à connaitre la taille d'une chaîne de caractères).
Je te conseille donc, pour voir une différence à l'affichage, d'utiliser memcpy plutôt que strcpy. Pour le nombre d'octets à copier, strlen de la chaîne avant le setSize fera l'affaire.

All right ca fonctionne, merci beaucoup :)