Question

我有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);
}

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

Answer 1

检查出来：

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后没有进行任何检查。

Answer 2

这是一个演示程序，展示如何使用标准算法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

使用基于声明的范围

可以完成相同的操作

#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 );
    }

正如您所看到的那样，如果使用您的方法，循环的主体只包含两个语句而不是六个或八个语句。

迭代到std :: vector <string> </string>

2 个答案: