Redefinition de variable genante.

**codas** · 17/12/2008, 15h06

Bonjour, ca fait un peu de temps que je fais un brickbreaker en opengl et sdl.
Tout allait bien jusqu'a ce que je tombe sur un os dans au niveau du test des collisions :
la balle est dirigee par balle.direc ( de la class ball ). Je passe ladresse de la classe dans la methode pour gerer les collisions et la ca ne marche que pour la deuxieme fonction appelee, car la variable de la premiere est redefinie directement apres.
voila le code pose probleme :

bricks.c

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
 
int brick::collide_haut_brique(ball balle)
{
 
// la balle touche le haut de la brique, il faut la faire remonter
//if(balle->direc==DOWN)
//{
	if(float(balle.ballposx/400)+0.1f >= float(brickposX/400) )// on teste que la balle est sur la case en x;
	{
		if ( float(balle.ballposx/400) <= float(brickposX/400)+0.1f )
		{
			if( float(balle.ballposy/400) <= float(brickposY/400)+0.1f)// on teste que la balle est sur la case en y;
			{
				if(float(balle.ballposy/400) >= float(brickposY/400)-0.1f )
				{
 
					//balle->direc=UP;
					return UP;
					//balle->direc=-balle->direc;
				}
			}
		}
	}
//}
 
return 0;
}
 
 
int brick::collide_bas_brique(ball balle)
{
 
// la balle touche le bas de la brique il faut la faire redescendre
//if(balle->direc==UP)
//{
	if(float(balle.ballposx/400)+0.1f >= float(brickposX/400) ) // on teste que la balle est sur la case en x;
	{
		if ( float(balle.ballposx/400) <= float(brickposX/400)+0.1f )
		{
			if( float(balle.ballposy/400) >= float(brickposY/400)-0.1f  )// on teste que la balle est sur la case en y;
			{
				if(float(balle.ballposy/400) <= float(brickposY/400)+0.1f )
				{
 
					//balle->direc=DOWN;
					return DOWN;
					//balle->direc=-balle->direc;
				}
			}
		}
	}
//}
 
return 0;
}
 
 
int level::checkcollides(ball *balle)
{
 
	for (int p=0; p<20 ; p++ )
	{
		if(bricks[p].isalive==true)
		{	
			if(bricks[p].collide_haut_brique(*balle)==UP)
			{
				balle->direc=UP;
				std::cout <<"UP ! balle.direc = " << balle->direc << "\n";
				bricks[p].isalive=false;
			}
 
			//std::cout <<"HAUT BRICK " << bricks[p].collide_bas_brique(balle) << "\n";
 
			if(bricks[p].collide_bas_brique(*balle)==DOWN) 
			{
				balle->direc=DOWN;
				std::cout <<"DOWN ! balle.direc = " << balle->direc << "\n";
				bricks[p].isalive=false;
			}
			//std::cout <<"BAS BRICK " << bricks[p].collide_bas_brique(balle) << "\n";
		}
	}
 
 
 
return 0;
}

et dans le main.c je l'apelle comme ca :

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
 
level Game;
 
Game.initbricks();
Game.bar.init(0,0);
Game.balle.init(Game.bar);
 
// init de ogl et sdl etc
 
unsigned int checkTime = SDL_GetTicks();
const unsigned int fps = 50;
 
bool run = true;
 
while (run)
{
	if(SDL_GetTicks() > (checkTime + 1000 / fps) )
	{	
		Events(event);
 
		Game.bar.updatepos();
 
		Game.checkcollides(&Game.balle);
 
		Game.balle.collide(Game.bar);
 
		Game.balle.upgrademove();
 
		Draw(Game);
 
    		/* On remet à jour le temps à contrôler */
    		checkTime = SDL_GetTicks();
	}
 
}
 
return 0;
}

Je ne sais pas trop pourquoi la boucle dans bricks.c redefinit la variable balle->direc donc si vous pouviez m'aider

.

**Médinoc** · 17/12/2008, 17h18

On peut voir la classe ball?

**codas** · 17/12/2008, 19h24

Envoyé par Médinoc

On peut voir la classe ball?

Oui la voila desole ;s

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
 
#ifndef BALL_H
#define BALL_H
 
#include <iostream>
#include <cmath>
#include <SDL/SDL.h>
#include <SDL/SDL_opengl.h>
#include <GL/gl.h>
#include <GL/glu.h>
#include "planche.h"
 
#define UP 1
#define DOWN -1
#define BEGIN 0
#define RIGHT 1
#define LEFT -1
 
class ball
{
public :
	int init(planche pl);
	int upgrademove();
	int collide(planche pla);
	int drawball();
 
//private :
	float ballposx, ballposy;
	bool ismoving;
	int direc;
	int rightorleft;
	float speedX;
	float speedY;
};
 
#endif

Sinon oui je vais preciser le probleme en fait quand je lance lapplication la variable balle->direc fait comme cela ( ce que jai avec les cout que jai mis dans le shell ) :

UP ! balle.direc = 1
DOWN ! balle.direc = -1
UP ! balle.direc = 1
DOWN ! balle.direc = -1

quand il y a une collision avec le haut dune brique. Ca devrait en fait juste faire UP ! balle.direc = 1 et donc faire remonter la balle, or ca redefinit directement apres par le if qui met le balle->direc a down, et je narrive pas a savoir pourquoi...

**3DArchi** · 18/12/2008, 09h11

Ou j'ai mal lu, ou les quatre tests sont identiques dans les deux fonctions (brick::collide_haut_brique et brick::collide_bas_brique). Donc s'ils sont vérifiés dans la première, ils le seront dans la seconde même si les 3ème et 4ème sont inversés.

**codas** · 18/12/2008, 19h45

Envoyé par 3DArchi

Ou j'ai mal lu, ou les quatre tests sont identiques dans les deux fonctions (brick::collide_haut_brique et brick::collide_bas_brique). Donc s'ils sont vérifiés dans la première, ils le seront dans la seconde même si les 3ème et 4ème sont inversés.

Oui mais ils ne sont pas identiques justement ... enfin je vais les revoir

**3DArchi** · 19/12/2008, 15h42

Envoyé par codas

Oui mais ils ne sont pas identiques justement ... enfin je vais les revoir

Salut,
Dans le code que tu fournis, moi, je vois les quatre mêmes tests dans les deux fonctions. La seule différence est l'inversion entre le troisième et le quatrième. Ce qui ne change rien s'ils sont vérifiés! :
int brick::collide_haut_brique(ball balle):

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
	if(float(balle.ballposx/400)+0.1f >= float(brickposX/400) )// on teste que la balle est sur la case en x;
	{
		if ( float(balle.ballposx/400) <= float(brickposX/400)+0.1f )
		{
			if( float(balle.ballposy/400) <= float(brickposY/400)+0.1f)// on teste que la balle est sur la case en y;
			{
				if(float(balle.ballposy/400) >= float(brickposY/400)-0.1f )
				{
 
					//balle->direc=UP;
					return UP;
					//balle->direc=-balle->direc;
				}
			}
		}
	}

int brick::collide_bas_brique(ball balle):

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
 
	if(float(balle.ballposx/400)+0.1f >= float(brickposX/400) ) // on teste que la balle est sur la case en x;
	{
		if ( float(balle.ballposx/400) <= float(brickposX/400)+0.1f )
		{
			if( float(balle.ballposy/400) >= float(brickposY/400)-0.1f  )// on teste que la balle est sur la case en y;
			{
				if(float(balle.ballposy/400) <= float(brickposY/400)+0.1f )
				{
 
					//balle->direc=DOWN;
					return DOWN;
					//balle->direc=-balle->direc;
				}
			}
		}
	}

Soit en les mettant côte à côte et en inversant 3 et 4:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
	if(float(balle.ballposx/400)+0.1f >= float(brickposX/400) )// on teste que la balle est sur la case en x;
	if(float(balle.ballposx/400)+0.1f >= float(brickposX/400) ) // on teste que la balle est sur la case en x;
	{
		if ( float(balle.ballposx/400) <= float(brickposX/400)+0.1f )
		if ( float(balle.ballposx/400) <= float(brickposX/400)+0.1f )
		{
			if(float(balle.ballposy/400) <= float(brickposY/400)+0.1f)// on teste que la balle est sur la case en y;
			if(float(balle.ballposy/400) <= float(brickposY/400)+0.1f )
			{
				if(float(balle.ballposy/400) >= float(brickposY/400)-0.1f )
			   if( float(balle.ballposy/400) >= float(brickposY/400)-0.1f  )// on teste que la balle est sur la case en y;
				{

**3DArchi** · 17/12/2008, 17h20

Bonjour et bienvenu,
Peux-tu préciser un peu plus ton problème?

Redefinition de variable genante.

C++

Vue hybride

Discussions similaires

Partager

Partager