std::multimap compile errors

Question

I am trying to use multimap for the first time but my app will not compile. TIA Paul..

// file dept.h typedef std::multimap <CString, std::map< CString, CString> > _DeparmentRecord; // also tryied replacing CString with LPCWSTR _DeparmentRecord DeparmentRecord; // file dept.cpp DWORD CIni::AddNameValue(LPCWSTR Section, LPCWSTR Name, LPCWSTR Value) { DeparmentRecord.insert(std::make_pair ( Section, std::make_pair(Name, Value)) ); <-- error here }

c:\program files\microsoft visual studio 9.0\vc\include\utility(57) : error C2664: 'std::map<_Kty,_Ty>::map(const std::less<_Ty> &)' : cannot convert parameter 1 from 'const std::pair<_Ty1,_Ty2>' to 'const std::less<_Ty> &'

1> with 1> [ 1> _Kty=CString, 1> _Ty=CString 1> ] 1> and 1> [ 1> _Ty1=LPCWSTR, 1> _Ty2=LPCWSTR 1> ] 1> and 1> [ 1> _Ty=CString 1> ] 1> Reason: cannot convert from 'const std::pair<_Ty1,_Ty2>' to 'const std::less<_Ty>' 1> with 1> [ 1> _Ty1=LPCWSTR, 1> _Ty2=LPCWSTR 1> ] 1> and 1> [ 1> _Ty=CString 1> ] 1> No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called 1> c:\dev\projects\migrator\jobbuilder\jobbuilder\ini.cpp(55) : see reference to function template instantiation 'std::pair<_Ty1,_Ty2>::pair>(const std::pair> &)' being compiled 1> with 1> [ 1> _Ty1=const CString, 1> _Ty2=std::map 1> ]

========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

Shino C G · Accepted Answer · 2009-05-28 11:23:02Z

3

Change the function as follows.

DWORD AddNameValue(LPCWSTR Section, LPCWSTR Name, LPCWSTR Value) { std::map<CString, CString> aTemp; aTemp.insert(std::make_pair (Name, Value)); DeparmentRecord.insert(std::make_pair (Section, aTemp)) ; }

answered May 28, 2009 at 11:23

Shino C G

1,26711 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

xtofl Over a year ago

Note: you will replace the entry for Section, not update it.

AngryWhenHungry Over a year ago

But DepartmentRecord is a multimap. So wouldn't a new entry be added?

xtofl · Accepted Answer · 2009-05-28 11:11:53Z

You're trying to insert a pair< section, pair<...> > into a map that takes pair< section, map<...> >.

Dietrich Epp · Accepted Answer · 2009-05-28 11:12:29Z

std::make_pair(Name, Value) is a pair... but it should be a map.

STL errors can be a true pain. Using the very latest version of GCC can help, its error messages are much improved, but I see you're using MSVC so that's not much help to you.

Dave Van den Eynde · Accepted Answer · 2009-05-28 11:13:09Z

In addition to the other answers, std::make_pair will return a std::pair. Don't expect the compiler to perform the conversion from LPCWSTR to CString for you.

James Hopkin · Accepted Answer · 2009-05-28 11:37:01Z

In addition to the other correct answers, you'll avoid conversion problems (and likely get better error messages) if you use _DeparmentRecord's value_type typedef rather than make_pair.

DWORD AddNameValue(LPCWSTR Section, LPCWSTR Name, LPCWSTR Value) { _DeparmentRecord::iterator i = DeparmentRecord.insert(_DeparmentRecord::value_type(Section, v)).first; i->second[Name] = Value; }

And a small point: don't use leading underscores for non-local names - they're reserved for the standard library.

Collectives™ on Stack Overflow

std::multimap compile errors

5 Answers 5

2 Comments

Comments

1 Comment

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

2 Comments

Comments

1 Comment

Comments

Comments

Related