Assigning an std::set as a tree's node problem

Hi,

I am having trouble creating a std::set as the datamember in an expression tree's node.

Here is what I have:

Code:

---

//an std::map is where the sets are held
map<string,set<int> > mapname;

//an iterator to the map
map<string,set<int> >::iterator mapnameit;

//create a std::set
set<int> setone;

setone.insert(1);
setone.insert(2);
setone.insert(3);

string keyone = "a1";

mapname[keyone] = setone;

//Now have a std::map containing one element of: <string,set<int> >

//Find the element in the map
mapnameit = mapname.find("a1");

//call the node constructor, setting the map's std:: set data member ( 1,2,3 ) as
//the nodes data member
ExTree* mynode = new ExTree(*mapnameit);

//Now the node should contain the set {1,2,3}, but fails to build.

---

I'm stuck with trying to assign the set from the map as the nodes value.

Visual Studio says:

Quote:

---

cannot convert parameter 1 from 'std::pair<_Ty1,_Ty2>' to 'std::string'

---

Thanks for any help, it's most appreciated :)

So, look at your error message: it is trying to convert a std::pair<> into a std::string. And if you look at the bolded line, *mapnameit is definitely a pair (being the dereference of a map iterator). Therefore you need to make a constructor for ExTree that accepts a std::pair<> item, instead of just the one that you apparently have that takes a std::string.

Why is your ExTree constructor expecting to take a string?
If you think that dereferencing the mapnameit iterator gives you a string, then wouldn't you also expect that string to be "a1", since that's the item you told it to find? It then makes no sense to call find at all since you are expecting to get what you started with.

In fact dereferencing the iterator actually gives you a std::pair so you can access both the key and the mapped value. use ->first to get the key (int your case a string) and ->second to get the mapped value (in your case a set).