La mémoire sature apparement..

**Nicovideo** · 31/03/2011, 20h02

Voilà je vous explique brièvement mon problème, je programme sur PSP où je suis en train de créer un jeu en C.

Actuellement j'ai un problème, quand je sors d'une fonction niv1() (qui se trouve dans niv1.c) je vais dans une fonction menu() (qui se trouve dans menu.c). Jusqu'à là aucun problème, mais quand je répète cette action 3 fois dans mon jeu, la PSP plante et s'éteint.

Je pense que c'est un problème de gestion de la RAM.

Je vous donne mon code du fichier niv1.c pour que vous puissiez me dire si il est vraiment mal codé et qu'est ce que je peux faire.

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
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
#include <stdio.h>
#include <stdlib.h>
#include <libge/libge.h>
 
 
typedef struct Boule {
        int x;
        int y;
				int h;
				int w;
        ge_Image* img;
} Boule;
 
typedef struct En {
        int x;
        int y;
				int h;
				int w;
        ge_Image* img;
} En;
 
typedef struct En2 {
        int x;
        int y;
				int h;
				int w;
        ge_Image* img;
} En2;
 
 
void niv1()
{
 
 
ge_Image* back = geLoadImage("niv1.png");
ge_Image* hero = geLoadImage("carre.png");
ge_Image* hero2 = geLoadImage("carre2.png");
ge_Image* ene = geLoadImage("carrerouge.png");
ge_Image* masque = geLoadImage("masque_niv1.png");
ge_Image* pause = geLoadImage("pause.png");
ge_Image* curs1 = geLoadImage("fleche1.png");
ge_Image* curs2 = geLoadImage("fleche2.png");
ge_Image* coeur = geLoadImage("coeur.png");
ge_Image* lost = geLoadImage("lose.png");
ge_Image* win = geLoadImage("win.png");
ge_Image* over = geLoadImage("gameover.png");
ge_Keys *pad = geCreateKeys(), *oldpad = geCreateKeys();
SceCtrlData ana;
 
FILE* fichier = NULL;
fichier = fopen("highscore.txt", "w");
 
 
ge_Font *font = geLoadFont("COOPBL.TTF");
ge_Font *high = geLoadFont("COOPBL.TTF");
geFontSize(font, 13);
geFontSize(high, 14);
 
Boule boule;
boule.x = 43;
boule.y = 57;
boule.h = 16;
boule.w = 16;
boule.img = hero;
 
En en;
en.x = 237;
en.y = 40;
en.h = 16;
en.w = 16;
en.img = ene;
 
En2 en2;
en2.x = 30;
en2.y = 240;
en2.h = 16;
en2.w = 16;
en2.img = ene;
 
extern unsigned char lance_menu;
unsigned char vie = 3;
int clignote=0;
int oldex;
int olde2x;
int oldey;
int oldx;
int oldy;
unsigned char vitesse = 2;
char VitesseIA = 2;
char VitesseIA2 = 1;
unsigned char selecteur = 1, maxselecteur = 2;
unsigned char state = 0;
 
// Collision map
 
void test()
{
unsigned int couleur = masque->data[(boule.x+10) + (boule.y+10) * masque->textureWidth];
if (couleur == RGB(0,255,0)){
boule.x = oldx;
boule.y = oldy;
}
}
 
// Deplacement ennemi 1
 
void dep_ennemi1()
{
en.x = en.x + VitesseIA;
if (en.x > 437) {
VitesseIA = VitesseIA * (-1);
}
else if(en.x < 237){
VitesseIA = VitesseIA * (-1) ;
}
}
 
// Deplacement ennemi 2
 
void dep_ennemi2()
{
en2.x = en2.x + VitesseIA2;
if (en2.x > 156) {
VitesseIA2 = VitesseIA2 * (-1);
}
else if(en2.x < 30){
VitesseIA2 = VitesseIA2 * (-1) ;
}
}
 
void deplacement()
{
if(pad->pressed[GEK_DOWN])
		{
       boule.y = boule.y + vitesse;
		}
if(pad->pressed[GEK_UP])
		{
       boule.y = boule.y - vitesse;
		}
if(pad->pressed[GEK_LEFT])
		{
       boule.x = boule.x - vitesse;
		}
if(pad->pressed[GEK_RIGHT])
		{
       boule.x = boule.x + vitesse;
		}
 
}
 
// Fontion deplacement analog
 
void analog(){
sceCtrlPeekBufferPositive(&ana, 1); 
if (ana.Ly < 128 - 56)
{ //Pad vers le haut
boule.y = boule.y - vitesse;
}
 
if (ana.Ly > 128 + 56)
{ //Pad vers le bas
boule.y = boule.y + vitesse;
}
 
if (ana.Lx < 128 - 56)
{
boule.x = boule.x - vitesse;
}
 
if (ana.Lx > 128 + 56)
{
boule.x = boule.x + vitesse;
}
 
}
 
 
 
void anim_col()
{
clignote++;
if (clignote < 10){
	boule.img = hero2;
	}
else if ((clignote > 10) && (clignote < 20)){
	boule.img = hero;
	}
if (clignote == 20){
	clignote = 0;
	}
}
 
 
// Boucle principale 
 
while (1)
{
 
		geClearScreen();
		geReadKeys(pad);
 
		geDrawImage(0, 0, back);
		geDrawImage(en2.x,en2.y,en2.img);
		geDrawImage(en.x,en.y,en.img);
		geDrawImage(boule.x,boule.y,boule.img);
		geDrawImage(400, 10, coeur);
 
		char co[10] = "";
		sprintf(co, "x%d",vie);
		geFontPrintScreen(420, 10, font, co, RGB(255, 255, 255));
 
 
		if(state == 0){
 
				deplacement();
				analog();
 
 
				test();
				dep_ennemi1();
				dep_ennemi2();
 
 
				if (boule.x >= 379 && boule.x <= 403 &&
				boule.y >= 222 && boule.y <= 246)
				{
				state = 3;
				}
 
 
				oldx = boule.x;
				oldy = boule.y;
				oldex = en.x;
				oldey = en.y;
				olde2x = en2.x;
 
 
 
				if(pad->pressed[GEK_START] && oldpad->released[GEK_START]){
 
				state = 1;}
				if ((boule.x + boule.w > en2.x) && (boule.x < en2.x + en2.w) &&
				(boule.y + boule.h > en2.y) && (boule.y < en2.y + en2.h) ){
 
				vie = vie - 1;
				state= 2;}
				if ((boule.x + boule.w > en.x) && (boule.x < en.x + en.w) &&
				(boule.y + boule.h > en.y) && (boule.y < en.y + en.h) ){
 
				vie = vie - 1;
				state= 2;}
 
				if (vie == 0){
				state = 4;}
		}
 
 
else if(state == 1){
  geDrawImage(0, 0, pause);
 
	  if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
		{
       selecteur++;
		}
    if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
		{
       selecteur--;
		}
		if (selecteur > maxselecteur)
		{
				selecteur = 1;
		}
		else if (selecteur < 1)
		{
		selecteur = maxselecteur;
		}
 
		if (selecteur == 1) 
		{
			geDrawImage(180, 115, curs1);
			geDrawImage(282, 115, curs2);
		}
		if (selecteur == 2)
		{
			geDrawImage(195, 140, curs1);
			geDrawImage(270, 140, curs2);
		}
 
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
		{
 
			state = 0;
		}
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
		{
				geFreeImage(back);
				geFreeImage(hero);
				geFreeImage(hero2);
				geFreeImage(ene);
				geFreeImage(masque);
				geFreeImage(pause);
				geFreeImage(curs1);
				geFreeImage(curs2);
				geFreeImage(coeur);
				geFreeImage(lost);
				geFreeImage(win);
				geFreeImage(over);	
				lance_menu=1;
				return;
		}
    if(pad->pressed[GEK_START] && oldpad->released[GEK_START]){
 
      state = 0;}
		}
 
else if(state == 2){
			geDrawImage(0, 0, lost);
			anim_col();
		if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
			{
       selecteur++;
			}
    if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
			{
       selecteur--;
			}
		if (selecteur > maxselecteur)
			{
				selecteur = 1;
			}
		else if (selecteur < 1)
			{
				selecteur = maxselecteur;
			}
 
		if (selecteur == 1) 
			{
				geDrawImage(182, 115, curs1);
				geDrawImage(282, 115, curs2);
			}
		if (selecteur == 2)
			{
				geDrawImage(195, 140, curs1);
				geDrawImage(270, 140, curs2);
			}
 
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
			{
				state = 0;
				boule.img = hero;
				boule.x = 43;
				boule.y = 57;
			}
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
			{
 
				geFreeImage(back);
				geFreeImage(hero);
				geFreeImage(hero2);
				geFreeImage(ene);
				geFreeImage(masque);
				geFreeImage(pause);
				geFreeImage(curs1);
				geFreeImage(curs2);
				geFreeImage(coeur);
				geFreeImage(lost);
				geFreeImage(win);
				geFreeImage(over);	
				lance_menu=1;
				return;
			}
 
			}
 
else if(state == 3){
			geDrawImage(0, 0, win);
 
		if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
			{
       selecteur++;
			}
    if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
			{
       selecteur--;
			}
		if (selecteur > maxselecteur)
			{
				selecteur = 1;
			}
		else if (selecteur < 1)
			{
				selecteur = maxselecteur;
			}
 
		if (selecteur == 1) 
			{
				geDrawImage(168, 112, curs1);
				geDrawImage(300, 112, curs2);
			}
		if (selecteur == 2)
			{
				geDrawImage(195, 140, curs1);
				geDrawImage(275, 140, curs2);
			}
 
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
			{
				state = 0;
				geFreeImage(back);
				geFreeImage(hero);
				geFreeImage(hero2);
				geFreeImage(ene);
				geFreeImage(masque);
				geFreeImage(pause);
				geFreeImage(curs1);
				geFreeImage(curs2);
				geFreeImage(coeur);
				geFreeImage(lost);
				geFreeImage(win);
				geFreeImage(over);	
 
			}
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
			{
				geFreeImage(back);
				geFreeImage(hero);
				geFreeImage(hero2);
				geFreeImage(ene);
				geFreeImage(masque);
				geFreeImage(pause);
				geFreeImage(curs1);
				geFreeImage(curs2);
				geFreeImage(coeur);
				geFreeImage(lost);
				geFreeImage(win);
				geFreeImage(over);	
				lance_menu=1;
				return;
			}
 
			}
 
else if(state == 4){
			geDrawImage(0, 0, over);
		if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
			{
       selecteur++;
			}
    if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
			{
       selecteur--;
			}
		if (selecteur > maxselecteur)
			{
				selecteur = 1;
			}
		else if (selecteur < 1)
			{
				selecteur = maxselecteur;
			}
 
		if (selecteur == 1) 
			{
				geDrawImage(155, 114, curs1);
				geDrawImage(313, 114, curs2);
			}
		if (selecteur == 2)
			{
				geDrawImage(195, 140, curs1);
				geDrawImage(272, 140, curs2);
			}
 
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
			{
				state = 0;
				geFreeImage(back);
				geFreeImage(hero);
				geFreeImage(hero2);
				geFreeImage(ene);
				geFreeImage(masque);
				geFreeImage(pause);
				geFreeImage(curs1);
				geFreeImage(curs2);
				geFreeImage(coeur);
				geFreeImage(lost);
				geFreeImage(win);
				geFreeImage(over);			
 
			}
		if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
			{
				geFreeImage(back);
				geFreeImage(hero);
				geFreeImage(hero2);
				geFreeImage(ene);
				geFreeImage(masque);
				geFreeImage(pause);
				geFreeImage(curs1);
				geFreeImage(curs2);
				geFreeImage(coeur);
				geFreeImage(lost);
				geFreeImage(win);
				geFreeImage(over);	
				lance_menu=1;
				return;
			}
 
			}
 
			geReadKeys(oldpad);
			geSwapBuffers();	
 
 
}
sceKernelExitGame();
 
}

Merci d'avance

**ram-0000** · 31/03/2011, 21h02

Déjà, ton code est assez horrible à lire car l'indentation n'est pas cohérente.

Je n'ai pas vu l'accolade fermante de la fonction niv1().

quand tu rentre dans niv1(), il y a des allocations (je suppose) de tes images. Que se passe t'il si tu rerentre dans niv1() sans avoir libéré ses images. Tes blocs de mémoires sont probablement perdus.

Autre chose, il te manque au moins 2 fonctions, une fonction alloc_image() et une fonction free_image(). car quand on voit les copier/coller des blocs geFreeImage(), je me dis que le jour où tu rajoutes une image, il y a bien un bloc que tu vas oublier et tu vas introduire une fuite de mémoire.

Au moins en faisant ces 2 fonctions, les modifications ne porteront que sur ces 2 fonctions.

**Nicovideo** · 31/03/2011, 21h13

Merci pour cette réponse

quand tu rentre dans niv1(), il y a des allocations (je suppose) de tes images. Que se passe t'il si tu rerentre dans niv1() sans avoir libéré ses images. Tes blocs de mémoires sont probablement perdus.

Je ne comprends pas exactement ce que vous dîtes a ce sujet.

Pour ce qui est des deux fonctions manquantes, j'ai juste à faire un void free(){} avec les gefreeimage à l'intérieur et l'appeler quand je sors à chaque fois ?

Et pour la fonction alloc je sais pas ce que vous voulais faire lol. En tout cas merci pour ces réponses j'espere que vous allez continuer à m'eclairer pour corriger mon soucis

**ram-0000** · 31/03/2011, 21h41

Déjà, pour commencer, je te propose de repartir sur ce code que je viens de réindenter afin qu'il soit plus facile à lire

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
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
#include <stdio.h>
#include <stdlib.h>
#include <libge/libge.h>
 
typedef struct Boule
{
   int x;
   int y;
   int h;
   int w;
   ge_Image* img;
} Boule;
 
typedef struct En
{
   int x;
   int y;
   int h;
   int w;
   ge_Image* img;
} En;
 
typedef struct En2
{
   int x;
   int y;
   int h;
   int w;
   ge_Image* img;
} En2;
 
void niv1()
{
   ge_Image* back = geLoadImage("niv1.png");
   ge_Image* hero = geLoadImage("carre.png");
   ge_Image* hero2 = geLoadImage("carre2.png");
   ge_Image* ene = geLoadImage("carrerouge.png");
   ge_Image* masque = geLoadImage("masque_niv1.png");
   ge_Image* pause = geLoadImage("pause.png");
   ge_Image* curs1 = geLoadImage("fleche1.png");
   ge_Image* curs2 = geLoadImage("fleche2.png");
   ge_Image* coeur = geLoadImage("coeur.png");
   ge_Image* lost = geLoadImage("lose.png");
   ge_Image* win = geLoadImage("win.png");
   ge_Image* over = geLoadImage("gameover.png");
   ge_Keys *pad = geCreateKeys(), *oldpad = geCreateKeys();
   SceCtrlData ana;
 
   FILE* fichier = NULL;
   fichier = fopen("highscore.txt", "w");
 
   ge_Font *font = geLoadFont("COOPBL.TTF");
   ge_Font *high = geLoadFont("COOPBL.TTF");
   geFontSize(font, 13);
   geFontSize(high, 14);
 
   Boule boule;
   boule.x = 43;
   boule.y = 57;
   boule.h = 16;
   boule.w = 16;
   boule.img = hero;
 
   En en;
   en.x = 237;
   en.y = 40;
   en.h = 16;
   en.w = 16;
   en.img = ene;
 
   En2 en2;
   en2.x = 30;
   en2.y = 240;
   en2.h = 16;
   en2.w = 16;
   en2.img = ene;
 
   extern unsigned char lance_menu;
   unsigned char vie = 3;
   int clignote=0;
   int oldex;
   int olde2x;
   int oldey;
   int oldx;
   int oldy;
   unsigned char vitesse = 2;
   char VitesseIA = 2;
   char VitesseIA2 = 1;
   unsigned char selecteur = 1, maxselecteur = 2;
   unsigned char state = 0;
 
// Collision map
void test()
{
   unsigned int couleur = masque->data[(boule.x+10) + (boule.y+10) * masque->textureWidth];
   if (couleur == RGB(0,255,0))
   {
      boule.x = oldx;
      boule.y = oldy;
   }
}
 
// Deplacement ennemi 1
void dep_ennemi1()
{
   en.x = en.x + VitesseIA;
   if (en.x > 437)
   {
      VitesseIA = VitesseIA * (-1);
   }
   else if(en.x < 237)
   {
      VitesseIA = VitesseIA * (-1) ;
   }
}
 
// Deplacement ennemi 2
void dep_ennemi2()
{
   en2.x = en2.x + VitesseIA2;
   if (en2.x > 156)
   {
      VitesseIA2 = VitesseIA2 * (-1);
   }
   else if(en2.x < 30)
   {
      VitesseIA2 = VitesseIA2 * (-1) ;
   }
}
 
void deplacement()
{
   if(pad->pressed[GEK_DOWN])
   {
      boule.y = boule.y + vitesse;
   }
   if(pad->pressed[GEK_UP])
   {
      boule.y = boule.y - vitesse;
   }
   if(pad->pressed[GEK_LEFT])
   {
      boule.x = boule.x - vitesse;
   }
   if(pad->pressed[GEK_RIGHT])
   {
      boule.x = boule.x + vitesse;
   }
}
 
// Fontion deplacement analog
void analog()
{
   sceCtrlPeekBufferPositive(&ana, 1); 
   if (ana.Ly < 128 - 56)
   {
      //Pad vers le haut
      boule.y = boule.y - vitesse;
   }
 
   if (ana.Ly > 128 + 56)
   {
      //Pad vers le bas
      boule.y = boule.y + vitesse;
   }
 
   if (ana.Lx < 128 - 56)
   {
      boule.x = boule.x - vitesse;
   }
 
   if (ana.Lx > 128 + 56)
   {
      boule.x = boule.x + vitesse;
   }
}
 
void anim_col()
{
   clignote++;
   if (clignote < 10)
   {
      boule.img = hero2;
   }
   else if ((clignote > 10) && (clignote < 20))
   {
      boule.img = hero;
   }
   if (clignote == 20)
   {
      clignote = 0;
   }
}
 
// Boucle principale
while (1)
{
   geClearScreen();
   geReadKeys(pad);
 
   geDrawImage(0, 0, back);
   geDrawImage(en2.x,en2.y,en2.img);
   geDrawImage(en.x,en.y,en.img);
   geDrawImage(boule.x,boule.y,boule.img);
   geDrawImage(400, 10, coeur);
 
   char co[10] = "";
   sprintf(co, "x%d",vie);
   geFontPrintScreen(420, 10, font, co, RGB(255, 255, 255));
 
   if(state == 0)
   {
      deplacement();
      analog();
 
      test();
      dep_ennemi1();
      dep_ennemi2();
 
 
      if (boule.x >= 379 && boule.x <= 403 && boule.y >= 222 && boule.y <= 246)
      {
         state = 3;
      }
 
      oldx = boule.x;
      oldy = boule.y;
      oldex = en.x;
      oldey = en.y;
      olde2x = en2.x;
 
      if(pad->pressed[GEK_START] && oldpad->released[GEK_START])
      {
         state = 1;
      }
      if ((boule.x + boule.w > en2.x) && (boule.x < en2.x + en2.w) &&   (boule.y + boule.h > en2.y) && (boule.y < en2.y + en2.h) )
      {
         vie = vie - 1;
         state= 2;
      }
      if ((boule.x + boule.w > en.x) && (boule.x < en.x + en.w) && (boule.y + boule.h > en.y) && (boule.y < en.y + en.h) )
      {
         vie = vie - 1;
         state= 2;
      }
 
      if (vie == 0)
      {
         state = 4;}
      }
      else if(state == 1)
      {
         geDrawImage(0, 0, pause);
 
         if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
         {
            selecteur++;
         }
         if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
         {
            selecteur--;
         }
         if (selecteur > maxselecteur)
         {
            selecteur = 1;
         }
         else if (selecteur < 1)
         {
            selecteur = maxselecteur;
         }
 
         if (selecteur == 1) 
         {
            geDrawImage(180, 115, curs1);
            geDrawImage(282, 115, curs2);
         }
         if (selecteur == 2)
         {
            geDrawImage(195, 140, curs1);
            geDrawImage(270, 140, curs2);
         }
 
         if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
         {
            state = 0;
         }
         if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
         {
            geFreeImage(back);
            geFreeImage(hero);
            geFreeImage(hero2);
            geFreeImage(ene);
            geFreeImage(masque);
            geFreeImage(pause);
            geFreeImage(curs1);
            geFreeImage(curs2);
            geFreeImage(coeur);
            geFreeImage(lost);
            geFreeImage(win);
            geFreeImage(over);   
            lance_menu=1;
            return;
         }
         if(pad->pressed[GEK_START] && oldpad->released[GEK_START])
         {
            state = 0;}
         }
         else if(state == 2)
         {
            geDrawImage(0, 0, lost);
            anim_col();
            if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
            {
               selecteur++;
            }
            if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
            {
               selecteur--;
            }
            if (selecteur > maxselecteur)
            {
               selecteur = 1;
            }
            else if (selecteur < 1)
            {
               selecteur = maxselecteur;
            }
 
            if (selecteur == 1) 
            {
               geDrawImage(182, 115, curs1);
               geDrawImage(282, 115, curs2);
            }
            if (selecteur == 2)
            {
               geDrawImage(195, 140, curs1);
               geDrawImage(270, 140, curs2);
            }
 
            if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
            {
               state = 0;
               boule.img = hero;
               boule.x = 43;
               boule.y = 57;
            }
            if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
            {
               geFreeImage(back);
               geFreeImage(hero);
               geFreeImage(hero2);
               geFreeImage(ene);
               geFreeImage(masque);
               geFreeImage(pause);
               geFreeImage(curs1);
               geFreeImage(curs2);
               geFreeImage(coeur);
               geFreeImage(lost);
               geFreeImage(win);
               geFreeImage(over);   
               lance_menu=1;
               return;
            }
         }
         else if(state == 3)
         {
         geDrawImage(0, 0, win);
 
         if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
         {
            selecteur++;
         }
         if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
         {
            selecteur--;
         }
         if (selecteur > maxselecteur)
         {
            selecteur = 1;
         }
         else if (selecteur < 1)
         {
            selecteur = maxselecteur;
         }
 
         if (selecteur == 1) 
         {
            geDrawImage(168, 112, curs1);
            geDrawImage(300, 112, curs2);
         }
         if (selecteur == 2)
         {
            geDrawImage(195, 140, curs1);
            geDrawImage(275, 140, curs2);
         }
 
         if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
         {
            state = 0;
            geFreeImage(back);
            geFreeImage(hero);
            geFreeImage(hero2);
            geFreeImage(ene);
            geFreeImage(masque);
            geFreeImage(pause);
            geFreeImage(curs1);
            geFreeImage(curs2);
            geFreeImage(coeur);
            geFreeImage(lost);
            geFreeImage(win);
            geFreeImage(over);   
 
         }
         if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
         {
            geFreeImage(back);
            geFreeImage(hero);
            geFreeImage(hero2);
            geFreeImage(ene);
            geFreeImage(masque);
            geFreeImage(pause);
            geFreeImage(curs1);
            geFreeImage(curs2);
            geFreeImage(coeur);
            geFreeImage(lost);
            geFreeImage(win);
            geFreeImage(over);   
            lance_menu=1;
            return;
         }
      }
      else if(state == 4)
      {
         geDrawImage(0, 0, over);
         if(pad->pressed[GEK_DOWN] && !oldpad->pressed[GEK_DOWN])
         {
            selecteur++;
         }
         if(pad->pressed[GEK_UP] && !oldpad->pressed[GEK_UP])
         {
            selecteur--;
         }
         if (selecteur > maxselecteur)
         {
            selecteur = 1;
         }
         else if (selecteur < 1)
         {
            selecteur = maxselecteur;
         }
 
         if (selecteur == 1) 
         {
            geDrawImage(155, 114, curs1);
            geDrawImage(313, 114, curs2);
         }
         if (selecteur == 2)
         {
            geDrawImage(195, 140, curs1);
            geDrawImage(272, 140, curs2);
         }
 
         if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 1)
         {
            state = 0;
            geFreeImage(back);
            geFreeImage(hero);
            geFreeImage(hero2);
            geFreeImage(ene);
            geFreeImage(masque);
            geFreeImage(pause);
            geFreeImage(curs1);
            geFreeImage(curs2);
            geFreeImage(coeur);
            geFreeImage(lost);
            geFreeImage(win);
            geFreeImage(over);         
         }
         if(pad->pressed[GEK_CROSS] && !oldpad->released[GEK_CROSS] && selecteur == 2)
         {
            geFreeImage(back);
            geFreeImage(hero);
            geFreeImage(hero2);
            geFreeImage(ene);
            geFreeImage(masque);
            geFreeImage(pause);
            geFreeImage(curs1);
            geFreeImage(curs2);
            geFreeImage(coeur);
            geFreeImage(lost);
            geFreeImage(win);
            geFreeImage(over);   
            lance_menu=1;
            return;
         }
      }
      geReadKeys(oldpad);
      geSwapBuffers();   
   }
   sceKernelExitGame();
}

Je suis maintenant sûr que ce code ne peut se compiler.

Je vois le début de fonction niv1() mais il n'y a pas de fin.

D'autres remarques :

tu ouvres un fichier mais tu ne fais rien du descripteur, ni lecture, ni écriture ni ce qui est plus grave fermeture
Ton code est bourré de valeurs "magiques qui font qu'il sera impossible de le modifier dans quelques semaines (par exemple : en.x = 237; if (ana.Lx > 128 + 56); if (boule.x >= 379 && boule.x <= 403 && boule.y >= 222 && boule.y <= 246))
ta structure while(1) fait 300 lignes (ligne 196 à 501), elle est 10 à 20 fois trop longue. Une structure de code doit pouvoir être englobée d'un seul regard sinon, on s'y perd.
Ton code devrait être découpé en plusieurs fonctions (à la louche, je dirais qu'il y a de la place pour 15 à 20 fonctions supplémentaires). Une fois mieux découpé, il sera nettement plus facile à comprendre, à faire comprendre et à deboguer.
500 lignes de code et 7 commentaires, je dis bel effort . Sérieusement, il doit y avoir plus de commentaires et surtout des commentaires utiles (un commentaire "// ouverture du fichier" qui précède un fopen() ne sert strictement à rien) qui aide à faire comprendre ton raisonnement.

**Nicovideo** · 31/03/2011, 21h58

Merci merci pour ces nouvelles réponses !

Au niveau de la compilation c'est bizarre car je compile et aucune erreur. Je vais regarder où pourrait il manquer une }

Au niveau du fichier j'étais en train de créer un système de highscore donc je n'en ai pas besoin je l'enlève d'ailleurs.

Les valeurs magiques sont des valeurs spécifiques à la PSP mais c'est vrai que je pourrai mettre des variables pour la lisibilité

Justement pour les fonctions je ne sais pas faire, c'est pour ça que je recherche quelqu'un qui peut prendre un peu de temps avec moi pour m'expliquer et peut être me guider dans le bon chemin

Pour ce qui est des commentaires... Effectivement je vous ai pas facilité la tâche car j'en ai pas mis des masses..

Ce qui me manque c'est se manque d'optimisation, créer des nouvelles fonctions (dans d'autres fichiers ?). Pouvez vous m'en dire plus au niveau de ça

Merci d'avance

**ram-0000** · 31/03/2011, 22h10

Envoyé par Nicovideo

Au niveau de la compilation c'est bizarre car je compile et aucune erreur. Je vais regarder où pourrait il manquer une }

Alors le code que tu présentes n'est pas le code que tu compiles (ou bien tu utilises un compilateur très particulier).

Envoyé par Nicovideo

Au niveau du fichier j'étais en train de créer un système de highscore donc je n'en ai pas besoin je l'enlève d'ailleurs.

Je m'en doutai un peu mais typiquement, ce système de highscore mérite 2 fonctions (au moins), une fonction read_high_score() et une fonction write_high_score()

Envoyé par Nicovideo

Les valeurs magiques sont des valeurs spécifiques à la PSP ...

Mauvaise réponse

Envoyé par Nicovideo

... mais c'est vrai que je pourrai mettre des variables pour la lisibilité

Bonne réponse enfin plutôt des constantes genre GEK_RIGHT, GEK_CROSS, ...

Envoyé par Nicovideo

Justement pour les fonctions je ne sais pas faire, c'est pour ça que je recherche quelqu'un qui peut prendre un peu de temps avec moi pour m'expliquer et peut être me guider dans le bon chemin

Ici, tu as quelqu'un qui pourra commencer à t'éclairer le chemin.

Envoyé par Nicovideo

Pour ce qui est des commentaires... Effectivement je vous ai pas facilité la tâche car j'en ai pas mis des masses..

C'est le moins que l'on puisse dire