Jeu de la vie

**Spiritus44** · 06/09/2014, 17h29

Bonjour,

Connaissant depuis peu le Jeu de la vie, je me suis lancer dans une réalisation personnel de se jeu, le tout en c++ procédural avec la sdl 1.2.

La génération de mon tableau fonctionne bien l'affichage aussi mais lorsque je rajoute la gestion des cellules au fur et a mesure des générations, cela m'affiche un écran noir, mon code :

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
 
#ifdef __cplusplus
    #include <cstdlib>
#else
    #include <stdlib.h>
#endif
#include <iostream>
#include <SDL/SDL.h>
#include <ctime>
 
using namespace std;
 
 
int main ( int argc, char** argv )
{
 
 
    int maxLargeur,maxHauteur;
 
    //Initialisation SDL
    if ( SDL_Init( SDL_INIT_VIDEO ) < 0 )
    {
        printf( "Unable to init SDL: %s\n", SDL_GetError() );
        return 1;
    }
 
    // make sure SDL cleans up before exit
    atexit(SDL_Quit);
 
    SDL_Surface* screen,*image;
    SDL_Rect positionImage;
 
 
    maxLargeur=SDL_GetVideoInfo()->current_w;
    maxHauteur=SDL_GetVideoInfo()->current_h;
 
 
    positionImage.x = 0;
    positionImage.y = 0;
 
    int maxX = maxHauteur/5;
    int maxY = maxLargeur/5;
    int grille[maxX][maxY];
    int alea = 0;
    srand(time(NULL));
    int j = 0;
 
    //Initialisation grille
    for(int i = 1; i < maxX; i++)
    {
        for(int j = 0; j < maxY; j++)
        {
 
            alea = rand() %(1-11);
 
            if(alea == 1)
            {
                grille[i][j]=1;
            }
            else
            {
                grille[i][j]=0;
            }
 
 
        }
    }
 
 
 
 
 
 
 
    // create a new window
    screen=SDL_SetVideoMode(maxLargeur, maxHauteur, 32, SDL_HWSURFACE| SDL_FULLSCREEN |SDL_DOUBLEBUF);
 
    if ( !screen )
    {
        printf("Unable to set 640x480 video: %s\n", SDL_GetError());
        return 1;
    }
 
 
 
 
    bool continuer = false;
 
    while (!continuer)
    {
 
 
        SDL_Event event;
        while (SDL_PollEvent(&event))
        {
 
            switch (event.type)
            {
 
            case SDL_QUIT:
                continuer = true;
                break;
 
 
            case SDL_KEYDOWN:
                {
 
                    if (event.key.keysym.sym == SDLK_ESCAPE)
                        continuer = true;
                    break;
                }
            }
        }
            int tempsActuel = SDL_GetTicks();
            int tempsPrecedent;
            if (tempsActuel - tempsPrecedent > 30) /* Si 30 ms se sont écoulées */
            {
 
 
                int resultat;
 
                // Affichage Map
                for(int i = 1; i < maxX; i++)
                {
                    for(int j = 0; j < maxY; j++)
                    {
 
 
 
                        if(grille[i][j]==0)
                        {
 
                            image = SDL_LoadBMP("vide.bmp");
                            /* On blitte par-dessus l'écran */
                            SDL_BlitSurface(image, NULL, screen, &positionImage);
                            SDL_FreeSurface(image);
                        }
                        else
                        {
                            image = SDL_LoadBMP("plein.bmp");
                            /* On blitte par-dessus l'écran */
                            SDL_BlitSurface(image, NULL, screen, &positionImage);
                            SDL_FreeSurface(image);
                        }
                        positionImage.x = positionImage.x +6;
                        SDL_Flip(screen);
 
                        //Gestion cellule
 
                        if(grille[i-1][j-1]==1 && i!=0 )
                        {
                            resultat++;
                        }
                        if(grille[i-1][j]==1 && i!=0 )
                        {
                            resultat++;
                        }
 
                        if(grille[i-1][j+1]==1 && i!=0 && j!=maxX)
                        {
                            resultat++;
                        }
 
 
 
                        if(grille[i][j-1]==1 && j!=0)
                        {
                            resultat++;
                        }
                        if(grille[i][j+1]==1 && j!=maxX )
                        {
                            resultat++;
                        }
 
 
                        if(grille[i+1][j-1]==1 && i!=maxY && j!=0)
                        {
                            resultat++;
                        }
                        if(grille[i+1][j]==1 && i!=maxY && j!=maxX)
                        {
                            resultat++;
                        }
                        if(grille[i+1][j+1]==1 && i!=maxY && j!=maxX)
                        {
                            resultat++;
                        }
 
 
                        if(grille[i][j]==0 && resultat==3)
                        {
                            grille[i][j]=1;
                        }
                        else if(grille[i][j]==1 && resultat==2 or resultat==3)
                        {
                            grille[i][j]=1;
                        }
                        else
                        {
                            grille[i][j]=0;
                        }
                        // Fin Gestion cellule
 
                    }
 
                    positionImage.x = 0;
                    positionImage.y = positionImage.y +6;
 
 
                }
 
 
                positionImage.x = 0;
                positionImage.y = 0;
 
            }
 
    }
 
 
 
 
 
    return 0;
}

Avez vous une idée de mon problème ? Au passage si vous avez des améliorations niveau rapidité ou autre, je veux bien.

**LittleWhite** · 07/09/2014, 10h30

Bonjour,

Votre premier gros soucis. En effet, à chaque tour vous rechargez l'image (SDL_LoadBMP) ce qui prend un temps monstre à chaque fois. L'idée est de charger l'image avant la boucle principale et de la garder en mémoire (une unique fois). Et après, vous ne devez pas oublier le SDL_FreeSurface() sur l'image.
De plus, vous devez vérifier ce que retourne SDL_LoadBMP. Rien ne prouve que vous arrivez à charger l'image (il ne trouve pas le fichier).

**Spiritus44** · 07/09/2014, 13h14

Merci d'avoir pris le temps de m'aider, j'ai tenté d'appliquer vos précieux conseil en rajoutant la gestion des erreurs de chargements d'images et en chargeant l'image avant ma boucle principal et la ré-allocation de la mémoire une fois la fin de boucle.
Constatez par vous même :

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
#ifdef __cplusplus
    #include <cstdlib>
#else
    #include <stdlib.h>
#endif
#include <iostream>
#include <SDL/SDL.h>
#include <ctime>
 
using namespace std;
 
 
int main ( int argc, char** argv )
{
 
 
    int maxLargeur,maxHauteur;
 
    //Initialisation SDL
    if ( SDL_Init( SDL_INIT_VIDEO ) < 0 )
    {
        printf( "Unable to init SDL: %s\n", SDL_GetError() );
        return 1;
    }
 
    // make sure SDL cleans up before exit
    atexit(SDL_Quit);
 
    SDL_Surface* screen,*imagePlein,*imageVide; // Ajout des images
    SDL_Rect positionImage;
 
 
    maxLargeur=SDL_GetVideoInfo()->current_w;
    maxHauteur=SDL_GetVideoInfo()->current_h;
 
    maxLargeur =500;
    maxHauteur =500;
 
    positionImage.x = 0;
    positionImage.y = 0;
 
    int maxX = maxHauteur/10;
    int maxY = maxLargeur/10;
    int grille[maxX][maxY];
    int alea = 0;
    srand(time(NULL));
    int j = 0;
 
    //Initialisation grille
    for(int i = 1; i < maxX; i++)
    {
        for(int j = 0; j < maxY; j++)
        {
 
            alea = rand() %(1-11);
 
            if(alea == 1)
            {
                grille[i][j]=1;
            }
            else
            {
                grille[i][j]=0;
            }
 
 
        }
    }
 
 
 
 
 
 
 
    // Création Fenêtre
 
// | SDL_FULLSCREEN
 
    screen=SDL_SetVideoMode(maxLargeur, maxHauteur, 32, SDL_HWSURFACE |SDL_DOUBLEBUF);
 
    if ( !screen )
    {
        printf("Unable to set 640x480 video: %s\n", SDL_GetError());
        return 1;
    }
 
 
 
 
    bool continuer = false;
 
    imagePlein = SDL_LoadBMP("plein.bmp");
    imageVide = SDL_LoadBMP("vide.bmp");
 
    // Gestion des erreurs, si problème on ferme tout
    if (!imagePlein)
    {
        fprintf(stderr, "Erreur d'initialisation de la SDL : %s\n", SDL_GetError());
        exit(EXIT_FAILURE);
        continuer = true;
    }
    if (!imageVide)
    {
        fprintf(stderr, "Erreur d'initialisation de la SDL : %s\n", SDL_GetError());
        exit(EXIT_FAILURE);
        continuer = true;
    }
 
    // Boucle principal
 
    while (!continuer)
    {
 
 
        SDL_Event event;
        while (SDL_PollEvent(&event))
        {
 
            switch (event.type)
            {
 
            case SDL_QUIT:
                continuer = true;
                break;
 
 
            case SDL_KEYDOWN:
                {
 
                    if (event.key.keysym.sym == SDLK_ESCAPE)
                        continuer = true;
                    break;
                }
            }
        }
 
            int tempsActuel = SDL_GetTicks();
            int tempsPrecedent;
 
            if (tempsActuel - tempsPrecedent > 30) /* Si 30 ms se sont écoulées */
            {
 
                // Initialisation variable,
                int resultat;
 
                // Affichage Map
                for(int i = 1; i < maxX; i++)
                {
                    for(int j = 0; j < maxY; j++)
                    {
 
 
 
                        if(grille[i][j]==0)
                        {
                            /* On blitte par-dessus l'écran */
                            SDL_BlitSurface(imageVide, NULL, screen, &positionImage);
                        }
                        else
                        {
                            SDL_BlitSurface(imagePlein, NULL, screen, &positionImage);
 
                        }
                        positionImage.x = positionImage.x +11;
 
                        //Affichage écran
                        SDL_Flip(screen);
 
                        //Gestion cellule En commentaire car ca bugait précédemment 
/*
                        if(grille[i-1][j-1]==1 && i!=0 )
                        {
                            resultat++;
                        }
                        if(grille[i-1][j]==1 && i!=0 )
                        {
                            resultat++;
                        }
 
                        if(grille[i-1][j+1]==1 && i!=0 && j!=maxX)
                        {
                            resultat++;
                        }
 
 
 
                        if(grille[i][j-1]==1 && j!=0)
                        {
                            resultat++;
                        }
                        if(grille[i][j+1]==1 && j!=maxX )
                        {
                            resultat++;
                        }
 
 
                        if(grille[i+1][j-1]==1 && i!=maxY && j!=0)
                        {
                            resultat++;
                        }
                        if(grille[i+1][j]==1 && i!=maxY && j!=maxX)
                        {
                            resultat++;
                        }
                        if(grille[i+1][j+1]==1 && i!=maxY && j!=maxX)
                        {
                            resultat++;
                        }
 
 
                        if(grille[i][j]==0 && resultat==3)
                        {
                            grille[i][j]=1;
                        }
                        else if(grille[i][j]==1 && resultat==2 or resultat==3)
                        {
                            grille[i][j]=1;
                        }
                        else
                        {
                            grille[i][j]=0;
                        }*/
                        // Fin Gestion cellule
 
                    }
 
                    positionImage.x = 0;
                    positionImage.y = positionImage.y +11;
 
 
                }
 
 
                positionImage.x = 0;
                positionImage.y = 0;
 
 
            }
    }
                //Vidage mémoire
                SDL_FreeSurface(imagePlein);
                SDL_FreeSurface(imageVide);
 
 
    return 0;
}

Seulement j'ai un autre problèmes qui est survenue, maintenant au chargement du programme il m'affiche un écran noir, comme si il ne pouvait pas blitter les images précédemment chargé, est ce possible ? Il est probable que c'est moi qui est mal fait les modifications.

En tout cas, je suis ravie que vous m'ayez aidé car pour un prochain programme je saurais qu'il faut d'abord déclaré les images pour plus de rapidité.

Dans l'attente du réponse, Spiritus44

**LittleWhite** · 07/09/2014, 13h44

Vous avez vu cette ligne :

Code :

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

alea = rand() %(1-11);

Essayez donc d'afficher alea, par curiosité

**Spiritus44** · 07/09/2014, 14h39

J'ai beau afficher, tout fonctionne parfaitement, il tire un nombre compris entre 1 et 10 et si le nombre est 1 on met un 1 dans la grille sinon un 0. Il y aurait t'il une erreur que je n'est pas vu ? Et quand bien même une erreur de ma variable aléa ne pourrait pas me causer de problème dans l'affichage, non ?

**LittleWhite** · 07/09/2014, 14h47

C'est mon compilateur qui a trouvé la raison :

main2.cpp:125:13: attention : ‘tempsPrecedent’ may be used uninitialized in this function [-Wuninitialized]

En effet, une variable non initialisée peut être différente de zéro. Si vous ne lui donnez pas de valeur, c'est une valeur aléatoire, qui peut casser votre test !

**Spiritus44** · 07/09/2014, 15h01

Olala, merci de votre aide, vous aviez raison il fallait initialisé le tout à 0 ! Sauf que je m’interroge, pourquoi mon compilateur ne m'a rien dit ? Faut t'il que j'en change ?

Par contre mon Jeu de la vie ne marche toujours pas ( même après avoir enlevé les parenthèse ) , sans doute parce que j'ai fait n'importe quoi en écrivant les règles ^^. Si je ne trouves vraiment pas je reposterais un sujet.

**LittleWhite** · 07/09/2014, 15h04

Pour le jeu de la vie, c'est du simple débogage (Mettez des printf, ou validez le comportement avec le débogueur

)
On peut aussi simplifier les tests, il me semble.

Après, vous devriez afficher le "resultat"

ça vous aidera.

Pour le compilateur, il faut activer tous les niveaux d'avertissements. Sous GCC/G++, toujours compiler avec :

-Wall -Wextra

Sinon, il suffit de chercher dans les options du projet.

**Spiritus44** · 07/09/2014, 15h22

Bhon j'ai finis par trouvé merci LittleWhite, pour ton aide sans vous je ni serais jamais arrivé. Surtout que vos réponses était toujours très pertinente, Si jamais le code intéresse quelqu'un je le poste en dessous, pensez juste à faire 2 images vide.bmp et plein.bmp.

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
#ifdef __cplusplus
    #include <cstdlib>
#else
    #include <stdlib.h>
#endif
#include <iostream>
#include <SDL/SDL.h>
#include <ctime>
 
using namespace std;
 
 
int main ( int argc, char** argv )
{
 
 
    int maxLargeur,maxHauteur;
 
    //Initialisation SDL
    if ( SDL_Init( SDL_INIT_VIDEO ) < 0 )
    {
        printf( "Unable to init SDL: %s\n", SDL_GetError() );
        return 1;
    }
 
    // make sure SDL cleans up before exit
    atexit(SDL_Quit);
 
    SDL_Surface* screen,*imagePlein,*imageVide;
    SDL_Rect positionImage ;
 
 
    maxLargeur=SDL_GetVideoInfo()->current_w;
    maxHauteur=SDL_GetVideoInfo()->current_h;
 
    maxLargeur =500;
    maxHauteur =500;
 
    positionImage.x = 0;
    positionImage.y = 0;
 
    int maxX = maxHauteur/10;
    int maxY = maxLargeur/10;
    int grille[maxX][maxY];
    int alea = 0;
    srand(time(NULL));
    int j = 0;
 
    //Initialisation grille
    for(int i = 1; i < maxX; i++)
    {
        for(int j = 0; j < maxY; j++)
        {
 
            alea = rand() %(1-11);
 
 
            if(alea == 1)
            {
                grille[i][j]=1;
            }
            else
            {
                grille[i][j]=0;
            }
 
 
        }
    }
 
 
 
 
 
 
 
    // Création Fenêtre
 
// | SDL_FULLSCREEN
 
    screen=SDL_SetVideoMode(maxLargeur, maxHauteur, 32, SDL_HWSURFACE |SDL_DOUBLEBUF);
 
    if ( !screen )
    {
        printf("Unable to set 640x480 video: %s\n", SDL_GetError());
        return 1;
    }
 
 
 
 
    bool continuer = false;
 
    imagePlein = SDL_LoadBMP("plein.bmp");
    imageVide = SDL_LoadBMP("vide.bmp");
 
    // Gestion des erreurs, si problèmes on ferme tout
    if (!imagePlein)
    {
        fprintf( stderr,"Impossible de charger l'image : plein.bmp");
        exit(EXIT_FAILURE);
        continuer = true;
    }
    if (!imageVide)
    {
        fprintf( stderr,"Impossible de charger l'image : vide.bmp");
        exit(EXIT_FAILURE);
        continuer = true;
    }
 
    // Boucle principal
    while (!continuer)
    {
 
 
        SDL_Event event;
        while (SDL_PollEvent(&event))
        {
 
            switch (event.type)
            {
 
            case SDL_QUIT:
                continuer = true;
                break;
 
 
            case SDL_KEYDOWN:
                {
 
                    if (event.key.keysym.sym == SDLK_ESCAPE)
                        continuer = true;
                    break;
                }
            }
        }
            int tempsActuel = SDL_GetTicks();
            int tempsPrecedent = 0;
            if (tempsActuel - tempsPrecedent > 30) /* Si 30 ms se sont écoulées */
            {
 
                // Initialisation variable,
                int resultat ;
 
                // Affichage Map
                for(int i = 1; i < maxX; i++)
                {
                    for(int j = 0; j < maxY; j++)
                    {
                if(grille[i-1][j-1]==1 && i!=0 )
                {
                    resultat++;
                }
                if(grille[i-1][j]==1 && i!=0 )
                {
                    resultat++;
                }
 
                if(grille[i-1][j+1]==1 && i!=0 && j!=maxX)
                {
                    resultat++;
                }
 
 
 
                if(grille[i][j-1]==1 && j!=0)
                {
                    resultat++;
                }
                if(grille[i][j+1]==1 && j!=maxX )
                {
                    resultat++;
                }
 
 
                if(grille[i+1][j-1]==1 && i!=maxY && j!=0)
                {
                    resultat++;
                }
                if(grille[i+1][j]==1 && i!=maxY && j!=maxX)
                {
                    resultat++;
                }
                if(grille[i+1][j+1]==1 && i!=maxY && j!=maxX)
                {
                    resultat++;
                }
 
 
                if(grille[i][j]==0 && resultat==3)
                {
                    grille[i][j]=1;
                }
                else if(grille[i][j]==1 && resultat==2 or resultat==3)
                {
                    grille[i][j]=1;
                }
                else
                {
                    grille[i][j]=0;
                }
 
 
                resultat = 0;
 
 
                        if(grille[i][j]==0)
                        {
                            /* On blitte par-dessus l'écran */
                            SDL_BlitSurface(imageVide, NULL, screen, &positionImage);
                        }
                        else
                        {
                            SDL_BlitSurface(imagePlein, NULL, screen, &positionImage);
 
                        }
                        positionImage.x = positionImage.x +11;
 
                        //Affichage écran
                        SDL_Flip(screen);
 
                    }
 
                    positionImage.x = 0;
                    positionImage.y = positionImage.y +11;
 
 
                }
 
 
                positionImage.x = 0;
                positionImage.y = 0;
 
 
            }
    }
                //Vidage mémoire
                SDL_FreeSurface(imagePlein);
                SDL_FreeSurface(imageVide);
 
    return 0;
}

Et si jamais quelqu'un trouve un moyen d'accélérer la vitesse d'affichage je prends !

**LittleWhite** · 07/09/2014, 15h34

La vitesse d'affichage dépend intégralement de ceci :

Code :

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

1
2
3
4
int tempsActuel = SDL_GetTicks();
            int tempsPrecedent = 0;
            if (tempsActuel - tempsPrecedent > 30) /* Si 30 ms se sont écoulées */
            {

Donc vous pouvez l'améliorer vous même

Pour éviter l'utilisation des images, j'avais fait ceci :

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
SDL_Rect dst;
						dst.x = positionImage.x;
						dst.y = positionImage.y;
						dst.w = 11;
						dst.h = 11;
 
 
                        if(grille[i][j]==0)
                        {
                            SDL_FillRect(screen, &dst, SDL_MapRGB(screen->format,196,32,32));
                        }
                        else
                        {
                            SDL_FillRect(screen, &dst, SDL_MapRGB(screen->format,32,32,196));
                        }

**Spiritus44** · 07/09/2014, 15h53

Merci de ton aide, si je comprends bien tu blit un pixel plutôt qu'appliquer une image ? Est-ce que les performances sont sensiblement changé ou çà reste presque pareil ?

Invité · 07/09/2014, 15h54

Bonjour,

Attention à l'indentation du code.

~~Plutôt que de faire une case par trame, tu devrais faire toutes les cases à la fois~~, et passer par une grille temporaire afin de te baser sur l'état antérieur et non un mélange nouveau/ancien.

Les lignes du style if(grille[i-1][j+1]==1 && i!=0 && j!=maxX) : la vérification de la condition n'est pas à la bonne place. Tu évalues d'abord la case, au risque de dépasser, puis, tu vérifies que l'on ne dépasse pas du tableau.
Pour commencer, tu as des boucles du style for(int i = 1; i < maxX; i++) donc i ne sera jamais nul.
Et ton programme devrait en principe planter sur les lignes où il y a un +1. N'oublie pas que l'on va de 0 à maxX-1 et pas maxX.

La bonne méthode : on vérifie d'abord la condition sur l'indice, puis on y accède, en profitant de l'opérateur && qui n'évalue pas le second terme si le premier est faux.

Code :

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

1
2
3
if(i!=0 && grille[i-1][j]) ...
if(i<maxY-1 && grille[i+1][j]) ...
etc.

Je te conseille aussi de décomposer ton code et de petites fonctions.
Et +1 LittleWhite pour les SDL_FillRect

EDIT : N'oublie pas de libérer SDL_Surface* screen à la fin, ainsi que lors de l'échec de chargement des images (idem la première image si la seconde ne peut pas être chargée).

**LittleWhite** · 07/09/2014, 16h00

Envoyé par Spiritus44

Merci de ton aide, si je comprends bien tu blit un pixel plutôt qu'appliquer une image ? Est-ce que les performances sont sensiblement changé ou çà reste presque pareil ?

Cela affiche un rectangle coloré, simplement.

Pour les performances : http://alexandre-laurent.developpez....ces-de-la-SDL/

**TiranusKBX** · 07/09/2014, 16h03

ça évite surtout au programme de ré-envoyer l’intégralité de la surface à afficher

**Spiritus44** · 07/09/2014, 16h05

Oui concernant l'indentation du code ca devient un peu n'importe quoi a force de faire des copier coller de mon propre code, désolé de ce problème.

Quand tu dis : "Plutôt que de faire une case par trame, tu devrais faire toutes les cases à la fois, et passer par une grille temporaire afin de te baser sur l'état antérieur et non un mélange nouveau/ancien."

Mais comment je peux vérifier ou afficher l'état de toute les cases en même temps ? Le c++ est un langage linéaire, je ne devrait pas pouvoir tout vérifier en parallèle ?

Les modifications faites :

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
#ifdef __cplusplus
    #include <cstdlib>
#else
    #include <stdlib.h>
#endif
#include <iostream>
#include <SDL/SDL.h>
#include <ctime>
 
using namespace std;
 
 
int main ( int argc, char** argv )
{
 
 
    int maxLargeur,maxHauteur;
 
    //Initialisation SDL
    if ( SDL_Init( SDL_INIT_VIDEO ) < 0 )
    {
        printf( "Unable to init SDL: %s\n", SDL_GetError() );
        return 1;
    }
 
    // make sure SDL cleans up before exit
    atexit(SDL_Quit);
 
    SDL_Surface* screen,*imagePlein,*imageVide;
    SDL_Rect positionImage ;
 
 
    maxLargeur=SDL_GetVideoInfo()->current_w;
    maxHauteur=SDL_GetVideoInfo()->current_h;
 
    positionImage.x = 0;
    positionImage.y = 0;
 
    int maxX = maxHauteur/10;
    int maxY = maxLargeur/10;
    int grille[maxX][maxY];
    int alea = 0;
    srand(time(NULL));
    int j = 0;
 
    //Initialisation grille
    for(int i = 1; i < maxX; i++)
    {
        for(int j = 0; j < maxY; j++)
        {
 
            alea = rand() %(1-20);
 
 
            if(alea == 1)
            {
                grille[i][j]=1;
            }
            else
            {
                grille[i][j]=0;
            }
 
 
        }
    }
 
 
 
 
 
 
 
    // Création Fenêtre
 
 
    screen=SDL_SetVideoMode(maxLargeur, maxHauteur, 32, SDL_HWSURFACE | SDL_FULLSCREEN |SDL_DOUBLEBUF);
 
    if ( !screen )
    {
        printf("Unable to set 640x480 video: %s\n", SDL_GetError());
        return 1;
    }
 
 
 
 
    bool continuer = false;
 
    imagePlein = SDL_LoadBMP("plein.bmp");
    imageVide = SDL_LoadBMP("vide.bmp");
 
    // Gestion des erreurs, si problèmes on ferme tout
    if (!imagePlein)
    {
        fprintf( stderr,"Impossible de charger l'image : plein.bmp");
        exit(EXIT_FAILURE);
        continuer = true;
    }
    if (!imageVide)
    {
        fprintf( stderr,"Impossible de charger l'image : vide.bmp");
        exit(EXIT_FAILURE);
        continuer = true;
    }
 
    // Boucle principal
    while (!continuer)
    {
 
 
        SDL_Event event;
        while (SDL_PollEvent(&event))
        {
 
            switch (event.type)
            {
 
            case SDL_QUIT:
                continuer = true;
                break;
 
 
            case SDL_KEYDOWN:
                {
 
                    if (event.key.keysym.sym == SDLK_ESCAPE)
                        continuer = true;
                    break;
                }
            }
        }
 
        // Initialisation variable,
        Uint32 tempsPrecedent = SDL_GetTicks();
 
 
        int resultat ;
 
        // Affichage Map
        for(int i = 1; i < maxX-1; i++)
        {
            for(int j = 0; j < maxY; j++)
            {
                // Applications des règles du Jeu de la vie
 
                if(i!=0 && grille[i-1][j-1]==1)
                {
                    resultat++;
                }
                if( i!=0 && grille[i-1][j]==1)
                {
                    resultat++;
                }
 
                if(grille[i-1][j+1]==1 && i!=0 && j!=maxX)
                {
                    resultat++;
                }
 
 
 
                if(j!=0 && grille[i][j-1]==1 )
                {
                    resultat++;
                }
                if(j!=maxX && grille[i][j+1]==1 )
                {
                    resultat++;
                }
 
 
                if( j!=0 && grille[i+1][j-1]==1 && i!=maxY )
                {
                    resultat++;
                }
                if(i!=maxY && j!=maxX && grille[i+1][j]==1 )
                {
                    resultat++;
                }
                if(i!=maxY && j!=maxX && grille[i+1][j+1]==1 )
                {
                    resultat++;
                }
 
 
                if(resultat==3 && grille[i][j]==0 )
                {
                    grille[i][j]=1;
                }
                else if(resultat==2 or resultat==3 && grille[i][j]==1 )
                {
                    grille[i][j]=1;
                }
                else
                {
                    grille[i][j]=0;
                }
 
 
                resultat = 0;
 
 
                if(grille[i][j]==0)
                {
                    /* On blitte par-dessus l'écran */
                    SDL_BlitSurface(imageVide, NULL, screen, &positionImage);
                }
                else
                {
                    SDL_BlitSurface(imagePlein, NULL, screen, &positionImage);
                }
                positionImage.x = positionImage.x +10;
 
                //Affichage écran
                SDL_Flip(screen);
 
            }
 
                    positionImage.x = 0;
                    positionImage.y = positionImage.y +10;
        }
 
 
        positionImage.x = 0;
        positionImage.y = 0;
 
 
 
        //  Attente si -30ms
        Uint32 tempsActuel = SDL_GetTicks();
 
 
        if (tempsActuel - tempsPrecedent < 30)
        {
            SDL_Delay(tempsActuel - tempsPrecedent);
        }
 
    }
 
    //Vidage mémoire
    SDL_FreeSurface(imagePlein);
    SDL_FreeSurface(imageVide);
 
    return 0;
}

Malheureusement, ces modifications ont entrainé un problèmes inexpliqué au niveau de la création de la grille, en effet maintenant le programme créer des sorte de structure qui se répète pas a l'identique mais à chaque démarrage de programme :

Nom : 12.png
Affichages : 599
Taille : 288,5 Ko

Nom : 12.png
Affichages : 599
Taille : 288,5 Ko

Invité · 07/09/2014, 16h31

Envoyé par Winjerome

Plutôt que de faire une case par trame, tu devrais faire toutes les cases à la fois

Autant pour moi j'ai lu un peu trop vite ton code.

Pour ce qui est des anciens/nouveaux états, actuellement tu n'utilises qu'un seul tableau.
Ce qui fait que tu modifies ta première case (mettons qu'elle était à 0 et passe à 1), puis passes à la case suivante : ce qui inclus de vérifier l"état de la première... mais étant déjà modifiée, tu prends la nouvelle valeur 1, et pas l'ancienne 0.
Avec deux tableaux, tu vérifies les valeurs de l'ancien tableau qui restent inchangées et mets à jour celles du nouveau.

**Spiritus44** · 07/09/2014, 16h38

Je vois ce que tu veux dire, mais cela veut dire que je dois faire 2 tableaux, un ancien et un nouveau. Mais comment mettre cela en place ? Je fais un tableau grille et grilleAncienne, quand j'initialise je met tout dans grilleAncienne, dans la boucle je fais mes calculs et je met le tout dans grille, j'affiche puis je refais 2 boucles pour dire que grilleAncienne=grille et ca recommence ? Si il y a une erreur de logique dites le moi. Mais en faisant çà, ca ne risque pas de ralentir encore plus le tout ?

Invité · 07/09/2014, 17h08

C'est bien ça, alors oui ça prend évidement un peu de ressources, mais c'est rien du tout

Par contre ce qui te bouffe tout en ce moment c'est ton SDL_Flip(screen); au sein des deux boucles.
Déplace cette ligne en dehors et tu verras que ça ira beaucoup mieux

PS : ton code est un mélange affreux de C et C++

**Spiritus44** · 07/09/2014, 17h17

Ok, je change la place du sdl_screen et oui je sais que c'est un mélange affreux de c/c++ mais je débute, et je ne connais pas les équivalents c/c++ donc je fais avec les moyens du bords ^^. Si tu connais les équivalence, je veux bien.

**Spiritus44** · 07/09/2014, 17h45

Bon, je penses avoir finis et grace a vos précieux conseil, tout fonctionne et ce très rapidement !

Je poste le code au cas où quelqu'un en aurait besoin ( notez au passage que j'ai fait un effort pour l'indentation du code ).

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
#ifdef __cplusplus
    #include <cstdlib>
#else
    #include <stdlib.h>
#endif
#include <iostream>
#include <SDL/SDL.h>
#include <ctime>
 
using namespace std;
 
 
int main ( int argc, char** argv )
{
 
 
    int maxLargeur,maxHauteur;
 
    //Initialisation SDL
    if ( SDL_Init( SDL_INIT_VIDEO ) < 0 )
    {
        printf( "Unable to init SDL: %s\n", SDL_GetError() );
        return 1;
    }
 
    // make sure SDL cleans up before exit
    atexit(SDL_Quit);
 
    SDL_Surface* screen;
    SDL_Rect positionImage, dst ;
 
    maxLargeur=SDL_GetVideoInfo()->current_w;
    maxHauteur=SDL_GetVideoInfo()->current_h;
 
 
    positionImage.x = 0;
    positionImage.y = 0;
 
    int maxX = maxHauteur/10;
    int maxY = maxLargeur/10;
    int grille[maxX][maxY];
    int grilleAncienne[maxX][maxY];
    int alea = 0;
    srand(time(NULL));
    int j = 0;
 
    //Initialisation grille
    for(int i = 1; i < maxX; i++)
    {
        for(int j = 0; j < maxY; j++)
        {
 
            alea = rand() %(1-15);
 
 
            if(alea == 1)
            {
                grilleAncienne[i][j]=1;
            }
            else
            {
                grilleAncienne[i][j]=0;
            }
 
            grille[i][j]=0;
        }
    }
 
 
 
    // Création Fenêtre
 
    screen=SDL_SetVideoMode(maxLargeur, maxHauteur, 32, SDL_HWSURFACE | SDL_FULLSCREEN |SDL_DOUBLEBUF);
 
    if ( !screen )
    {
        printf("Unable to set 640x480 video: %s\n", SDL_GetError());
        return 1;
    }
 
 
    bool continuer = false;
 
 
    int tempsPrecedent = 0;
    // Boucle principal
    while (!continuer)
    {
 
        Uint32 tempsPrecedent = SDL_GetTicks();
 
        SDL_Event event;
        while (SDL_PollEvent(&event))
        {
 
            switch (event.type)
            {
 
            case SDL_QUIT:
                continuer = true;
                break;
 
 
            case SDL_KEYDOWN:
                {
 
                    if (event.key.keysym.sym == SDLK_ESCAPE)
                        continuer = true;
                    break;
                }
            }
        }
 
 
        // Initialisation variable,
        int resultat ;
 
        // Affichage Map
 
        for(int i = 1; i < maxX; i++)
        {
            for(int j = 0; j < maxY; j++)
            {
                //Calcul Cellule
                if(grilleAncienne[i-1][j-1]==1 && i!=0 )
                {
                    resultat++;
                }
                if(grilleAncienne[i-1][j]==1 && i!=0 )
                {
                    resultat++;
                }
 
                if(grilleAncienne[i-1][j+1]==1 && i!=0 && j!=maxX)
                {
                    resultat++;
                }
 
 
 
                if(grilleAncienne[i][j-1]==1 && j!=0)
                {
                    resultat++;
                }
                if(grilleAncienne[i][j+1]==1 && j!=maxX )
                {
                    resultat++;
                }
 
 
                if(grilleAncienne[i+1][j-1]==1 && i!=maxY && j!=0)
                {
                    resultat++;
                }
                if(grilleAncienne[i+1][j]==1 && i!=maxY && j!=maxX)
                {
                    resultat++;
                }
                if(grilleAncienne[i+1][j+1]==1 && i!=maxY && j!=maxX)
                {
                    resultat++;
                }
 
 
                if(grilleAncienne[i][j]==0 && resultat==3)
                {
                    grille[i][j]=1;
                }
                else if(grilleAncienne[i][j]==1 && resultat==2 or resultat==3)
                {
                    grille[i][j]=1;
                }
                else
                {
                    grille[i][j]=0;
                }
 
 
                resultat = 0;
 
                dst.x = positionImage.x;
                dst.y = positionImage.y;
                dst.w = 10;
                dst.h = 10;
 
                if(grille[i][j]==0)
                {
                    SDL_FillRect(screen, &dst, SDL_MapRGB(screen->format,255,255,255));
                }
                else
                {
                    SDL_FillRect(screen, &dst, SDL_MapRGB(screen->format,196,32,32));
                }
                positionImage.x = positionImage.x +10;
 
 
 
            }
 
            positionImage.x = 0;
            positionImage.y = positionImage.y +10;
 
 
        }
 
        //Affichage écran
        SDL_Flip(screen);
        positionImage.x = 0;
        positionImage.y = 0;
 
        //Changement grille->grilleAncienne
 
        for(int i = 1; i < maxX; i++)
        {
            for(int j = 0; j < maxY; j++)
            {
                grilleAncienne[i][j]=grille[i][j];
            }
        }
 
        Uint32 tempsActuel = SDL_GetTicks();
 
        if (tempsActuel - tempsPrecedent < 1000)
        {
            SDL_Delay(150);
        }
    }
    return 0;
}

Merci de votre aide !

Jeu de la vie [SDL 1.2]

SDL

Discussions similaires

Partager

Partager