Problème d'utilisation d'une map

Salut à tous,

je tente vainement d'utiliser une map ainsi :

Code:

---

std::map<std::string,cStatus> m_status;

---

Code:

---

1 2 3 4 5 6

int cElement::AddStatus (std::string name, int nbframes, int visible, int animated, int looping, float x, float y) { cStatus s(name, nbframes, visible, animated, looping, x, y); m_status["test"] = s; return m_status.size(); }

---

Et là, il me sort ca à la fin de la compil :

Citation:

---

F:\Projets\mi\element.cpp:: In member function `int cElement::AddStatus(std::string, int, int, int, int, float, float)':
F:\Projets\mi\element.cpp:28: error: no match for 'operator[]' in '((cElement*)this)->cElement::m_status["test"]'
C:\Dev-Cpp\bin\..\lib\gcc\mingw32\3.4.2\..\..\..\..\include\c++\3.4.2\bits\stl_map.h:332: note: candidates are: _Tp& std::map<_Key, _Tp, _Compare, _Alloc>::operator[](const _Key&) [with _Key = std::string, _Tp = cStatus, _Compare = std::less<std::string>, _Alloc = std::allocator<std::pair<const std::string, cStatus> >] <near match>
:: === Build finished: 2 errors, 0 warnings ===

---

Quelqu'un peut-il m'aider ? :aie:

Je soupçonne une déclaration foreward sans inclusion du fichier "map" de std::map.
Mais si ce n'est pas ça, c'est difficile a dire, peux tu envoyer la totalité du code (le .h et le .cpp), s'il te plait?

et en remplacant cela ?

Code:

---

m_status["test"] = s;

---

par

Code:

---

m_status[std::string("test")] = s;

---

car dans l'erreur
C:\Dev-Cpp\bin\..\lib\gcc\mingw32\3.4.2\..\..\..\..\include\c++\3.4.2\bits\stl_map.h:332: note: candidates are: _Tp& std::map<_Key, _Tp, _Compare, _Alloc>::operator[](const _Key&) [with _Key = std::string, _Tp = cStatus, _Compare = std::less<std::string>, _Alloc = std::allocator<std::pair<const std::string, cStatus> >]

Je pense qu'il n'arrive pas a remplacer "test" en const _Key& , vu que Key est une std::string et non un tableau de char

Eh bien, c'est quasiment la totalité du code...

ma classe cElement contient la map (private).
le main crée une instance de cette classe mais n'utilise pas la fonction AddStatus (pas encore, j'étais en train de la coder justement).

voici le constructeur de la classe cStatus si ca peut aider :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

cStatus::cStatus (std::string name, int nbframes, int visible, int animated, int looping, float x, float y) : m_name(name), m_nbframes(nbframes), m_visible(visible), m_animated(animated), m_looping(looping), m_x(x), m_y(y) { m_nextframeupdate = 0; m_actualframe = 0; }

---

Rien que de très normal quoi... je pense...

Mongaulois> Je viens d'essayer, mais même message. Normalement je voudrais ca :

Code:

---

1 2 3 4 5 6

int cElement::AddStatus (std::string name, int nbframes, int visible, int animated, int looping, float x, float y) { cStatus s(name, nbframes, visible, animated, looping, x, y); m_status[name] = s; return m_status.size(); }

---

histoire de retrouver mon statut facilement par la suite. mais j'ai strictement la même erreur. J'avais remplacé name par "test" juste pour voir... Désolé du mauvais copier-coller :?

Bizarre. Par contre tu devrait rajouter l'implémentation de "operator =" pour cStatus.

juste une question bête.
tu as bien
#include <string>

J'ai bien l'include, sinon il me planterait avant, dès le hpp...
Je vais voir un construteur de recopie par le = en effet...

peux tu envoyer le .h et le .cpp complet ?

n'as tu pas inclus string ou map a l'interieur d'un namespace a toi ou un truc du genre ?

Merci MonGauois !
Un operator semble être la solution :

Code:

---

1 2 3 4 5

cStatus& cStatus::operator = (cStatus& s) { cStatus st(s.m_name,s.m_nbframes,s.m_visible,s.m_animated,s.m_looping,s.m_x,s.m_y); return (st); }

---

La seule chose c'est qu'il me fait un warning comme quoi je retourne la référence de st... ?

Citation:

---

Envoyé par Mindiell

Merci MonGauois !
Un operator semble être la solution :

Code:

---

1 2 3 4 5

cStatus& cStatus::operator = (cStatus& s) { cStatus st(s.m_name,s.m_nbframes,s.m_visible,s.m_animated,s.m_looping,s.m_x,s.m_y); return (st); }

---

La seule chose c'est qu'il me fait un warning comme quoi je retourne la référence de st... ?

---

Dans ton code st est créé au debut de ta fonction puis détruite aprés le return. Donc retourné une reference sur un objet détruit c'est soit ca passe parce que t'as pris un café sous une lune verte et que t'as de la change, soit ça fera un segmentation fault (ou un truc comme ca) :lol:
ton "operator =" devrait plustôt ressemblé a :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

cStatus& cStatus::operator = (const cStatus& s) { m_name = s.m_name; m_nbframes = s.m_nbframes; m_visible = s.m_visible; m_animated = s.m_animated; m_looping = s.m_looping; m_x = s.m_x; m_y = s.m_y; return (*this); }

---

Pour la surcharge de l'operateur = le mieux est encore d'utiliser cette technique qui permet d'éviter l'auto-affectaion et autres pièges.

Sauf que là c'est inutile, la classe ne gère aucune ressource. D'ailleurs cet opérateur d'affectation est lui-même inutile, celui généré par défaut sera exactement identique. J'ai du mal à croire que le fait de le définir explicitement change quelque chose au problème :|

Citation:

---

Envoyé par Laurent Gomila

D'ailleurs cet opérateur d'affectation est lui-même inutile, celui généré par défaut sera exactement identique.

---

tu est sur? vu que name est une std::string

Citation:

---

Envoyé par Laurent Gomila

J'ai du mal à croire que le fait de le définir explicitement change quelque chose au problème :|

---

J'avais déjà eu des problèmes similaire... mais cela dépendais du compilot...

Citation:

---

tu est sur? vu que name est une std::string

---

Du moment que std::string possède un opérateur = ce qui est évidemment le cas), ça ne pose pas de problème puisque c'est ce que le compilo appelle pour chaque membre.

Bon,
je vérifierais ca ce soir tranquillement. Au bureau, c'est pas pratique :)
Merci pour vos posts très intéressants...

En effet !
Sans ces maudits constructeurs de copie et autres opérateur, ca passe :)
En fait, il semble qu'il n'est pas prévu d'avoir des membres const dans une classe :aie:

Et j'en avais mis beaucoup, un peu n'importe comment... J'ai tout retiré, ca passe sans problème ! Je vais donc conserver les const pour les retours de valeur à travers ma classe et c'est tout :mrgreen:

Un membre constant empêche ta classe d'avoir un opérateur d'affectation, ce qui la rend incompatible avec les conteneurs standards.

Oui, j'ai bien compris maintenant :)
Je laisse donc seulement certaine fonctions en const et je n'en mets plus partout, merci ! :king: