Discussion :

[izguit]

Yop all,
Je suis débutant en C++, en pointeurs aussi, soyez indulgents
Alors le problème :
J'ai une classe "square" (une case) avec les coordonnées et compagnie comme champs. Une autre classe path, qui doit contenir une liste de pointeurs vers des cases ("square")
J'ai donc ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
class path {
    private:
        square* squares[50];
        int size;
    public:
        void appendSquare(square _square);
        int getSize();
        square* getSquare(int id);
};

et

Code :

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

1
2
3
4
5
 
void path::appendSquare(square _square)
{
    *squares[size] = &_square;
}

mais pan :

error: no match for 'operator=' in '*((path*)this)->path::squares[((path*)this)->path::size] = &_square'

sur la ligne de la fonction appendSquare. Je comprends pas trop :/

Des idées?
A vot' bon coeur

[Laurent Gomila]

Code :

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

*squares[size] = &_square;

L'étoile est en trop. Tu ne dois l'utiliser que lorsque tu veux accéder à l'objet square pointé ; or là ce n'est pas un objet square que tu veux réaffecter, mais une adresse d'objet square.

Ceci-dit, ton paramètre _square est passé par valeur à la fonction, ce qui signifie que ce n'est qu'une copie de l'original, et qu'elle sera détruite dès la sortie de la fonction. Stocker son adresse est donc une erreur, puisqu'immédiatement après l'appel elle pointera sur un objet détruit.

[izguit]

merci de vos réponses rapides

L'étoile est en trop. Tu ne dois l'utiliser que lorsque tu veux accéder à l'objet square pointé ; or là ce n'est pas un objet square que tu veux réaffecter, mais une adresse d'objet square.

Ceci-dit, ton paramètre _square est passé par valeur à la fonction, ce qui signifie que ce n'est qu'une copie de l'original, et qu'elle sera détruite dès la sortie de la fonction. Stocker son adresse est donc une erreur, puisqu'immédiatement après l'appel elle pointera sur un objet détruit.

Ok, donc ma méthode est pourrie. Ce que je voudrais, c'est pouvoir retrouver les coordonnées de mon chemin, sans pour autant les restocker bêtement alors que j'ai déjà mes cases de créées. il faudrait plutôt que je fasse un truc comme ça à ce moment là? :

Code :

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

1
2
3
4
5
 
void path::appendSquare(square *_square)
{
    squares[size] = _square;
}

merci

/me, qui a vraiment du mal avec tout ce qui a des étoiles...

[Laurent Gomila]

Ce que je voudrais, c'est pouvoir retrouver les coordonnées de mon chemin, sans pour autant les restocker bêtement alors que j'ai déjà mes cases de créées. il faudrait plutôt que je fasse un truc comme ça à ce moment là? :

Code :

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

1
2
3
4
5
 
void path::appendSquare(square *_square)
{
    squares[size] = _square;
}

Oui

[izguit]

Merci beaucoup

[ZeRiL]

Utiliser un vecteur ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#include <vector>
 
class path {
    private:
        vector<square> squares;
        int size;
    public:
        void appendSquare(square _square);
        int getSize();
        square getSquare(int id);
};

Code :

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

1
2
3
4
5
 
void path::appendSquare(square _square)
{
    squares.push_back(_square);
}

[Laurent Gomila]

Utiliser un vecteur ?

Un vecteur peut-être (bien qu'ici si la dimension 50 est fixe ça ne servirait à rien), mais de pointeurs alors (relis l'énoncé).

[ZeRiL]

Un vecteur peut-être (bien qu'ici si la dimension 50 est fixe ça ne servirait à rien), mais de pointeurs alors (relis l'énoncé).

Au temps pour moi. Si effectivement la taille est fixe... mea culpa. J'lis trop vite