Discussion :

[Mic75]

Bonjour à tous,
Je suis actuellement confronter à un problème que je ne comprend pas. Mon but étant ici de pouvoir mettre en pause une animation toute simple (une image qui traverse la fenêtre en diagonale).

Voici le code de la boucle principale (où se situe le prob à mon avis) :

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
 while(continuer)
    {
        SDL_PollEvent(&event) ;
        switch(event.type)
        {
            case SDL_QUIT :
                continuer = 0 ;
            break ;
            case SDL_KEYDOWN :
                if(event.key.keysym.sym == SDLK_p) 
                {    
                    switch(enPause) 
                    {
                        case 0 :
                            enPause = 1 ;
                        break ;
                        case 1 :
                            enPause = 0 ;
                        break ;
                    }
                }    
            break ;
        }
 
        if(!enPause)
        {
            if((positionZozor.y + zozor->h) >= 480)
                versHaut = 1 ;
            else if(positionZozor.y <= 0)
                versHaut = 0 ;                  
 
            tempsAct = SDL_GetTicks() ;
            if(tempsAct - tempsPrec > 20)
            {
                tempsPrec = tempsAct ;                
                if(versHaut)
                {
                    positionZozor.x++ ;
                    positionZozor.y-- ;
                }
                else
                {
                    positionZozor.x-- ;
                    positionZozor.y++ ;
                }
            }
            else
                SDL_Delay(20-(tempsAct-tempsPrec)) ;
        }
 
            SDL_FillRect(ecran,NULL,SDL_MapRGB(ecran->format,255,255,255)) ;
            SDL_BlitSurface(zozor,NULL,ecran,&positionZozor) ;
            SDL_Flip(ecran) ;
    }

Le problème est que la pause marche aléatoirement, en gros, le booléen en pause semble changer de valeur avant d'arriver à (!enPause).
En gros j'appuie sur "p", défois le prog ce met en pause et défois pas, pareil pour enlever la pause.

Bon je sais que c'est plutot confus comme explication alors n'hésitez pas à me demander des précisions.

Merci d'avance pour vos réponses

[Laurent Gomila]

Lorsque tu appuies sur P, l'évènement SDL_KEYDOWN est généré en boucle jusqu'à ce que tu relâches. Si le nombre de frames écoulé est impair alors tu auras bien une pause, sinon non -- ce qui correspond au phénomène "aléatoire" que tu observes.

Quant à la solution, cela dépend de SDL, je ne sais pas si elle permet de faire la distinction entre un évènement keydown et un évènement keypressed (c'est ce dernier que tu veux).

[Mic75]

Merci bcp laurent.
Si j'utilise SDL_KEYUP à la place de SDL_KeyDown cela devrait résoudre le problème nan ?

Edit : Ca marche pas, et la sur le coup, je comprend pas trop.

Autant je peux comprendre que plusieur evènement KEYDOWN sont générés quand j'appuie, mais quand je relache, 1 seul évènement devrait être généré, nan ?

Edit :
Une solution envisageable , mais qui oblige à utiliser une autre touche que celle utiliser pour mettre en pause :

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
switch(event.type)
        {
            case SDL_QUIT :
                continuer = 0 ;
            break ;
            case SDL_KEYUP :
                if(event.key.keysym.sym == SDLK_p)
                {     
                    enPause = 1 ;
 
                    while(enPause && continuer)
                    {
                        SDL_WaitEvent(&event) ;
                        switch(event.type)
                        {
                            case SDL_QUIT : 
                                continuer = 0 ;
                            break ;
                            case SDL_KEYUP :
                                if(event.key.keysym.sym == SDLK_o)
                                    enPause = 0 ;
                            break ;
                        }
                    }
                }
            break ;
        }

[fearyourself]

Lorsque tu appuies sur P, l'évènement SDL_KEYDOWN est généré en boucle jusqu'à ce que tu relâches.

Faux, enfin du moins sauf si on met systèmatiquement en place SDL_EnableKeyRepeat... Et je crois par défaut il est à faux (donc pas de répétitions)

Voir la FAQ :
http://jeux.developpez.com/faq/sdl/?...tition_touches

L'erreur ici se trouve sur le fait que la réception des événements est mal faite...

En effet, cela devrait être :

Code :

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

1
2
3
4
5
6
7
8
 
while(SDL_PollEvent(&event) )
{
        switch(event.type)
        {
        ...
        }
}

Le fait de mettre le while autour permet de capter le résultat de la fonction SDL_PollEvent. Ce résultat disant s'il y a ou non un nouvel événement.

En effet, s'il n'y a pas d'événements, le contenu de event est indéfini (en principe inchangé...)...

Jc

[Mic75]

Merci à vous 2 pour vos réponses.
En effet le keyRepeat n'était pas activer, voilà pourquoi je ne comprenais pas trop le fait que SDL_KEYDOWN serait répéter en boucle.

Par contre tu as tout à fait raison FearYourself, il m'a suffit de faire un test sur la valeur de retour de SDL_PollEvent pour résoudre le problème, cependant étant donné qu'il s'agit d'une animation, le code de la boucle doit tout de même s'éxécuter qu'il y ai un évènement ou pas, d'où ma variante :

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
 while(continuer)
    {
        if(SDL_PollEvent(&event))
        {
            switch(event.type)
            {
                case SDL_QUIT :
                    continuer = 0 ;
                break ;
                case SDL_KEYDOWN :
                    if(event.key.keysym.sym == SDLK_p)
                        enPause = (enPause+1)%2 ;
                    break ;
            }
        }
 
        if(!enPause)
       {......

Merci encore

[fearyourself]

d'où ma variante :

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
 while(continuer)
    {
        if(SDL_PollEvent(&event))
        {
            switch(event.type)
            {
                case SDL_QUIT :
                    continuer = 0 ;
                break ;
                case SDL_KEYDOWN :
                    if(event.key.keysym.sym == SDLK_p)
                        enPause = (enPause+1)%2 ;
                    break ;
            }
        }
 
        if(!enPause)
       {......

Non, il faut mieux mettre un while comme je l'ai montré :

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
 while(continuer)
    {
        while(SDL_PollEvent(&event))
        {
            switch(event.type)
            {
                case SDL_QUIT :
                    continuer = 0 ;
                break ;
                case SDL_KEYDOWN :
                    if(event.key.keysym.sym == SDLK_p)
                        enPause = (enPause+1)%2 ;
                    break ;
            }
        }
 
        if(!enPause)
       {......

Jc

[Mic75]

Pourquoi 1 while est-il préférable qu'un simple if ?
Est ce que s'est du au fait que event est à considérer comme une pile ?
Le while permettrait de traiter tout les évènements de la pile ?

[fearyourself]

Pourquoi 1 while est-il préférable qu'un simple if ?
Est ce que s'est du au fait que event est à considérer comme une pile ?
Le while permettrait de traiter tout les évènements de la pile ?

SDL_PollEvent remplit la structure event. Le problème est qu'entre deux itérations de ta boucle tu peux avoir plus de deux événements il faut donc effectivement tous les traiter.

Un while le permet, un if non.

Jc