Trình lặp lặp bản đồ không đổi sẽ không được đặt thành mymap.begin()

Question

map<string,Shopable*>::iterator it = mymap.begin(); 

Trình lặp có vẻ không đổi, nhưng items.begin() không trả về trình lặp liên tục. Hoặc, đó là những gì tôi nghĩ vì lỗi mouseover là một cái gì đó như:

"No conversion from 'std::Tree_const_iterator<...> to std::Tree_iterator<...> exists'". 

Tại sao?

Answer 1

Sử dụng const_iterator như:

map<string,Shopable*>::const_iterator it = mymap.begin(); 

Từ lỗi, rõ ràng rằng mymap.begin() lợi nhuận của nó const_iterator. Đó là bởi vì mymap là const trong hàm mà bạn đã viết này, một cái gì đó như sau:

void f(const std::map<int,int> & m) 
{ //^^^^^ note this 

     std::map<int,int>::const_iterator it = m.begin(); //m is const in f() 
         //^^^^^ note this 
} 

void g(std::map<int,int> & m) 
{ 
     std::map<int,int>::iterator it = m.begin(); //m is non-const in g() 
} 

Đó là, const container (cho dù nó std::map, std::vector vv) trả const_iterator và container không const lợi nhuận iterator.

Mỗi vùng chứa có chức năng quá tải là begin() và end(). Vì vậy, const container gọi quá tải begin() trả về const_iterator và thùng chứa không const gọi quá tải khác begin() trả về iterator. Và tương tự cho end() chức năng quá tải.

Answer 2

Vấn đề là mymap trong mã ở trên là bản đồ không đổi, không phải bản đồ có thể thay đổi (có thể là thành viên của lớp và mã đó nằm trong hàm thành viên không đổi?). Vì vậy, các cuộc gọi đến mymap.begin() sẽ pichup quá tải mà trả về một const_iterator thay vì quá tải mà trả về một iterator.

Nếu bạn không cần thay đổi vùng chứa thông qua trình lặp, hãy sử dụng const_iterator. Nếu bạn dự định sửa đổi bản đồ, hãy đảm bảo rằng bạn đang sử dụng đối tượng không phải const cho vòng lặp (có thể hàm thành viên (nếu đó là trường hợp) không phải là const?)