Boost smart pointers：我能用更简洁的方式表达吗？

Question

今天我一直在使用 Boost::shared_ptr，我有一个问题。

vector<shared_ptr<KlasaA> > vec;
vec.push_back(shared_ptr<KlasaA>(new KlasaB));
vec.push_back(shared_ptr<KlasaA>(new KlasaC));
vec.push_back(shared_ptr<KlasaA>(new KlasaC));
vec.push_back(shared_ptr<KlasaA>(new KlasaA));

for (vector<shared_ptr<KlasaA> >::const_iterator c_it = vec.begin();
    c_it != vec.end(); ++c_it)
{
    cout << c_it->get()->foo(10) << endl;
}

上面的循环通过一个向量并多态调用foo(10)。

我的问题是：

可以...

for (vector<shared_ptr<KlasaA> >::const_iterator c_it = vec.begin();
    c_it != vec.end(); ++c_it)

和

cout << c_it->get()->foo(10) << endl;

用更简洁的方式表达？提前致谢。

Answer 1

typedef boost::shared_ptr<KlasaA> SharedKlasaA;
typedef vector<SharedKlasaA> KlasaAVector;

KlasaAVector vec;
vec.push_back(SharedKlasaA(new KlasaB));
...
for (KlasaAVector::const_iterator ...

cout << (*c_it)->foo(10);

Answer 2

您可以使用Boost.Foreach 库。然后它看起来会非常简洁明了：

BOOST_FOREACH( boost::shared_ptr<KlasaA> v, vec )
{
    std::cout << v->foo(10) << std::endl;
}

Answer 3

据我所知，您的迭代器初始化似乎尽可能紧凑。如果你真的想，你可以把循环函数写成：

cout << (*c_it)->foo(10) << endl;

这样只节省了 4 个字符，但看起来确实清晰了一些。

Answer 4

shared_ptr 有一个隐式向下转型的构造函数，所以你可以写

vec.push_back(shared_ptr<KlasaB>(new KlasaB));

注意 shared_ptr 类型是 shared_ptr<KlasaB> 并且 vec 期望 shared_ptr<KlasaA>。

考虑到这一点，您可以将 shared_ptr 构造函数替换为 make_shared 函数调用（无论如何，这实际上可能是首选）。

vec.push_back(make_shared<KlasaB>());

这将为您省去输入一堆类名的麻烦。 make_shared 接受它将转发的参数，因此 make_shared<X> 是 new X 的一个很好的替代品。

您可能会注意到这里存在潜在的性能影响：make_shared 数据结构可能（会？）不同，并且存在额外的隐式转换，但编译器可能会优化掉它。配置文件并检查。

Answer 5

依赖于编译器，你可能会做这样的事情：

std::foreach(vec.begin(),vec.end(),[](const boost::shared_ptr<KlasaA> &p){
  cout<<p->foo(10);
});