STL, map et operateur '+'

Version imprimable

13/05/2006, 15h50
insomniak

STL, map et operateur '+'
Bonjour.

J'ai un problème avec cette ligne de code :
Code:

1 2 3 4 5 for (int pos_x = x; pos_x <= x+(m_pFileData->Get_DataDecors(m_Sprites[DECORS])[TAILLEX]) && pos_x < longueur; pos_x++) {
A la ligne "x+(m_pFileData[...]", le compilateur me donne une erreur très longue (je met que le début): "no match for operator+ in 'x+(mp_FileData[...]"

mp_FileData retourne un map<int, int>... D'ou vient le problème ?? Merci d'avance. :)
13/05/2006, 16h00
Swoög

tu es totalement sûr que

Code:

m_pFileData->Get_DataDecors(m_Sprites[DECORS])[TAILLEX]

tu renvois un int ?
13/05/2006, 16h17
insomniak

Logiquement oui, puisque Get_DataDecors(m_Sprites[DECORS]) renvoit un map<int, int>, donc j'y rajoute l'opérateur [] avec [TAILLEX], je devrais donc obtenir un int...
13/05/2006, 16h21
Swoög

en effet...

ça serait possible que tu donnes le message d'erreur en entier s'il te plaît ? (c'est vrai qu'il sont très longs... ^^)
13/05/2006, 16h46
insomniak

Pas moyen de copier-coller le message d'erreur sous Code::Blocks :?

Alors le voici :

Code:

error: no match for 'operator+' in 'x + *(((C_Case*)this->C_Case::m_pFileData->C_FileData::Get_DataDecors((((std::map<int, int, std::less<int>, std::allocator<std::pair<const int, int> > >*)((C_Case*(this))+8u)->std::map<_Key, _Tp, _Compare, _Alloc>::opera

J'ai pas la suite...
13/05/2006, 16h56
Laurent Gomila
Dans ce cas là décompose pour mieux cerner l'erreur. De toute façon, faire une recherche dans une map à chaque tour de boucle est une mauvaise idée. Et ton code n'en sera que plus clair.
Code:

1 2 3 4 const std::map<int, int>& Data = m_pFileData->Get_DataDecors(m_Sprites[DECORS]); int Size = Data[TAILLEX]; for (int pos_x = x; (pos_x <= x + Size) && (pos_x < longueur); pos_x++)
13/05/2006, 17h10
insomniak

A la ligne

Code:

int Size = Data[TAILLEX];

C::B me donne cette erreur :

Code:

passing 'const std::map<int, int [...] >' as 'this' argument pf '_Tp& std::map<_Key, _Tp, _Compare, _Alloc>::operator[](const Key&) [with _Key = int, _Tp = int, _Compare = std::less<int>, _Alloc

Là encore, je n'est pas la suite...

Ah oui zut... Si tu ne veux pas t'embêter, il ne faut pas que ta map soit constante.

Tant qu'on y est, je suppose que m_pFileData->Get_DataDecors(...) renvoie une map par valeur et non par référence ? Bonjour les performances :aie:

Donc pour tester avec ton code actuel sans te faire ch*** tu peux faire ça :

Code:

std::map<int, int> Data = m_pFileData->Get_DataDecors(m_Sprites[DECORS]);

Mais dans le code final, idéalement il faudrait ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 // Get_DataDecors renvoie un const std::map<int, int>& const std::map<int, int>& Data = m_pFileData->Get_DataDecors(m_Sprites[DECORS]); std::map<int, int>::const_iterator It = Data.find(TAILLEX); if (It == Data.end()) { // Erreur : clé TAILLEX non présente dans la table } int Size = It->second;

13/05/2006, 17h42
Matthieu Brucher

C'est vrai, on a pas le droit de faire [] sur une map constante :|
Vire le const devant une fois.

Code:

1
2
3
4
const std::map<int, int>& Data = m_pFileData->Get_DataDecors(m_Sprites[DECORS]);
int Size = *Data.find(TAILLEX);
 
for (int pos_x = x; (pos_x <= x + Size) && (pos_x < longueur); pos_x++)

13/05/2006, 17h52
insomniak

Citation:

Tant qu'on y est, je suppose que m_pFileData->Get_DataDecors(...) renvoie une map par valeur et non par référence ? Bonjour les performances :aie:

Comment ça ? Ca renvoit juste une map<int, int>...

Sinon, pourquoi y a-t-il un & à :

Code:

const std::map<int, int>& Data = m_pFileData->Get_DataDecors(m_Sprites[DECORS]);

Merci de m'aider, je vais essayer ça.
13/05/2006, 17h55
Swoög

Les map sont des gros objets, leur copie prend énormément de temps processus et de mémoire...

c'est pour ça qu'ils sont généralement renvoyés par référence, et non par valeur

le & est justement là pour créer une référence (ici à la valeur de retour de la fonction), et donc épargner le temps et la mémoire d'une copie de map
13/05/2006, 17h57
insomniak

D'accord.
Merci :)
13/05/2006, 18h11
Laurent Gomila

Donc finalement, d'où venait ton erreur ?

Citation:

int Size = *Data.find(TAILLEX);

Data.find(TAILLEX)->second, plutôt, vu que les itérateurs de map pointent sur des std::pair. Mais ne pas vérifier le retour d'un find est dangereux.
13/05/2006, 18h12
Swoög

je pense que c'était le [] sur un const map<...> ;)
13/05/2006, 18h14
Laurent Gomila

Citation:

Envoyé par Swoög

je pense que c'était le [] sur un const map<...> ;)

A la base il n'y avait pas de const, c'est moi qui l'ait introduit.
13/05/2006, 18h16
insomniak

Citation:

Envoyé par Swoög

je pense que c'était le [] sur un const map<...> ;)

Il y avait de ça, mais c'est aussi parce que j'avais déclarer Get_DataDecors en tant que retournant un pointeur -_-''

Par contre, comment faut-il que je fasse pour que Get_DataDecors retourne par référence ?? Au sein meme de la fonction, ça ne change pas ?
13/05/2006, 18h18
Laurent Gomila
Citation:

Par contre, comment faut-il que je fasse pour que Get_DataDecors retourne par référence ??
Code:

1 2 3 4 const std::map<int, int>& C_Case::Get_DataDecors(...) const { return la_meme_chose_qu_avant; }
13/05/2006, 18h19
Swoög

pour que ça retourne une référence, il faut déjà que ta fonction ait la particulaité de renvoyer une variable qui ne sera pas détruite à la fin de la fonction (si avant elle renvoyait un pointeur ça doit être bon)

ensuite, au lieu de déclarer ta fonction comme renvoyant :
map<int, int>

tu la déclares renvoyant
map<int, int>&

et d'ailleurs, comme c'est un getter, je suppose qu'il ne faut pas pouvoir modifier la map qui est renvoyée...

donc fait la renvoyer un
const map<int, int>&

ensuite, dans le corps de la fonction lui-même il n'y a rien à modifier ;)

Citation:
Envoyé par Laurent Gomila
Code:

1 2 3 4 const std::map<int, int>& C_Case::Get_DataDecors(...) const { return la_meme_chose_qu_avant; }
De fois const ?? o_O

13/05/2006, 18h21
Swoög

le second const est là pour préciser que tu peux utiliser la fonction sur une instance constante de C_Case
13/05/2006, 18h23
insomniak

Citation:

Envoyé par Swoög

le second const est là pour préciser que tu peux utiliser la fonction sur une instance constante de C_Case

Excusez-moi, mais qu'est-ce qu'une "instance constante de C_Case" ?
Ca fait 2 ans que je programme en C++, mais je ne m'en suis jamais servit de vocabulaire pareil ^^'
13/05/2006, 18h29
Swoög

ça fait DEUX ANS que tu fais du C++ et tu as CE niveau !? 8O...

Bon, passons...

C_Case est une classe.

Une variable de type C_Case est un objet, car son type est une classe, et plus précisément, c'est une instance de la classe C_Case on dit aussi une instance de C_Case.

Une variable de type const C_Case est une instance constante de C_Case
13/05/2006, 18h39
insomniak

Tout ça je savais, donc ça signifique que je peux faire :
const C_Case Case si j'ai bien compris...

Ok, sinon, oui, en fait j'ai passé 6 mois à faire du C++ sur de la syntaxe C, un an à faire du C++ en utilisant les classes et en m'améliorant petit à petit, et depuis 1 semaine, je me met à utiliser la STL ^^'
13/05/2006, 18h41
Swoög

OK ;) :P ceci expliquant cela ^^

Dans la rubrique tuto du site, il y a un excellent tuto sur la STL (tu peux pas te gourrer d'après le nom ^^)

Bonne chance et bon courage pour la suite ^^
13/05/2006, 18h47
insomniak

Ok, c'est bon, j'ai plus d'erreurs de compilation :)