2017-01-10 7 views
3

Попытка создать вектор shared_ptr для int.Создайте вектор shared_ptr для ints

Куда я иду не так? Благодарю. Кит: ^)

#include <iostream> 
#include <vector> 
#include <memory> 

int main() { 
    std::vector<std::shared_ptr<int> > w; 
    std::vector<std::shared_ptr<int> >::iterator it_w; 
    w.push_back(new int(7)); 

    std::cout << std::endl; 
} 

Компилятор Результат:

pickledegg> g++ -std=c++11 -o shared_ptr shared_ptr.cpp 
shared_ptr.cpp:29:4: error: no matching member function for call to 'push_back' 
     w.push_back(new int(7)); 
     ~~^~~~~~~~~ 
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/vector:697:36: note: 
     candidate function not viable: no known conversion from 'int *' to 'const value_type' (aka 
     'const std::__1::shared_ptr<int>') for 1st argument 
    _LIBCPP_INLINE_VISIBILITY void push_back(const_reference __x); 
           ^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/vector:699:36: note: 
     candidate function not viable: no known conversion from 'int *' to 'value_type' (aka 
     'std::__1::shared_ptr<int>') for 1st argument 
    _LIBCPP_INLINE_VISIBILITY void push_back(value_type&& __x); 
           ^
1 error generated. 

ответ

9

std::shared_ptr<T> конструктор, который принимает сырые указатели отмечен явным. source. Вы не сможете позвонить push_back с необработанным указателем, так как он неявно конвертируется в std::shared_ptr<int>, что и принимает push_back. Решением является либо использовать emplace_back вместо push_back, либо использовать std::make_shared<int>.

emplace_back соответствует параметрам, заданным одному из конструкторов Т.

w.emplace_back(new int(7)); 

std::make_shared<int> возвращает объект типа std::shared_ptr<int>, избежать этой проблемы.

w.push_back(std::make_shared<int>(7)); 

Вы можете комбинировать эти решения.

#include <iostream> 
#include <vector> 
#include <memory> 

int main(int argc, char** argv) { 
    std::vector<std::shared_ptr<int> > w; 
    std::vector<std::shared_ptr<int> >::iterator it_w; 
    w.emplace_back(std::make_shared<int>(7)); 

    std::cout << std::endl; 
} 

Edit: В качестве дополнительной записке, всегда предпочитают std::make_shared<T>(...) над std::shared_ptr<T>(new T(...)). Он разработан, чтобы избежать очень тонких потенциальных утечек памяти. Он также элегантно избегает ситуации, когда у вас есть new без delete, который может беспокоить некоторых людей.

Редактировать 2: Кроме того, std::make_shared<T>(...) имеет преимущество в производительности, когда оно позволяет избежать дополнительного распределения в `std :: shared_ptr (новый T (...)) 'see this answer.

+0

«Он также элегантно избегает ситуации, когда у вас есть« новый »без« удаления », который может беспокоить некоторых людей.« Я нахожу, что он чаще всего старается использовать инструменты анализа статического кода, чем люди в эти дни. – YoungJohn

+1

'emplace_back (std :: make_shared (7))' вызывает copy-ctor 'std :: shared_ptr', который является бит-clupsy, потому что emplace_back существует для возможности обойти это. – Zereges

+0

Необходимы ли «make_shared» и «emplace_back»? Является '' w.push_back (std :: make_shared (7)); '' приемлемо? – kmiklas

Смежные вопросы