防止循环内外的代码重复

Question

我在重写循环时遇到问题：

else if( "d" == option || "debug" == option )
{
    debug(debug::always) << "commandline::set_internal_option::setting debug options: "
                         << value << ".\n";
    string::size_type index = 0;
    do
    {
        const string::size_type previous_index = index+1;
        index=value.find( ',', index );
        const string item = value.substr(previous_index, index);
        debug::type item_enum;
        if( !map_value(lib::debug_map, item, item_enum) )
            throw lib::commandline_error( "Unknown debug type: " + item, argument_number );

        debug(debug::always) << "commandline::set_internal_option::enabling " << item
                             << " debug output.\n";
        debug(debug::always) << "\n-->s_level=" << debug::s_level << "\n";
        debug::s_level = static_cast<debug::type>(debug::s_level ^ item_enum);
        debug(debug::always) << "\n-->s_level=" << debug::s_level << "\n";
    } while( index != string::npos );
}

value 类似于 string("commandline,parser") ，问题是在第一次运行时，我需要 substr (previous_index, index)，但在后续的每次迭代中，我需要 substr(previous_index+1, index) 来跳过逗号。是否有一些我忽略的简单方法，或者我是否必须在初始迭代的循环外部重复调用 find ？

Answer 1

由于您的目标是防止代码重复：

std::vector<std::string> v;
boost::split(v, value, [](char c) { c == ','; });

如果您想创建自己的 split 函数，您可以执行以下操作：

template<typename PredicateT>
std::vector<std::string> Split(const std::string & in, PredicateT p)
{
    std::vector<std::string> v;
    auto b = in.begin();
    auto e = b;
    do {
        e = std::find_if(b, in.end(), p);
        v.emplace_back(b,e);
        b = e + 1;        
    } while (e != in.end());

    return v;
}

Answer 2

为什么不在获取 substr 后更新 previous_index？

string::size_type index = 0;
string::size_type previous_index = 0;
do {
  index=value.find( ',', previous_index );
  const string item = value.substr(previous_index, index);
  previous_index = index+1;
} while( index != string::npos );

未经检查，但这应该可以解决问题（只需多记住一个单词）。

Answer 3

从 -1 开始？

string::size_type index = -1;
do
{
    const string::size_type previous_index = index + 1;
    index=value.find(',', previous_index);
    const string item = value.substr(previous_index, index - previous_index);
} while( index != string::npos );

Answer 4

一个愚蠢的（而且有点不可读）的解决方案是这样的：

string::size_type once = 0;
/* ... */
const string::size_type previous_index = index+1 + (once++ != 0); // or !!once

Answer 5

首先，我认为有一个小错误：

在您的代码中，表达式 index=value.find( ',', index ); 不会更改 index 的值如果它已经是字符串中逗号字符的索引（除了第一次循环迭代之外总是这种情况）。

因此，您可能需要将 while(index != string::npos ); 替换为 while(index++ != string::npos ); 和 previous_index = index +1 与 previous_index = index。

这也应该可以解决您原来的问题。

澄清一下：

string::size_type index = 0;
do
{
    const string::size_type previous_index = index;
    index = value.find( ',', index );
    const string item = value.substr(previous_index, index - previous_index);
} while( index++ != string::npos );