Discussion :

[insomniak]

Alors voila, dans mon programme, j'ai fait ceci pour tester la position de mes sprites :

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
int Menu::Pos(int numSprite, int type)
{
	if((numSprite < taille) && (image[numSprite] != NULL))
	{
    int x = (int)&rect[numSprite].x;
    int y = (int)&rect[numSprite].y;
    int *returnx = &x;
    int *returny = &y;
    if (type == 1)
    {
         return -(*returnx);
    }
    if (type == 2)
    {
         return -(*returny);
    }
    }
}

Et dans mon programme, il y a ça :

Code :

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

1
2
     PrintSDL(font,screen,500,10,"positionx : %d", menu.Pos(i, 1));
     PrintSDL(font,screen,500,30,"positiony : %d", menu.Pos(i, 2));

(printsdl est une fonction qui affiche du texte. elle n'est pas de moi)

Et il m'affiche 0 !!! Meme si le rect est affiché completement en bas a droite de l'écran !!!

Ou est mon problème ?

[Swoög]

Salut !

il faudrait peut-être savoir ce que tu fais :

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
int Menu::Pos(int numSprite, int type)
{
   if((numSprite < taille) && (image[numSprite] != NULL))
   {
    int x = (int)&rect[numSprite].x;
    int y = (int)&rect[numSprite].y;//Là tu as récupéré des pointeurs castés en entiers sur &rect[numSprite].(x et y)
    int *returnx = &x;
    int *returny = &y;//Là, tu récupère des pointeurs sur les pointeurs pré castés
    if (type == 1)
    {
         return -(*returnx); //là tu revoies -x càd - (int)&rect[numSprite].x
    }
    if (type == 2)
    {
         return -(*returny); //là tu revoies -y càd - (int)&rect[numSprite].y
    }
    }
}

Je sais pas ce que tu tentes de faire, mais ta fonction est équivalente à ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
int Menu::Pos(int numSprite, int type)
{
   if((numSprite < taille) && (image[numSprite] != NULL))
   {
       if (type == 1)
            return -(int)&rect[numSprite].x;
       else if (type == 2)
            return -(int)&rect[numSprite].x;
    }
}

Je ne pense pas que ce soit tout à fait ce que tu tentes de faire... puisque ça ne correspond à rien... un pointeur ne renseignera que sur la position en mémoire, et l'opposé d'un pointeur ne correspond absoluement à rien...

Tu as du confondre pointeurs et références... ou quelque chose dans ce style...

de plus prévoit des retours par défaut, ou (mieux) des exceptions au cas où les arguments seraient invalides... (type différent de 1 et de 2, numSprite >= taille, ou image[numSprite] == NULL)

en tous cas, il y a un micmac quelque part... :\

[jmv]

si j'ai bien compris ton code, tu retourne une référence sur une variable locale. Le pb est que les variables locales sont détruites à la sortie de la fonction, une référence sur une de ces variables locales n' a donc plus de sens.

[insomniak]

@Swoög C'est ce que je faisais au début, mais il m'affichait une ligne de chiffres (surrement l'adresse, c'est pour ça que je voulais récupérer ce qu'il y avait sous l'adresse.
@jmv Mmmh... Donc apperrement, il faudrait que je donne a returnx la valeur dans la fonction, puis la renvoyer ? Je sais pas si ce que je dit est très clair...

[jmv]

je crois que j'ai rien compris à ton code on s'y perd avec les pointeurs et références.
que veux tu faire exactement ?
c'est quoi image ? rect ? taille ? je ne vois nulle part leur déclaration

[insomniak]

Je suis parti d'une classe un peu bizzare, une classe sprite, et je l'ai modifiée.
Enfin bon, je vais me débrouiller tout seul pour l'instant.

[jmv]

et pourquoi pas tout simplement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
int Menu::Pos(int numSprite, int type)
{
    if((numSprite < taille) && (image[numSprite] != NULL))
    {
        if (type == 1)
            return rect[numSprite].x;
        if (type == 2)
            return rect[numSprite].y;
    }
    return -1; // il faut bien retourner qqchose !
}