5

我正在创建自己的自定义过滤器类以在 boost::filtered_graph 中使用。WeightMap 概念必须具有默认构造函数、复制构造函数和赋值运算符。

我创建了下面的类，它有一个 std::shared_ptr 私有成员。我的问题是我应该如何编写赋值运算符。复制构造函数没有问题，但赋值运算符不起作用。

class BFDMFilter
{
private:
const BGraph* m_battlemap;
const std::shared_ptr<MoveAbility> m_mv_ab;

public:
BFDMFilter() : m_battlemap(nullptr), m_mv_ab() { }
BFDMFilter(const BGraph* bmap, std::shared_ptr<MoveAbility> mv) : m_battlemap(bmap), m_mv_ab(mv) { }

BFDMFilter(const BFDMFilter& filter) : m_battlemap(filter.m_battlemap), m_mv_ab(filter.m_mv_ab) { }
BFDMFilter& operator=(const BFDMFilter& filter) 
{
  if(this != &filter)
  {
m_battlemap = filter.m_battlemap;
m_mv_ab = filter.m_mv_ab;
  }

  return *this;
}

bool operator()(const Edge& edge) const 
{ 
  Tile::TileEdge path = (*m_battlemap)[edge];

  return m_mv_ab->CanMove(path.TerrainType()) > 0.0;
}

bool operator()(const Vertex& vertex) const 
{ 
  Tile tile = (*m_battlemap)[vertex];

  return m_mv_ab->CanMove(tile.TerrainType()) > 0.0;
}
};

然后给我一个编译错误：

error: passing ‘const std::shared_ptr<momme::battle::MoveAbility>’ as ‘this’ argument of ‘std::shared_ptr<_Tp>& std::shared_ptr<_Tp>::operator=(std::shared_ptr<_Tp>&&) [with _Tp = momme::battle::MoveAbility, std::shared_ptr<_Tp> = std::shared_ptr<momme::battle::MoveAbility>]’ discards qualifiers [-fpermissive]

我明白为什么；赋值运算符在进行赋值时修改 shared_ptr 的引用计数，以便它可以跟踪有多少打开的引用。但是，我该如何编写赋值运算符呢？std::weak_ptr 具有相同的行为，如果我将引用设为非常量，则 boost 库会抱怨该函数已被删除。

4

1 回答 1

12

您的代码似乎没有任何理由声明m_mv_ab为

const std::shared_ptr<MoveAbility> m_mv_ab;

这是以下的智能指针版本：

MoveAbility * const m_mv_ab;

（指向非常量的常量指针MoveAbility）

如果您不想修改指向的MoveAbility对象并希望它是const，您应该这样做：

std::shared_ptr<const MoveAbility> m_mv_ab;

这是以下的智能指针版本：

const MoveAbility * m_mv_ab;

（指向常量的非常量指针MoveAbility）

为了使它更直观一点，您可以始终将const其用作后缀并且始终从右到左读取，除了std::shared_ptr<X>读取“（智能）指向 X 的指针”：

std::shared_ptr<MoveAbility> const m_mv_ab; // const ptr to non-const MoveAbility
MoveAbility * const m_mv_ab; // const ptr to non-const MoveAbility

std::shared_ptr<MoveAbility const> m_mv_ab; // non-const ptr to const MoveAbility
MoveAbility const * m_mv_ab; // non-const ptr to const MoveAbility

但是大多数人const尽可能使用前缀，这会让人感到困惑。

于 2013-03-03T23:00:05.993 回答