PGA : RPG EN C++/OPENGL

Commentaire de Kirua le 17/05/2005 17:13:09

Plusieurs choses:

* l'ambiance sonore de ton jeu est géniale ^^ j'avais pas envie de quitter à cause de la musique et des cui cui des oiseaux :D
* tu dois subdiviser ton code en plusieurs (PLEIN!) de fichiers: c'est illisibles
   tu dois avoir deux fichiers par classe!
* tu définis plein de char[] au début de ton main.cpp (qui est indigeste :/ 1500 lignes :/) alors que tu utilises aussi des std::string: fais un choix! (et choisis les std::string ;))
* bravo: t'as qd même réussi à faire un machin qui tourne, sans menus et sans script engine, c'est bien joué ^^.
* ton fichier globales.h me fait rire: t'as qu'un seul fichier :D sois logique :) ne corrige pas ça, mais fais plutôt une belle subdivision en classes ;)
* tu as un énorme enum où tu définis toutes des valeurs qui se suivent: tu ne dois préciser que le premier, tous les suivants sont obligatoirement le précédent + 1 sauf si tu précises autre chose
* en C++ on écrit: enum type {}; et pas enum {} objet;
* tu définis que un ennemi EST UN héros. non: tu devrais faire une classe Perso d'où héritent: Héros et PNJ. Après quoi, tu as quand même Ennemi qui est assez différent, tu es d'accord? (doit pas se déplacer etc)
* Définis une classe Sprite qui se charge de gérer l'affichage d'une image (avec potentiellement un état d'animation: frame 1 - 2 - 3...), et tu donnes à la classe Perso et à la classe Ennemi un Sprite correspondant.


Extrait de globales.h:


//SOURIS
class claviers
{
      public:
      bool gauche,droite,haut,bas,espace,echap,enter,a,z,e;
};  

sans compter que tu as fait un immonde copier coller (visez moi le commentaire :p), tu trouves ça élégant? ;) habituellement, on a un tableau qui sauvegarde l'état des touches pour tout le programme (enfin, avec les bonnes libs c'est géré en interne)

* tu écris toujours le this-> quand tu accèdes à un élément de ta classe dans une méthode: c'est facultatif. je le dis, mtnt, pê que tu préfères comme ça (ça se défent), mais tu tiens pê ça du PHP, et en C++ à ce que je sache, peu de gens le font.

* comme ton format de maps est simple, tu peux redéfinir les opérateurs istream>> et ostream<< pour lire et écrire ldes données du jeu (maps, objets ...): ça rend les routines de lecture et d'écritures terriblement simples et claires (mais c'est franchement facultatif)

* ton indentation est dégueu (désolé ^^): tu fais des

if()
{
dffgfg
}

tu devrais mettre d'office le bloc dans la if en indenté, honnêtement.
Ceci par exemple:

     if(this->hero.posy==y)
     this->hero.afficher();

est illisible.

* pour le reste c'est pas mal. juste dommage que ce soit bcp hardcodé et peu évolutif.

Enfin, suis vrmnt content que t'aies posté le résultat en cours ^^.

Commentaire de Kirua le 17/05/2005 18:15:24

Le mot de la fin: je trouve très (vraiment) positif que tu aies fait ce RPG, même à l'arrachée comme tu dis. Ça t'a sûrement permis d'avoir une vue d'ensemble des éléments complexes et de l'imbrication des idées, et ça te permettra certainement de bien prévoir l'organisation du prochain. Alors pr le prochain, puisque prochain IL Y AURA (;)), offre-toi une belle phase papier, plusieurs jours. Considère l'intégration d'un moteur de scripts, même très réduit et très orienté pour ton jeu. Pense à isoler les éléments souvent réutilisés (sprites, menus), et puis: définis-toi un VRAI format de maps :D.

Suis impatient de voir la suite :)

Commentaire de kenjimax le 20/05/2005 18:52:11

OpenGlut est disponible sur devpaks.org et fmod sur www.fmod.org, tu trouveras un tutoriel pour l'installer et l'utiliser sur http://glinfrench.apinc.org/
Pour fmod.dll et glut.dll, ils sont dispo sur mon autre source qui s'apelle RPG Allende. En tout cas je suis content que ca te plaise :D
A bientot

Commentaire de noSnoR le 21/05/2005 22:40:09

merci pour le tout, ça vas m'aider (je pense...)

Commentaire de kenjimax le 23/05/2005 19:54:09

:D c'est de la POO, crade il est vrai, mais c'en est ! Efin...ca n'en serait pas ce serait presque pareil, c'est juste un poil plus simple... Et puis c'est un mixte C/C++ qui n'est pas du meilleur genre, mais c'est 100% fonctionnel pour ce que je voulais en faire, donc ;)
Merci pour ton commentaire

Commentaire de J0ebarteam le 12/07/2005 14:28:46

Il me manque le fichier Glut32.dll malgrès le dl des librairies Glut alors si kelkun pouvait me l'envoyer a johan.fenix@voila.fr merci.

Commentaire de kenjimax le 21/05/2006 17:22:00

Comme ce lien arrive dans les premiers quand on tape : "C++ RPG" sur Google, je rajoute juste un commentaire pour indiquer le site web de mon projet bien plus avancé que celui la :
http://www.saveourlife.c.la
C'est encore en chantier mais ca avance ;)

Commentaire de kenjimax le 21/05/2006 20:08:37

C'est un peu hors sujet, mais si tu voyais le nombre de message que j'ai sur mon compte sur cppfrance qui me demande : "T'as un site web ?", "J'aimerais faire un RPG, tu veux pas le faire avec moi". Et bien j'espere qu'avec ca, les gens posteront sur mon forum un peu trop vide :D (6membres :D)