是否有更簡潔的方式將std :: string合併到std :: vector <char>？

Question

我有一些代碼將各種元素組合到一個緩衝區中。我的代碼看起來是這樣的：

static void CreatePacket(const std::string& source, const std::string id, const std::string payload, std::vector<char>& buffer) 
{ 
    buffer.resize(source.size() + id.size() + payload.size()); 
    std::vector<char>::iterator bufferDest = buffer.begin(); 

    // Start the message 
    char MessageStart = '$'; 
    *bufferDest = MessageStart; 
    ++bufferDest; 

    // Copy the message source 
    std::copy(source.begin(), source.end(), bufferDest); 
    bufferDest += source.size(); 

    // Copy the message id 
    std::copy(id.begin(), id.end(), bufferDest); 
    bufferDest += id.size(); 
} 

調用該方法如下：

std::vector<char> buffer; 

std::string source = "AB"; 
std::string id = "CDE"; 
std::string payload = "payload"; 

CreatePacket(source, id, payload, buffer); 

我還是有點綠做事的方式std，但我感覺實現有點笨重（具體而言，必須在每個副本之後明確增加bufferDest）。有沒有更乾淨的方法來做到這一點？

如果這有所幫助，我的編譯器不支持C++ 11。

Answer 1

你可以只用一個適當的vector::insert() overload到的內容追加string在的vector結束（不需要在其他的答案顯示使用std::copy或std::back_inserter代碼複雜化），例如：

buffer.insert(buffer.end(), source.begin(), source.end()); 

所以，你的功能應該是這樣的：

void CreatePacket(const std::string& source, 
        const std::string& id, 
        const std::string& payload, 
        std::vector<char>& buffer) 
{ 
    buffer.clear(); 
    buffer.reserve(source.size() + id.size() + payload.size() + 1); 

    buffer.push_back('$'); 

    buffer.insert(buffer.end(), source.begin(), source.end()); 
    buffer.insert(buffer.end(), id.begin(),  id.end() ); 
    buffer.insert(buffer.end(), payload.begin(), payload.end()); 
} 

Answer 2

我覺得這樣更清晰。

void CreatePacket(const std::string& source, const std::string& id, const std::string& payload, std::vector<char>& buffer) 
{ 
    buffer.clear(); 
    buffer.reserve(source.size() + id.size() + payload.size() + 1); 

    buffer.push_back('$'); 

    std::copy(source.begin(), source.end(), std::back_inserter(buffer)); 
    std::copy(id.begin(), id.end(), std::back_inserter(buffer)); 
    std::copy(payload.begin(), payload.end(), std::back_inserter(buffer)); 
} 

Answer 3

這是除了你可以從std::copy使用返回的值，從而擺脫bufferDest明確的增量幾乎是乾淨的：

static void CreatePacket(const std::string& source, const std::string id, const std::string payload, std::vector<char>& buffer) 
{ 
    buffer.resize(source.size() + id.size() + payload.size()); 
    std::vector<char>::iterator bufferDest = buffer.begin(); 

    // Start the message 
    char MessageStart = '$'; 
    *bufferDest = MessageStart; 
    ++bufferDest; 

    // Copy the message source 
    bufferDest = std::copy(source.begin(), source.end(), bufferDest); 

    // Copy the message id 
    bufferDest= std::copy(id.begin(), id.end(), bufferDest); 
}