堆上的字符串对象需要 C++ 智能指针吗？

Question

我试图证明我的问题。我不能提供一个完整的例子，但我想我的问题会很清楚。

我有这个构造函数：

mlt::Client::Client(const string& hostname, struct sockaddr *hostaddr)
        : content(nullptr),
          hostname(hostname),
          ip_and_port(mlt::Client::prepareIPandPort(hostaddr))
{ /* ... */ }

如你所见，我使用了一个转换hostaddr的静态函数：

#include <iostream>
#include <libnet.h>

const string& mlt::Client::prepareIPandPort(struct sockaddr *hostaddr) {
    assert(hostaddr != nullptr);

    string *ipport;
    char clienthost[NI_MAXHOST];
    char clientport[NI_MAXSERV];
    int result = getnameinfo(hostaddr, sizeof(*hostaddr),
                             clienthost, sizeof(clienthost),
                             clientport, sizeof(clientport),
                             NI_NUMERICHOST | NI_NUMERICSERV);

    if (result != 0) {
        cerr << "Error: " << gai_strerror(result) << endl;
        ipport = new string {"unknown"};
    }
    else {
        switch (hostaddr->sa_family) {
            case AF_INET:
                ipport = new string {string(clienthost) + ":"
                                     + string(clientport)};
                break;
            case AF_INET6:
                ipport = new string {"[" + string(clienthost) + "]:"
                                     + string(clientport)};
                break;
            default:
                ipport = new string {"unknown"};
        }
    }

    return *ipport;
}

mlt::Client::ip_and_port 声明为：

const string ip_and_port;

据我了解，我在堆上创建了一个字符串，并将取消引用的内容作为对 const 字符串的引用返回。此结果在初始化程序中用于初始化 const 字符串成员 ip_and_port。

这会产生内存泄漏吗？如果我调用“delete some_client_instance”并调用析构函数 mlt::Client::~Client() 会发生什么？

我是否必须在 mlt::Client::prepareIPandPort 中使用智能指针 (std::shared_ptr)？如果是这样，这段代码会是什么样子？

提前非常感谢

Answer 1

是的，这会导致内存泄漏。您需要在函数结束时调用delete some_client_instance。

理想情况下，您应该：

• 在mlt::Client::prepareIPandPort 中返回一个纯字符串（不是引用）
• 对ip_and_port 使用纯字符串
• 如下初始化ip_and_port

这样，字符串只会被创建一次，并且永远不会被复制。

mlt::Client::Client(const string & hostname, struct sockaddr *hostaddr)
        : content(nullptr),
          hostname(hostname),
          ip_and_port(std::move(mlt::Client::prepareIPandPort(hostaddr)))
{ /* ... */ }