“类 std::map 在没有模板参数的情况下使用” 错误

Question

我不得不说我不是使用 STL 的专家。 这是我的问题，我有一个名为 LdapClientManager 的类，它维护许多由 ID 管理的 LDAP 客户端。 保存 LdapClients 的容器被声明为成员变量，即

typedef std::map<int, LdapClient *> LdapClientMap;
LdapClientMap _ldapClientMap;

以下函数无法编译并出现错误：

LdapClient * LdapClientManager::getLdapClient(unsigned int templateID)
{
    // Do we have an LdapClient
    LdapClientMap::const_iterator it = _ldapClientMap.find(templateID);
    if (it == std::map::end) {
        // no existing client, lets create it
        LdapClient * ldapClient = new LdapClient();
        if (ldapClient == NULL) {
            // TODO: handle out of memory condition
        }

        _ldapClientMap[templateID] = ldapClient;
        return ldapClient;
    }

    return it->second;
}

不幸的是，我在编译时收到以下错误，这是什么意思。 我目前google还没有找到解决方案。

LdapClientManager.cc：在成员函数 LdapClient* 中 LdapClientManager::getLdapClient(unsigned int)': LdapClientManager.cc:33:模板类 std::map' 在没有模板参数的情况下使用

Answer 1

将 std::map::end 替换为 _ldapClientMap.end()。
另外，new 永远不会返回 0，如果分配失败，它会抛出异常。

请注意，程序可以变得更短。

LdapClient * LdapClientManager::getLdapClient(unsigned int templateID)
{
    LdapClient *& value = _ldapClientMap[templateID];
    if (value == 0)
        value = new LdapClient();
    return value;
}

Answer 2

它的意思正是它所说的意思。 std::map 是一个类模板。 它本身并不是一个类。 它需要模板参数，就像您在定义 LdapClientMap 类型时使用的那样。 后来，你说std::map::end，编译器说它也需要参数。

但您可能指的是 _ldapClientMap.end()。 每张地图都有自己的终点； end 不是静态函数，因此您需要在实例上调用它。 如果它是静态的，您将需要提供模板参数，就像定义类型时一样：std::map::end 。

Answer 3

std::map::end() 是容器实例的成员函数，而不是通用值，因此您需要根据 _ldapClientMap.end() 检查 std::map::find() 的结果。

改进代码的另外一些建议：

• 标准 C++ 容器具有值语义（它们想要存储实际对象而不是指向对象的指针）。 如果您确实需要存储指向 LdapClient 的指针而不是 LdapClient 对象本身，我强烈建议将它们包装在适当的智能指针中，例如 boost::shared_ptr （不是 std::auto_ptr，这不起作用）。 这样，std::map 的自动内存管理仍然可以工作并按预期销毁对象。 如果您不想使用智能指针或将实际的 LdapClient 对象放入容器中，则必须手动管理对象的生命周期并在适当的时候调用删除以防止内存泄漏。 我的偏好是将映射的类型更改为 std::map ，除非 LdapClient 对象是多态的。
• 除非您使用的是非常过时的编译器，否则检查常规 new() 的结果是否为 0 或 NULL 将不会产生任何新的见解，因为如今当 new 由于某种原因无法分配内存时，它会抛出 std::bad_alloc 。
• 而不是使用 _ldapClientMap[x] = y; 要插入新元素，我将使用 _ldapClientMap.insert(LdapClientMap::value_type(x,y)) 因为后者不会覆盖键 x 的现有值（前者会这样做）并且会返回“false”以防万一该密钥已存在于地图中。 当然，如果这是您的意图的话。

Answer 4

LdapClientMap _ldapClientMap;

您应该避免使用以下划线开头的名称。 从技术上讲，这是未定义的行为，即使编译器允许它，因为使用它会与当前或将来的保留名称发生冲突。