Question

我有一个简单的数组类，它使用模板并具有大小。我想在它充满时调整它的大小，但是我的程序一直在崩溃。代码是这样的：

    template<class T>
class Buffer:public File_ptr
{

    unsigned int siz;
    T *data;


public:

    ///konstruktor(size,filename,openmode)
    Buffer(unsigned int s=50,const char* n, const char* m):File_ptr(n,m),siz(s)
    {
        data= new T[siz];
        for(unsigned int i=0; i<siz; ++i)
        {
            data[i]=0;
        };
    };

    ///destruktor
    ~Buffer()
    {
        delete[] data;
    }

    ///operator[]
    T& operator[](unsigned int i)
    {
        if(i>siz)
        {
            unsigned int newsize=siz*2;
            T* tmp=new T[newsize];
            for(unsigned int j = 0; j < siz; j++)
            {
                tmp[j] = data[j];
            }
            siz=newsize;
            delete[] data;
            data=tmp;
            delete[] tmp;
        };
        return data[i];
    }

};

错误来自运营商[]。（至少我认为）。主程序是这样的：

int main()
{
   Buffer<int> k(20,"k.txt","w");
   Buffer<char*> s(20,"s.txt","w");
   k.printfile("HEY");
   s.printfile(5);
   for(unsigned int i=0;i<23;i++){
    s[i]="Hey";
    cout << s[i] << endl;
   }


return 0;
}

任何帮助都会很棒！ :) 谢谢！

Answer 1

您收到分段错误，因为您要删除内存两次。

data=tmp;
delete[] tmp;

在此，您为data指定tmp的内存位置。到目前为止，data和tmp指向相同的内存区域。但现在你delete[] temp;，所以删除了内存区域。但由于data也指向它，data也不再有效用作指针（如tmp）。

然后，使用return data[i];您尝试访问该内存，但是您删除了它！所以你得到了一个分段错误。

您可以通过不 delete[] tmp;解决此问题，因为data指向有效的内存位置，一切都很好：）

Answer 2

您删除了tmp中仍由operator[]引用的新阵列。由于这是data引用的新数组，您不可避免地会遇到麻烦。

Answer 3

你有几个问题。第一个问题是您删除了tmp指向哪个内容data现在指向的内存。这意味着下次数组增长时，您将访问一个未定义行为的已删除指针。

第二个问题是，if(i>siz)检查i已经过了数组的末尾，但您从未查看i是否也大于newsize。您需要将newsize设置为siz * 2和i中的最大值。此外，操作员[]不能直观地生长容器。当您调用大小大于容器的[]时，应该是错误而不是重新调整数组大小。

C ++数组调整大小崩溃

3 个答案: