problème avec la gestion du temps

**jurams** · 23/06/2011, 19h11

bonjour à tous je suis encore débutant, je me suis lancé dans cette bibliothèque il y a peu de temps et j'ai un problème avec mon code une erreur se produit,
le programme consiste à faire bouger un petit cube (une surface) dans direction différente en fonction d'un nombre pris au hasard.

code 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
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
#include <stdlib.h>
#include <stdio.h>
#include <SDL/SDL.h>
#include <time.h>
 
 
const int MIN = 1, MAX=4;
 
int aleatoire(int NBaleatoire);//fontion qui renvoi un nobre aléatoire
void bouger_gauche(int positionCube);//fontion permettant de diriger le cube
void bouger_droite(int positionCube);
void bouger_haut(int positionCube);
void bouger_bas(int positionCube);
 
 
int main(int arcgc, char *argv[])
{
 
    SDL_Surface *ecran = NULL, *lifeCube = NULL;
    int continuer = 1;
    int nombreMystere = 0;
    SDL_Event event;
    SDL_Rect positionCube;
 
    positionCube.x = (640/2)-(30/2);
    positionCube.y = (480/2)-(30/2);
 
 
    SDL_Init(SDL_INIT_VIDEO);
    ecran = SDL_SetVideoMode(640,480,32,SDL_HWSURFACE|SDL_DOUBLEBUF);
 
 
    lifeCube = SDL_CreateRGBSurface(SDL_HWSURFACE, 30, 30, 32, 0,0,0,0);
    SDL_FillRect(lifeCube,NULL,SDL_MapRGB(lifeCube->format,34,66,196));
 
    aleatoire(nombreMystere);
 
    while(continuer)
    {
        SDL_PollEvent(&event);
        switch(event.type)
        {
            case SDL_QUIT:
            continuer = 0;
        }
 
 
        if (nombreMystere==1)
        {
            bouger_gauche(positionCube.x);
 
        }
 
        else if (nombreMystere == 2)
        {
            bouger_droite(positionCube.x);
        }
 
        else if (nombreMystere==3)
        {
            bouger_haut(positionCube.y);
        }
 
        else if (nombreMystere==4)
        {
            bouger_bas(positionCube.y);
        }
 
 
 
         SDL_FillRect(ecran,NULL,SDL_MapRGB(ecran->format,179,255,176));
         SDL_BlitSurface(lifeCube,NULL,ecran,&positionCube);
         SDL_Flip(ecran);
    }
 
 
 
    SDL_FreeSurface(lifeCube);
    SDL_QUIT;
}
 
 
 
 
int aleatoire(int NBaleatoire)
{
    srand(time(NULL));
    NBaleatoire = (rand()%(MAX-MIN+1))+MIN;
 
    return NBaleatoire;
 
}
 
 
int bouger_droite(int positionCube)
{
 
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube++;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
int bouger_gauche(int positionCube)
{
    int positionCube;
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube--;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
int bouger_haut(int positionCube)
{
    int positionCube;
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube++;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
 
 
int bouger_bas(int positionCube)
{
    int positionCube;
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube--;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}

**Kannagi** · 24/06/2011, 14h38

Pour un débutant , tu programme déjà bien (code propre et clair) c'est plutôt rare et je t'en félicite.

Une remarque , on utilise rarement

Code :

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

const int MAX = 4;

mais plutot :

Code :

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

#define MAX 4

pour ton code déjà par logique ,la valeur de la position reste inchangé
tu a ecrit :

Code :

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

bouger_droite(positionCube.x);

Donc déjà la positionCube.x n’étant pas en pointeur , la variable dans le main reste inchangé.

Après pour la fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int bouger_droite(int positionCube)
{
 
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube++;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}

Tu renvoie une valeur mais tu ne utilise pas ,il faudrait que le code dans le main soit :

Code :

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

positionCube.x = bouger_droite(positionCube.x);

Personnellement je te conseillerai t'utiliser le pointeur , plus logique et plus formateur (ça t'obligera a t'en habitué).

**jurams** · 24/06/2011, 16h10

Merci pour le compliment

j'ai écrit donc positionCube.x = bouger_gauche(positionCube.x);
mais encore 15 erreur...
je n'ai pas mis de pointeur, je suis pas encore à l'aise avec eux.
Merci pour ton aide

**jurams** · 26/06/2011, 14h16

aucune réponse ?

**LittleWhite** · 27/06/2011, 02h36

Envoyé par jurams

Merci pour le compliment

j'ai écrit donc positionCube.x = bouger_gauche(positionCube.x);
mais encore 15 erreur...
je n'ai pas mis de pointeur, je suis pas encore à l'aise avec eux.
Merci pour ton aide

Bonjour,

Tout d'abord, il faut mieux comprendre l'histoire des pointeurs, car d'une part c'est pratique (notamment pour le point soulever par Kanagi) et parce que c'est souvent utilisé (et bien souvent car on ne peut pas éviter).
(Voir cours C / pointeurs sur DVP )

Pour l'histoire du MAX:
Il faut mieux un const int MAX = 4 que le #define (voir la FAQ C, je pense).

Il faudra voir aussi à programmer dans plus d'un fichier (des fichiers .c / .h)

Code :

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

1
2
3
4
5
6
7
8
int aleatoire(int NBaleatoire)
{
    srand(time(NULL));
    NBaleatoire = (rand()%(MAX-MIN+1))+MIN;
 
    return NBaleatoire;
 
}

srand ne s'appelle qu'une fois au début du programme.

Les 4 fonctions bouger répètent le même code, il serait très judicieux de factoriser le code (enfin, c'est une remarque pour avancer dans la "bonne programmation")

Lorsque l'on indique que l'on a 15 erreurs, il serait bien de préciser quelles sont elles...

**jurams** · 01/07/2011, 18h36

Merci pour ton aide,
je sais qu'il faut répartir le code dans plusieurs fichiers,
mais comme c'est un petit projet je met tout dans le même
fichier.
Sinon j'ai corrigé quelques erreurs débiles, mes prototypes déclaraient
des fonctions de types 'void' alors que elles sont de types 'int',
donc maintenant j'ai plus d'erreurs mais le cube ne bouge toujours pas.
Je redonne mon code corrigé :

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
#include <stdlib.h>
#include <stdio.h>
#include <SDL/SDL.h>
#include <time.h>
 
 
const int MIN = 1, MAX=4;
 
int aleatoire(int NBaleatoire);//fontion qui renvoi un nobre aléatoire
int bouger_gauche(int positionCube);//fontion permettant de diriger le cube
int bouger_droite(int positionCube);
int bouger_haut(int positionCube);
int bouger_bas(int positionCube);
 
 
int main(int arcgc, char *argv[])
{
 
    SDL_Surface *ecran = NULL, *lifeCube = NULL;
    int continuer = 1;
    int nombreMystere = 0;
    SDL_Event event;
    SDL_Rect positionCube;
 
    positionCube.x = (640/2)-(30/2);
    positionCube.y = (480/2)-(30/2);
 
 
    SDL_Init(SDL_INIT_VIDEO);
    ecran = SDL_SetVideoMode(640,480,32,SDL_HWSURFACE|SDL_DOUBLEBUF);
 
 
    lifeCube = SDL_CreateRGBSurface(SDL_HWSURFACE, 30, 30, 32, 0,0,0,0);
    SDL_FillRect(lifeCube,NULL,SDL_MapRGB(lifeCube->format,34,66,196));
 
 
    srand(time(NULL));//initialisation du temps
 
    aleatoire(nombreMystere);
 
    while(continuer)
    {
        SDL_PollEvent(&event);
        switch(event.type)
        {
            case SDL_QUIT:
            continuer = 0;
        }
 
 
        if (nombreMystere==1)
        {
 
            positionCube.x = bouger_gauche(positionCube.x);
 
        }
 
        else if (nombreMystere == 2)
        {
 
            positionCube.x = bouger_droite(positionCube.x);
        }
 
        else if (nombreMystere==3)
        {
 
            positionCube.y = bouger_haut(positionCube.y);
        }
 
        else if (nombreMystere==4)
        {
 
            positionCube.y = bouger_bas(positionCube.y);
        }
 
 
 
         SDL_FillRect(ecran,NULL,SDL_MapRGB(ecran->format,179,255,176));
         SDL_BlitSurface(lifeCube,NULL,ecran,&positionCube);
         SDL_Flip(ecran);
    }
 
 
 
    SDL_FreeSurface(lifeCube);
    SDL_QUIT;
}
 
 
 
 
int aleatoire(int NBaleatoire)
{
 
    NBaleatoire = (rand()%(MAX-MIN+1))+MIN;
 
    return NBaleatoire;
 
}
 
 
int bouger_droite(int positionCube)
{
 
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube++;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
int bouger_gauche(int positionCube)
{
 
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube--;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
int bouger_haut(int positionCube)
{
 
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube++;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
 
 
int bouger_bas(int positionCube)
{
 
    int tempsPrecedent = 0, tempsActuel = 0;
    //gérer le temps :
        tempsActuel = SDL_GetTicks();
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube--;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}

Merci encore.

**LittleWhite** · 01/07/2011, 19h45

Dans vos fonctions de mouvement, il faut que le tempsPrecedent soit en static.

Code c :

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

static int tempsPrecedent = 0, tempsActuel = 0;

Vous pouvez lire des informations sur ce mot clé, dans la FAQ / cours C.
Sinon, la description rapide (et incomplète) le mot clé static rend une variable crée une fois. Cela veut dire, pour tempsPrecedent, qu'elle va être mise à 0 qu'une seule fois, et ce au premier appel de cette ligne.
A chaque prochain appel, elle va garder sa valeur précédente, faisant que votre code marchera mieux.

**jurams** · 01/07/2011, 20h44

Merci encore et encore...
Malheureusement mon cube ne bouge toujours pas,
j'ai essayé de mettre "static int" dans le mains,
en dehors du main (au début du code) et dans chaque fonction
mais rien ne se passe.
J'ai essayé aussi comme ça :

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
#include <stdlib.h>
#include <stdio.h>
#include <SDL/SDL.h>
#include <time.h>
 
 
const int MIN = 1, MAX=4;
 
int tempsPrecedent = 0, tempsActuel = 0;
 
 
int aleatoire(int NBaleatoire);//fontion qui renvoi un nobre aléatoire
int bouger_gauche(int positionCube);//fontion permettant de diriger le cube
int bouger_droite(int positionCube);
int bouger_haut(int positionCube);
int bouger_bas(int positionCube);
 
 
int main(int arcgc, char *argv[])
{
 
    SDL_Surface *ecran = NULL, *lifeCube = NULL;
    int continuer = 1;
    int nombreMystere = 0;
    SDL_Event event;
    SDL_Rect positionCube;
 
    positionCube.x = (640/2)-(30/2);
    positionCube.y = (480/2)-(30/2);
 
 
    SDL_Init(SDL_INIT_VIDEO);
    ecran = SDL_SetVideoMode(640,480,32,SDL_HWSURFACE|SDL_DOUBLEBUF);
 
 
    lifeCube = SDL_CreateRGBSurface(SDL_HWSURFACE, 30, 30, 32, 0,0,0,0);
    SDL_FillRect(lifeCube,NULL,SDL_MapRGB(lifeCube->format,34,66,196));
 
 
    srand(time(NULL));//initialisation du temps
 
    aleatoire(nombreMystere);
 
    tempsActuel = SDL_GetTicks();
 
    while(continuer)
    {
        SDL_PollEvent(&event);
        switch(event.type)
        {
            case SDL_QUIT:
            continuer = 0;
        }
 
 
        if (nombreMystere==1)
        {
 
            positionCube.x = bouger_gauche(positionCube.x);
 
        }
 
        else if (nombreMystere == 2)
        {
 
            positionCube.x = bouger_droite(positionCube.x);
        }
 
        else if (nombreMystere==3)
        {
 
            positionCube.y = bouger_haut(positionCube.y);
        }
 
        else if (nombreMystere==4)
        {
 
            positionCube.y = bouger_bas(positionCube.y);
        }
 
 
 
         SDL_FillRect(ecran,NULL,SDL_MapRGB(ecran->format,179,255,176));
         SDL_BlitSurface(lifeCube,NULL,ecran,&positionCube);
         SDL_Flip(ecran);
    }
 
 
 
    SDL_FreeSurface(lifeCube);
    SDL_QUIT;
}
 
 
 
 
int aleatoire(int NBaleatoire)
{
 
    NBaleatoire = (rand()%(MAX-MIN+1))+MIN;
 
    return NBaleatoire;
 
}
 
 
int bouger_droite(int positionCube)
{
 
 
    //gérer le temps :
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube++;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
int bouger_gauche(int positionCube)
{
 
 
    //gérer le temps :
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube--;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
int bouger_haut(int positionCube)
{
 
 
    //gérer le temps :
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube++;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}
 
 
 
 
 
int bouger_bas(int positionCube)
{
 
 
    //gérer le temps :
        if (tempsActuel - tempsPrecedent > 30)
        {
            positionCube--;
            tempsPrecedent = tempsActuel;
        }
    return positionCube;
 
}

pour que la variable puisse être modifier en ayant la même valeur.
J'ai l'impression que l'erreur ne vient pas de là.

**LittleWhite** · 02/07/2011, 01h41

Encore un problème, avec le temps.
Car le temps actuel n'est pas mis à jour à chaque tour de boucle.
En effet, il a été initialisé une fois, avant la boucle principale, puis n'ai plus jamais mis à jour ... bizarre, non ?

**jurams** · 02/07/2011, 12h06

enfin j'y suis!
merci à vous deux pour votre aide.
Une autre erreur, la fonction avait le return mais elle donné le résultat
à aucune variable.
Merci beaucoup mon programme peut enfin avancer!

**skeud** · 08/07/2011, 10h24

N'oublie pas le petit bouton

ça m'aurait éviter de regarder ce sujet

problème avec la gestion du temps

SDL

Discussions similaires

Partager

Partager