Discussion :

[bringer]

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 :

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
#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

[Invité(e)]

Bonjour,

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

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
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 :

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
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;
        }
    }
}

[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 :

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

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

[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.

[bringer]

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 :

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

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;
}

[Invité(e)]

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

Code :

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

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

[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.

[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...