0 
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))];
}

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

没有运算符 "[]" 匹配这些操作数操作数类型是： 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[] 不是。

4

1 回答 1

4

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

要么将地图声明为mutable（不推荐），要么使用迭代器搜索并在搜索返回时返回 NULL keywordItems.end()。否则，返回迭代器的对象（in 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：晚餐吃什么。

于 2013-08-19T08:00:52.767 回答