Discussion :

[Zeeraptor]

Bonjour tout le monde.

j'utilise Visual c++ 2008.
Je m'attaque sérieusement au pointeur et j'aurais aimé savoir si ce petit programme (qui fonctionne) ne crée pas de problème (fuite mémoire et autre):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <stdio.h>
#include <stdlib.h>
#include <string.h>[/B]
 
int main(int argc, char *argv[])
{	
	void* vp;
	char* cp;
	vp=(void*)malloc(sizeof(char*));
	vp="a333";
	printf("%s\n",vp);
	vp="a22";
	cp=(char*)vp;
	vp=NULL;
	free(vp);
	printf("%s\n",cp);
	cp="a1";
	printf("%s\n",cp);
	system("PAUSE");	
	return 0;
}

[CGi]

Oui, il y a une fuite mémoire !

Tu alloues de la mémoire et tu mets l'adresse de cette zone mémoire dans le pointeur vp. Là c'est bon !
Ensuite tu fait pointer ce pointeur sur une chaîne de caractère. Donc à ce moment là, tu perds l'adresse de la zone mémoire que tu y avais mis auparavant !

[Lintel-oo]

A première vue je dirais que si , même si je suis loin d'être un grand spécialiste du domaine

Code :

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

void* vp;

Pourquoi ne pas mettre char* ? Même si c'est correct je pense qu'il vaut mieux éviter des variables de type void* sauf dans des cas particuliers. Après je conçois tout à fait que tu souhaites t'entraîner

Code :

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

1
2
vp=(void*)malloc(sizeof(char*));
vp="a333";

Tu déclares allouer la taille d'un pointeur de char, c'est à dire sur beaucoup de machines 4 octets. Cependant cela ne sert à rien puisque tu déclares ensuite

Code :

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

vp="a333";

Donc il y a fuite de mémoire puisque tu ne libères pas ce que tu as alloué avant d'écrire vp="a333";

Il faut donc ici écrire :

Code :

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

vp=(void*)malloc(X*sizeof(char));

Avec X le nombre de caractères que tu souhaites mettre dans ta chaîne (ici 4 donc) et faire ensuite :

Code :

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

1
2
3
4
5
 
*vp = 'a';
*(vp+1) = '3';
*(vp+2) = '3';
*(vp+3) = '3';

Ou sinon tu déclares directement :

Code :

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

vp = "a333"

Ensuite :

Code :

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

1
2
3
 
vp=NULL;
free(vp);

Mettre à nul le pointeur ne sert à rien si tu libères ce pointeur après donc ici le

Code :

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

free(vp);

est totalement inutile. D'ailleurs si tu l'avais mis avant la mise à 0 du pointeur tu aurais eu une fuite de mémoire, car, lorsque tu déclares

Code :

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

cp=(char*)vp;

cela signifie que le pointeur cp va prendre la valeur du pointeur vp, donc si tu libères vp à cet instant, cp ne va pointer sur plus rien.
Donc "Heureusement" encore une fois que tu ais mis le pointeur vp à NULL avant de le libérer, sinon tu aurais eu une fuite

Enfin, tu as une dernière fuite de mémoire puisque tu n'appelle pas free() pour libérer le pointeur cp, anciennement vp, mais comme tu as mis vp à NULL tu dois libérer cp, sinon tu aurais eu le choix entre libérer l'un ou libérer l'autre mais pas les 2 ! Si tu as besoin d'autres explications n'hésite pas !

Edit : CGi dit plus ou moins la même chose en omettant toutefois la dernière fuite de mémoire (ou alors c'est moi qui me trompe )

Edit bis : je dirais que ce code est correct :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
void* vp;
char* cp;
vp=(void*)malloc(sizeof(char)*//ce que tu veux//); //ligne ici totalement inutile
free(vp); //inutile si ligne précédente absente
vp="a333";
printf("%s\n",vp);
vp="a22";
cp=(char*)vp;
printf("%s\n",cp);
free(cp); // ou free(vp) puis vp=NULL;
cp="a1";
printf("%s\n",cp);
//plus besoin de libérer cp ni de le mettre à NULL
system("PAUSE");	
return 0;
}

[CGi]

Voir commentaires dans code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#include <stdio.h>
#include <stdlib.h>
//#include <string.h> // Pas utile ici
int main(void)
{
    void* vp;
    char* cp;
    vp = (void*)malloc(sizeof(char*)); // Transtypage pas necessaire et taille pas bonne.
    vp = "a333"; // ICI tu perds l'adresse de ton bloc mémoire
    printf("%s\n",(char*)vp); //Là tu peux mettre un transtypage
                                // pour éviter le warning
    vp="a22";
    cp=(char*)vp;
    vp=NULL;  // On met le pointeur à null
    free(vp); // après avoir libéré la mémoire.
    printf("%s\n",cp);
    cp="a1";
    printf("%s\n",cp);
    system("PAUSE"); // "Pause" spécifique à Windows
    return 0;
}