TETRIS EN OPENGL

Commentaire de Funto66 le 07/08/2004 13:37:34

Très bonne source, félicitations ;)
C'est orienté objet, hyper-commenté, tout bien fait...franchement bravo ;)

Bon, forcément, en tant qu'éternel insatisfait, et comme je fais de façon rituelle maintenant, je vais t'énumérer ce que j'ai trouvé bizarre ou les améliorations qui peuvent être apportées ;)

D'abord : pourquoi tu inclus glaux? Ça sert à rien...
Après, tu fournis l'exe en Debug, ça serait mieux en Release, simplement parce que ceux qui n'ont pas Visual C++ ne peuvent pas exécuter l'exe en Debug car il utilise une DLL MSVCRTD.dll qu'ils n'ont pas.
J'ai aussi trouvé le jeu plutôt lent, j'étais au niveau 5 et franchement ça bouge pas :/ Pourtant il me semble bien que tu utilises un timer, j'ai vu ça dans les sources (avec clock() ...etc).
Un autre truc : tes pièces sont pas séparées visuellement, je veux dire par là, chaque "morceau" de pièce remplit le carré qui lui es attribué. C'est selon les goûts, moi je préfère quand elles sont séparées, enfin bon...sinon tu peux aussi utiliser des textures ;) Si tu vois pas ce que je veux dire par séparées visuellement, tu peux regarder le Tetris qui est dans mes sources, peut-être que ça te parlera plus ^^

Dans tes commentaires, j'ai vu ça :

//===========================================================================//
// Global pour ce fichier                                                    //
//===========================================================================//
CJeu g_Jeu;

Il me semble bien que l'objet n'est pas global pour le fichier mais pour tous les fichiers, pour peu qu'ils le déclarent en extern. Pour que ça soit global pour le fichier il faudrait le déclarer en static.

Bon, et pour finir mon habituelle liste, manque le son ;)

Bref, pour résumer, c'est une super source malgré tout ce que je dis, t'inquiète pas c normal avec moi, je trouverai toujours des défauts ou des trucs qui me plaisent pas :p
Je mets 10/10 ;) (Si jamais t'as pas 10/10 c'est à cause d'un bug de CppFrance, j'ai remarqué qu'il avait des problèmes avec les 10/10, je l'ai signalé aux admins, pas de réponse...).

Bon, allez,je sais que je vous glonfle alors j'arrête mon speech là :p

Commentaire de Funto66 le 07/08/2004 23:40:38

De rien ;)
Tes sources sont un plaisir à parcourir, contrairement à la plupart des autres, alors j'en profite ;)

Pour les textures, au début aussi j'utilisais glaux mais cette lib n'est pas faite pour ça, le chargement de BMP est juste une fonction annexe, faut se rappeler qu'à la base c'est un GLUT-like ^^
Allez, je te refais le coup de la pub : là j'utilise un loader d'images TGAs, ce qui permet d'avoir des textures avec transparence et compressées, et le loader est dans mes sources...

Bonne prog
++