Fuite de memoire.

**razonback** · 01/04/2008, 20h12

Bonjour a tous.

Voila, je suis en ce moment entrain de développer un Dr Mario en C. Pour ceux ou celles qui ne connaissent pas, il s'agit en tout et pour tout d'un Tetris modifié.

Voila mon probleme est que pour afficher les éléments qui tombent je dois charger et recharger des images a chaque instant. Le probleme c'est que je pense qu'il y a des fuites de mémoires durant ces chargements. (Je vous montre la fonction d'affichage après). Ce probleme ne gêne pas tant que je ne fait pas tomber les gellules toutes seules puisque le programme attend que j'appuie sur une touche (SDL_WatiEvent). Mais lorsque je dois mettre des SDL_PollEvent, le programme continue et bug au bout d'un certain temps... Donc pouvez-vous me dire si j'ai des fuites de memoires dans le code qui suit et comment les resoudre. Sinon dites moi pourquoi ca rame au bout d'un certain moment...

Merci d'avance pour toutes vos reponses et desolé pour le roman.

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
 
 
void Interprete_bouteille(Joueur joueur)//autre choix blitter la surface a cahque if!
{
    int i;
    int j;
    SDL_FillRect(menu, NULL, blanc);
    SDL_BlitSurface(menu, NULL, ecran, &positionMenu);
    //Liberation_memoire_surface(menu);
    for(i = 0; i < H; i++)
    {
        for(j = 0; j < L; j++)
        {
           // fprintf(stderr,"%d\n",Liaison_gellule(direction, i, j));
 
            positionG_V.x = positionMenu.x + (j * 32);
            positionG_V.y = positionMenu.y + (i * 32);
            switch (joueur.bouteille[i][j])
            {
                case 1 : //on dessine un virus bleu. 
                    Gellules_Virus = SDL_LoadBMP("Virus_bleu.bmp");
                    break;
                case 2 : //un virus blanc.               
                    Gellules_Virus = SDL_LoadBMP("Virus_blanc.bmp");
                    break;
                case 3 : //un virus vert.
                    Gellules_Virus = SDL_LoadBMP("Virus_vert.bmp");
                    break;
                case 4 : //une gellule bleue
                    if( (joueur.bouteille[i+1][j] >= 4 )&& (Liaison_gellule(joueur.direction, i, j) == 8) )
                    { 
                        Gellules_Virus = SDL_LoadBMP("G_haut_bleue.bmp");
                    }   
                    if( (joueur.bouteille[i][j+1] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 4) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_gauche_bleue.bmp");
                    }
                    if( (joueur.bouteille[i][j-1] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 6) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_droite_bleue.bmp");
                    }
                    if( (joueur.bouteille[i-1][j] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 2) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_bas_bleue.bmp"); 
                    }
                    if(Liaison_gellule(joueur.direction, i, j) == 0)
                    {
                        Gellules_Virus = SDL_LoadBMP("G_seule_bleue.bmp");
                    }
                    break;
                case 5 : //une gellule blanche.
                    if( (joueur.bouteille[i+1][j] >= 4 ) && (Liaison_gellule(joueur.direction, i, j) == 8) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_haut_blanc.bmp");
                    }
                    if( (joueur.bouteille[i][j+1] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 4) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_gauche_blanc.bmp");
                    }
                    if( (joueur.bouteille[i][j-1] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 6) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_droite_blanc.bmp");
                    }
                    if( (joueur.bouteille[i-1][j] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 2) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_bas_blanc.bmp");
                    }
                    if(Liaison_gellule(joueur.direction, i, j) == 0)
                    {
                        Gellules_Virus = SDL_LoadBMP("G_seule_blanche.bmp");
                    }
                    break;
                case 6 : //une gellule verte.
                    if( (joueur.bouteille[i+1][j] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 8) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_haut_vert.bmp");
                    }
                    if( (joueur.bouteille[i][j+1] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 4) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_gauche_vert.bmp");
                    }
                    if( (joueur.bouteille[i][j-1] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 6) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_droite_vert.bmp");
                    }
                    if( (joueur.bouteille[i-1][j] >= 4) && (Liaison_gellule(joueur.direction, i, j) == 2) )
                    {
                        Gellules_Virus = SDL_LoadBMP("G_bas_vert.bmp");
                    }
                    if(Liaison_gellule(joueur.direction, i, j) == 0)
                    {
                        Gellules_Virus = SDL_LoadBMP("G_seule_verte.bmp");
                    }
                    break;
                default : 
                    break;      
            }
            SDL_SetColorKey(Gellules_Virus, SDL_SRCCOLORKEY, SDL_MapRGB(Gellules_Virus->format, 255, 0, 0));
            SDL_BlitSurface(Gellules_Virus , NULL , ecran , &positionG_V);
            SDL_FillRect(Gellules_Virus , NULL, blanc); 
        }
    }
    SDL_Flip(ecran);
}
 
// Et voici la fonction qui gère les évènements.
 
int Touches_utiles()
{
    int tps;
    tps = 0;
    SDL_Event event;
    SDL_WaitEvent(&event);
    if(event.type == SDL_QUIT)
    {
        exit(0);
    }
    if (event.type == SDL_KEYDOWN)
	{
	   switch (event.key.keysym.sym)
		{
            case SDLK_ESCAPE:
                exit(0);
                break;
			case SDLK_DOWN : 
                tps = 1;  
                break;
            case SDLK_RIGHT :
                tps = 2;
                break;
            case SDLK_LEFT :
                tps = 3;
                break;
            case SDLK_v :
                tps = 4;
                break; 
            case SDLK_n :
                tps = 5;
                break;  
            case SDLK_SPACE :
                tps = 6;
                break; 
            case SDLK_r :
                tps = 7;
                break; 
            case SDLK_s :
                tps = 8;
                break; 
            case SDLK_l :
                tps = 9;
                break;  
            default :
                break;
		  }
    }
    return tps;	
}

**fearyourself** · 01/04/2008, 20h21

C'est logique, tu charges tout le temps des images. Charge une seule fois chaque ressource et utilise ensuite le même élément chargé.

Jc

**razonback** · 01/04/2008, 20h38

Peut tu me donner un exemple s'il te plait car je ne comprend pas tout... tu veux dire que je crée N Surfaces et que je charge la bonne surface quand il faut ???
Bien sure lors de la création je fais FreeSurface juste après?? c'est ça?

**fearyourself** · 01/04/2008, 20h49

Envoyé par razonback

Peut tu me donner un exemple s'il te plait car je ne comprend pas tout... tu veux dire que je crée N Surfaces et que je charge la bonne surface quand il faut ???
Bien sure lors de la création je fais FreeSurface juste après?? c'est ça?

Il faut que ton code ressemble à ceci :

- Initialisation de la SDL

- Chargement des textures (donc TOUS tes SDL_LoadBMP) dans un tableau de surfaces que je vais nommés images_chargees

- Boucle de jeu (sans SDL_LoadBMP).

Normalement, on se débrouille pour que dans la table de jeu (ton tableau bouteille), la valeur à l'intérieur de chaque soit :

- 0 donc rien
- i pour dire que c'est la case i du tableau images_chargees.

Jc

**razonback** · 01/04/2008, 21h48

Ok je crois avoir tout compris, j'essaie de le faire et je reviens si j'ai des problèmes merci a toi!

Voila j'ai un probleme...

Est ce que ce code est juste ???
Sinon peux tu me donner un exemple de code car je vois pas trop la difference de faire comme cela...

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
void Images_chargees()
{
   SDL_Surface *tab[19];
   int i;
 
   for (i = 0; i < 19 ; i++)
   {
      tab[i] = NULL;
   }
 
   tab[1] = SDL_LoadBMP("Virus_bleu.bmp");
}
 
// je n'ai mis qu'un cas pour simplifier la chose.
 
void Interprete_bouteille(Joueur joueur)//autre choix blitter la surface a cahque if!
{
    int i;
    int j;
    SDL_FillRect(menu, NULL, blanc);
    SDL_BlitSurface(menu, NULL, ecran, &positionMenu);
    SDL_Surface *tab[19];
    tab[1] = SDL_LoadBMP("Virus_bleu.bmp");
    //Liberation_memoire_surface(menu);
    for(i = 0; i < H; i++)
    {
        for(j = 0; j < L; j++)
        {
           // fprintf(stderr,"%d\n",Liaison_gellule(direction, i, j));
 
            positionG_V.x = positionMenu.x + (j * 32);
            positionG_V.y = positionMenu.y + (i * 32);
            switch (joueur.bouteille[i][j])
            {
                case 1 : //on dessine un virus bleu. 
                    tab[1];
                    break;
             default : 
                    break;      
            }
            SDL_SetColorKey(Gellules_Virus, SDL_SRCCOLORKEY, SDL_MapRGB(Gellules_Virus->format, 255, 0, 0));
            SDL_BlitSurface(Gellules_Virus , NULL , ecran , &positionG_V);
            SDL_FillRect(Gellules_Virus , NULL, blanc); 
        }
    }
    SDL_Flip(ecran);
}

**razonback** · 03/04/2008, 16h35

re bonjour.
Bon j'ai crée mon tableau de surface mais je ne sais pas le code pour les "blitter"
Enfin je sais qu' il faut faire SDL_BlitSurface( X , NULL, ecran , &position)
mais qu'est ce que je dois mettre à la place du X ???
Si vous pouviez m'aider...

Fuite de memoire.

SDL

Vue hybride

Discussions similaires

Partager

Partager