Копировать массив LPTSTR

Question

У меня есть структура, которая представляет собой массив LPSTR и определяются как так:

struct MyStruct { 
    DWORD cbString; 
    LPTSTR * pbString; 
} 

Мне нужно скопировать эту структуру на другую идентичную структуру. Мой вопрос: как распределить память для целевой структуры? Моим первым инстинктом было бы следующее:

destStruct.pbString = (LPTSTR*)malloc(totalSizeOfSrcStrings * sizeof(TCHAR)) 

for(int i = 0; i < srcStruct.cbString; i++) { 
    destStruct.pbString[i] = (LPTSTR)malloc(_tcslen(srcStruct.pbString[i] * sizeof(TCHAR)); 
} 

Я не уверен, нужен ли код в цикле for.

Answer 1

Как обычно, в то время как у вас нет выбора но для того, чтобы привести результат malloc в C++, вы все же можете избежать бесполезной практики использования typenames под sizeof. В вашем случае ошибка в вашем первом malloc вызвана именно тем, что: для некоторого необъяснимого резонанса вы использовали sizeof(TCHAR), тогда как правильный размер элемента на самом деле sizeof(LPTSTR).

Опять же, не используйте typenames под sizeof. Ваш оригинальный malloc можно переписать в более безопасным способом, как

destStruct.pbString = 
    (LPTSTR*) malloc(totalSizeOfSrcStrings * sizeof *destStruct.pbString); 

Цикл может выглядеть следующим образом

for(int i = 0; i < srcStruct.cbString; i++) 
    destStruct.pbString[i] = 
    (LPTSTR) malloc((_tcslen(srcStruct.pbString[i]) + 1) * sizeof *destStruct.pbString[i]); 

P.S. Конечно, приведение в стиле C в этом случае также может быть плохой идеей, но код выглядит как нечто, что, возможно, необходимо перекрестно скомпилировать.

Answer 2

Как Brightstar отметил, что должно быть SizeOf (Char), не SizeOf (TCHAR)

LPTSTR = TCHAR *

LPSTR = CHAR *