迭代到 std::vector<string>

Iteration into std::vector<string>

我有一个 vector 个字符串参数...

|name1|value1|name2|value2|...

我想迭代并将名称缓存到一个字符串中,并将其添加到名称的 vector 中,并用 value 做同样的事情。他们在 std::vector<string>.

我做到了:

std::vector<string> names;
std::vector<string> values;
std::vector<string>::iterator pit = p.begin();

while(pit != p.end()){
   string name = *pit;
   pit++;
   string value = *pit;
   pit++;
   names.push_back(name);
   values.push_back(value);
}

但它 returns vector 中的访问冲突。它正在访问返回 <BadPtr> 的错误位置。 这个迭代怎么做? 它有办法为每个人使用吗?

看看这个:

std::vector<string> names;
std::vector<string> values;
std::vector<string>::iterator pit = p.begin();

while(pit != p.end()){
   string name = *pit;
   pit++;
   if(pit == p.end())
   break;
   string value = *pit;
   pit++;
   names.push_back(name);
   values.push_back(name);
}

正如我在评论中所说,问题可能是,您在第二次递增 pit 后没有进行任何检查。

这是一个演示程序,展示了如何使用标准算法完成 std::partition_copy

#include <iostream>
#include <vector>
#include <string>
#include <iterator>
#include <algorithm>

int main()
{
    std::vector<std::string> p = { "name1", "value1", "name2", "value2" };
    std::vector<std::string> names;
    std::vector<std::string> values;

    names.reserve( ( p.size() + 1 ) / 2 );
    values.reserve( p.size() / 2 );

    unsigned int i = 0;

    std::partition_copy( p.begin(), p.end(),
                         std::back_inserter( names ),
                         std::back_inserter( values ),
                         [&]( const std::string & ) { return i ^= 1; } );

    for ( const auto &s : p ) std::cout << s << ' ';
    std::cout << std::endl;

    for ( const auto &s : names ) std::cout << s << ' ';
    std::cout << std::endl;

    for ( const auto &s : values ) std::cout << s << ' ';
    std::cout << std::endl;

    return 0;
}

程序输出为

name1 value1 name2 value2 
name1 name2 
value1 value2 

同样可以使用基于范围的 for 语句

#include <iostream>
#include <vector>
#include <string>

int main()
{
    std::vector<std::string> p = { "name1", "value1", "name2", "value2" };
    std::vector<std::string> names;
    std::vector<std::string> values;

    names.reserve( ( p.size() + 1 ) / 2 );
    values.reserve( p.size() / 2 );

    unsigned int i = 0;

    for ( const std::string &s : p )
    {
        if ( i ^= 1 ) names.push_back( s );
        else values.push_back( s );
    }

    for ( const auto &s : p ) std::cout << s << ' ';
    std::cout << std::endl;

    for ( const auto &s : names ) std::cout << s << ' ';
    std::cout << std::endl;

    for ( const auto &s : values ) std::cout << s << ' ';
    std::cout << std::endl;

    return 0;
}

因此您的循环看起来更简单,如

    unsigned int i = 0;

    for ( const std::string &s : p )
    {
        if ( i ^= 1 ) names.push_back( s );
        else values.push_back( s );
    }

如您所见,如果使用您的方法,循环体仅包含两个语句而不是六个或八个语句。