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

3

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

answered May 28, 2009 at 11:11

xtofl

41.7k13 gold badges112 silver badges203 bronze badges

Comments

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

2

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.

answered May 28, 2009 at 11:12

Dietrich Epp

216k39 gold badges366 silver badges426 bronze badges

1 Comment

Dave Van den Eynde Over a year ago

Using the very latest MSVC helps too, you know.

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

1

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.

answered May 28, 2009 at 11:13

Dave Van den Eynde

17.5k7 gold badges63 silver badges91 bronze badges

Comments

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

1

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.

answered May 28, 2009 at 11:37

James Hopkin

14k1 gold badge46 silver badges73 bronze badges

Collectives™ on Stack Overflow

std::multimap compile errors

5 Answers 5

2 Comments

Comments

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

2 Comments

Comments

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related