将继承类型的 boost::shared_ptr 的引用传递给函数[关闭]

Question

我有以下代码：

struct A : boost::noncopyable {
  virtual A& copy (const A&);
};
struct B : A {
  virtual B& copy (const B&);
};

typedef boost::shared_ptr <A> APtr; 
typedef boost::shared_ptr <B> BPtr; 
typedef std::string Key;

struct AFactory {
static APtr s_get (const Key& key) {
APtr a;
//... //some factory code
return a;     
}
};

这是C类的旧实现，以及用法

class COld {
public:
  void get (APtr a) {
    a->copy (*_a) ;
  }
private:
  Key _key;  
  APtr _a;
};

int main (void) {
 APtr a = AFactory::s_get ("b"));
 BPtr b = boost::dynamic_pointer_cast <B> (AFactory::s_get ("b"));
 COld c;

 c.get (a); //works good
 c.get (b); //works good

 return 1;
}

我试图对用户更友好，如果传递的对象为NULL，则分配它

class CNew {
public:
  void get (APtr& a) {
    if (!a) {
      a = AFactory::s_get (_key);
    }
    a->copy (*_a) ;
  }
private:
  Key _key;  
  APtr _a;
};

int main (void) {
 //current usage
 APtr a;
 BPtr b;
 APtr ap = boost::dynamic_pointer_cast <A> (b);

 CNew c;

 c.get (a);
 c.get (b); // here I get the error: 
 c.get (ap); // ok, but i don't like it

 return 1;
}

错误：“APtr &”类型的引用（非 const 限定）不能用“BPtr”类型的值初始化

确定用户可以将 BPtr 转换为 APtr 以传递给 c.get()：

APtr ap = boost::dynamic_pointer_cast <A> (b);
c.get (ap);

但这不是很优雅

我不想共享来自 C::_a 的指针，只是复制数据 如何使 C::get () 的新实现工作方便？

Answer 1

让我们看看有问题的函数。

void get (APtr& a) {
  if (!a) {
    a = AFactory::s_get (_key);
  }
  a->copy (*_a) ;
}

它接受一个智能指针引用，可能将它指向智能指针资源的一个实例，然后将一些值复制到其中。

问题是APtr& 不能引用BPtr 的实例。这不应该真的那么令人惊讶......如果你可以传入一个伪装成APtr 的BPtr，然后为其分配一个新的A，那么你就违反了类型系统。考虑到从c.get(b) 返回时，您可以调用b->some_method_A_does_not_have()...如果b 以某种方式持有A 的实例，您最终会触发某种运行时错误。毕竟，您不会期望B* 持有指向A 的指针。

最简单的解决方案是通过值传入a，并要求对其进行初始化，这样您就不必替换它的托管资源。

void get (APtr a) {
  if (!a)
    throw std::exception();

  a->copy (*_a) ;
}

这对于您的库的使用者来说可能不是那么方便，但是所需的样板代码很少，并且行为很容易描述。传递shared_ptr 的副本的开销很小，除非get 被大量使用，否则应该不是问题。如果它被大量使用，以效率的名义要求您的用户采取额外措施是合理的。

另一种可能性是模板化您的工厂类和get 方法。

template <class T>
struct Factory
{
  static boost::shared_ptr<T> s_get()
  {
    boost::shared_ptr <T> t;
    // whatever
    return t;
  }
};

class C
{
public:
  template <class T>
  void get(boost::shared_ptr<T>& t) {
    if (!t) {
      t = Factory<T>::s_get();
    }
    dynamic_cast<A*>(t.get())->copy(*_a);
  }

  // etc