Question

我遇到了运算符=的重载问题。我的代码是

#include <stdlib.h>
#include <iostream>


const std::size_t DIM = 10;


template <typename T>

class rowvec
  {
    private:
      T* m_pnt;
      std::size_t m_dim;

    public:
      rowvec();
      rowvec(std::size_t);
      ~rowvec();

      rowvec<T>& operator=(const rowvec<T>& x);

      T* pnt();
  };


template <typename T>
rowvec<T>::rowvec()
  {
    m_dim = DIM;
    m_pnt = (T*) calloc (DIM ,sizeof(T));
  }


template <typename T>
rowvec<T>::rowvec(std::size_t n)
  {
    m_dim = n;
    m_pnt = (T*) calloc(m_dim,sizeof(T));
  }


template <typename T>
rowvec<T>::~rowvec()
      {free(m_pnt);}


template <typename T>
rowvec<T>& rowvec<T>::operator=(const rowvec<T> &x)
  {
    std::cout << "hello" << std::endl;
    if (this != &x)
      {
        free (m_pnt);
        this->m_dim=x.m_dim;
        m_pnt = (T*) calloc (m_dim,sizeof(T));
        for (int i=0; i!=m_dim; i++)
          *(m_pnt+i)=*(x.m_pnt+i);
      }
    return *this;
  }


template <typename T>
T* rowvec<T>::pnt()
  {return m_pnt;}  




int main()
  {
    rowvec<int> k(3);

    rowvec<int> c=k;

    std::cout << "ok" << std::endl;

    return 0;
  }

没有编译错误，但是当我运行结果时：

ok
*** Error in `./blog': double free or corruption (fasttop): 0x0000000001a5e010 ***

如果我以这种方式更改代码：

int main()
  {
    rowvec<int> k(3);

    rowvec<int> c;

    c=k;

    std::cout << "ok" << std::endl;

    return 0;
  }

一切正常（输出）

hello
hello
ok

有没有办法允许声明像＆＃34; rowvec c = k;＆＃34; ？

Answer 1

即使其中有=，rowvec<int> c=k;也是复制初始化并使用复制构造函数而不是复制赋值运算符。

由于您的班级没有定义一个，因此使用简单执行成员复制的默认班级。这只会复制指针，后来导致双重释放，因为c和k的析构函数都会在同一个指针上调用free()。

为您的班级提供正确的复制构造函数（制作深层复制品），双重删除应该消失。

此外，您正在使用C ++。请勿使用calloc / free。使用new和delete。

c ++重载：operator =的重载

1 个答案: