Синтаксис указателя строки, это правильно?

Question

Я иду с фона C#, и мой C++ очень ржавый, поэтому, пожалуйста, несите меня.

Следующее определение функции принимает указатели (LPTSTR) на строки, представляющие входное имя файла (szIn) и имя выходного файла (szOut). Вот прототип функции:

ConvertFile(LPTSTR szIn, LPTSTR szOut); 

В настоящее время функция выполняется внутри некоторого кода пользовательского интерфейса, вот пример того, как имя выходного файла получается из указателя szOutput:

TCHAR szOutput[255]; // output filename 

if (g_szFilename[0] != 0) 
{ 
    SetDlgItemText(hMainDlg, IDC_EDIT_INPUT, g_szFilename); 
    _tcscpy(szOutput, g_szFilename); 
    TCHAR * szExt = _tcsrchr(szOutput, '.'); 
    if (szExt != NULL) *szExt = 0; 
    _tcscat(szOutput, _T(".png")); 
} 

Я хочу сделать что-то в этих строках:

TCHAR inputFilename[256]; 
TCHAR outputFilename[256]; 

inputFilename += "somefile.txt"; 
outputFilename += "someotherfile.txt"; 

char *inputPtr; 
char *outputPtr; 

inputPtr = inputFilename; 
outputPtr = outputFilename; 

ConvertFile(inputPtr, outputPtr); 

Является ли это правильным синтаксисом для вкуса Microsoft C++ в Visual Studio 2008?

Answer 1

Нет, вы не можете конкатенировать такую ​​строку. Вы пытаетесь добавить const char[] на конец другого массива ... который даже не будет компилироваться. Вам нужно использовать _tcscat() или (еще лучше, поскольку вы добавили тег C++), std::basic_string<TCHAR>.

Answer 2

Не совсем. Массивы TCHAR уже должны распадаться на указатели TCHAR, что и есть LPTSTR (a TCHAR*). Таким образом, вы не должны использовать указатели char*, если только функция не объявлена ​​для принятия LPCSTR. Короче говоря, вы можете напрямую использовать inputFilename и outputFilename.

Помимо этого незначительного вопроса, реальная проблема заключается в том, что += не будет работать (он попытается выполнить арифметику указателя, если что-нибудь, что совсем не то, что вы хотите). Вместо этого используйте _tcscat():

_tcscat(inputFilename, TEXT("somefile.txt")); 
// ... 

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

еще одна вещь: Вы никогда не инициализировать inputFilename и outputFilename - - Они должны быть инициализированы до некоторой строки, прежде чем вы сможете объединить их.

Answer 3

Это незаконно:

inputFilename += "somefile.txt"; 
outputFilename += "someotherfile.txt"; 

Вы можете использовать _tcscpy() скопировать строки:

TCHAR inputFilename[256] = { 0 }; /* FYI, the '= { 0 }' initialises the array.*/ 
TCHAR outputFilename[256] = { 0 }; 

_tcscpy(inputFilename, "somefile.txt"); 

и использовать _tcscat() конкатенировать:

_tcscat(inputFilename, "another-bit.txt"); 

Вы можете затем перейти к ConvertFile():

ConvertFile(inputFilename, outputFilename); 

Нет необходимости создавать другой указатель на эти переменные (поскольку массивы будут распадаться на указатели).

EDIT:

После изменения языка вы можете использовать std::basic_string:

std::basic_string<TCHAR> inputFilename(TEXT("somefile.txt")); 
std::basic_string<TCHAR> outputFilename(TEXT("someotherfile.txt")); 

И вы можете выполнить конкатенацию, как вы изначально хотели:

inputFilename += TEXT("a-bit-more"); 

И использовать c_str() метод доступ к массиву внутренних символов для перехода на ConvertFile():

ConvertFile(inputFilename.c_str(), outputFilename.c_str()); 

Обратите внимание, что c_str() возвращает const, изменить ConvertFile() к (я сделать предположение, что ConvertFile() не изменяет буферы, передаваемые в):

ConvertFile(LPCTSTR szIn, LPCTSTR, szOut); 

Answer 4

Неа. строки C++ может делать то, что вы пытаетесь сделать:

// Since we don't know whether to use string or wstring, we're stuck 
// with basic_string<TCHAR> 
std::basic_string<TCHAR> inputFilename, outputFilename; 

// Note the TEXT macro...it makes sure your chars are TCHARs 
inputFilename += TEXT("somefile.txt"); 
outputFilename += TEXT("someotherfile.txt"); 

// basic_string::c_str() returns you a pointer suitable for C stuff 
// (since this is a string of TCHARs, we'll get a const TCHAR* back) 
ConvertFile(inputFilename.c_str(), outputFilename.c_str()); 

C символьные массивы не могут быть объединены, как это. Вам нужно будет использовать _tcscat или другие подобные функции.