Discussion :

[tasna]

Bonjour,

J'ai un problème avec ce code qui provoque une erreur de segmentation au niveau de la fonction strcpy() :

Code :

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

1
2
3
4
5
6
7
8
 
char *Text=new char[9];
if(Text!=0)
{
     memset(&Text,0,sizeof(Text));
     strcpy(Text,"01234567");
     delete[] Text;
}

Et je ne comprends pas pourquoi TextSz n'est pas égal à 9 dans cet exemple :

Code :

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

1
2
3
4
5
6
7
 
char *Text=new char[9];
if(Text!=0)
{
     int TextSz=sizeof(Text);
     delete Text[];
}

D'avance merci de votre aide.

[Bousk]

Bonjour,

pour ton premier cas, tu donnes l'adresse de Text en paramètre à memset, il va donc remplacer la valeur pointée, soit le pointeur vers le char*.

Code :

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

1
2
3
4
5
6
7
char *Text=new char[9];
if(Text!=0)
{
     memset(&Text,0,sizeof(Text)); // l'adresse de text est maintenant 0
     strcpy(Text,"01234567"); // tu essayes donc de copier la chaîne à l'emplacement mémoire 0
     delete[] Text; // delete 0 ne plantera pas mais la mémoire allouée ne sera pas libéré
}

Code :

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

1
2
3
4
5
6
7
char *Text=new char[9];
if(Text!=0)
{
     memset(Text,0,sizeof(Text));
     strcpy(Text,"01234567");
     delete[] Text;
}

memset prend un pointeur et affectera la mémoire pointée, Text est déjà un pointeur, qui pointe sur les 9 char alloués.

deuxième problème:
Text* est un pointeur sur char*, sizeof te retourne la taille d'un char*, indépendament du fait que derrière se cache un char, char[2] ou char[4000].


ps: C++ => std::string.

[tasna]

ça veut dire que la mémoire allouée lors des essais n'a pas été libérée ?

Etant donné que je travaille avec des chaînes de caractères de 65536 octets, ça fait beaucoup de mémoire perdue.

Est-ce que la mémoire reste allouée à la fin de l'exécution du programme ou y'a-t-il un ramasse-miettes intégré au système ?

Pour info je développe sous Debian 2.6.32-5-686.

[Kalith]

ça veut dire que la mémoire allouée lors des essais n'a pas été libérée ?

D'après le code que tu nous as donné, si. L'opérateur delete (ou delete[]) "sait" combien d'octets ont été alloués dans Text. Si tu oublies de les appeler, alors la mémoire restera utilisée jusqu'à la fin du programme.

Ton code fonctionnerait si tu avais écris :

Code :

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

1
2
3
char Text[9];
memset(Text, 0, sizeof(Text));
strcpy(Text, "01234567");

ou :

Code :

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

1
2
3
4
char* Text = new char[9];
memset(Text, 0, sizeof(char)*9); // Car ici sizeof(Text) vaut 4 ou 8 (la taille du pointeur), mais pas la taille de la chaîne
strcpy(Text, "01234567");
delete[] Text;

ou, quand même vachement mieux :

Code :

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

1
2
std::string Text;
Text = "01234567";

[tasna]

Oui mais mon memset() avait changé l'adresse sur 0.

Mais du moment que la mémoire est libérée à la fin de l'exécution du programme ça m'arrange. Je n'aurais pas a éteindre mon PC.

Bon je me lance...
Qu'est ce que c'est std:: ?
De la STL ?
Est qu'est ce que la STL ?

[Kalith]

Oui mais mon memset() avait changé l'adresse sur 0.

Tout à fait. Ton exemple plante sur le strcpy, mais la mémoire est néanmoins toujours libérée à la sortie du programme (en principe ça dépend du système d'exploitation, mais c'est le cas pour Windows et linux).

Mais du moment que la mémoire est libérée à la fin de l'exécution du programme ça m'arrange. Je n'aurais pas a éteindre mon PC.

Ca dépend de ton programme. Si tu fais du traitement lourd, alors il y aura nécessairement un moment ou tu n'auras plus de mémoire disponible, et là tout fout le camp. Si c'est juste un petit programme, oui ça passera. C'est très moche, et je te déconseille de prendre cette mauvaise habitude, mais ça fonctionnera.

Bon je me lance...
Qu'est ce que c'est std:: ?
De la STL ?
Est qu'est ce que la STL ?

J'ai comme un doute : tu programmes bien en C++ ?
La STL est l'acronyme de "Standard Template Library". C'est un morceau de la bibliothèque standard du C++, qui constitue je pense l'un des plus gros atouts du langage. Comme tu peux le voir dans l'exemple ci-dessus, utiliser la STL te permet bien souvent de faire la même chose qu'en C avec moitié moins de code (et donc au minimum deux fois moins d'erreurs).
Le préfixe std:: est un namespace (espace de nommage) réservé à la STL, dans lequel tu trouveras beaucoup de classes très utiles, comme ici string.
Si tu veux en apprendre plus, je te conseille de lire la FAQ sur la STL.