map / set迭代器是不可引用的C ++ map

Question

请看看我的代码。

int main () {
    Program* allcommand = new Program;
    allcommand->addCommand("add", new Add);
    allcommand->addCommand("convert", new Convert);
    allcommand->addCommand("exit", new Exit);
    allcommand->addCommand("help", new Help);
    allcommand->addCommand("show", new Show);
    allcommand->addCommand("modify", new Modify);

    std::string input;
    Command* obj;
    while (true) {
        std::cout << "\nCommand >> ";
        std::getline(std::cin, input);
        std::map<std::string, Command*> :: iterator it;
        std::vector<std::string> parsedinput = allcommand->parse(input);

        it = allcommand->getCommands().find(parsedinput[0]);
        obj = it->second;

        obj->start(parsedinput);
        delete obj;
    }

    return 0;
}

它将命令注册到包含其命令名称和指向其类的指针的映射中。 编译没有问题，但是当我输入命令时，它崩溃，并显示“ map / set迭代器不可取消引用”。 我是地图新手（几分钟），所以请帮忙。

编辑。 好吧，我发现问题不在主要...这是Program类的代码（其中一些）

    void Program::addCommand(std::string command1, Command* obj) {
    m_allCommands[command1] = obj;
}

std::map<std::string, Command*> Program::getCommands () {
    return m_allCommands;
}

我认为问题出在这里，因为在main中注册命令后，我无法指出任何命令的名称（同样的问题）

Answer 1

std::map<std::string, Command*> Program::getCommands () {
    return m_allCommands;
}

返回m_allcommands映射的副本。 因此，当您这样做时：

it = allcommand->getCommands().find(parsedinput[0]);

您会在allcommand->getCommands()返回的临时对象上获得一个迭代器，该迭代器在分配完成后会被销毁。 因此，它什么都没有指向。

将getCommands()更改为：

std::map<std::string, Command*>& Program::getCommands () {
    return m_allCommands;
}

甚至更好：

const std::map<std::string, Command*>& Program::getCommands () const {
    return m_allCommands;
}

Answer 2

调用find()您需要检查是否

if(it == allcommand->getCommands().end()) {
   //Not Found
} else {
   obj = it->second;
   obj->start(parsedinput);
}