在std :: list中找到

Question

嗨，我从以下代码获取编译错误。 我不知道：

g++ stl_list_1.cc
stl_list_1.cc: In function \u2018int main()\u2019:
stl_list_1.cc:16: error: \u2018struct std::_List_iterator<int>\u2019 has no member named \u2018clist\u2019
stl_list_1.cc:19: error: \u2018struct std::_List_iterator<int>\u2019 has no member named \u2018clist\u2019
stl_list_1.cc:25: error: no match for call to \u2018(std::list<std::_List_const_iterator<int>, std::allocator<std::_List_const_iterator<int> > >) (int&)\u2019

码：

#include <iostream>
#include <list>
#include <algorithm>
using namespace std;

main()
{
    std::list<int> clist;
    for (int i =0; i<10; i++)
    {
        clist.push_back(i);
    }

    std::list<std::list<int>::const_iterator> list1;

    std::list<int>::iterator itr1;
    itr1 = std::find(clist.begin().clist.end(),1);
    std::list<int>::iterator itr2 ;

    itr2 = std::find(clist.begin().clist.end(),7);

    list1.push_back(itr1);
    list1.push_back(itr2);

    for(int j =0; j< list1.size(); j++)
    {
       int k = *list1[j];
       std::cout << "cvalue " << k <<std::endl;
       j++;
     }
}

谢谢Ruchi

Answer 1

这是一个错字，您使用了点而不是逗号：

itr1 = std::find(clist.begin().clist.end(),1);
                              ^

应该是这样的：

itr1 = std::find(clist.begin(), clist.end(),1);

您在两次调用std::find都犯了这个错误。

另外，您正在尝试在列表上使用operator[] ，这将不起作用。 代替：

int k = *list1[j];

您可以使用：

itr1 = list1.begin();
std::advance(itr1, j);
int k = *itr1;

如JohnB所述，以上代码效率低下。 隔离，是一样的，但是由于您是在循环内使用它，因此最好只使用列表迭代器执行迭代：

for(itr1 = list1.begin(); itr1 != list1.end(); ++itr1)
{
   int k = *itr1;
   std::cout << "cvalue " << k <<std::endl;
}

Answer 2

另外，请记住

list1[j]

不起作用，因为列表没有索引运算符，因此您必须对其进行迭代。

建议：使用std::vector< int >代替std::list< int > 。

Answer 3

至于其他人都注意到有一个在一个错字find电话（ .代替, ）。

但是，一旦您超过该list就没有[]运算符，因此您会发现该部分也不起作用。 您将需要使用迭代器在最终循环中遍历列表。

Answer 4

int k = *list1[j];

那也不会编译。

也许这样的事情可能（不检查任何安全性等，这是一种废话，而只是显示）

int k = *(find(clist.begin(), clist.end(), j));

同样，您将j递增两次，这是您的意思吗？

for(int j =0; j< list1.size(); j++) //here
{
   int k = *list1[j];
   std::cout << "cvalue " << k <<std::endl;
   j++; //and here?
}