Как пройти unique_ptr <char[]> к функции использовать

Question

Вот в основном то, что я хочу сделать:

SerialPort::ReadBytes(int32& errcode, Message::string& msg, uint32 num) 
{ 
    DWORD numBytesRead = 0; 
    LPDWORD pNumBytesRead = &numBytesRead; 
    errcode = 0;   

    std::unique_ptr <char[]> buff (new char[num]); 
    // ^^^^ pass this char buffer to the ReadFile function below 

    if (!ReadFile(m_sp_pointer,  // Handle to device 
      buff,     // Receives data from device 
      num,     // num bytes to read (in) 
      (LPDWORD)pNumBytesRead, // num bytes read (out) 
      NULL)) 
     { 
      errcode = GetLastError(); 
     } 

    if (numBytesRead > 0) 
    { 
     return true; 
    } 
    return false; 
} 

Я знаю, что я делаю это не правильно, так что мои вопросы: как я могу сделать это правильно и есть ли что-нибудь, что делает это плохой идеей? Заранее спасибо.

Редактировать: Я действительно должен проходить в unique_ptr в параметрах, а не локально объявлять его и передавать в Message::string& msg.

Моя первоначальная попытка состояла в том, чтобы передать Message::string (std::string) ссылкой, так что это вариант тоже, т. Е. Вместо использования unique_ptr вообще. В , что случае я использовал бы регулярный char[] локально, а затем установить msg содержимое на char[] и вернуть его.

Я не уверен, что было бы лучше, кажется, что несколько ответов рекомендуют vector<char>. (Было бы очень похоже на использование std::string).

Answer 1

Я не думаю, что вам нужно std::unique_ptr. Я предпочел бы std::vector в этом случае:

std::vector<char> buff(num); //and pass buff.data() to function 
           //which accepts char* 

if (!ReadFile(m_sp_pointer,    // Handle to device 
     buff.data(),      // Receives data from device 
     buff.size(), //(CHANGED THIS TOO) //num bytes to read (in) 
     (LPDWORD)pNumBytesRead,   // num bytes read (out) 
     NULL)) 
    { 
     errcode = GetLastError(); 
    } 

Обратите внимание, что std::vector::data() доступен с C++ 11 только. В C++ 03 вместо этого вы можете использовать &buff[0].

Answer 2

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

Я предполагаю, что ReadFile использует буфер и не хранит его нигде - так что на самом деле нет аргумента для умного указателя, содержащего эту функцию. Просто passt сырой указатель с

buff.get() 

Далее комментарий:
Это имеет смысл использовать unique_ptr вне ReadFile, потому что он может помочь правильно обрабатывать память. С помощью «ручного управления памятью» вам нужно будет удалить каждый путь, возвращающийся из функций (например, исключений).