Discussion :

[As2piK]

Bonsoir,

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

Code :

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

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

[JolyLoic]

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

[As2piK]

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

[dragonjoker59]

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

[Invité]

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.

[phi1981]

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

[As2piK]

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

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.

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

[phi1981]

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.