※ 引述《Keitaro (动き出す时间...)》之铭言：
: 开发平台(Platform): (Ex: Win10, Linux, ...)
: Ubuntu 18.04 LTS
: 编译器(Ex: GCC, clang, VC++...)+目标环境(跟开发平台不同的话需列出)
: gcc
: 程式码(Code)：(请善用置底文网页, 记得排版，禁止使用图档)
: https://ideone.com/jugcYC
: 重新上传完整原始码
后面恕删, 因为看起来不像 C++ 所以路过分享一下拙见.
C++ 物件不仅仅是一块内存, 因为生命周期中会有建构子/解构子参与, 简单
说就是至少会比 C 语言多两次函式呼叫. 所以撰写的时候不会轻易地创造物件
, 更不会创完而不用它. array new/delete 这种昂贵的操作其实是蛮罕见的.
你的程式码主要存在两个问题:
1. ParseCmd() 的实作没有弹性, 且效率不高
2. 准备 execve() 引数时做了许多不必要的操作
一般为了减少对 STL container 的依赖, 函式接口会倾向于接受迭代器而不是
特定容器型别物件, 而且除非你很确定要用 std::char_traits<CharT>::eq()
来比较字符是否相等, 不然我们在搜寻的时候倾向于呼叫 STL algorithm 而非
std::basic_string 这种内嵌比较逻辑的型别, 所以通常接口会这样设计:
template <typename ForwardIterator, typename ForwardSentinel,
typename OutputIterator>
void ParseCmd(ForwardIterator first, ForwardSentinel last,
OutputIterator output) {
while (first != last) {
// call std::find() here
// other code goes here
(*output++) = std::string(first, last);
}
}
#include <iterator>
const std::string str = "-vsync 0 -i file.cfg Compare.yuv";
std::vector<std::string> vCmdSet;
ParseCmd(begin(str), end(str), std::back_inserter(vCmdSet));
std::basic_string 只作为字符的载体, 而不是兼做比较的角色.
再来是准备 execve() 引数的逻辑, 如果非得用 array new/delete 来管理物
件, 那前面呼叫 ParseCmd() 创建一堆 std::string 物件有什么用呢? 有没
有办法重复使用这些物件的内容呢? 实际上你可以呼叫 std::string::data()
成员函式来达成这件事情, 然后把拿到的指标存进另一个 std::vector 内:
#include <algorithm>
#include <iterator>
std::vector<char*> ppCmdArg;
std::transform(begin(vCmdSet), end(vCmdSet),
std::back_inserter(ppCmdArg),
[](std::string& s) { return data(s); });
ppCmdArg.push_back(nullptr);
execve(..., data(ppCmdArg), ...);
这里用到的概念主要有两个: 1) std::vector 的 data() 成员函式会回传第
0 个元素的位址, 而且每个成员的位址保证是连续的, 所以回传的东西和你用
array new 创建出来的 array of pointers 是相同的; 2) 再者 std::string
的 data() 成员函式会回传第 0 个字符的位址, 每个字符的位址也是连续的,
而且回传的内存和 c_str() 相同 (只差在 constness), 包含结束字符.
下次撰码的时候可以试着写出无 new/delete 的程式码, 因为偏底层的函式呼
叫次数愈多, 发生错误的机会也跟着变多.