Question

在以下代码中，我想在input类中存储vector<double> derived。我这样做是通过应用std::vector的副本分配作为向量传递给setIT函数。我需要它来使用在派生中实现的计算。在此复制分配期间出现内存泄漏。

使用：vector<double> * input代替vector<double> input可以避免此泄漏，但我不明白为什么。

有人可以澄清一下吗？提前谢谢。

#include "utilities.h"
#include <fstream>

using namespace std;
using namespace astro;

class base
{
  public:
    base () { cout<<" in base default constructor "<<endl; }
    virtual void setIT (void *v) = 0;
    virtual double compute () = 0;
};

class derived : public base
{
  protected:
    vector<double> input;

  public:
    derived ();
    virtual void setIT (void *v);
    virtual double compute () { /* using input vector to return something */ return 0; }
};

derived::derived () : base()
{
    cout<<" in derived default constructor "<<endl;
    input.resize(0);
}

void derived::setIT (void *v)
{
  cout<<" in derived setIT "<<endl;
  vector<double> * a = reinterpret_cast<vector<double>* >(v);
  input = *a;
  for (uint i = 0; i<input.size(); i++)
    cout<<i<<" "<<input[i]<<endl;
}

int main ()
{
  vector<double> test;
  fill_linear(test,5,1.,6.); // linear filling of test vector by '5' values between 1 and 6

  base * t = new derived;
  t->setIT (&test);
  cout<<t->compute()<<endl;

  delete t;
  t = NULL;
  return 0;
}

OUTPUT：

 in base default constructor 
 in derived default constructor 
 in derived setIT 
0 1
1 2.25
2 3.5
3 4.75
4 6
1

Answer 1

实际上你的程序会调用未定义的行为。

base类的析构函数必须为virtual才能明确定义。

只需将析构函数定义为：

virtual ~base() {}

即使它是空的，也要这样做！

详情请阅读：

When should my destructor be virtual?

Answer 2

避免在C ++中使用void指针。如果要处理不同类型，请改用模板。

class Base
{
public:
  virtual ~Base(){}
  virtual double compute() const=0;
};

template<typename T>
class Derived : public Base
{
private:
  std::vector<T> m_input;
public:
  void set_it(const T& in)
  {
    m_input = in;
  }
  double compute() const{/*Do the computation and return*/;}
};

当以多态方式使用时，派生类的std :: vector成员的复制赋值会导致内存泄漏

2 个答案: