Discussion :

[katia_heraklion]

Bonjour,
J'ai une erreur dans mon 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
 
void CoordClick(int *xx, int *yy)
{  int go=1;
    SDL_Event event;
    event.button.x=0;
    event.button.y=0;
 
    while(go)
    {
        SDL_WaitEvent(&event);
        if(event.type==SDL_MOUSEBUTTONUP)
            {
                  *xx = event.button.x;
                  *yy = event.button.y;
                   go=0;
            }
    }
}
 
int main(int argc, char *argv[])
{
    int *xx=NULL;
    int *yy=NULL;    
 
    SDL_SetVideoMode(640, 480, 32, SDL_HWSURFACE);
 
    SDL_Surface *ecran = NULL;
    ecran = SDL_SetVideoMode(640, 480, 32, SDL_HWSURFACE);
 
    SDL_FillRect(ecran, NULL, SDL_MapRGB(ecran->format, 17, 206, 112));
    SDL_Flip(ecran); 
 
    CoordClick(xx,yy);
    printf("x = %d y = %d",*xx,*yy);
 
    SDL_Quit(); 
 
    return EXIT_SUCCESS;
}

et il y a un probleme a la ligne

Code :

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

*xx = event.button.x;

, qui fait buguer tout le programme.
L'affectation ne doit pas marcher, pourtant il me semble bien respecter les types et les regles des pointeurs.

Quelqu'un saurait-il m'aider ?
Merci beaucoup.

[souviron34]

Dans ton main tu déclares des pointeurs..

Qui de plus sont initialisés à NULL..

Ces pointeurs ne SONT PAS des entiers... mais des adresses potentielles.

Or nulle part tu leur affectes une valeur , un espace de stockage correspondant...

Il te faut relire ton cours sur les pointeurs......

[la drogue c'est mal]

je te conseillerais en plus de créer une strucuture "coordonne" avec x et y plutot que de te trimballer 2 variables

[katia_heraklion]

Oui je vais essayer avec une structure. Du coup, je n'ai plus besoin d'avoir de pointeurs, mais je peux utiliser un return dans ma fonction. Merci

[souviron34]

mais pour ton instruction, voici ce que tu aurais dû faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int main(int argc, char *argv[])
{
    int xx=0;
    int yy=0;    
 
 ...
 
    CoordClick(&xx,&yy);
    printf("x = %d y = %d",xx,yy);
.....
}

Comme je le disais, tu as déclaré des POINTEURS, pas des variables. Donc là je déclare des VARIABLES.

Ensuite, tu passes l'adresse de ces variables, puisqu'elle vont être modifiées dans ta fonction.

Ta fonction était correcte.

Enfin tu imprimes la VARIABLE..

[katia_heraklion]

Ok, merci beaucoup, bien sur avec les explications je comprends mon erreur... Merci encore.