更改函数以返回指针

Question

我对 C++ 很陌生，我已经编写了下面的类和函数：

class Person {
    public:
        boost::shared_ptr<Details> GetDetails(const std::string& name) const;
        ...
    private:
        std::map<std::string, boost::shared_ptr<Details> > map;
        ...
};

inline
boost::shared_ptr<Details>
Person::GetDetails(const std::string& name) const {
    return map.find(name)->second;
}

这工作正常，但我被告知让函数返回指针，如果查找失败则返回 NULL。

我尝试了几件事，其中之一是：

class Person {
    public:
        boost::shared_ptr<Details> *GetDetails(const std::string& name) const;
        ...
    private:
        std::map<std::string, boost::shared_ptr<Details> > map;
        ...
};

inline
boost::shared_ptr<Details>
*Person::GetDetails(const std::string& name) const {
    if (!map.find(name)->first.empty()) {
        return *map.find(name)->second;
    }
    return NULL;
}

这给了我错误：

error: cannot convert 'Details' to 'boost::shared_ptr<Details>*' in return

我不完全确定在这里做什么。任何帮助或资源都会有很大帮助。

谢谢。

Answer 1

您想要返回共享指针的地址，因此您需要使用&，而不是*。

请注意，处理指向共享指针的指针有点奇怪。如果找不到该项目，为什么不直接返回一个空的共享指针呢？

Answer 2

您需要返回空的 boost::shared_ptr

return boost::shared_ptr<Details>();

而不是这个

return NULL;

另外返回指向 shared_ptr 的指针是个坏主意。你以前的版本看起来更好。

Answer 3

您应该在访问 first 或 second 之前测试 find 得到的内容：

inline
boost::shared_ptr<Details> 
Person::GetDetails(const std::string& name) const {
    std::map<std::string, boost::shared_ptr<Details> >::iterator i = map.find(name);

    if (i != map.end()) return i->second;
    return boost::shared_ptr<Details>(); // empty shared_ptr if not found
}

但不要返回裸指针。这会很危险。

基本原理：

• find() 在最后一个条目之后返回 map.end() 迭代器。所以没有入口可以访问。
• map.find(name)->first.empty() 将是未定义的行为，因为现在地图末尾可能有空字符串。
• shared_ptr 负责所有内存管理。返回原始指针可能会毁掉这一切。