«MessageBoxA»: не удается преобразовать параметр 2 из «станд :: вектор <_Ty>» к «» LPCSTR

Question

Следующий код работает:

void CMyPlugin8::myMessageBox(std::string& myString) 
{ 
    myString = "Received the following string\n" + myString; 
    char * writable = new char[myString.size() + 1]; 
    std::copy(myString.begin(), myString.end(), writable); 
    writable[myString.size()] = '\0'; // don't forget the terminating 0 "delete[] writable;" 

    int msgboxID = MessageBox(
     NULL, 
     writable, 
     "Notice", 
     MB_OK 
    ); 
    delete[] writable; 
} 

Для очистки автоматически я информацию из: How to convert a std::string to const char* or char*?.

Следующий код выдает ошибку:

void CMyPlugin8::myMessageBox(std::string& myString) 
{ 
    myString = "Received the following string\n" + myString; 
    std::vector<char> writable(myString.begin(), myString.end()); 
    writable.push_back('\0'); 

    int msgboxID = MessageBox(
     NULL, 
     writable, 
     "Notice", 
     MB_OK 
    ); 
} 

Я получаю эту ошибку: 'MessageBoxA': не удается преобразовать параметр 2 из 'станд :: вектор < _Ty>' на '' LPCSTR

Answer 1

MessageBox принимает const char*. Вам не нужно сначала копировать строку. Просто используйте c_str:

void CMyPlugin8::myMessageBox(std::string& myString) 
{ 
    myString = "Received the following string\n" + myString; 
    int msgboxID = MessageBox(
     NULL, 
     myString.c_str(), 
     "Notice", 
     MB_OK 
    ); 
} 

Обратите внимание, что я думаю, что ваш API беден. Вы изменяете значение строки, переданной в Обычно звонящий не будет ждать, что. Я думаю, что ваша функция должна выглядеть следующим образом, вместо:

void CMyPlugin8::myMessageBox(const std::string& myString) 
{ 
    std::string message = "Received the following string\n" + myString; 
    int msgboxID = MessageBox(
     NULL, 
     message.c_str(), 
     "Notice", 
     MB_OK 
    ); 
} 

Answer 2

Вы не можете передать вектор в качестве LPCSTR, вам необходимо. Использование:

&writable[0] 

или:

writable.data() 

вместо этого. Или просто используйте myString.c_str()

Answer 3

void CMyPlugin8::myMessageBox(const std::string& myString) 
{ 
    std::string message = "Received the following string\n" + myString; 
    int msgboxID = MessageBox(
     NULL, 
     message.c_str(), 
     "Notice", 
     MB_OK 
    ); 
} 

Спасибо всем & @Falcon