Discussion :

[Callehabana]

Bonjour à tous et à toutes,

Je viens vous exposer un petit soucis survenu pendant la programmation d'un jeu 2D en SDL dans le cadre d'un projet scolaire.
C'est un DuckHunt, le principe du jeu est de tuer des canards.
Dans un premier temps, j'affiche un menu permettant de jouer ou de quitter puis le jeu se lance (si le joueur remplit la condition "play").

Mon codé est divisé en plusieurs fichiers .cc et .h.

Voici mon soucis : dans mon Main.cc,je fais appel à un PollEvent afin qu'il gère la fermeture de mon programme tout au long de celui-ci.
Cependant, je suis obligé de relancer une boucle "while PollEvent" pour fermer le programme pendant le jeu.
Ceci a pour conséquence une imbriquation de deux PollEvents et pour inconvénients d'obtenir un code non optimisé et un evenement assez surprenant : un SDL_Rect sensé bouger continuellement n'est en déplacement que lorsque je bouge ma souris.

Ma question est donc la suivante :

Comment faire pour n'avoir à déclarer qu'une seule boucle while (PollEvent) gérant la fermeture du programme sur tout le long du jeu ?

Mon "Main.cc"

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
 
#include <iostream>
#include <cstdlib>
#include <string>
#include <SDL.h>
#include <SDL_image.h>
#include <sstream>
#include <SDL/SDL_ttf.h>
#include <SDL_ttf.h>
#include "menu.h" 
#include "jouer.h"
#include "image.h"
 
using namespace std;
 
SDL_Surface *ecran;//Initialisation de l'écran
SDL_Event evennement;//Initialisation des évènements à venir sur la fenêtre SD
 
int main ()
{
	bool quit=false;
	bool play;
 
	SDL_Init(SDL_INIT_EVERYTHING);
	ecran=SDL_SetVideoMode(SCREEN_WIDTH,SCREEN_HEIGHT,SCREEN_BPP,SDL_SWSURFACE);
 
	play = menu();//J'appelle ici mon menu qui demande à l'utilisateur de jouer ou de //quitter, c'est un booléen retournant play
 
	while(!quit)
	{
		while(SDL_PollEvent(&evennement))//1er appel de mon PollEvent
		{
			if( evennement.type == SDL_QUIT )
			{
				quit = true;
			}
				if (play)
					jouer();//On lance le jeu contenu dans un autre fichier
 
				else
					quit = true;
		}
 
	}
	return 0;
}

Et voici mon "jouer.cc" :

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
#include <iostream>
#include <cstdlib>
#include <string>
#include <SDL.h>
#include <SDL_image.h>
#include <sstream>
#include <SDL/SDL_ttf.h>
#include <SDL_ttf.h>
#include "jouer.h"
using namespace std;	
 
SDL_Surface *surfaceJeu;
extern SDL_Event evennement;//J'appelle le SDL-Event déclaré dans le main
extern SDL_Surface *canard;
 
void jouer()
{
	int cptPhaseVol =0;
	Canard c;
	bool quit=false;
	applySurface( 0, 0, surfaceJeu, screen, NULL );
	surfaceJeu = load_image( "backGame.png");
	c.canard = loadImageWithColorKey ("duck.png",228,255,0);
	initImageCanard (c);
	initCanard(c);
 
		while(SDL_PollEvent(&evennement))//Deuxieme boucle PollEvent
		{
			if( evennement.type == SDL_QUIT )
			{
				quit = true;
			}
 
			if (!quit)
			{
				if (cptPhaseVol == 0)
					c.canLect = c.NDos1;
				else if (cptPhaseVol==50)
					c.canLect = c.NDos2;
				else if(cptPhaseVol == 100)
					c.canLect = c.NDos3;
				else if (cptPhaseVol >150)
					cptPhaseVol =0;
				applySurface( 0, 0, surfaceJeu, screen, NULL );
				applySurface( c.x, c.y, c.canard, screen,&c.canLect );
				SDL_Flip(screen);
				moveCanard(c);
				showCanard(c, screen);
				SDL_FreeSurface(screen);
				cptPhaseVol++;
			}
		}
}

A noter que nous sommes obligés de fragmenter ainsi notre code cela fait partie des eigeances de nos professeurs.
Par ailleurs, le code est très loin d'être terminé, ce soucis me blogue légèrement pour continuer la suite.
Enfin, dans mon menu.cc (non affiché ici), j'ai également appelé une autre boucle while (PollEvent).
Cette dernière n'affecte cependant pas le bon déroulement du programme,hors-mis peut-être une perte d'optimisation de l'execution.
Je ne sais pas si c'est vraiment très clair, n'hésitez pas à me demander de clarifier certains points si necéssaire.
Merci par avance !
Cordialement.

[ternel]

Il y a sur le net un tuto pour un tetris en sdl que j'avais suivi il y a un an.

Je te le recommande, mais je ne le retrouve pas, gràce au proxy de mon poste au boulot.

Je l'avais trouvé rapidement, demande donc à notre ami à tous.

[Callehabana]

J'ai trouvé des tetris en un seul fichier code entier ou n'utilisant qu'un simple .h. Ils ne traitent donc pas de mon probleme.

Un seul d'entre eux était intérressant cependant hors-mi un code illisible le programme n'a pas "besoin" d'imbriquer les PollEvent comme ici.
Dans mon code, j'aimerai ne pouvoir avoir à déclarer qu'une seule boucle qui s'applique à tous les autres fichiers.cc de mon executable, me permettant ainsi de pouvoir quitter la fênetre par SDL_Quit à tout moment, que ce soit dans le menu ou en plein jeu.

Enfin les tutos ne traitent pas ce problème, ils ne parlent même pas de PollEvent.

Cela-dit, merci de ta réponse

[koala01]

Salut,

Finalement, les conditions de sorties de la boucle et le poolEvent sont identiques dans la fonction principale et dans la fonction joue...

Dés lors, pourquoi ne pas passer à la fonction joue des références sur les données qui nous permettent de sortir de la boucle, au moment de l'appeler depuis la boucle principale

Ta fonction joue pourrait très bien avoir un prototype proche de

Code :

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

void joue(PollEvent & poll , bool & quit)

et ressembler dés lors à quelque chose comme

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
 
void jouer(PollEvent & poll , bool & quit)
{
	int cptPhaseVol =0;
	Canard c;
        /* plus besoin de cette variable, on l'a en paramètre ;)
	bool quit=false;
        */
	applySurface( 0, 0, surfaceJeu, screen, NULL );
	surfaceJeu = load_image( "backGame.png");
	c.canard = loadImageWithColorKey ("duck.png",228,255,0);
	initImageCanard (c);
	initCanard(c);
 
		while(poll) // je ne connais pas assez SDL pour savoir comment 
                            // rajouter un événement à ce pollEvent :P
                            // mais est-ce encore nécessaire ???
		{
			if( evennement.type == SDL_QUIT )
			{
				quit = true;
			}
 
			if (!quit)
			{
				if (cptPhaseVol == 0)
					c.canLect = c.NDos1;
				else if (cptPhaseVol==50)
					c.canLect = c.NDos2;
				else if(cptPhaseVol == 100)
					c.canLect = c.NDos3;
				else if (cptPhaseVol >150)
					cptPhaseVol =0;
				applySurface( 0, 0, surfaceJeu, screen, NULL );
				applySurface( c.x, c.y, c.canard, screen,&c.canLect );
				SDL_Flip(screen);
				moveCanard(c);
				showCanard(c, screen);
				SDL_FreeSurface(screen);
				cptPhaseVol++;
			}
		}
}

et ta fonction main se simplifierait sous la forme de

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
int main ()
{
	bool quit=false;
	bool play;
 
	SDL_Init(SDL_INIT_EVERYTHING);
	ecran=SDL_SetVideoMode(SCREEN_WIDTH,SCREEN_HEIGHT,SCREEN_BPP,SDL_SWSURFACE);
 
	play = menu();//J'appelle ici mon menu qui demande à l'utilisateur de jouer ou de //quitter, c'est un booléen retournant play
        /* prenons quand meme en compte le fait que le joueur veut jouer
         * autrement, on passe directement à la fin ;)
         */
    if(play)
    {
	while(!quit)
	{
            SDL_PollEvent poll(&evennement);
            joue(poll, quit);
        }
    }
	return 0;
}

[Bousk]

Bonjour,

en général on laisse le poll event dans la boucle principal et on effectue des actions sur le jeu en fonction des évènements qu'on y trouve.

Dans ton cas, il est inutile de réaliser cette boucle dans le main amha, il faut voir les évènements comme une liste ordonnée dans l'ordre où ils ont été émis.
Donc s'il y a un évènement de fermeture de fenêtre, tu le retrouveras dans jouer().

btw, avec ce code, il est impossible de quitter par le menu sans fermer la fenêtre manuellement. Est-ce voulu ?

enfin, pourquoi déclarer ecran en globale ? pourquoi evenements en globale ?
N'oublie pas non plus de libérer les ressources, en particulier les SDL_Surface.
T'as déjà réussi à le compiler et le lancer ?
Avec un beau free surface sur l'écran dans jouer, j'ai des doutes quant à l'absence de crash.

[r0d]

Callehabana, je te conseille de jeter un coup d'oeil au modèle MVC. Ce sera peut-être un peu ardu (je ne connais pas ton niveau), mais c'est exactement ce qu'il te faut pour ce type de programme.