Discussion :

[bachir006]

Salut ,
je suis entrain de creer un jeu en SFML, mais voial je bloque a l'affichage des sprite.

mais malheureusement cette fonction fait bugger mon programme ,
buggé ça veut dire la fenetre reste gelé un peu, puis il y a le traditionnel message d'erreur qui apparait: envoyer, ne pas envyer...

Voici plus d'infos:
voila comment j'alloue de la memoir pour mes objets:

Code :

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

1
2
3
4
5
void Niveau::initObjet(Objet* obj, char c, int x, int y){
//[...]
    obj = new Mur();//Mur erite de la classe objet
 
}

la partie censé afficher _tous les sprites_:

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
23
typedef std::map < int, Objet*, std::less<int> > C_Objet;
 
//c'est la fonction qui doit être appelé lorsqu'on veut un rafraichissement de l'écran
void Moteur_graphique::affiche() {
    C_Objet objets = m_niveau.getTableauObjet();
    for(C_Objet::const_iterator itr = objets.begin(); itr != objets.end(); itr ++){
        itr->second->show(m_app);
        }
    bool Running = true;
     while (Running)
     {
         // Process events
         sf::Event Event;
         while (m_app.GetEvent(Event))
         {
             // Close window : exit
             if (Event.Type == sf::Event::Close)
                 Running = false;
         }
         // Update the window
         m_app.Display();
     }
}

et voici la fonction show de la classe objet:

Code :

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

1
2
3
void Objet::show(sf::RenderWindow& app){
    app.Draw(sprite);
}

je ne sais pas trop d'ou vieznt le probleme . Pourriez vous m'aider?

pour voir l'ensemble des sources, voici le lein:https://man-fan.svn.sourceforge.net/...n-fan/sources/

merci de m'aider

[Laurent Gomila]

Code :

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

1
2
3
4
5
void Niveau::initObjet(Objet* obj, char c, int x, int y){
//[...]
    obj = new Mur();//Mur erite de la classe objet
 
}

Si tu t'attends à ce que obj soit initialisé au retour de la fonction, c'est loupé, il faut le passer par référence si tu veux que ce soit le cas. Ou alors le faire renvoyer par ta fonction.

[bachir006]

Si tu t'attends à ce que obj soit initialisé au retour de la fonction, c'est loupé, il faut le passer par référence si tu veux que ce soit le cas. Ou alors le faire renvoyer par ta fonction.

Et pourquoi, le pointeur permet la meme chose, c'est l'adresse de mon objet qui est passé en parametre, et non pas une copie de ce dernier
Sinon je ne sais pas (encore) allouer de la memoire par pasage de reference

[Laurent Gomila]

Et pourquoi, le pointeur permet la meme chose, c'est l'adresse de mon objet qui est passé en parametre, et non pas une copie de ce dernier

Oui, donc ce serait bon si tu modifiais l'objet. Or là c'est le pointeur que tu modifies, et le pointeur lui est bien copié dans ta fonction puisque tu le passes par valeur.
Si tu veux passer ton pointeur par référence ce n'est pas plus compliqué que pour n'importe quel autre type :

Code :

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

1
2
3
4
5
void Niveau::initObjet(Objet*& obj, char c, int x, int y){
//[...]
    obj = new Mur();//Mur erite de la classe objet
 
}

Mais pourquoi ne pas le renvoyer, puisque c'est une création ?

Code :

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

1
2
3
4
5
Objet* Niveau::initObjet(char c, int x, int y){
//[...]
    return new Mur();//Mur erite de la classe objet
 
}

[bachir006]

Oui, donc ce serait bon si tu modifiais l'objet. Or là c'est le pointeur que tu modifies, et le pointeur lui est bien copié dans ta fonction puisque tu le passes par valeur.
Si tu veux passer ton pointeur par référence ce n'est pas plus compliqué que pour n'importe quel autre type :

Code :

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

1
2
3
4
5
void Niveau::initObjet(Objet*& obj, char c, int x, int y){
//[...]
    obj = new Mur();//Mur erite de la classe objet
 
}

Mais pourquoi ne pas le renvoyer, puisque c'est une création ?

Code :

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

1
2
3
4
5
Objet* Niveau::initObjet(char c, int x, int y){
//[...]
    return new Mur();//Mur erite de la classe objet
 
}

ah oui, t'as raison, ça marche mnt.
mais j'ai quand meme un probleme: l'image ne s'affiche pas, il y seulement un carrée blanc qui a les meme dimension que mes images qui s'affiche

[Laurent Gomila]

Tu as vérifié que l'image était bien chargée ? Tu ne fais pas de copies de ton image après l'avoir affectée à ton sprite ?

[bachir006]

Tu as vérifié que l'image était bien chargée ? Tu ne fais pas de copies de ton image après l'avoir affectée à ton sprite ?

ben ce que je ne comprends pas, c'est que si je charge mon image dans la fonction Objet::Show(), elle s'affiche.
alors que si je le fait dans le constructeur, ça ne marche pas
Je crois que ce code sera plus claire:

Code :

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

1
2
3
4
5
6
//ce code ne marche pas:
Objet::Objet(){
    sf::Image image;
    image.LoadFromFile("mur.png");
    sprite.SetImage(image);
}

Code :

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

1
2
3
4
5
6
7
//ce code marche:
void Objet::show(sf::RenderWindow& app){
    sf::Image image;
    image.LoadFromFile("mur.png");
    sprite.SetImage(image);
    app.Draw(sprite);
}

[Laurent Gomila]

C'est normal, tu déclares ton image localement à ton constructeur, dès la sortie de celui-ci l'image sera détruite et le sprite pointera sur une variable qui n'existe plus. Il faut que ton image vive aussi longtemps que ton sprite en aura besoin.

[bachir006]

C'est normal, tu déclares ton image localement à ton constructeur, dès la sortie de celui-ci l'image sera détruite et le sprite pointera sur une variable qui n'existe plus. Il faut que ton image vive aussi longtemps que ton sprite en aura besoin.

Une derniere question: est ce que c'est toi qui a crée la SFML?
si oui, ben elle vraiment super

[Laurent Gomila]

Une derniere question: est ce que c'est toi qui a crée la SFML?

Oui oui.

si oui, ben elle vraiment super

Merci, ça fait plaisir

[Xanto]

SFML est une librairie naissante.