Discussion :

[bachir008]

Bonjour,

Presentation:
Je suis en train de realiser un jeu 2D en C++ à l'aide de la lib SDL ou j'essaye d'incarner plusieurs personnage celebres des dessins animés( mario, goku, naruto ...), d'ou le nom"manga fantaisie" ou encore en abreviation: "Man-Fan" .

Ou j'en suis arrivé?:
Je suis deja arrivé à créer le premier niveau, à faire deplacer mon personnage, a gerer les collission, a mettre un systeme de gravité et à lui permettre de sauter( ça fait deja beaucoup, hein ).

Des screens?
Voila quelques screen de mon jeu:
ecran d'accueil:
image du jeu


Vous voulez tester?:

Pas de problemes , voila le lien:

sourceforge

Ce que je cherche:

Des commentaires et suggestion surtout a propos de la gestion des collisions
Des remarques a propos du graphisme
votre opinion sur l'idée generale de mon projet
et pourquoi pas une equipe qui m'aiderait a mener a bien mon projet?


Conclusions
Bon bref, j'attends vos avis sur ce projet, et surtout est ce qu'il pourra tenir

[bachir008]

alors, qu'est ce que vous en pensez?

[luckyvae]

J'ai téléchargé les sources, mais il y a pas de MakeFile

[bachir008]

Si tu es sous windows, prends l'executable.
si tu es sous linux, tu n'as qu'a recompiler toit meme , malheureusement je n'ai pas linux.
les lib utilisé sont la SDL et la SDL_image

[Mat.M]

Testé sous Win XP
*les graphismes sont attrayants ceci dit attention aux problèmes de copyright on dirait qu'ils sont repompés
*le jeu est fluide

Par contre
*le saut du perso serait perfectible
*manque plus qu'à gérer la collection d'objet et l'interaction avec des ennemis
*pas d'aide encore

C'est un bon début bonne continuation

[bachir008]

Testé sous Win XP
*les graphismes sont attrayants ceci dit attention aux problèmes de copyright on dirait qu'ils sont repompés
*le jeu est fluide

Par contre
*le saut du perso serait perfectible
*manque plus qu'à gérer la collection d'objet et l'interaction avec des ennemis
*pas d'aide encore

C'est un bon début bonne continuation

Merci beaucoup pour tes remarques, sinon je ne sais vraiment pas comment perfectionner le saut.

[fearyourself]

Ok je viens de le tester sous linux, premier problème :

Code :

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

#include <SDL\SDL.h>

doit être :

Code :

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

#include <SDL/SDL.h>

pour fonctionner sous linux.

Ensuite,

le fichier gfx/decor/nounou.PNG doit s'appeler gfx/decor/nounou.png sinon ton programme plante (il manque un test dans le constructeur Niveau pour vérifier si le chargement s'est bien déroulé).

Enfin, j'ai un problème dans le jeu, il semblerait que tu affiches mal le niveau. Voir l'image attachée à ce post.

Dernier point, le code compile correctement, il y a que trois warnings, donc c'est déjà un bon point :

main.cpp:15: warning: unused parameter ‘argc’
main.cpp:15: warning: unused parameter ‘argv’

Niveau.cpp: In member function ‘bool Niveau::pouvoirAllerEnHaut(Personnage&)’:
Niveau.cpp:118: warning: unused variable ‘longueur’

Jc

[bachir008]

Pour le bug, je crois que c'est specifique a linux, sous windows ça marche nikel.
pour les warning, je vais corriger ça.

[fearyourself]

Pour le bug, je crois que c'est specifique a linux, sous windows ça marche nikel.
pour les warning, je vais corriger ça.

Ce genre d'erreur est souvent dû au fait que des variables ne sont pas correctement initialisées.

Si j'ai un peu de temps je regarderais cela,
Jc

[bachir008]

ok merci, c'est gentil de ta part. J'ai deja rencontré ce genre de probleme,il suffit de changer l'ordre de declaration des variables pour que ça marche.

[fearyourself]

ok merci, c'est gentil de ta part. J'ai deja rencontré ce genre de probleme,il suffit de changer l'ordre de declaration des variables pour que ça marche.

Ca c'est inquiétant comme remarque tout de même.

Mais j'ai trouvé le problème.

1) Pourquoi déclarer ton tableau comme ceci :

Code :

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

char m_decor[HAUTEUR_PAR_CASES][LARGEUR_PAR_CASES+1];//les sprites du niveau

Tu n'utilises pas la dernière case de chaque ligne par la suite. En fait tu fais cela à cause de '\n' qui se trouve dans le fichier de données. Est-ce que tu trouves vraiment que c'est de la bonne programmation de stocker une case de plus parce que ton fichier d'entrée a une donnée en plus par ligne ? Ne serait-ce pas mieux de virer le '\n' à chaque lecture ?

Ensuite dans ta lecture tu faisais ceci :

Code :

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

                fgets(m_decor[i], LARGEUR_PAR_CASES+2, niveau);

Ce qui est encore pire puisque tu dis au programme que ton tableau est plus grand que ce qu'il est.

La solution plus propre serait de faire :

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
 
char buf[2*LARGEUR_PAR_CASES], *ptr;
...
 
    for(int i = 0; i < HAUTEUR_PAR_CASES; i++){
                fgets(buf, sizeof(buf), niveau);
 
                ptr = strchr(buf, '\n');
 
                if(ptr == NULL) {
                    std::cerr << "Probleme avec chargement de la carte" << std::endl;
                    return EXIT_FAILURE;
                }
 
                *ptr = '\0';
 
                int j = 0;
                while((j<LARGEUR_PAR_CASES) && ( buf[j] != '\0')) {
                    m_decor[i][j] = buf[j];
                    j++;
                }
    }

Ensuite attention à ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void jouer(SDL_Surface *ecran, bool &continuer){
...
            jouer(ecran, continuer);
        SDL_PollEvent(&event);
        switch(event.type){
            case SDL_QUIT://si le joueur appuie sur la croix
                continuer = false;
                break;
        }
...
}

- Tu fais une récursive qui pourrait un jour faire sauter la pile
- Il faut tester le retour de SDL_PollEvent

Jc

[bachir008]

Merci beaucoup pour tes remarques, je vais corriger ça.
Il ya une chose que je ne comprends pas, pourquoi tu declare buf comme ceci:

Code :

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

buf[2*TAILLE_PAR_CASES]

au lieu de faire comme ça:

Code :

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

buf[TAILLE_PAR_CASES+1]

et puis j'ai edité mon code:

Code :

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

1
2
3
4
5
6
7
 
...
  for(int i = 0; i < HAUTEUR_PAR_CASES; i++){
                fgets(m_decor[i], LARGEUR_PAR_CASES, niveau);//lecture d'une ligne du fichier
                fseek(niveau, 1, SEEK_CUR);//positionner le curseur a la ligne suivante
    }
...

et je ne sais pas pk ça marche pas

[fearyourself]

Merci beaucoup pour tes remarques, je vais corriger ça.
Il ya une chose que je ne comprends pas, pourquoi tu declare buf comme ceci:

Code :

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

buf[2*TAILLE_PAR_CASES]

au lieu de faire comme ça:

Code :

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

buf[TAILLE_PAR_CASES+1]

Car j'avais la flemme de réfléchir. Vu que fgets prends le '\n' aussi et ajoutes un '\0', il faudrait en fait faire

Code :

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

buf[TAILLE_PAR_CASES+2]

Pour ces raisons, j'ai préféré * 2, j'étais encore endormi ce matin...

Code :

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

1
2
3
4
5
6
7
 
...
  for(int i = 0; i < HAUTEUR_PAR_CASES; i++){
                fgets(m_decor[i], LARGEUR_PAR_CASES, niveau);//lecture d'une ligne du fichier
                fseek(niveau, 1, SEEK_CUR);//positionner le curseur a la ligne suivante
    }
...

Normal, tu joues avec le feu là. Vu que tu lui dis que la taille du buffer fait LARGEUR_PAR_CASES alors si ton fichier a vraiment des lignes de LARGEUR_PAR_CASES caractères (sans le '\n'), la fonction fgets va lire LARGEUR_PAR_CASES-1 caractères et mettre le '\0' à la fin.

Du coup, il faudrait sauter 2 caractères pour revenir à la prochaine ligne... C'est pour cela que je lit plus et je me débrouille pour que fgets cherche la fin de ligne, c'est nettement plus simple...

Jc

[bachir008]

Merci beaucoup.

[bachir008]

et pour ce bout de code, ça compile sous linux?:

Code :

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

1
2
3
4
5
6
7
8
    for(int i = 0; i < HAUTEUR_PAR_CASES; i++){
                fgets(buf, sizeof(buf), niveau);//lecture d'une ligne du fichier
                for(int j = 0; j < LARGEUR_PAR_CASES; j++)
                    {
                        m_decor[i][j] = buf[j];
                        cout<<buf[j];
                    }
    }

en tout cas, chez moi, ça compile nikel

et puis quand tu mets:

Code :

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

buf[j] != '\0'

Il fallait plutot mettrebuf

Code :

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

[j] != '\n'

.
et enfon ton "*ptr" ne fais rien.

[fearyourself]

Chez moi aussi mais je préfére ma version

Jc

[bachir008]

mais dans ta version, tu n'utilise pas ton *ptr!

[fearyourself]

et puis quand tu mets:

Code :

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

buf[j] != '\0'

Il fallait plutot mettre

Code :

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

buf[j] != '\n'

.
et enfon ton "*ptr" ne fais rien.

Faux, puisque j'enlève le '\n' en mettant un '\0' à la place.

Code :

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

1
2
3
4
5
6
7
8
                ptr = strchr(buf, '\n');
 
                if(ptr == NULL) {
                    std::cerr << "Probleme avec chargement de la carte" << std::endl;
                    return EXIT_FAILURE;
                }
 
                *ptr = '\0';

Jc

[bachir008]

OK merci beaucoup pour ton aide.

[Bibicy]

Essayé sous WinXP, pas regardé le code encore.

C'est un bon début.

La dynamique du saut est perfectible, tu peux lire :
http://www.gamedev.net/reference/art...article694.asp

Pour faire un poil plus réaliste il faut gérer la vitesse de ton perso (lorsque le personnage monte sa vitesse décroit...)