Discussion :

[Polyfructol]

Bonjour, j'ai un problème d'utilisation de vecteurs dans le cadre d'un petit jeu de casse brique.

Pour résumer, le jeu contient une classe abstraite Object, sur laquelle dérivent 3 autres classes. Une classe Player, Ball et Brick.

Ces 3 classes sont gérées par une classe ObjectManager qui contient un vecteur qui pointe sur tous les objets de la scène.

La classe ObjectManager :

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
class ObjectManager
{
private:
    std::vector<Object *> list;
public:
    ~ObjectManager();
    void addObject(Object*);
    Object* getObject(int i) { return list[i]; }
    int getSize() { return list.size(); }
};
ObjectManager::~ObjectManager()
{
    while (!list.empty())
        list.pop_back();
}
void ObjectManager::addObject(Object *t_object)
{
    list.push_back(t_object);
}

La déclaration des objets et l'ajout des objets dans ObjectManager :

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
    // Quelque part dans le main ...
    // Déclaration des Objets de la scène. ->Objets(int position_x, int position_y)
    Player player(240, 460); // le player s'affiche en bleu
    Brick bricktest(50, 50); // Brique s'affiche en rouge
    Ball ball(320, 430);     // La balle s'affiche en rouge
 
    std::vector<Brick> bricklist; // Toutes les briques sont dans un vecteurs
    for (int i = 10; i < 620; i += 25)
        for (int j = 10; j < 300; j += 15)
            bricklist.push_back(Brick(i, j));
 
    // Ajout des objets précédents 
    objectmanager.addObject(&ball);
    objectmanager.addObject(&bricktest);
    objectmanager.addObject(&player);
 
    for (int i = 0; i < bricklist.size(); i++)
        objectmanager.addObject(&bricklist[i]); //<--- PAS BON, mais fonctionne ...
        // De plus les briques s'affichent en bleue, c'est la couleur de l'objet précédents soit player...

Je soupçonne la dernière ligne d'être la cause de tout ça.

Ce code compile, pas d'erreur avec MinGW, mais lors de l'affichage des briques, elles ne sont pas de la bonne couleur -> toutes les briques qui étaient dans le vecteur sont bleues, alors que la brique déclarée normallement est rouge (la couleur étant générée dans le constructeur), et ça rame bien.

Voilà, donc je pense qu'il y a un gros problèmes dans la mémoire, mais je n'arrive pas a le régler. Surement une mauvaise utilisation du vecteur pour faire passé l'adresse dans la dernière ligne.

Merci d'avance d'avoir lu !

EDIT : Aïe, je me suis trompé de section, si un admin passe par ici, pourrait-il le déplacer dans la section SL & STL, merci.

[JolyLoic]

Est-tu certain que le problème n'est pas quand tu mets les briques dans le vecteur ? Comment est défini le constructeur par recopie de cette classe ?

Autrement, prendre l'adresse d'un élément dans le vecteur devrait marcher à condition que par la suite tu ne touches plus du tout au nombre d'élément dans le vecteur.

[Polyfructol]

Je n'ai pas défini de constructeur par recopie à vrai dire, je vais suivre cette piste en tout cas !

Je montre un code plus explicite, il faut que je sache si c'est censé marcher, parce que dans mon cas ça ne marche pas (bon ça marche a moitié c'est pas normal quand même )

Je déclare des objets qui sont tous dérivés de ma classe Object

Code :

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

1
2
3
4
5
6
7
    // Déclaration des Objets de la scène
    Player player(240, 460);
    Ball ball(320, 430);
    Brick bricktest(200, 200);
 
    std::vector<Brick> bricklist;
    bricklist.push_back(Brick(260, 200)); // Je met une brique dans le vecteur

J'envois leurs adresses dans le vecteur de la classe ObjectManager

Code :

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

1
2
3
4
    objectmanager.addObject(&ball);
    objectmanager.addObject(&bricktest);
    objectmanager.addObject(&player);
    objectmanager.addObject(&bricklist.at(0)); // J'envois l'adresse du premier membre du vecteur

Et là ça marche moyennement ... Alors que si je n'utilise pas le vecteur et place 30 briques (à la main), ça marche à merveille.

Deux images pour expliquer la bizarrerie engendrée par ce truc :
Si je fais :

Code :

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

1
2
3
4
    objectmanager.addObject(&ball);
    objectmanager.addObject(&bricktest);
    objectmanager.addObject(&player);
    objectmanager.addObject(&bricklist.at(0));

Ca m'affiche ça :


Alors que si je fais

Code :

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

1
2
3
4
    objectmanager.addObject(&bricktest);
    objectmanager.addObject(&player);
    objectmanager.addObject(&ball);
    objectmanager.addObject(&bricklist.at(0));

J'obtiens :


La brique du vecteur prend la couleur de l'objet situé avant elle !! alors que c'est pas possible, ça doit être rouge, le dessin du rectangle (dont la couleur) est dans le constructeur.

Donc j'aimerais votre avis sur les deux petits morceaux de code au début du message. Si ce n'est pas un problème de vecteur, j'irais poster dans la rubrique consacré à la librairie graphique associée.

[Sylvain Togni]

Je n'ai pas défini de constructeur par recopie à vrai dire, je vais suivre cette piste en tout cas !

Dans ton cas, le vecteur n'est pas responsable de la durée de vie des objets, donc le constructeur de copie, l'opérateur d'affectation et le destructeur par défaut conviennent, pas la peine de les redéfinir.

Donc j'aimerais votre avis sur les deux petits morceaux de code au début du message.

Tout a l'air correct, à condition que tu n'utilise pas l'objectmanager en dehors du bloc où sont déclaré les objets dont tu prend l'adresse. Si c'est le cas, alors l'erreur doit venir d'ailleurs dans le code.

[Polyfructol]

Merci de la précision. Je n'utilise pas ObjectManager en dehors du bloc où je déclare les objets. Si c'est correct je n'y comprends plus rien.

Je viens de faire la même chose en utilisant un tableau à double dimension. Et ça fonctionne bien. Mais je préfèrerais comprendre pourquoi avec mon vecteur ça marche pas.

La déclaration

Code :

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

1
2
3
4
5
    const unsigned int nb_bricks = 500;
    Brick **bricklist = new Brick*[nb_bricks];
    for(int i = 10, a = 0; i < 620; i += 25)
        for(int j = 10; j < 300; j += 15, a++)
            bricklist[a] = new Brick(i,j);

Équivaut-elle à ça ?

Code :

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

1
2
3
4
    std::vector<Brick> bricklist;
    for (int i = 10; i < 620; i += 25)
        for (int j = 10; j < 300; j += 15)
            bricklist.push_back(Brick(i, j));

Et la connection à l'objectmanager

Code :

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

1
2
    for(unsigned int i = 0; i < nb_bricks; i++)
        objectmanager.addObject(bricklist[i]);

Équivaut-elle à ça ?

Code :

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

1
2
    for(int i = 0; i < bricklist.size(); i++)
        objectmanager.addObject(bricklist.at(i));

Parce que le résultat n'est pas le même.

[loufoque]

La déclaration

Code :

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

1
2
3
4
5
    const unsigned int nb_bricks = 500;
    Brick **bricklist = new Brick*[nb_bricks];
    for(int i = 10, a = 0; i < 620; i += 25)
        for(int j = 10; j < 300; j += 15, a++)
            bricklist[a] = new Brick(i,j);

Équivaut-elle à ça ?

Code :

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

1
2
3
4
    std::vector<Brick> bricklist;
    for (int i = 10; i < 620; i += 25)
        for (int j = 10; j < 300; j += 15)
            bricklist.push_back(Brick(i, j));

Pas du tout.
Partons du code avec le vecteur.
Le code suivant est déjà plus proche :

Code :

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

1
2
3
4
5
    const unsigned int nb_bricks = 500;
    Brick* bricklist = new Brick[nb_bricks];
    for (int i = 10; i < 620; i += 25)
        for (int j = 10; j < 300; j += 15)
            bricklist[i] = Brick(i, j);

vector<T> n'est pas un tableau de pointeurs vers T, il s'agit d'un tableau de T, tout simplement.
Dans ce code, par exemple, les briques sont défaut-construites puis ensuite l'opérateur d'affectation est appelé.
En vrai néanmoins, ce n'est pas comme ça. La mémoire est non-initialisée puis quand on insère un élément celui est copié directement dans la mémoire.

C'est donc plutôt comme ceci :

Code :

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

1
2
3
4
5
    const unsigned int nb_bricks = 500;
    Brick* bricklist = static_cast<Brick*>(::operator new(sizeof(Brick)*nb_bricks));
    for (int i = 10; i < 620; i += 25)
        for (int j = 10; j < 300; j += 15)
            new(bricklist[i]) Brick(Brick(i, j));

Code :

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

1
2
    for(int i = 0; i < bricklist.size(); i++)
        objectmanager.addObject(bricklist.at(i));

Ce code ne compile pas a priori, il manque un &.

[JolyLoic]

Dans ton cas, le vecteur n'est pas responsable de la durée de vie des objets, donc le constructeur de copie, l'opérateur d'affectation et le destructeur par défaut conviennent, pas la peine de les redéfinir.

Le std::vector<Object *> list; n'est certes pas responsable de la durée de vie, mais il y a un autre vecteur dans le code, qui lui utilise le constructeur de copie, c'est le std::vector<Brick> bricklist;

[poukill]

Code :

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

1
2
3
4
5
ObjectManager::~ObjectManager()
{
    while (!list.empty())
        list.pop_back();
}

Attention, ici tu n'effaces pas du tout tes Objets !
Il te faut un delete, sinon tu auras une belle fuite de mémoire.

Une autre solution pourrait être d'utiliser les conteneur de pointeurs intelligents de Boost qui t'éviteront de remplir ce destructeur.

[Polyfructol]

Merci à vous, j'ai enfin réussi à régler mon problème, et c'était tout bête en plus :

Code :

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

1
2
3
4
5
6
7
8
    std::vector<Brick*> bricklist;
    for (float i = 10; i < 620; i += 25)
        for (float j = 10; j < 300; j += 15)
            bricklist.push_back(new Brick(i, j)); // au lieu de .push_back(Brick(i, j));
...
    // Suppression du tableau de brique
    for(int i = 0; i < bricklist.size(); i++)
        delete bricklist[i];

Tout fonctionne à ce niveau. Mais j'ai un doute pour la suppression des briques...

C'est donc plutôt comme ceci :

Code :

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

1
2
3
4
5
    const unsigned int nb_bricks = 500;
    Brick* bricklist = static_cast<Brick*>(::operator new(sizeof(Brick)*nb_bricks));
    for (int i = 10; i < 620; i += 25)
        for (int j = 10; j < 300; j += 15)
            new(bricklist[i]) Brick(Brick(i, j));

Merci pour cette solution, cependant, ne maitrisant pas vraiment ces "aspects" du C++ je n'ai pas réussi à le faire marcher.
Ca me donne une erreur de compilation là : new(bricklist[i]) Brick(Brick(i, j));

Ce code ne compile pas a priori, il manque un &.

En effet, erreur de frappe en recopiant.

Attention, ici tu n'effaces pas du tout tes Objets !
Il te faut un delete, sinon tu auras une belle fuite de mémoire.

En fait, la classe objectmanager ne gère pas la mémoire des objets pointés par le vecteur list. Donc ce que j'ai fait ne sert a rien, les objets sont créés puis supprimés à l'extérieur de la classe objectmanager, je vais supprimer le destructeur.

Voilà, merci de vous être penché sur mon cas

[screetch]

loufoque ton code contenant un new me parait bien compliqué