probleme avec une structure contenant un tableau

Version imprimable

bonjour à tous,

je débute ne c, et je développe un sokoban d'entrainement. La partie logique est fonctionnelle et je m'attaque maintenant à la partie graqhique.
Je créé pour cela une structure de tuile pour créé une map.
voici le code :
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 #include <stdio.h> #include <stdlib.h> #include "SDL/SDL.h" #include "constantes.h" typedef struct S_Tile { SDL_Surface *image; int symbole; }S_Tile; typedef struct S_Map { S_Tile map[HEIGHT_ARRAY][WIDTH_ARRAY]; }S_Map; S_Map* initMap(void); void displayMap(const S_Map *level); int main() { S_Map *level; level=initMap(); displayMap(level); return EXIT_SUCCESS; } S_Map* initMap(void) { S_Map *initLevel={NULL}; int i, j; for (i=0; i<HEIGHT_ARRAY; i++) { for (j=0; j<WIDTH_ARRAY; j++) { initLevel->map[i][j].image=NULL; } } return initLevel; } void displayMap(const S_Map *levelMap) { int i, j; for (i=0; i<HEIGHT_ARRAY; i++) { for (j=0; j<WIDTH_ARRAY; j++) { printf("map : initlevel.map[%d][%d]=%p\n", i,j,levelMap->map[i][j].image); } } }
Pour des raison de faciliter, le fait des allocation statique de tableau.
HEIGHT_ARRAY et WIDTH_ARRAY valent 12.
A la compilation, aucun message d'avertissement ni à l'édition de lien.
Par contre à l'execution, je reçoit un segment fault avec GCC 4.2 sur ubuntu.

Quelqu'un pourrait-il m'expliquer le pourquoi ?
Merci d'avance

Bonjour,

Il faut réserver de la mémoire pour la structure qui contient ton tableau (dans ce cas penser au free):

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
int main()
{
    S_Map *level;
    level = initMap();
    displayMap(level);
 
    return EXIT_SUCCESS;
}
 
S_Map *initMap(void)
{
    S_Map *initLevel = malloc(sizeof*initLevel);
    int i, j;
    for (i = 0; i < HEIGHT_ARRAY; i++) {
        for (j = 0; j < WIDTH_ARRAY; j++) {
            initLevel->map[i][j].image = NULL;
        }
    }
    return initLevel;
}

OU tout faire en statique (c'est moins élégant) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
int main()
{
    S_Map level;
    initMap(&level);
    displayMap(&level);
 
    return EXIT_SUCCESS;
}
 
 
void initMap(S_Map *initLevel)
{
    int i, j;
    for (i = 0; i < HEIGHT_ARRAY; i++) {
        for (j = 0; j < WIDTH_ARRAY; j++) {
            initLevel->map[i][j].image = NULL;
        }
    }
}

31/05/2010, 13h21
ram-0000
Tu n'alloues pas de structure S_map dans ton programme, c'est pour cela que cela plante.

il faudrait un truc du genre :
Code:

1 2 3 4 5 6 7 8 S_Map* initMap(void) { S_Map *initLevel= malloc(sizeof(S_Map)); if(initLevel == NULL) { /* traitement de l'erreur malloc } ...
[Edit] Grillé par mabu :aie:
31/05/2010, 13h31
bringer

merci à tous les 2,

je pensais que le fait d'avoir déclarer un tableau statique au lieu d'un pointeur double dans la structure permettait de s'affranchir d'un malloc/free.
En fait, il faut comprendre que la déclaration de structure ne fait que déclarer la structure et n'alloue aucune ressource, c'est bien ça?

merci encore.

a oui, et pendant qu'on y est.

dans ma fonction initMap(), j'ai placer le free juste avant le return level;
On pourrait penser que cela entrainerait la perte de la valeur de l'adresse mémoire.
Comment cela fonctionne t-il ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 S_Map* initMap(void) { S_Map *initLevel=malloc(sizeof(S_Map)); int i, j; for (i=0; i<HEIGHT_ARRAY; i++) { for (j=0; j<WIDTH_ARRAY; j++) { initLevel->map[i][j].image=NULL; } } free(initLevel); return initLevel; }

Le free est a appeler quand ton application n'a plus besoin du tout du tableau : à la fin du main :
Code:

1 2 3 4 5 6 7 8 9 10 int main() { S_Map *level; level = initMap(); displayMap(level); free(level); return EXIT_SUCCESS; }

31/05/2010, 14h04
bringer

oui, mais là, je parle du level initialiser dans la fonction initMap et pas du levelMap déclarer dans le main.
D'où, la question du free.
31/05/2010, 14h07
haraelendil

Vu que ta fonction renvoie un pointeur, elle ne recopie pas ton tableau, elle renvoie juste son adresse, donc les données en mémoire sont les mêmes dans ta fonction et dans ton main, donc si tu fait ton free, le pointeur dans ton main ne pointera plus sur rien, et donc segfault...
31/05/2010, 16h18
bringer

merci beaucoup à tous, vous venez de ma permettre de comprendre un peu plus la notion de pointeur.:ccool: