Problème avec les timers

**Mouse Slider** · 21/08/2006, 20h08

Bonjour, j'ai codé un petit programme pour m'habituer aux timers, et j'ai un problème dans le code suivant :

( je code en C sous windows XP SP 2 avec code::blocks et son compilateur gcc fourni, et la dernière version stable de SDL)

J'ai cru comprendre qu'une pause de thread ne coupait pas les timers ? (exécutés dans un autre thread)

Mais ici la fenêtre s'affiche et disparaît rapidement...je ne comprends pas pourquoi.

Merci de votre aide.

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
 
#include <stdio.h>
#include <stdlib.h>
#include <SDL/SDL.h>
#include <time.h>
 
typedef struct surfaces{
 
    SDL_Surface* surface1;
    SDL_Surface* surface2;
    SDL_Surface* surface3;
 
    SDL_Rect* position;
 
}SURFACES;
 
Uint32 mouvement(Uint32 intervalle, void* param){
 
    SDL_Rect* pos = param;
    pos->x+=5;
    pos->y+=5;
 
    return intervalle;
}
 
Uint32 afficher(Uint32 intervalle, void* param){
 
    SURFACES* surface = param;
 
    SDL_Rect p;
    p.x=0;
    p.y=0;
 
    SDL_BlitSurface(surface->surface1,NULL,surface->surface2,&p);
 
      SDL_BlitSurface(surface->surface3,NULL,surface->surface2,surface->position);
 
    SDL_Flip(surface->surface2);
 
    return intervalle;
}
 
int main(int argc, char *argv[]){
 
    SDL_Surface* ecran=NULL;
    SDL_Surface* fond=NULL;
    SDL_Init(SDL_INIT_VIDEO | SDL_INIT_TIMER);
 
 
    ecran=SDL_SetVideoMode(750,500,32,SDL_HWSURFACE | SDL_DOUBLEBUF);
 
    SDL_Rect position, posi;
 
    position.x=200;
    position.y=100;
 
    posi.x=0;
    posi.y=0;
 
    SDL_WM_SetCaption("Test timer",NULL);
 
    fond=SDL_LoadBMP("fond.bmp");
 
    SDL_BlitSurface(fond,NULL,ecran,&posi);
 
    SDL_Flip(ecran);
 
 
 
    SDL_Surface* piques;
    piques = SDL_LoadBMP("boulepique.bmp");
 
    SURFACES* surfaces;
 
    surfaces->surface1=fond;
    surfaces->surface2=ecran;
    surfaces->surface3=piques;
 
 
 
    SDL_TimerID timer = SDL_AddTimer(500, mouvement, &position);
    SDL_TimerID frame = SDL_AddTimer(400, afficher, surfaces);
 
SDL_Delay(10000);
 
    SDL_RemoveTimer(timer);
    SDL_RemoveTimer(frame);
 
    SDL_FreeSurface(piques);
    SDL_FreeSurface(fond);
    SDL_FreeSurface(ecran);
 
    SDL_Quit();
 
    return EXIT_SUCCESS;
}

**fearyourself** · 21/08/2006, 21h50

Envoyé par Mouse Slider

Bonjour, j'ai codé un petit programme pour m'habituer aux timers, et j'ai un problème dans le code suivant :

( je code en C sous windows XP SP 2 avec code::blocks et son compilateur gcc fourni, et la dernière version stable de SDL)

D'accord mais visiblement, il faudrait d'abord travaillé son C, il y a des erreurs assez énormes dans le code présenté...

J'ai cru comprendre qu'une pause de thread ne coupait pas les timers ? (exécutés dans un autre thread)

C'est exact (du moins à ma connaissance)

Mais ici la fenêtre s'affiche et disparaît rapidement...je ne comprends pas pourquoi.

C'est du à une erreur de segmentation dûe au C...

Il faudrait d'abord revoir son C... Toutes les erreurs sont dûes à des erreurs C et non à un problème SDL...

Voici mes commentaires du code:

Code :

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

1
2
3
4
5
 
#include <stdio.h>
#include <stdlib.h>
#include <SDL/SDL.h>
#include <time.h>

time.h et stdio.h sont inutiles dans le code présenté....

Code :

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

1
2
3
4
5
6
7
8
 
typedef struct surfaces{
    SDL_Surface* surface1;
    SDL_Surface* surface2;
    SDL_Surface* surface3;
 
    SDL_Rect* position;
}SURFACES;

Pas de majuscules pour le nom d'une structure... Du moins c'est déconseillé... C'est mieux de garder ce genre d'écriture pour les macros et constantes...

Et une nommenclature plus explicite que surface1 surface2 et surface3 serait plus adaptée..

Code :

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

1
2
3
4
5
6
 
    SDL_Rect p;
    p.x=0;
    p.y=0;
 
    SDL_BlitSurface(surface->surface1,NULL,surface->surface2,&p);

La déclaration et utilisation de p est inutile ici... Cela revient au même de faire :

Code :

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

1
2
 
    SDL_BlitSurface(surface->surface1,NULL,surface->surface2,NULL);

Code :

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

1
2
 
    SDL_Init(SDL_INIT_VIDEO | SDL_INIT_TIMER);

Il faut tester le retour de SDL_Init...

Code :

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

1
2
 
    ecran=SDL_SetVideoMode(750,500,32,SDL_HWSURFACE | SDL_DOUBLEBUF);

Il faut tester la valeur de ecran avant de continuer...

Code :

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

1
2
 
    SDL_Rect position, posi;

Je trouve plus propre de déclarer les variables au début de la fonction (subjectif)

Code :

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

1
2
3
4
5
6
 
    posi.x=0;
    posi.y=0;
 
    ....
    SDL_BlitSurface(fond,NULL,ecran,&posi);

Même remarque qu'avant, posi ne sert à rien ici mais je suppose que tu voulais le changer après...

Code :

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

1
2
 
    fond=SDL_LoadBMP("fond.bmp");

Il faut tester le retour de SDL_LoadBMP...

Code :

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

1
2
3
 
    SDL_Surface* piques;
    piques = SDL_LoadBMP("boulepique.bmp");

Il faut tester le retour de SDL_LoadBMP...

Code :

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

1
2
3
4
5
6
 
    SURFACES* surfaces;
 
    surfaces->surface1=fond;
    surfaces->surface2=ecran;
    surfaces->surface3=piques;

Si tu compilais avec les options -Wall -Wextra -O2 tu aurais comme message :

timer.c:74: warning: ‘surfaces’ is used uninitialized in this function

Ceci devrait t'interpeler et te dire... Ou est-ce que j'ai alloué de l'espace pour cette structure ?? Plus simple et correct vu que t'as oublié de mettre position en place :

Code :

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

1
2
3
4
5
6
7
 
    SURFACES surfaces;
 
    surfaces.surface1=fond;
    surfaces.surface2=ecran;
    surfaces.surface3=piques;
    surfaces.position = &position;

Code :

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

1
2
3
 
    SDL_TimerID timer = SDL_AddTimer(500, mouvement, &position);
    SDL_TimerID frame = SDL_AddTimer(400, afficher, surfaces);

Bien sûr, le deuxième appel devient :

Code :

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

1
2
 
    SDL_TimerID frame = SDL_AddTimer(400, afficher, &surfaces);

Code :

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

1
2
 
    SDL_FreeSurface(ecran);

Non! SDL_Quit s'en occupe de la surface de l'ecran...

Jc

**Mouse Slider** · 21/08/2006, 21h50

Ah oui en effet !

J'ai oublié certaines choses que je connaissais , mais d'autres que je ne connais pas ...

Désolé j'apprends le C depuis deux semaines, et SDL depuis quelques jours...avant je faisais du java. Je vais donc apprendre plus avant de continuer à coder !

Mais en tout cas merci pour ta réponse, car il m'est difficile d'identifier mes lacunes.

**shenron666** · 22/08/2006, 08h56

Envoyé par fearyourself

Je trouve plus propre de déclarer les variables au début de la fonction (subjectif)

De ce que je me souviens des contraintes du C, il falait déclarer les variables au début de la fonction.
La déclaration de variables en plein milieu d'une source n'est-elle pas une spécificité du C++ ?

**Laurent Gomila** · 22/08/2006, 09h14

La déclaration de variables en plein milieu d'une source n'est-elle pas une spécificité du C++ ?

Et du C99 il me semble. Lorsque l'on peut déclarer les variables n'importe où, une bonne pratique est de ne pas les déclarer avant d'en avoir besoin. Surtout en C++, où on pourra utiliser le constructeur adéquat directement.

**fearyourself** · 22/08/2006, 09h34

Envoyé par shenron666

De ce que je me souviens des contraintes du C, il falait déclarer les variables au début de la fonction.
La déclaration de variables en plein milieu d'une source n'est-elle pas une spécificité du C++ ?

Comme le dit Laurent :

Et du C99 il me semble.

et c'est une des choses que les compilateurs C acceptent de nos jours si on ne met pas les options adéquates...

Lorsque l'on peut déclarer les variables n'importe où, une bonne pratique est de ne pas les déclarer avant d'en avoir besoin. Surtout en C++, où on pourra utiliser le constructeur adéquat directement.

Je suis d'accord sauf qu'il faut bien aérer le code pour que cela soit lisible. Souvent, je trouve que cela allourdi le code et le rend indigeste (lorsque c'est mal fait)...

Jc

Problème avec les timers

SDL

Discussions similaires

Partager

Partager