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を使用することができますvector（他の回答で示したようにstd::copy又はstd::back_inserterを使用してコードを複雑にする必要はありません）の終わりにstring、例えば：

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); 
}