LIBRAIRIE GRAPHIQUE SDL

Commentaire de mickbad le 28/02/2006 14:38:42

salut,

j'ai peut-être quelques remarques sur la librairie.

Déjà, pourrais-tu mettre un exemple d'utilisation ? non pas que cela soit très difficile de mettre en oeuvre un programme test, mais pour les néophytes que nous sommes ... un ch'tite pensée :)

Ensuite, personnellement ça m'ennuie de voir à ta fonction initSDL un exit si l'initialisation SDL ne se fait pas : en fait, tu ne laisses pas le soin au programmeur de gérer justement les erreurs ! Oui, on peut nous même changer le code mais on peut imaginer que cela ne soit pas possible (dll, lib ...)

Dans la fonction :
void pause(void)
{
long i=0;
while (i<PAUSE) i++;
}

oulala, tu n'as pas peur pour ton CPU!
le mien (les miens car il est bi le cochon ;) commence à activer son ventilateur dès que le CPU dépasse les (admettons) 35%. Alors si dans un jeu (surtout un p'tit), je fais une pause d'affichage, j'entends mon ventilo s'affoler .. je ne vois plus l'intérêt de la pause.
Nan .. pour résumer ... ta boucle ne propose pas trop au système de passer à un autre processus (programme) donc il va demander la main au processeur tout le temps alors que c'est inutile:
solution, un ch'tite pause :
while (++i < PAUSE) Sleep(10); /* par exemple */
sachant que Sleep est défini dans windows.h et un équivalent sur linux usleep dans unistd.h (aux unités d'argument près)


Que dire d'autre : visiblement tu dois faire un initSDL() puis un initCouleurs dans ton programme .. moui, autant faire un appel d'initCouleurs dans initSDL à ce moment là (vive l'automatisme);
Mais de manière globale, et ça n'engage que moi naturellement, quitte à encapusler les appels SDL, pourquoi ne pas faire de l'objet ?
effectivement ça permettait bcp de choses et d'éviter à se poser des questions quant au nommage des fonctions de la librairie car on peut facilement avec une fonction pause() ailleurs ;)

m'enfin, le C c'est bien aussi, je te rassure .. jamais obligé d'employer du C++

Dernière chose : MAIS OU SONT LES COMMENTAIRES ??
héhé! ben oui je suis toujours étonné d'en voir rarement sur ce genre de code (censé être réutilisé ailleurs). Ne me dis pas qu'ils sont dans le .h
Oui j'ai vu mais le code par lui-même n'a que très peu de commentaire !

Je pense que tu comprends ce que je veux dire. Sinon, un jour tu le comprendra surtout si tu dois par exemple reprendre un code d'une autre personne pour le débugger et/ou le faire évoluer !


une dernière chose, comment tu utilises ta fonction quitter() par exemple ? en gros, pour moi, ta fonction n'est pas bonne car elle me raconte que si je l'emploi le programme devrait s'arrêter et là ... pas vraiment !

bonne continuation
.Mick.

Commentaire de PLA le 01/03/2006 08:53:28

L'horloge a été faite avant la librairie et j'ai décidé de mettre les deux codes en même temps mais c'est pour cette raison que je n'utilise pas la librairie.
De plus je n'aime pas trop les structures avec les noms à rallonge et c'est vrai qu'un de ces jours il faudrait que je fasses de l'orienté objet.

Commentaire de jean84 le 04/03/2006 15:33:59

C'est vrai qu'uitliser une boucle pour faire attendre n'est pas tres precis parce que suivant la puissance du processeur, la boucle sera plus ou moins longue... il vaut mieux utiliser comme l'a suggerer mickbad la fonction Sleep() mais tant qu'a faire, autant utiliser SDL_Delay(int) qui fait fait la meme chose mais plus propre a mon avis (appartient directement a la lib sdl) et qui fait une pose de x milisecondes....
@++ et bon code !!