2

我刚开始学习提升共享指针。

我写了一个简短的程序,结果看起来不错,但我不确定我的代码是否能很好地释放内存。我想问一下,是否有人可以查看我的代码并判断我是否正确使用了共享指针。

#include <boost/shared_ptr.hpp>
#include <iostream>
#include <vector>
#include <string>

#define VECTSIZE 10

typedef boost::shared_ptr<std::string> StringPtr;
typedef std::vector<StringPtr> StringVect;

///////////////////////////////////////////////////////////////

std::string random_string (size_t length);

///////////////////////////////////////////////////////////////

int main()
{
    StringVect vect;

    for (int i = 0; i < VECTSIZE; i++)
    {
        std::string * stdstr;
        stdstr = new std::string;
        *stdstr = random_string(10);
        std::cout << *stdstr << "\r\n";

        StringPtr str(stdstr);
        vect.push_back(str);
    }

    std::cout << "\r\n\r\n";

    for (int i = 0; i < VECTSIZE; i++)
    {
        std::cout << *vect[i] << "\r\n";
    }

    vect.clear();
    system("pause");
    return 0;
}

///////////////////////////////////////////////////////////////

std::string random_string (size_t length)
{
    auto randchar = []() -> char
    {
        const char charset[] =
        "0123456789"
        "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
        "abcdefghijklmnopqrstuvwxyz";
        const size_t max_index = (sizeof(charset) - 1);
        return charset[ rand() % max_index ];
    };

    std::string str(length,0);
    std::generate_n( str.begin(), length, randchar );
    return str;
}

感谢您的任何建议;我希望它对我和其他人有帮助。

4

2 回答 2

4

从没有直接内存泄漏的意义上说,您的使用是正确的。然而,你并不是真正的异常安全——如果random_string抛出,你会泄漏stdstr。完全绕过 rwa 指针会更好(也更惯用)。这是一个使用的例子std::shared_ptr

for (int i = 0; i < VECTSIZE; i++)
{
    StringPtr str = std::make_shared<std::string>();  // Encapsulates new
    *str = random_string(10);
    std::cout << *str << '\n'; //No \r here: text streams insert it on Windows automatically

    vect.push_back(str);
}

std::string此外,正如@ForEveR 所指出的,在现实世界的应用程序中没有理由动态分配。但是我假设您将它用作智能指针的练习,这当然很好。

于 2013-06-05T06:51:56.237 回答
3

一切都好,但你不需要vect.clear()串。但是,string是值类型,不要使用shared_ptr字符串。

于 2013-06-05T06:37:49.660 回答