Question

大家好，我有以下代码
在PointArray.h

#ifndef POINTARRAY_H_INCLUDED
#define POINTARRAY_H_INCLUDED

template <typename T>
class PointArray
{
private:
    int nSize;
    T *array[];
public:
    PointArray();
    PointArray(const T *points[],const int size);
    PointArray(const PointArray &pv);
    ~PointArray();
    int * get(const int position);//is same to array[position]
    const int * get(const int position) const;//is same to array[position]
    const int getSize() const;//get size of array
};

#endif // POINTARRAY_H_INCLUDED

PointArray.cpp

中的

#include "PointArray.h"
#include<iostream>
#include <assert.h>
using namespace std;

template<class T>
PointArray<T>::PointArray(const T *points[],const int size)
{
    nSize=size;
    for (int i=0;i<size;i++)
    {
        array[i]=new T;
        *array[i]=*points[i];
    }
}

template<class T>
PointArray<T>::PointArray()
{
    nSize=0;
}

template<class T>
PointArray<T>::PointArray(const PointArray &pv)
{
    nSize=pv.getSize();
    for (int i=0;i<nSize;i++)
    {
        array[i]=new T;
        *array[i]=*(pv.get(i));
    }
}

template<class T>
const int PointArray<T>::getSize() const
{
    return nSize;
}

template<class T>
PointArray<T>::~PointArray()
{
    delete[] array;
    nSize=0;
}

template<class T>
int * PointArray<T>::get(const int position)
{
    assert(position>-1 && position<nSize);
    return array[position];
}

template<class T>
const int * PointArray<T>::get(const int position) const
{
    return array[position];
}

main.cpp

中的

#include<iostream>
#include "PointArray.h"
#include "PointArray.cpp"
using namespace std;

int main()
{
    int x=22;
    int y=3;
    const int * a[2]={&x,&y};
    PointArray<int> p;
    PointArray<int> p2(a,2);
    PointArray<int> p3(p2);
    cout << p.getSize() << endl;
    cout << p2.getSize() << endl;
    cout << p3.getSize() << endl;
    return 0;
}

上面的代码应该打印

0
2
2

但输出

9244616
9244600
2

当我再次运行9244616时，9244600会发生变化什么是问题？

Answer 1

你的问题是

T *array[];

您定义了一个没有元素的指针数组。然后，您尝试访问其（不存在的）元素，这会导致未定义的行为。

另一个未定义的行为是在析构函数

中

delete[] array;

你应该只delete[]个new[]的东西。 array不是。相反，你应该遍历所有array元素（如果有的话）并删除它们。

最佳解决方案是使用std::vector而不是数组。如果您坚持使用数组，请将T* array[]更改为T **array，在构造函数中为其分配内存

array = new T*[size];

并在析构函数中删除所有数组元素，然后删除数组本身。

Answer 2

问题在于你将cpp文件包含在main中并在那里实现，模板实现应该在h文件中完成。你的h文件应该是：

 #ifndef POINTARRAY_H_INCLUDED
#define POINTARRAY_H_INCLUDED

template <typename T>
class PointArray
{
private:
    int nSize;
    T *array[];
public:
    PointArray()
    {
        nSize=0;
    }
    PointArray(const T *points[],const int size)
    {
        nSize=size;
        for (int i=0;i<size;i++)
        {
            array[i]=new T;
            *array[i]=*points[i];
        }
    }
    PointArray(const PointArray &pv)
    {
        nSize=pv.getSize();
        for (int i=0;i<nSize;i++)
        {
            array[i]=new T;
            *array[i]=*(pv.get(i));
        }
    }
    ~PointArray()
    {
        delete[] array;
        nSize=0;
    }
    int * get(const int position)
    {
        assert(position>-1 && position<nSize);
        return array[position];
    }
    const int * get(const int position) const
    {
        return array[position];
    }
    const int getSize() const { return nSize;}
};

#endif // POINTARRAY_H_INCLUDED

代码仍然很难，分配很糟糕（注意你在进程结束时得到堆断言）..但它对我有用： 0 2 2 打印... 欢呼声。

（请注意：）

delete[] array;

是一个内存泄漏，因为你的对象不是一个数组，但是你试图定义指向数组的指针，你应该将它改为

T** array

然后在数组第一维的析构函数循环中，然后执行delete[] array

编写一个与指针数组相同的类的代码中的错误？

2 个答案: