Problème avec les réfèrences

pour le passage à une fonction

11 mars 2008 à 18:38:27

Bonjour tout le monde !

Je commence le C++ depuis peu et j'ai un petit problème avec les réfèrences :
Dans ma fonction main (), j'ai un pointeur de type SDL_Surface et je voudrais passer ce pointeur en paramètre à une fonction qui va lui allouer de la mémoire afin que je puisse l'utiliser dans le reste de mon main (). o_O

En gros j'ai fait ceci :

#include <iostream>
#include <SDL/SDL.h>
#define COULEUR SDL_MapRGB (ecran->format, 220, 220, 220)
using namespace std;
void InitialiserEcran (SDL_Surface *&ecran, char *titre, int w, int h, Uint32 couleur);
int main (int argc, char* argv [])
{
SDL_Init (SDL_INIT_VIDEO);
SDL_Surface *ecran = NULL;
InitialiserEcran (ecran, "MonTitre", 450, 275, COULEUR);
SDL_Flip (ecran);
SDL_Quit ();
return EXIT_SUCCESS;
}
void InitialiserEcran (SDL_Surface *&ecran, char *titre, int w, int h, Uint32 couleur)
{
ecran = SDL_SetVideoMode (w, h, 32, SDL_HWSURFACE);
SDL_WM_SetCaption (titre, NULL);
SDL_FillRect (ecran, NULL, couleur);
}

Mais cela ne marche pas et je ne voit pas la solution !

Donc si vous pouviez m'aider, ce serait super sympa !

Merci d'avance !

Anonyme

12 mars 2008 à 15:03:34

void InitialiserEcran (SDL_Surface *&ecran, char *titre, int w, int h, Uint32 couleur)

je croix que sa serai plutot &ecran et pas *&ecran enfin tu peut esseyer.

Ensuite ton #define COULEUR SDL_MapRGB (ecran->format, 220, 220, 220)
sa fait pas trop c++ mais plutot c en c++ on utilise plutot inline
voila un exemple avec inline a toi de l'adapté sa fait plus propre que avec ton define
http://www.ann.jussieu.fr/courscpp/Sections/Sect05-D0.html

MatteX

12 mars 2008 à 15:17:19

Est-ce qu'il y a une erreur?

Parce que je crois que ton #define COULEUR va te poser problème...

@prsieux -> sont passage de pointeur par référence est bon puisqu'il cherche à modifier la valeur d'une adresse pointée et qu'elle soit conservée au retour de la fonction.

Pour ce qui est de la corrélation des #define et des fonction inline cela ne s'applique pas ici. Parce qu'il veut pourvoir remplacer "COULEUR" par le texte qui suis... Il n'a pas défini une fonction.

Reste qu'un #define ça ne fait pas très C++ et que je crois que tu as une mauvaise définition de ce qu'il va arriver avec ton code.

Essai de faire :

Uint32 couleur = SDL_MapRGB( /*...*/ );
void InitialiserEcran ( /*...*/, couleur );

Voir si tu as le même problème.

groscask

12 mars 2008 à 19:32:52

Je ne suis pas sur que les références sur pointeurs soient d'un très bon goût...
Essaye avec un bon vieux

Init_Ecran(SDL_Surface ** ecr, /*... */);
/* MAIN */
SDL_Surface * ecran = NULL;
Init_Ecran(&ecran, /* ... */);

Prend bien garde à déréférencer ecran dans la fonction par contre.

chlablak

12 mars 2008 à 20:09:19

pourquoi ne pas faire tout simplement 'Init_Ecran(SDL_Surface*, /* ... */);'

C++ Références | Idiomes | FAQ

groscask

12 mars 2008 à 21:14:00

A tout hasard parce que SDL_SetVideoMode() va modifier la valeur du pointeur ?

chlablak

12 mars 2008 à 21:16:14

c'est le but

C++ Références | Idiomes | FAQ

MatteX

13 mars 2008 à 14:18:48

Chlab_lak, si tu fais ça, au retour de ta fonction, l'ecran va encore valloir 0 et l'adresse retournée par SDL_SetVideoMode() sera perdue!

groscask : Que ce soit un pointeur de pointeur (**) ou une référence sur pointeur (*&) c'est du pareille au même au final. Sauf que (*&) permet dévité les opérations supplémentaires de déréférencement et de référencement à l'appel et à l'intérieur de la fonction.

chlablak

13 mars 2008 à 16:19:18

<rien>

C++ Références | Idiomes | FAQ

groscask

13 mars 2008 à 22:44:52

Tout à fait d'accord MatteX...

Considère ça comme les vieilles habitudes d'un programmeur C =)

(même si je me sers massivement des références... hein ! Faut vivre avec son temps vindju)

int21h

14 mars 2008 à 9:13:54

Et pourquoi ne pas tout simplement renvoyer le pointeur ?

SDL_Surface * InitialiserEcran (const std::string & titre, int w, int h, Uint32 couleur);

Personnellement je trouve cela beaucoup plus élégant, et plus pratique.

Mettre à jour le MinGW Gcc sur Code::Blocks. Du code qui n'existe pas ne contient pas de bug