1

有没有更简单的方法来解决这个问题,或者我做错了什么?我认为我的问题的核心在于代码,

vector<int> &v = miss_words[*i];

但也许我只是把整个概念弄错了。有什么建议么?

代码:

void print_map(map<string, vector<int> > miss_words)    // Prints out dictionary set
{
    map<string, vector<int> >::iterator it = miss_words.begin();    // Creates an iterator
    while(it != miss_words.end())           // While hasn't reached the end
    {
        vector<int> &v = miss_words[*it];   // Accesses Vector in map
        for(unsigned int g = 0; g <= v.size(); g++)
        {
            cout<<v.at(g)<<": ";            
            cout<<v.at(g)<<" "<<endl;   // Print out data at i
        }
        it++;                   // Increment iterator
    }
}

编译器表示“miss_words 中的 'operator[]' 不匹配。

4

1 回答 1

4

你想说vector<int> &v = it->second;

迭代器的值类型是映射的值类型,即pair<string, vector<int>>.


实际上,在现代 C++ 中,您可以更简单地编写此代码,并且更不容易出错,如下所示:

for (auto const & p : miss_words)
{ 
    for (auto const & x : p.second)
    {
        cout << x << ": " << x << " \n";
    }
}

这样,您甚至不必担心矢量的大小。

于 2013-07-02T23:32:48.430 回答