Discussion :

[insomniak]

Bonjour.

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

Code :

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

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.

[Swoög]

tu es totalement sûr que

Code :

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

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

tu renvois un int ?

[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...

[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... ^^)

[insomniak]

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

Alors le voici :

Code :

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

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...

[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 :

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

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++)

[insomniak]

A la ligne

Code :

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

int Size = Data[TAILLEX];

C::B me donne cette erreur :

Code :

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

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...

[Laurent Gomila]

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

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

Code :

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

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

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

Code :

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

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;

[Matthieu Brucher]

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

[Swoög]

Code :

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

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++)

[insomniak]

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

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

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

Code :

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

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

Merci de m'aider, je vais essayer ça.

[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

[insomniak]

D'accord.
Merci

[Laurent Gomila]

Donc finalement, d'où venait ton erreur ?

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.

[Swoög]

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

[Laurent Gomila]

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.

[insomniak]

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 ?

[Laurent Gomila]

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

Code :

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

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

[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

[insomniak]

Code :

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

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