используйте вектор <wchar>вместо динамически распределяемого массива wchar - PullRequest
Новая
       1

используйте вектор <wchar>вместо динамически распределяемого массива wchar

0 голосов
/ 18 апреля 2011

На днях меня уволили (по поводу stackoverflow!) За то, что я не использовал vector вместо динамически размещаемого массива wchar.

Так что я взглянул на использование этого метода манипуляции со строками, как кажетсякак хорошая идея для предотвращения возможных утечек памяти.

Я придумал, что если я не использую класс векторного шаблона неправильно, использование вектора гораздо менее гибко, чем использование массива, выделенного в куче, и старого доброгоmemcpy. 

#include <shlobj.h>
HRESULT ModifyTheme()
{
using namespace std;

vector <WCHAR>  sOutput;
vector <WCHAR>  sPath;      
vector <WCHAR>  sThemesLocation;
vector <WCHAR>  sThemeName; 

const WCHAR sThemesPath []  = _T("\\Microsoft\\Windows\\Themes");
const WCHAR sFileName []    = _T("\\darkblue.theme");

sOutput.resize(MAX_PATH);
sPath.resize( MAX_PATH );   
sThemesLocation.resize( MAX_PATH );
sThemeName.resize( MAX_PATH );

// Get appdata\local folder
SHGetFolderPath(NULL, CSIDL_LOCAL_APPDATA, NULL, 0, &sPath[0] );

// copy consts to vectors   
memcpy( &sThemesLocation[0],    sThemesPath,    sizeof(sThemesPath) );
memcpy( &sThemeName[0],         sFileName,      sizeof(sFileName) );    

// append themes path & filename
sOutput.insert( sOutput.begin(), sPath.begin(), sPath.end() );
sOutput.insert( sOutput.end()-1, sThemesLocation.begin(), sThemesLocation.end() );
sOutput.insert( sOutput.end()-1, sThemeName.begin(), sThemeName.end() );    

wcout << &sThemeName[0] << endl;
wcout << &sThemesLocation[0] << endl;
wcout << &sPath[0] << endl;
wcout << &sOutput[0] << endl;

return S_OK;
}

Я ожидаю, что вектор sOutput будет содержать конкатенацию всех строк.Вместо этого он содержит только первую вставленную строку.

Кроме того, я помню, что слышал, что, хотя невозможно присвоить значения вектора в списке инициализатора, это может быть функцией c ++ 0x.Это правильно - и есть ли способ (на минуту) сделать следующее: 

vector<wchar> sBleh = { _T("bleh") };

Наконец, для того, что я хочу достичь с помощью простой процедуры, описанной выше, я был бы лучше с динамически распределенныммассив, или я должен остаться с, казалось бы, негибким вектором wchar?

Ответы [ 2 ]

4 голосов
/ 18 апреля 2011

Если вы используете std::vector<WCHAR>, вам, вероятно, следует использовать std::wstring, так как он также является контейнером WCHAR элементов.

Вам могут помочь следующие ссылки:
std :: wstring (typedef of std::basic_string<WCHAR>)
std :: basic_string

1 голос
/ 18 апреля 2011

Используйте лучший инструмент для работы. Некоторые ситуации требуют использования статических массивов, а некоторые для динамических массивов. Когда ситуация требует динамического массива, используйте вместо него вектор.

Пометить Ingram правильно, что вы можете использовать wstring, но только если wchar_t того же размера, что и WCHAR.

Что-то вроде этого лучше для того, что вы хотите (заметьте, ниже я не запускал компилятор из-за слишком большого количества специфических конструкций Microsoft.) 

WCHAR sPath[MAX_PATH]; // doesn't need to be a dynamic array, so don't bother with a vector.
SHGetFolderPath(NULL, CSIDL_LOCAL_APPDATA, NULL, 0, &sPath[0] );

const WCHAR sThemesPath[] = _T("\\Microsoft\\Windows\\Themes"); // doesn't need to be a dynamic array, so don't bother with a vector.
const WCHAR sFileName[] = _T("\\darkblue.theme"); // doesn't need to be a dynamic array, so don't bother with a vector.
vector<WCHAR> sOutput; // this needs to be dynamic so use a vector.

// wcslen should probably be replaced with an MS specific call that gets the length of a WCHAR string
copy(sPath, sPath + wcslen(sPath), back_inserter(sOutput));
copy(sThemesPath, sThemesPath + wcslen(sThemesPath), back_inserter(sOutput));
copy(sFlieName, sFileName + wcslen(sFileName), back_inserter(sOutput));
...