views:

59

answers:

3

I figured out a somewhat convoluted way to convert a CStringW to a std::string, but I was wondering if there's a cleaner way than:

CStringW cwstr;
std::wstring stdwstr = cwstr;
std::string stdstr = CW2T(stdwstr.c_str());
+3  A: 

You can cut out the intermediate std::wstring:

CStringW cwstr;
std::string stdstr = CW2A(cwstr);

Also note that you want the CW2A macro for correctness. CW2T converts to a TCHAR string, so the code you posted would only compile for an ANSI build (where TCHAR is char).

Nick Meyer
+1  A: 

std::string has the following constructor:

template<class InputIterator> string (InputIterator begin, InputIterator end);

So you could use it:

CStringW cwstr;
std::string x( &cwstr.GetString()[0], &cwstr.GetString()[cwstr.GetLength()] );

Implicit convertion wchar_t to char will take place.

Kirill V. Lyadvinsky
GMan
yes.surely.yes.
Kirill V. Lyadvinsky
adam_0
It doesn't work without the ampersands either. :/
adam_0
Something changed in `CString` since I've used it last time. Fixed. But now it's not so clear.
Kirill V. Lyadvinsky
+2  A: 
std::string str = CStringA( cwstr );

Job done.

Goz