Erstmal der Code:

int MageTalentBaum::getTalentPoints(const string &talent) const{
	return talents[talent];
}

talents ist eine std::mapstd:.string,int.

Als Fehlermeldung kommt dann:

.\talentbaum.cpp(89) : error C2678: Binärer Operator '[': Es konnte kein Operator gefunden werden, der einen linksseitigen Operanden vom Typ 'const std::map<_Kty,_Ty>' akzeptiert (oder keine geeignete Konvertierung möglich)
1> with
1> [
1> _Kty=std::string,
1> _Ty=int
1> ]
1> C:\Programme\Microsoft Visual Studio 8\VC\include\map(166): kann 'int &std::map<_Kty,_Ty>::operator [](const std::basic_string<_Elem,_Traits,_Ax> &)' sein
1> with
1> [
1> _Kty=std::string,
1> _Ty=int,
1> _Elem=char,
1> _Traits=std::char_traits<char>,
1> _Ax=std::allocator<char>
1> ]
1> bei Anpassung der Argumentliste '(const std::map<_Kty,_Ty>, const std::string)'
1> with
1> [
1> _Kty=std::string,
1> _Ty=int
1> ]

Naja, ich vermute in der Fehlermeldung steht, was ich falsch mache, ich finde es aber nicht heraus. Ich hätte jetzt ja gedacht, dass es daran liegt, dass ich die Methode const gemacht habe, weisz aber nicht, was ich dann dagegen machen könnte, wenn es so wäre.

Bitte um Hilfe (wen wunderts, sonst hätte ich es nicht hier gepostet) Sollte der Fehler woanders liegen bitte sagen, dann versuche ich es mit etwas mehr code.

PS: in welchem Header befindet sich make_pair?

Edit: PS: std::pair<> und make_pair() findest du in <utility>

Darf ich fragen inwiefern der unter Umständen ein neues Element anlegen muss? Legt der, wenn es das Element nicht gibt, ein neues an??? Dann bräuchte man insert ja gar nicht mehr.

EDIT: achja, sorry wegen der headerfrage, aber irgendwie scheint es den utility Header und seine Inhalte auf www.cppreference.com (was ich oft als Quick-Referenz nutze) nicht zu geben.

Vielen Dank CStoll, dann hantiere ich eben mit find rum.

Darf ich fragen inwiefern der unter Umständen ein neues Element anlegen muss? Legt der, wenn es das Element nicht gibt, ein neues an??? Dann bräuchte man insert ja gar nicht mehr.

Ja, wenn du op[] einen Schlüssel gibst, der noch nicht existiert, trägt er ihn neu in die map ein (den Wert dazu holt er sich über den Default-Ctor). Und ja, das kann insert() in gewissen Grenzen ersetzen, macht es aber nicht überflüssig.

EDIT: achja, sorry wegen der headerfrage, aber irgendwie scheint es den utility Header und seine Inhalte auf www.cppreference.com (was ich oft als Quick-Referenz nutze) nicht zu geben.

Ist vielleicht nicht der richtige Zeitpunkt, aber ich würde dir meine Artikelserie "Aufbau der STL" ans Herz legen (zu finden bei uns im Magazin).

Darf ich fragen inwiefern der unter Umständen ein neues Element anlegen muss? Legt der, wenn es das Element nicht gibt, ein neues an??? Dann bräuchte man insert ja gar nicht mehr.

map kann nicht zwischen den beiden folgenden Situationen unterscheiden:

map<string, string>foo;
foo["a"] = "b"; // wird ein neues Element angelegt
foo["c"] = "d";

map<string, string>bar;
if(bar["a"] == "rrr"){
  //...
}

In der ersten erwartet man dass op[] ein Element hinzufügt und bei der zweiten will man hauptsächlich den Wert testen. Da es sich aber um den gleichen op[] handelt muss er eben in beiden Situationen ein neues Objekt anlegen auch wenn es im zweiten Fall suboptimal ist.

Da der op[]const aber nichts hinzufügen kann (soll) gibt es für ihn kein gescheites Verhalten im Fall wo der Schlüssel nicht in der map enthalten wäre. Eine Ausnahme ist keine Option da sonst das Verhalten vom op[] und vom op[]const sehr verschieden wären.