Gestion de la mémoire [Débutant(e)]

**yan** · 03/09/2007, 23h02

Envoyé par Aszarsha

Pas avec des systèmes qui datent de plus d'une dixaine d'années, ok ; et encore...
M'enfin tous les systèmes actuels le font sans aucun probleme, et je ne pense pas que jo_le_coco vise Win9x.

Une fuite mémoire ne se passe que lors de l'execution du programme, à sa terminaison, toute les resources sont automatiquement recupérés par le système (à moins de programmer pour de très vieux systemes, comme dit plus haut).

tu me rassure.

**Polyfructol** · 04/09/2007, 00h06

Avec SDL les fuites de mémoires sont souvent dues aux chargements d'images à répétition. Es-tu sûr que tu n'utilises pas SDL_LoadBMP en boucle ?

De cette façon :

Code :

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

1
2
3
4
5
6
SDL_Surface *surface;
surface = SDL_LoadBMP("test.bmp"); <-- 1ère allocation
...
surface = SDL_LoadBMP("test.bmp"); <-- 2ème allocation
...
SDL_FreeSurface(surface); <-- désallocation de la 2ème et pas de la 1ère

--> fuites de mémoire

Il faut faire :

Code :

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

1
2
3
4
5
6
7
SDL_Surface *surface;
surface = SDL_LoadBMP("test.bmp");
...
SDL_FreeSurface(surface);
surface = SDL_LoadBMP("test.bmp");
...
SDL_FreeSurface(surface);

Bon je peux me tromper ça fait quelque temps que je n'ai pas touché à la SDL, mais j'en suis quasiement sûr.

**Aszarsha** · 04/09/2007, 01h50

Erreur qu'une classe bien codée ne permet plus de faire.

screetch · 04/09/2007, 07h59

tu utilises quel compilateur/debuggeur ?

Invité · 04/09/2007, 09h33

Envoyé par screetch

tu utilises quel compilateur/debuggeur ?

J'utilise Code::Blocks et son débuggeur intégré. J'ai vu aussi récemment la news sur "DDD", je ne sais pas si ça pourrait m'aider...

En ce qui concerne l'allocation d'une surface SDL en boucle, je pense plutôt que ça doit être l'appel en boucle d'une méthode qui alloue, vu que je ne suis pas encore tout à fait à l'aise avec le mécanisme des classes. En tout cas, je ne trouve pas l'erreur.

Je peux vous donner le code source si vous voulez, il n'est pas très long. Bon quand même je vous préviens : il doit y avoir plein de maladresses, ou d'erreurs de programmation, dues notamment aux dernières modifications que j'ai faites pour chercher à résoudre le problème. Et également, je ne vérifie pas que mes allocations fonctionnent bien, je comptais le faire à la fin. Enfin bref voilà la source (avec images, projet Code::Blocks et exécutable) : http://kokospace.free.fr/autres/jodowsmediaplayer.zip

Si quelqu'un arrivait à trouver l'erreur (ou les erreurs ?), ce serait vraiment chouette. A mon avis, il doit y avoir une histoire de classes. Mais de toute façon, je vais certainement recoder le tout avec des classes bien robustes

**yan** · 04/09/2007, 11h53

En regardant vite fait ton code, dans la fonction,
void DialogOpen::dessiner(SDL_Event *event), je n'est pas trouvé la destruction de rectangle.

Sinon, juste une remarque, dans la fonction void Dialog::dessiner(), est tu vraiment obligé de recharger tes images stockées dans le tableau fenetre?

Invité · 04/09/2007, 12h44

Envoyé par Mongaulois

En regardant vite fait ton code, dans la fonction,
void DialogOpen::dessiner(SDL_Event *event), je n'est pas trouvé la destruction de rectangle.

Ah mais bien sûr ! La voilà, l'erreur

J'ai pensé à détruire la surface dans le destructeur, et je n'ai pas pensé à le faire à chaque appel de la fonction... Maintenant que ça marche, c'est merveilleux...

Enfin bien sûr, il subsiste toujours quelques fuites de mémoires, bien moins importantes, mais bon... Je vais donc essayer ces fameux RAII.

Envoyé par Mongaulois

Sinon, juste une remarque, dans la fonction void Dialog::dessiner(), est tu vraiment obligé de recharger tes images stockées dans le tableau fenetre?

Euh en fait c'est une de ces modifications dont je parlais. Je me suis dit que le problème aurait pu venir de là, et j'ai évité de garder quoi que ce soit en mémoire

Enfin bon le problème est résolu, et je vous remercie tous pour votre précieuse aide

**yan** · 04/09/2007, 13h59

Envoyé par jo_le_coco

Ah mais bien sûr ! La voilà, l'erreur

J'ai pensé à détruire la surface dans le destructeur, et je n'ai pas pensé à le faire à chaque appel de la fonction... Maintenant que ça marche, c'est merveilleux...

Enfin bien sûr, il subsiste toujours quelques fuites de mémoires, bien moins importantes, mais bon... Je vais donc essayer ces fameux RAII.

Euh en fait c'est une de ces modifications dont je parlais. Je me suis dit que le problème aurait pu venir de là, et j'ai évité de garder quoi que ce soit en mémoire

Enfin bon le problème est résolu, et je vous remercie tous pour votre précieuse aide

C'est une type erreur assez récurrente, et pas facile à trouver. Surtout quand on mélange du C dans du C++.
Par contre il n'existe de bonne interface C++ de la SDL??

Invité · 04/09/2007, 19h46

Envoyé par Mongaulois

C'est une type erreur assez récurrente, et pas facile à trouver. Surtout quand on mélange du C dans du C++.
Par contre il n'existe de bonne interface C++ de la SDL??

Euh c'est une bonne question

Ce serait en effet bien utile, notamment à cause de tous les ".c_str()" que je suis obligé d'utiliser à tout bout de champ...

Mais apparemment il n'y en a pas (officiellement en tout cas) :

Envoyé par http://www.libsdl.org/

SDL is written in C, but works with C++ natively

Mais de toute façon, les RAII facilitent considérablement les choses. J'ai déjà implémenté une ébauche de classe RAII pour les surfaces SDL, et le code du main est trente fois plus simple :

Code :

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

1
2
3
4
5
RAII_Surface image("images/test.bmp");
image.setColorKey(32,59,84);
image.setAlpha(128);
image.fill(255,255,0);
image.blit(ecran, 30, 30);

**bogoss91** · 04/09/2007, 20h04

Il y a la SFML qui est semblable à la SDL mais en orienté objet.

Invité · 04/09/2007, 20h50

Oui, j'en ai entendu parler, mais (je me trompe peut-être) ça me semble moins complet que la SDL. Ce qu'il y a de bien, c'est qu'on peut ouvrir plusieurs fenêtres

**bogoss91** · 04/09/2007, 20h54

Moi justement je trouve ça plus complet que la SDL (mais bon, je connais pas vraiment toutes les fonctions de la SDL), et il faut pas oublier que le projet est jeune et qu'il va s'étoffer au fil du temps.

**Aszarsha** · 05/09/2007, 02h54

De plus, SFML est developpé principalement par un des membres du forum, Laurent "Loulou" Gomila pour ne pas le citer ; ce qui assure un service après-téléchargement du tonerre !

Et si tu veux voir du code très bien ecrit en C++, il te suffira d'aller regarder du côté des sources en plus !

Invité · 05/09/2007, 10h50

Envoyé par Aszarsha

De plus, SFML est developpé principalement par un des membres du forum, Laurent "Loulou" Gomila pour ne pas le citer ; ce qui assure un service après-téléchargement du tonerre !

Et si tu veux voir du code très bien ecrit en C++, il te suffira d'aller regarder du côté des sources en plus !

Ah je ne savais pas, tout ça

Bah ça mérite sûrement de s'y intéresser alors. Et puis j'ai vu que la gestion des vidéos est "A venir"

En voilà une bonne nouvelle

Et question puissance, c'est vraiment rapide ?