用char *构建动态结构的更好方法？

Question

我想使用C++其采用可变数量结构与字符C风格的API *成员如下：

typedef struct _FOO 
{ 
    const char * name; 
    const char * value; 
} FOO; 

void ApiFunc(FOO const* args, unsigned count); 

为了填补这个参数，我需要循环一些其他的数据，并创建FOO在飞行中的条目。什么是最优雅的方式来做到这一点？

下面的办法似乎在第一个简单的，但不工作（因为字符串实例走出去的范围和调用之前被破坏到ApiFunc（））：

// Approach A: this does *not* work 
std::vector<FOO> args; 

for (...) 
{ 
    string name = ... // something which gets 
    string value = ... // calculated in the loop 
    args.push_back(FOO{name.c_str(), value.c_str()}); 
} 

ApiFunc(args.data(), args.size()); 

把字符串对象中矢量（以防止它们被破坏）无法正常工作或 - 作为投入的载体时，琴弦被复制，并与原有的仍然破坏：

// Approach B: this also does *not* work 
std::vector<string> strings; 
for (...) 
{ 
    string name = ... // something which gets 
    string value = ... // calculated in the loop 
    strings.push_back(name); 
    strings.push_back(value); 
    args.push_back(FOO{name.c_str(), value.c_str()}); 
} 

ApiFunc(args.data(), args.size()); 

我可以防止通过创建字符串对象在堆上并使用auto_ptr来保持trac其中有k个，但是有更好的方法吗？

// Approach C: this does work 
std::vector<auto_ptr<string>> strings; 
for (...) 
{ 
    string* name = new ... 
    string* value = new ... 
    strings.push_back(auto_ptr<string>(name)); 
    strings.push_back(value); 
    args.push_back(FOO{name.c_str(), value.c_str()}); 
} 

ApiFunc(args.data(), args.size()); 

虽然方法C.似乎工作，我觉得它很不明显/难以理解。任何建议如何我可以改善它？

Answer 1

我很肯定std::vector<auto_ptr<T>>是不允许的标准。改为使用unique_ptr。或者，在向量中构建字符串，然后从中构建参数。

std::vector<std::pair<std::string, std::string>> strings; 
for (...) 
{ 
    const std::string name = ...; 
    const std::string value = ...; 
    strings.push_back(std::make_pair(name, value)); 
} 
// Note: This loop must be separate so that 'strings' will not reallocate and potentially 
// invalidate the pointers returned by elements in it. 
for (const auto& pair : strings) 
{ 
    args.push_back(FOO{pair.first.c_str(), pair.second.c_str()}); 
} 

ApiFunc(args.data(), args.size()); 

Answer 2

您可以使用存储在矢量字符串：

string name = ... // something which gets 
    string value = ... // calculated in the loop 
    strings.push_back(name); 
    strings.push_back(value); 
    args.push_back(FOO{strings[strings.size()-2].c_str(), strings[strings.size()-1].c_str()}); 

Answer 3

可以使用strdup功能。

string name = ... // something which gets 
string value = ... // calculated in the loop 
args.push_back(FOO{strdup(name.c_str()), strdup(value.c_str())}); 

之后，您将不得不删除字符串。

free(foo.name); 
free(foo.value); 

Answer 4

由于您使用的是C API接口，因此您将不得不采用C API方式进行操作。也就是说，分配堆缓冲区并在之后释放它们。这实际上是unique_ptr<T[]>效用的一个最好的例子：

//Helper function 
std::unique_ptr<char[]> cpp_strdup(const std::string &str) 
{ 
    std::unique_ptr<char[]> ret = new char[str.size() + 1]; 
    //Must copy the NUL terminator too. 
    std::copy(str.data(), str.data() + str.size() + 1, ret.get()); 
    return ret; 
} 

//In your function: 
std::vector<unique_ptr<char[]>> strings; 

for (...) 
{ 
    string name = ... // something which gets 
    strings.push_back(cpp_strdup(name)); 

    string value = ... // calculated in the loop 
    strings.push_back(cpp_strdup(value)); 

    args.push_back(FOO{strings[strings.size() - 2].get(), strings[strings.size() - 1].get()}); 
} 

通过在strings矢量使用unique_ptr代替std::string，你巧妙地避免重新分配字符串本身时strings进行再分配。