Discussion :

[Kr00pS]

Bonjour !

Je rencontre un problème quand je veux "encapsuler" mes surfaces (comme la surface de la fenêtre) dans des structures.

J'ai deux structures :

Code :

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

1
2
3
4
5
6
7
8
9
10
typedef struct
{
    SDL_Surface *Perso, *Ennemi;
} Exemple2;
 
typedef struct
{
    SDL_Surface *Ecran;
    Exemple2 *Ex2;
} Exemple1;

Le code complet (exemple) :

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
#include <SDL/SDL.h>
 
typedef struct
{
    SDL_Surface *Perso, *Ennemi;
} Exemple2;
 
typedef struct
{
    SDL_Surface *Ecran;
    Exemple2 *Ex2;
} Exemple1;
 
void Exemple1_InitialiserFenetre(Exemple1 *this)
{
    this->Ecran = SDL_SetVideoMode(640,480,32, SDL_HWSURFACE);
}
 
void Exemple2_InitialiserChamps(Exemple1 *this)
{
    this->Ex2->Perso = SDL_CreateRGBSurface(SDL_HWSURFACE, 25, 25, 32, 0,0,0,0);
    this->Ex2->Ennemi = SDL_CreateRGBSurface(SDL_HWSURFACE, 25, 25, 32, 0,0,0,0);
}
 
int main(int argc, char *argv[])
{
    SDL_Init(SDL_INIT_VIDEO);
 
    Exemple1 *p = NULL;
 
    Exemple1_InitialiserFenetre(p);
    Exemple2_InitialiserChamps(p);
 
    SDL_Quit();
 
    return 0;
}

Ce code ne marche pas.

Je sais qu'il faut modifier un pointeur sur SDL_Surface donc il me faut un pointeur sur le pointeur de la SDL_Surface. Mais quand c'est imbriqué dans une structure (voir plusieurs), j'en ai aucune idée.

J'ai déjà essayé beaucoup de choses, mais je vois pas. :/

Merci pour votre aide.
Kr00pS

PS: Obligé de coder en C, en C++ j'aurais pu utiliser une référence, ce qui aurait simplifier la chose mais bon.

[Eusebe]

A priori, ton problème doit venir du fait que tu n'as pas alloué de mémoire pour tes objets.

Mais pourquoi dans ta structure 1 avoir un pointeur sur ta structure 2, et dans ton main créer un pointeur vers une structure 1 et pas simplement directement une structure 1 ?

[Kr00pS]

A priori, ton problème doit venir du fait que tu n'as pas alloué de mémoire pour tes objets.

Je peux pas faire ça avec une structure automatique alors ? Bien dommage.

Mais pourquoi dans ta structure 1 avoir un pointeur sur ta structure 2, et dans ton main créer un pointeur vers une structure 1 et pas simplement directement une structure 1 ?

Tout simplement car dans mon code Exemple1 est une structure qui regroupe tout. Il y aura forcément des structures imbriquées.

Merci de ta réponse.

[Eusebe]

Ce que je voulais dire, c'est pourquoi utiliser des pointeurs vers tes structures, et pas, par exemple :

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
typedef struct
{
    SDL_Surface *Perso, *Ennemi;
} Exemple2;
 
typedef struct
{
    SDL_Surface *Ecran;
    Exemple2 Ex2;
} Exemple1;
 
void Exemple1_InitialiserFenetre(Exemple1 *this)
{
    this->Ecran = SDL_SetVideoMode(640,480,32, SDL_HWSURFACE);
}
 
void Exemple2_InitialiserChamps(Exemple1 *this)
{
    this->Ex2.Perso = SDL_CreateRGBSurface(SDL_HWSURFACE, 25, 25, 32, 0,0,0,0);
    this->Ex2.Ennemi = SDL_CreateRGBSurface(SDL_HWSURFACE, 25, 25, 32, 0,0,0,0);
}

Et dans ton main, il suffit de faire :

Code :

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

1
2
3
4
    Exemple1 ex1;
 
    Exemple1_InitialiserFenetre(&ex1);
    Exemple2_InitialiserChamps(&ex1);

Mais bon, je ne suis pas un spécialiste du C...

[Kr00pS]

Ce que je voulais dire, c'est pourquoi utiliser des pointeurs vers tes structures, et pas, par exemple :

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
typedef struct
{
    SDL_Surface *Perso, *Ennemi;
} Exemple2;
 
typedef struct
{
    SDL_Surface *Ecran;
    Exemple2 Ex2;
} Exemple1;
 
void Exemple1_InitialiserFenetre(Exemple1 *this)
{
    this->Ecran = SDL_SetVideoMode(640,480,32, SDL_HWSURFACE);
}
 
void Exemple2_InitialiserChamps(Exemple1 *this)
{
    this->Ex2.Perso = SDL_CreateRGBSurface(SDL_HWSURFACE, 25, 25, 32, 0,0,0,0);
    this->Ex2.Ennemi = SDL_CreateRGBSurface(SDL_HWSURFACE, 25, 25, 32, 0,0,0,0);
}

Et dans ton main, il suffit de faire :

Code :

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

1
2
3
4
    Exemple1 ex1;
 
    Exemple1_InitialiserFenetre(&ex1);
    Exemple2_InitialiserChamps(&ex1);

Mais bon, je ne suis pas un spécialiste du C...

On m'a toujours dis qu'en C, passer des structures sans pointeur, c'est mal.

[Eusebe]

Si tu tiens à utiliser des pointeurs, il faut que tu alloues de l'espace mémoire.
Voir http://c.developpez.com/faq/c/?page=pointeurs.

Quelque chose comme ça (non testé) :

Code :

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

1
2
Exemple1 *p = malloc(sizeof *p);
p->Ex2 = malloc(sizeof *(p->Ex2));

...Et il faudra penser aussi à libérer cet espace...

[Eusebe]

On m'a toujours dis qu'en C, passer des structures sans pointeur, c'est mal.

Une précision : dans ma proposition, je ne passais pas des structures, mais bien un pointeur vers une structure.

Si "c'est mal" de ne pas utiliser des pointeurs pour passer des structures, je suppose que c'est parce que ça entraîne une duplication de la structure, et que de ce fait les modifications du paramètre ne changent rien sur la structure passée en paramètre.

Mais il existe des cas où au contraire c'est ce que l'on recherche (sémantique par valeur)...