无法在 print() 方法中遍历 map

Question

在循环遍历地图没问题之前，我已经做过一些练习。 现在我在 class 中这样做，我收到了我根本不理解的奇怪错误。 它似乎发生在 print() 方法中迭代器的实例化时。

错误：从 'std::mapstd::__cxx11::basic_string<char, bool>::const_iterator' {aka 'std::_Rb_tree_const_iterator<std::pair<const std::__cxx11::basic_string, bool> >' 转换} 到非标量类型 'std::mapstd::__cxx11::basic_string<char, bool>::iterator' {aka 'std::_Rb_tree_iterator<std::pair<const std::__cxx11::basic_string, bool> >'} 要求 33 | for (map<string, bool>::iterator i = jobs.begin(); i.= jobs;end(); i++) {

这是我的代码：

class Employee {
private:
    string name;
    map<string, bool> jobs;

public:

    Employee() {
        name = "";
        jobs[""] = false;
    }
    
    Employee(const Employee &other) {
        cout << "Copied." << endl;
        name = other.name;
        jobs = other.jobs;
    }

    Employee(string name, string task, bool trained) {
        this->name = name;
        jobs[task] = trained;
    }
    
    void setTask(string task, bool trained) {
        jobs[task] = trained;
    }
    
    void print() const {
        for (map<string, bool>::iterator i = jobs.begin(); i != jobs.end(); i++) {
            pair<string, bool> jobs = *i;
            cout << name << " is trained on " << jobs.first << "? " << jobs.second << endl;
        }
    }
};

我promise 来之前尽量在网上找解决方法。 对不起，如果我误解了一些非常基本的东西。

我试过使打印方法为常量或非常量，并且我已经尝试使用 const_iterator 和 .cbegin()/.cend() 以防它对此很挑剔。 可悲的是，我还不太擅长调试。

Answer 1

成员function print是一个常量成员function。也就是说在function数据成员中class是常量。 因此，您需要使用map<string, bool>::const_iterator而不是map<string, bool>::iterator

在 for 循环中重新声明名称jobs也是一个坏主意。 而且这个声明是多余的。

你应该写

void print() const {
    for (map<string, bool>::const_iterator i = jobs.cbegin(); i != jobs.cend(); i++) {
        cout << name << " is trained on " << i->first << "? " << i->second << endl;
    }
}

如果您的编译器支持 C++ 17 那么您可以通过以下方式使用基于范围的 for 循环

void print() const {
    for ( const auto &[first, second] : jobs ) {
        cout << name << " is trained on " << first << "? " << second << endl;
    }
}

Answer 2

您的print成员 function 被声明为const ，因此在其中发生的 class 的数据成员的所有出现/使用也将被视为const 。

因此，对jobs.begin()和jobs.end()的调用将调用这些函数的const重载，这些函数的返回类型为const_iterator 。 因此，要解决您的问题，请将i设为const_iterator ：

    void print() const {
        for (map<string, bool>::const_iterator i = jobs.begin(); i != jobs.end(); i++) {
            pair<string, bool> localjob = *i; // Avoid "shadowing" member variable!
            cout << name << " is trained on " << localjob.first << "? " << localjob.second << endl;
        }
    }

Answer 3

void print() const

最后那个const就是说这个class方法是一个const class方法。 它可以在这个 class 的const实例上调用。它的意思是，就这个方法而言，它的 class 的所有成员都是const 。

map<string, bool>::iterator i = jobs.begin(); 

由于this是const ， begin()重载返回一个const_iterator ，而不是iterator ，这就是编译错误的原因。

我试过使打印方法为 const 或不是 const，

但是您没有尝试做的是使用当前的 C++ 标准，您显然是在使用非常非常过时的教科书来学习 C++。当前的 C++ 使所有这些变得更加容易：

   void print() const {
        for (auto &job:jobs)
            cout << name << " is trained on " << job.first << "? " << job.second << endl;
        }

这甚至不是最好的方法。 我会留给你去更新你的教科书，并了解结构化绑定，使它更容易阅读。