0

我定义了一个名为 ClusterSet 的类，它只有一个字段，名为clusters：

class ClusterSet {
   std::map<std::string, std::map<std::string, float>* > clusters;

   public:
      typedef std::map<std::string, std::map<std::string, float> *>::iterator iterator;
      typedef std::map<std::string, std::map<std::string, float> *>::const_iterator const_iterator;

      iterator begin() { return clusters.begin(); }
      const_iterator begin() const { return clusters.begin(); }
      iterator end() { return clusters.end(); }
      const_iterator end() const { return clusters.end(); }

      void create_cluster(std::string representative);
      void add_member(std::string representative, std::string member, float similarity);
      int write_to_file(std::string outputfile);
      int size();
      ~ClusterSet();
};

在我的create_cluster方法中，我使用new为内部映射分配内存，并将此指针存储在clusters. 我定义了一个析构函数，以便我可以释放所有这些内存：

ClusterSet::~ClusterSet() {
  ClusterSet::iterator clust_it;
  for (clust_it = clusters.begin(); clust_it != clusters.end(); ++clust_it) {
      std::cout << "Deleting members for " << clust_it->first << std::endl;
      delete clust_it->second;
  }
}

当我的析构函数被调用时，它似乎正确地释放了所有内部映射（它为每个映射打印出“删除成员...”）。但是，一旦完成，我会收到一个运行时错误，上面写着“无法“munmap”1068 字节：无效参数”。这是什么原因造成的？

我已经简要地查看了“三规则”，但我不明白为什么我需要一个复制构造函数或一个赋值运算符，或者这如何解决我的问题。我永远不需要直接使用任何一个。

4

1 回答 1

1

动态分配内部映射没有充分的理由（也有很多缺点）。将外部地图类型更改为

std::map<std::string, std::map<std::string, float> >

然后您根本不需要实现自己的析构函数和复制/移动语义（除非您想更改从地图中获得的那些，也许是为了防止复制您的类）。

如果在其他情况下，您确实需要存储指向对象的指针并将它们的生命周期与它们在映射中的存在联系起来，请存储智能指针：

std::map<std::string, std::unique_ptr<something> >

如果您出于某种原因真的想手动管理它们的生命周期，那么您将需要遵循三规则并为您的类提供有效的复制语义（通过删除复制构造函数/赋值运算符来防止复制，或者实现您想要的任何语义）。即使你不认为你在复制对象，编写这样的代码也很容易。

于 2013-08-08T18:12:08.753 回答