c++ Map [] 运算符未找到

Question

void Library::addKeywordsForItem(const Item* const item, int nKeywords, ...)
{
    // the code in this function demonstrates how to handle a vararg in C++

    va_list     keywords;
    char        *keyword;

    va_start(keywords, nKeywords);
    for (int i = 0; i < nKeywords; i++)
    {
        keyword = va_arg(keywords, char*);
        ((Item*)const_cast<Item*>(item))->addKeyword(string(keyword)); 

        ItemSet* itemSet = keywordItems[string(keyword)];
        if (itemSet == NULL)
        {
            itemSet = new ItemSet();
            keywordItems[keyword] = itemSet;
        }
        bool isNull = (itemSet == NULL) ? true : false;
        itemSet->insert(((Item*)const_cast<Item*>(item)));
    }
    va_end(keywords);
}

const ItemSet* Library::itemsForKeyword(const string& keyword) const
{
    return keywordItems[((string)const_cast<string&>(keyword))];
}

在上面的代码中，第一种方法按预期工作。第二种方法没有，并显示错误@“[”

no operator "[]" 匹配这些操作数操作数类型是： const StringToItemSetMap [ std::string ]

StringToItemSetMap 只是 map 的 typedef。我尝试了不同的强制转换，并创建了一个本地字符串变量，但不是运气。甚至像 keywordItems[string("test")];在第二种方法中不起作用，但在第一种方法中起作用。有什么我可能错过的吗？

编辑：

const ItemSet* Library::itemsForKeyword(const string& keyword) const
{
    std::map<std::string, ItemSet*>::const_iterator it = keywordItems.find(keyword);
    if (it != keywordItems.end())
    {
        return it->second;
    }
    return NULL;
}

正如在答案中指出的那样，问题是因为第二种方法是 const 而 map::operator[] 不是。

Answer 1

您正在使用覆盖的operator[]，它必须对底层映射具有非常量访问权限，因为如果请求的关键位置不存在新条目，它将添加一个新条目。在您的情况下，将添加一个 NULL 指针（不说明原因）。您的成员函数被声明为const，因此映射不可修改。

要么将映射声明为 mutable（不推荐），要么使用迭代器搜索并在搜索返回 keywordItems.end() 时返回 NULL。否则，返回迭代器的对象（it->second）。

示例

const ItemSet* Library::itemsForKeyword(const string& keyword) const
{
    std::map<std::string, ItemSet*>::const_iterator it =  keywordItems.find(keyword);
    if (it != keywordItems.cend())
       return it->second;
    return nullptr;
}

注意：我强烈建议对地图对象内容使用直接对象，或者至少使用智能指针（例如std::shared_ptr<ItemSet>）。 RAII: 晚餐吃什么。