Нулевой указатель с повышением:: shared_ptr?

Хотя возможно, что ваша среда выполнения не сообщит об этой ошибке, используя надлежащую проверку выполнения ошибок, например, Valgrind предупредит вас об использовании памяти после ее освобождения.

Я рекомендую, чтобы если вы пишете код с помощью new / delete и необработанных указателей (вместо std::make_shared() и т.п.), вы выполняете свои модульные тесты под Valgrind, чтобы хотя бы иметь возможность обнаружить такие ошибки.

55
задан 7 March 2009 в 03:03
поделиться

5 ответов

Ну, это законно:

shared_ptr<Foo> foo;  /* don't assign */

И в этом состоянии, это ни на что не указывает. Можно даже протестировать это свойство:

if (foo) {
    // it points to something
} else {
    // no it doesn't
}

Итак, почему бы не это:

std::vector < shared_ptr<Foo> > vec;
vec.push_back (shared_ptr<Foo>);   // push an unassigned one
17
ответ дан Larry Gritz 7 November 2019 в 17:25
поделиться

Да, объявите глобального статического нулевого указателя.

0
ответ дан Christopher Dolan 7 November 2019 в 17:25
поделиться

Вы могли объявить глобальное nullPtr для shared_ptr<Foo>. Но если бы Вы загрязняете глобальное пространство имен, что Вы назвали бы глобальным nullPtr для shared_ptr<Bar>?

Обычно я объявляю пустой указатель ptr как помехи в классе указателя.

#include <boost\shared_ptr.hpp>

class Foo; // forward decl
typedef boost::shared_ptr<Foo> FooPtr;
class Foo
{
public:
    static FooPtr Null;
}
...
// define static in cpp file
FooPtr Foo::Null;
...
// use Foo Null
vec.push_back(Foo::Null);

Тот путь каждый класс имеет статический Пустой указатель.

4
ответ дан m-sharp 7 November 2019 в 17:25
поделиться

Ваше предложение (звонящий shared_ptr<T> конструктор без аргумента) корректно. (Вызов конструктора со значением 0 эквивалентен.) Я не думаю, что это было бы немного медленнее, чем вызов vec.push_back() с существованием ранее shared_ptr<T>, так как конструкция требуется в обоих случаях (или прямая конструкция или конструкция копии).

, Но если Вы хотите "более хороший" синтаксис, Вы могли бы попробовать следующий код:

class {
public:
    template<typename T>
    operator shared_ptr<T>() { return shared_ptr<T>(); }
} nullPtr;

Это объявляет единственный глобальный объект nullPtr, который включает следующий естественный синтаксис:

shared_ptr<int> pi(new int(42));
shared_ptr<SomeArbitraryType> psat(new SomeArbitraryType("foonly"));

...

pi = nullPtr;
psat = nullPtr;

Примечание, что при использовании этого в нескольких единицах перевода (исходные файлы) необходимо будет дать классу имя (например, _shared_null_ptr_type), перемещает определение эти nullPtr объект в отдельный .cpp файл и добавляет extern объявления в заголовочном файле, где класс определяется.

52
ответ дан j_random_hacker 7 November 2019 в 17:25
поделиться

Вот кое-что, что, на мой взгляд, немного проще и отлично работает

(помните, что typedef - ваш друг):

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

typedef boost::shared_ptr< std::vector<char> > CharVecHandle;

inline CharVecHandle newCharVec(std::vector<char>::size_type size) {
    return CharVecHandle(new std::vector<char>(size));
}

inline CharVecHandle newCharVec(void) {
    return CharVecHandle();
}

int main ( void )
{
    CharVecHandle cvh = newCharVec();

    if (cvh == NULL) 
        std::cout << "It's NULL" << std::endl;
    else 
        std::cout << "It's not NULL" << std::endl;

    std::vector< CharVecHandle > cvh_vec;

    cvh_vec.push_back(newCharVec(64));
    cvh_vec.push_back(newCharVec());

    // or call the NULL constructor directly
    cvh_vec.push_back(CharVecHandle());

    return EXIT_SUCCESS;
}
1
ответ дан 26 November 2019 в 17:26
поделиться
Другие вопросы по тегам:

Похожие вопросы: