Discussion :

[Hybrix]

Bonjour à tous,

Je me suis remis à programmer depuis hier, afin de partir sur de bonne bases, j'ai décidé de refaire entièrement le code de mon petit projet.

Mes principaux problèmes rencontrés pour le moment viennent de l'organisation de celui-ci.

J'ai donc lu les tutoriels sur l'architecture d'un jeu-vidéo 3D, même si le mien est en 2D, et j'ai regarder le code source de différents projets Open source.
Je souhaite d'ailleur rendre le projet Open source, c'est pourquoi j'essai d'obtenir un code lisible et le plus simple possible.


Explications :
J'aimerais donc avoir votre avis quand à la lisibilité du code et plus spécialement sur l'architecture du jeu, du moins les bases ( Architecture correct ? Classes au bonne endroit ? Bonne façon de faire, des erreurs ? Chose à améliorer ?, ... ).

Le lien : Ici ( 651 Ko )


Quelques informations :
Langage : C++.
Bibliothèque : SFML 1.1
Compatible ( Théoriquement ): Window, Mac, Unix.
Environnement développement : CodeBlock ( R.4807)

L'archive comprend l'espace de travails ainsi que toutes les sources du projet.

Au final le jeu ressemblera à cette démonstration que j'avais faite :
Lien : Ici




Merci à tous ce qui auront la gentillesse de regarder les sources et de laisser leur avis.
Hybrix.

[Mat007]

Salut,

Personnellement je ne suis pas fan de tout couper en ouate mille répertoires avec 3 classes dans chaque...
Surtout que là c'est un découpage largement artificiel et qui ne repose pas vraiment sur une organisation des dépendances. D'ailleurs y'a plein d'include avec des ../.. ce qui est quand même un peu louche (ou au mieux pénible à utiliser).
Si vraiment tu veux découper il faut essayer d'isoler un bout qui ne dépend de rien d'autre (que des dépendances entrantes) à mon avis...

Sinon ben c'est pas évident de faire une revue de code vu que y'a pas vraiment de code
Juste quelques trucs :
. en général on remet les virtual sur les destructeurs des classes dérivées pour éviter aux lecteurs de se poser la question (mais ce n'est techniquement pas une faute car si le destructeur de la classe de base est virtuel, c'est bon)
. ça ne sert à rien de mettre un ; après les implémentations de méthodes (par exemple à la fin de la ligne du constructeur de Vector dans le .hpp ou encore le destructeur de PhysicListener dans le .hpp)
. Object_list.hpp c'est bizarre, et ça va masquer les dépendances puisque tous ceux qui auront besoin d'un .hpp dedans vont les inclure tous
. il vaut mieux respecter la casse quand on inclut un fichier (cf. bloc.hpp dans Object_list.hpp)
. inclure iostream partout est une drôle d'idée...
. il vaut mieux rendre les interfaces (classes virtuelles) pures non copiables, sinon on peut avoir des surprises

Sur l'architecture générale, j'avoue que c'est pas ma tasse de thé d'avoir au centre un tas d'accesseurs sur une classe qui n'a pas d'autre rôle mais bon...
De toute façon le design émerge à force de remaniements, donc je dirais lance-toi et tu verras bien

MAT.

[ash.ice.loky]

Au contraire de Mat007 j'aime découper au maximum mes sources par une bonne arborescence, mais comme lui cette arborescence doit avoir un comportement logique or la ... Sprite dans maths ? Game dans game et base ? Map dans game ? En fait je ne comprend pas trop ton decoupage.

Ensuite tu utilises une facade pour ton "engine" ou tu charges tes differents modules, pourquoi pas, mais c'est pas tres bien structuré, et tes module comme phisique, devrait etre singleton.

[Hybrix]

Je vous remercie d'avoir pris le temps de regarder les sources .

Personnellement je ne suis pas fan de tout couper en ouate mille répertoires avec 3 classes dans chaque...
Surtout que là c'est un découpage largement artificiel et qui ne repose pas vraiment sur une organisation des dépendances. D'ailleurs y'a plein d'include avec des ../.. ce qui est quand même un peu louche (ou au mieux pénible à utiliser).
Si vraiment tu veux découper il faut essayer d'isoler un bout qui ne dépend de rien d'autre (que des dépendances entrantes) à mon avis...

Personnellement, je préfère bien structurer mon programme, je trouve cela plus agréable. L'inclusion des fichiers ne me gène pas vraiment, ça ne sais fait pas énormément de fois.

. en général on remet les virtual sur les destructeurs des classes dérivées pour éviter aux lecteurs de se poser la question (mais ce n'est techniquement pas une faute car si le destructeur de la classe de base est virtuel, c'est bon)

Merci pour le conseil, je ne le savais pas, j'ai donc rajouté les "virtual" aux destructeurs.

ça ne sert à rien de mettre un ; après les implémentations de méthodes (par exemple à la fin de la ligne du constructeur de Vector dans le .hpp ou encore le destructeur de PhysicListener dans le .hpp)

La encore, je n'avais jamais fait attention à ce détail, merci je me coucherais moins bête .

Object_list.hpp c'est bizarre, et ça va masquer les dépendances puisque tous ceux qui auront besoin d'un .hpp dedans vont les inclure tous

J'avais un problème d'inclusion de fichiers ( Ici ), je n'ai pas trouvé de solutions adaptés mieux que celle-ci.

il vaut mieux respecter la casse quand on inclut un fichier (cf. bloc.hpp dans Object_list.hpp)

Corrigé .

inclure iostream partout est une drôle d'idée.

Les copiés-collés c'est le mal :s, corrigé.

il vaut mieux rendre les interfaces (classes virtuelles) pures non copiables, sinon on peut avoir des surprises

&&

... et tes module comme phisique, devrait etre singleton.

J'ai ajouté un Singleton, merci pour le conseil.

Ensuite tu utilises une facade pour ton "engine" ou tu charges tes differents modules, pourquoi pas, mais c'est pas tres bien structuré

Jutilise la technique employé dans le tutoriel "Archticture d'un jeu-vidéo 3D", si il y a mieux je suis preneur .


Merci beaucoup pour vos remarques, elles m'ont permis de corrigé des détails.
Serait t-il possible d'avoir d'autre avis ???

J'ai mis à jour les sources : Les sources ( 640.3 Ko )

Merci encore.
Hybrix

[Hybrix]

Pas d'autres avis ?

[kain_tn]

Euh... Le lien vers les sources est mort?

[_FaFa_]

Je me suis remis à programmer depuis hier, afin de partir sur de bonne bases, j'ai décidé de refaire entièrement le code de mon petit projet.

Tu as codés ça en une soirée ?

[Davidbrcz]

refaire n'est pas faire.
Le jeu était déjà codé (avec SDL si je ne me trompe pas).
Il à juste porté le code de SDL à SFML.
Et c'est assez rapide à faire si le jeu est bien conçus.

[_FaFa_]

Merci de ta précision Davidbrcz.

Mais il est écrit "refaire entièrement le code" ce qui m'a induit en erreur vu que je ne connaissait pas le projet.

[Davidbrcz]

C'est vrai que les source sont dead.
Est il possible de les avoir ?