ベクトル<string>をベクトルに変換する方法<char*>

Question

これはHow to convert a vector<char*> to a vector<string>/stringの反対です。

私はvector<char*>で動作するいくつかのレガシールーチンを持っていますので、vector<string>を変換する必要があります。ここで

は、私が出てくるものです：

std::vector<char*> charVec(strVec.size(),nullptr); 
for (int i=0; i<strVec.size();i++) { 
    charVec[i]= new char(strVec[i].size()+1); 
    charVec[i][strVec[i].copy(charVec[i], strVec[i].size())] = '\0'; 
} 

は、この正しいですか？

これを実装するより良い方法はありますか？

---

p.s.もちろん最後に私が持っている：

for (int i=0; i<strVec.size();i++) { 
    delete charVec[i]; 
} 

Answer 1

これを行うためのより高速な方法は

std::vector<const char*> charVec(strVec.size(),nullptr); 
for (int i=0; i<strVec.size();i++) { 
    charVec[i]= strVec[i].c_str(); 
} 

、その後、得られたベクターを使用しています。これにより、大きなデータセットのメモリ割り当てに多くの時間が節約されます。

Answer 2

あなたは単にのstd ::文字列からc文字列を取得するためにstring::c_str()を使用することができます。 How to convert a std::string to const char* or char*?

また、関数の本体をちょっと修正することもできますが、これが価値があるかどうかだけ知ることができます。

---

もちろん、コメントにssellが記載されているので、この方法では文字列を処理する際に文字列を破壊する必要はありません。それはトレードオフですが、あなたはコピーを避けるので（涼しい）、迷惑メールを読まないように注意する必要があります（あなたの文字列を削除する場合に備えて）！

Answer 3

なぜあなたの文字列に新しいバッファを割り当てていますか？関数への文字ポインタのvectorを渡し、最後に割り当てられたバッファを削除しています。これは、これらのバッファが一時的であり、ソースvector（vector<string>）が存在する限り、メモリを割り当てる必要はないことを意味します。

あなたは簡単にこれを行うことができます。

std::vector<std::string> vec1 = { "1", "2", "3" }; 
std::vector<const char*> vec2; 

vec2.resize(vec1.size(), nullptr); 

std::transform(std::begin(vec1), std::end(vec1), std::begin(vec2), [&](const std::string& str) 
{ 
    return str.c_str(); 
}); 

Answer 4

あなたnew文は間違って見えます。文字列を保持するのに十分な長さのcharの配列を割り当てたいとします。文字列の長さを値とする単一のcharを割り当てています。私はあなたがそれについてコンパイラの警告を受け取っていないのに驚いています。

はこれを試してみてください：ここ

std::vector<char*> charVec; 
for (const auto &str : strVec) { 
    char *charStr = new char[str.size() + 1]; 
    std::strcpy(charStr, str.c_str()); 
    charVec.push_back(charStr); 
} 

唯一の欠点は、文字列のいずれかがnull文字が埋め込まれている場合は、それらを越えて何もコピーされません、ということです。しかし、私は、char *のベクトルをとる関数は、とにかくそれについてはほとんど気にしないと思う。