Problème fonction SDL

Version imprimable

Bonsoir, j'ai fini de coder le jeu mario sokoban mais quand je compile j'ai une (pour l'instant ) erreur :

fonction
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void moveBox(int *first_frame, int *second_frame) { if(*first_frame== BOX || *first_frame== BOX_OK) { if(*second_frame== TARGET) *second_frame== BOX_OK; else *second_frame==BOX; if (*first_frame== BOX_OK) *first_frame= TARGET; else *first_frame= BLANK; } }
quand j'utilise la fonction

Code:

moveBox(&map[pos->x][pos->y-1], &map[pos->x][pos->y-1]);

j'ai ce message d'erreur: Undefinied reference to '_moveBox'
Je ne comprend pas.

La déclaration de la fonction est faite après l'appelle mais j'ai mis le prototype en haut.(l'erreur ne viens pas de là je pense)

Merci d'avance.

13/11/2009, 21h31
Pouet_forever

Ton prototype de fonction ne doit pas correspondre à la fonction elle même.
Vérifie que les 2 sont bien identiques, sinon montre nous plus de code :)

Un petit détail dans ton code, fais attention à bien faire la différence entre = et ==, le premier est fait pour l'affectation, et le second pour la comparaison :ccool:
Un autre détail aussi, tu envoies à ta fonction moveBox la même valeur pour first_frame et second_frame ;)
13/11/2009, 21h38
Bobsleigh
Vite comme ça, tu devrais essayer
Code:

1 2 moveBox(&(map[pos->x][pos->y-1]), &(map[pos->x][pos->y-1]));
pour être sûr que tu envoies bien l'adresse de la case qui t'intéresse, et non pas (&map)[pos->x][pos->y-1]. Je ne suis pas sûr de la priorité des opérateurs ici, se serait à vérifier, mais je crois que ça change quelque chose d'ajouter des parenthèses.

Assure toi que ton code compile en C, aussi. Je connais pas vraiment le C++, mais il me semble que les références s'utilisent aussi avec l'opérateur &.
13/11/2009, 21h50
SofEvans
Citation:
Envoyé par Bobsleigh

Vite comme ça, tu devrais essayer

Code:

1 2 moveBox(&(map[pos->x][pos->y-1]), &(map[pos->x][pos->y-1]));

pour être sûr que tu envoies bien l'adresse de la case qui t'intéresse, et non pas (&map)[pos->x][pos->y-1]. Je ne suis pas sûr de la priorité des opérateurs ici, se serait à vérifier, mais je crois que ça change quelque chose d'ajouter des parenthèses.
Code:

1 2 moveBox(&map[pos->x][pos->y-1], &map[pos->x][pos->y-1]);
Citation:

Priorité des opérateurs
() []
-- ++ ! ~ -
* / %
+ -
<< >>
< <= >= >
== !=
&
^
|
&& ||
?:
= += -= *= /= %= <<= >>= &= ^= |=

Citation:

Envoyé par Bobsleigh

Assure toi que ton code compile en C, aussi. Je connais pas vraiment le C++, mais il me semble que les références s'utilisent aussi avec l'opérateur &.

Pour tester cela, ajouter au debut de main.c :
Code:

1 2 3 4 #ifdef _cplusplus #error Compilation avec un compilateur C++ pour du code C #endif
Pour le coup des référence, c'est exact. Mais en general, on ne les trouve que dans les prototype/declaration de fonction du style :
Code:

1 2 void FonctionBidon (Class &Instance, ...);

Merci pour vos réponses , j'ai résolu ce problème (il y avait les == et =) mais je n'avais pas vus que les déclarations des fonctions movePlayer et moveBox étaient dans la fonction play.
Mais maintenant le fenêtre s"ouvre et se referme tout de suite ( j'ai crée une fonction pause)

main.c

Code:

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
#include <stdio.h>
#include <stdlib.h>
#include <SDL/SDL.h>
#include <SDL_image.h>
 
#include "constantes.h"
#include "jeu.h"
#include "editeur.h"
 
int main(int argc, char *argv[])
{
    SDL_Surface *screen=NULL, *menu=NULL;
    SDL_Rect position_menu;
    SDL_Event event;
    int continu=1;
 
 
    SDL_Init(SDL_INIT_VIDEO);
 
    SDL_WM_SetIcon(IMG_Load("caisse.jpg"),NULL);
 
    SDL_SetVideoMode(WINDOW_WIDTH, WINDOW_HEIGHT, 32, SDL_HWSURFACE | SDL_DOUBLEBUF);
    SDL_WM_SetCaption("Mario Sokoban", NULL);
 
    menu=IMG_Load("menu.jpg");
    position_menu.x=0;
    position_menu.y=0;
 
    while(continu)
    {
        SDL_WaitEvent(&event);
        switch(event.type)
        {
            case SDL_QUIT:              /*On qui le programme*/
                continu=0;
                break;
            case SDL_KEYDOWN:
                switch(event.key.keysym.sym)
                {
                    case SDLK_ESCAPE:   /*On quitte le programme*/
                        continu=0;
                        break;
                    case SDLK_KP1:      /*On veut jouer*/
                        play(screen);
                        break;
                    case SDLK_KP2:      /*On veut aller à l'éditeur de niveaux*/
                        editor(screen);
                        break;
                }
        }
        /*Effacement de l'écran*/
        SDL_FillRect(screen, NULL, SDL_MapRGB(screen->format, 0, 0, 0));
        SDL_BlitSurface(menu, NULL, screen, &position_menu);
        SDL_Flip(screen);
 
    }
 
    SDL_FreeSurface(menu);
    SDL_Quit();
 
    return EXIT_SUCCESS;
}

jeu.c

Code:

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
#include <stdio.h>
#include <stdlib.h>
#include <SDL/SDL.h>
#include <SDL_image.h>
 
#include "constantes.h"
#include "jeu.h"
 
void play(SDL_Surface* screen)
{
    SDL_Surface *mario[4]={NULL};
    SDL_Surface *wall=NULL, *box=NULL, *boxOK=NULL, *target=NULL, *current_mario=NULL;
    SDL_Rect position, player_position;
    SDL_Event event;
 
    int continu=1, left_target=0, i=0, j=0;
    int map[NB_BLOCKS_WIDTH][NB_BLOCKS_HEIGHT]={0};
 
    /*Chargement des sprites*/
    wall = IMG_Load("mur.jpg");
    box = IMG_Load("caisse.jpg");
    boxOK = IMG_Load("caisse_ok.jpg");
    target = IMG_Load("objectif.png");
    mario[DOWN] = IMG_Load("mario_bas.gif");
    mario[LEFT] = IMG_Load("mario_gauche.gif");
    mario[UP] = IMG_Load("mario_haut.gif");
    mario[RIGHT] = IMG_Load("mario_droite.gif");
 
    current_mario=mario[DOWN];
 
    player_position.x=11;
    player_position.y=11;
 
    for(i=0;i<NB_BLOCKS_HEIGHT;i++)
    {
        for(j=0;j<NB_BLOCKS_WIDTH;j++)
        {
            if(map[i][j]==MARIO)
            {
                player_position.x=i;
                player_position.y=j;
                map[i][j]=BLANK;
            }
        }
    }
 
    SDL_EnableKeyRepeat(100,100);
    while(continu)
    {
        SDL_WaitEvent(&event);
        switch(event.type)
        {
            case SDL_QUIT:
                continu=0;
                break;
            case SDL_KEYDOWN:
                switch(event.key.keysym.sym)
                {
                    case SDLK_ESCAPE:
                        continu=0;
                        break;
                    case SDLK_UP:
                        current_mario = mario[UP];
                        movePlayer(map,&player_position,UP);
                        break;
                    case SDLK_DOWN:
                        current_mario = mario[DOWN];
                        movePlayer(map,&player_position,DOWN);
                        break;
                    case SDLK_LEFT:
                        current_mario = mario[LEFT];
                        movePlayer(map,&player_position,LEFT);
                        break;
                    case SDLK_RIGHT:
                        current_mario = mario[RIGHT];
                        movePlayer(map,&player_position,RIGHT);
                        break;
                }
                break;
        }
    }
    /*Effacement de l'écran*/
    SDL_FillRect(screen,NULL,SDL_MapRGB(screen->format,255,255,255));
 
    /*Placement des oblets sur l'écran*/
    left_target=0;
 
    for(i=0;i<NB_BLOCKS_WIDTH;i++)
    {
        for(j=0;j<NB_BLOCKS_HEIGHT;j++)
        {
            position.x= i*BLOCK_SIZE;
            position.y= j*BLOCK_SIZE;
            switch(map[i][j])
            {
                case WALL:
                    SDL_BlitSurface(wall,NULL,screen,&position);
                    break;
                case BOX:
                    SDL_BlitSurface(box,NULL,screen,&position);
                    break;
                case BOX_OK:
                    SDL_BlitSurface(boxOK,NULL,screen,&position);
                    break;
                case TARGET:
                    SDL_BlitSurface(target,NULL,screen,&position);
                    break;
            }
        }
    }
 
    if(!left_target)
        continu=0;
 
    position.x= player_position.x * BLOCK_SIZE;
    position.y= player_position.y * BLOCK_SIZE;
    SDL_BlitSurface(current_mario,NULL,screen,&position);
 
    SDL_Flip(screen);
 
    /*Désactivation de la répétition des touches*/
    SDL_EnableKeyRepeat(0,0);
 
    /*Libération des surfaces chargées*/
    SDL_FreeSurface(wall);
    SDL_FreeSurface(box);
    SDL_FreeSurface(boxOK);
    SDL_FreeSurface(target);
    for(i=0;i<4;i++)
        SDL_FreeSurface(mario[i]);
 
}
 
    void movePlayer(int map[][NB_BLOCKS_HEIGHT], SDL_Rect *pos, int direction)
    {
        switch(direction)
        {
            case UP:
                if(pos->y-1<0)/*Si le joueur dépasse l'écran, on arrête*/
                    break;
                if(map[pos->x][pos->y-1]==WALL)/*S'il y a un mur, on arrête*/
                    break;
                /*On vérifie qu'il n'y a pas un mur, ou la fin du monde ou une autre caisse devant la caisse qu'on veut bouger*/
                if(map[pos->x][pos->y-1]==BOX || map[pos->x][pos->y-1]==BOX_OK && map[pos->x][pos->y-2]==BOX || map[pos->x][pos->y-2]==BOX_OK)
                    break;
                /*On déplace une caisse*/
                moveBox(&map[pos->x][pos->y-1], &map[pos->x][pos->y-1]);
                /*On fait monter le joueur*/
                pos->y--;
                break;
            case DOWN:
                if(pos->y+1>NB_BLOCKS_HEIGHT*BLOCK_SIZE)
                    break;
                if(map[pos->x][pos->y+1]==WALL)
                    break;
                /*On vérifie qu'il n'y a pas un mur, ou la fin du monde ou une autre caisse dèrriere la caisse qu'on veut bouger*/
                if((map[pos->x][pos->y+1]==BOX || map[pos->x][pos->y+1]==BOX_OK) && (map[pos->x][pos->y+2]==BOX || map[pos->x][pos->y+2]==BOX_OK))
                    break;
                /*On deplace une caisse*/
                pos->y++;
                break;
            case LEFT:
                if(pos->x-1<0)
                    break;
                if(map[pos->x-1][pos->y]==WALL)
                    break;
                /*On vérifie qu'il n'y a pas un mur, ou la fin du monde ou une autre caisse dèrriere la caisse qu'on veut bouger*/
                if((map[pos->x-1][pos->y]==BOX || map[pos->x-1][pos->y]==BOX_OK) && (map[pos->x-2][pos->y]==BOX || map[pos->x-2][pos->y]==BOX_OK))
                    break;
                /*On deplace une caisse*/
                pos->x--;
                break;
            case RIGHT:
                if(pos->x+1>NB_BLOCKS_WIDTH*BLOCK_SIZE)
                    break;
                if(map[pos->x+1][pos->y]==WALL)
                    break;
                /*On vérifie qu'il n'y a pas un mur, ou la fin du monde ou une autre caisse dèrriere la caisse qu'on veut bouger*/
                if((map[pos->x+1][pos->y]==BOX || map[pos->x+1][pos->y]==BOX_OK) && (map[pos->x+2][pos->y]==BOX || map[pos->x+2][pos->y]==BOX_OK))
                    break;
                /*On deplace une caisse*/
                pos->x++;
                break;
        }
    }
 
    int *first_frame, *second_frame;
 
    void moveBox(int *first_frame, int *second_frame)
    {
        if(*first_frame==BOX || *first_frame==BOX_OK)
        {
            if(*second_frame==TARGET)
                *second_frame=BOX_OK;
            else
                *second_frame=BOX;
            if (*first_frame==BOX_OK)
                *first_frame=TARGET;
            else
                *first_frame=BLANK;
        }
    }

Tu devrais verifier le retour des fonctions SDL. S'il y a une erreur, affiche la avec SDL_GetError() qui renvoie un char* . Generalement, on obtient ce type de code :
Code:

1 2 3 4 5 6 if(SDL_Init(SDL_INIT_VIDEO) == -1) // Si l'initiation renvoie une erreur { fprintf(stderr , "Erreur d'initialisation de la SDL : %s\n" , SDL_GetError() ); return EXIT_FAILURE; }
Fait de meme en t'aidant de la doc pour connaitre les code d'erreur des fonctions

SDL_Init
SDL_SetVideoMode
IMG_Load

Autre chose, la SDL_Surface retourné par SDL_SetVideoMode est en realité une variable globale (plus ou moins, mais dans les grandes ligne c'est ca).
Pour la recuperer, utilise :

SDL_GetVideoSurface();

Ca te permettra de ne pas passer la SDL_Surface screen a tout bout de champ.

13/11/2009, 22h44
kimimsc

Je n'ai pas de message d'erreur ,

je ne comprend pas pourquoi tu me dis d'utiliser SDL_GetVideoSurface();
13/11/2009, 22h50
SofEvans
Citation:

Envoyé par kimimsc

Je n'ai pas de message d'erreur ,

je ne comprend pas pourquoi tu me dis d'utiliser SDL_GetVideoSurface();

Post ton code avec ta gestion des erreurs.

Pour SDL_GetVideoSurface : Apparemment, tu n'as pas compris quand je t'ai expliquer, alors voici un petit exemple :
Code:

1 2 3 4 5 void play(SDL_Surface* screen) { ... }
Devient :
Code:

1 2 3 4 5 6 void play(void) { SDL_Surface* screen = SDL_GetVideoSurface(); ... }
Voila, ca sert juste a utiliser screen uniquement la ou tu le veux et a alleger les fonctions.
13/11/2009, 23h33
kimimsc

ah d'accord merci
14/11/2009, 13h05
kimimsc

j'avais oublié de mettre screen=...
mais j'ai encore un autre problème la fenêtre s'ouvre mais quand j'appuie sur 1 pour jouer la fenêtre ne change pas , je reste dans le menu mais quand je veux fermer la fenêtre j'ai le terrain de jeu qui s'affiche rapidement et la fenêtre se referme
14/11/2009, 13h14
Pouet_forever

Regarde comment tu as placé ta boucle while(continu) :ccool:
Tu ne blittes rien dedans :aie:
14/11/2009, 19h47
SofEvans

En rapport a une remarque de kimimsc :

C'est normal que le compilateur te colore en blue le mot "continue", c'est un mot clé au meme titre que "int", "char", "struct", ....

Pour info, "continue" a l'effet contraire de "break".
14/11/2009, 20h07
kimimsc

Merci pour cette précision, pour les tutoriels SDL, et pour les tutoriels C :bravo: