我的析构函数是不是给了我这个错误：*** `./main' 中的错误：双重释放或损坏（fasttop）：？

Posted 2023-02-22

技术标签:

【中文标题】我的析构函数是不是给了我这个错误：*** `./main\' 中的错误：双重释放或损坏（fasttop）：？【英文标题】：Is my destructor giving me this error: *** Error in `./main': double free or corruption (fasttop):?我的析构函数是否给了我这个错误：*** `./main' 中的错误：双重释放或损坏（fasttop）：？ 【发布时间】：2018-04-26 03:55:26 【问题描述】：

这是我的作业说明：复制构造函数。复制构造函数应该执行参数对象的深层复制，即它应该构造一个与参数具有相同大小和容量的 IntCollection，并具有参数数据数组的完整副本。

赋值运算符 (=)。赋值运算符还应该执行参数对象的深层复制。它必须返回自身（或者更有效地，对自身的引用）以支持同一行上的多个赋值，例如a = b = c。如果你首先实现你的赋值运算符，它可以在复制构造函数中使用，但这不是必需的。

Is 等于运算符 (==)。如果参数对象的大小与接收对象的大小相同，并且两个对象的数据数组中的值相同，则“等于”运算符应返回 true。

插入运算符 (

析构函数。函数 add() 调用 addCapacity() 以在需要更多空间时分配内存。在这个程序中，没有任何地方使用 delete [] 释放内存，这意味着我们有内存泄漏！添加一个可以正确处理此问题的析构函数。

增加容量。注意 addCapacity() 是一个私有成员函数。如果您尝试从类外部调用它，即通过将下面的行添加到 main()，会发生什么？

c.addCapacity();

这是我的代码： IntCollection.h：

#ifndef INTCOLLECTION_H
#define INTCOLLECTION_H

// Allocate memory in chunks of ints of this size.
const int CHUNK_SIZE = 5;

class IntCollection

  private:
  // The number of ints currently stored in the int
    int size;
  // the total number of elements available for storage
  // in the data array
    int capacity;
  // A pointer to the dynamically allocated data array
    int* data;
  // a private member function to allocate more memory 
  // if necessary
    void addCapacity();
  public:
  // Constructor
    IntCollection();
  // Destructor
    ~IntCollection();
  // Copy constructor:
    IntCollection(const IntCollection &c);

    void add(int value);
    int get(int index);
    int getSize();
    IntCollection& operator=(const IntCollection &c);
    bool operator==(const IntCollection &c);
    IntCollection& operator<<(int value);
;

#endif

IntCollection.cpp：

#include "IntCollection.h"
#include <cstdlib>
#include <iostream>
using namespace std;

IntCollection::IntCollection() 

    // Initialize member data to reflect an empty
    // IntCollection
    size = capacity = 0;
    data = NULL;


IntCollection::~IntCollection() 

    delete [] data;


IntCollection::IntCollection(const IntCollection &c) 
    size = c.size;
    capacity = c.capacity;
    data = c.data;

    for (int i = 0; i < c.size; i++) 
    
        data[i] = c.data[i];
    


void IntCollection::addCapacity() 

    // Create a new, bigger buffer, copy the current data to
    // it, delete the old buffer, and point our data
    // pointer to the new buffer
    int *newData;
    data = new int[capacity];
    capacity += CHUNK_SIZE;
    newData = new int[capacity];

    for (int i = 0; i < size; i++) 
    
        newData[i] = data[i];
        delete[] data;
        data = newData;
    


void IntCollection::add(int value) 

    // first, allocate more memory if we need to
    if (size == capacity) 
    
        addCapacity();
    
    // Now, add the data to our array and increment size
    data[size++] = value;


int IntCollection::get(int index) 

    if (index < 0 || index >= size) 
    
        cout << "ERROR: get() trying to access index out of range.\n";
        exit(1);
    
    return data[index];


int IntCollection::getSize() 

    return size;


IntCollection &IntCollection::operator=(const IntCollection &c) 

    size = c.size;
    capacity = c.capacity;
    data = c.data;

    return *this;


bool IntCollection::operator==(const IntCollection &c) 

    if ((size == c.size) && (capacity == c.capacity)) 
    
        for (int m = 0; m < size; m++) 
        
            if (data[m] == c.data[m]) 
            
                continue;
             else 
            
                return false;
            
        
    
    return true;


IntCollection &IntCollection::operator<<(int value) 

    add(value);
    return *this;

main.cpp：

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

int main() 

  IntCollection c;

  c.add(45);
  c.add(-210);
  c.add(77);
  c.add(2);
  c.add(-21);
  c.add(42);
  c.add(7);

  for (int i = 0; i < c.getSize(); i++)
  
    cout << c.get(i) << endl;
  

  IntCollection d(c);
  for (int i = 0; i < c.getSize(); i++)
  
    cout << c.get(i) << endl;
  

  IntCollection e;
  e = c;
  cout << "Testing = Overload" << endl;

  for(int i = 0; i < c.getSize(); i++)
  
    cout << c.get(i) << endl;
  

  IntCollection f;
  f<<8<<9<<10;
  cout << "Testing<<Overload" << endl;
  for(int i = 0; i < f.getSize(); i++)
  
    cout << f.get(i) << endl;
  

  cout << "Testing == Overload" << endl;

  c.add(10);

  if(f == c)
  
    cout << "Both objects match" << endl;
  
  else
  
    cout << "They don't match" << endl;
  

  return 0;

在我相信我修复了大部分错误之后，我得到了以下输出：

*** Error in `./main': double free or corruption (fasttop): 0x0000000000b2ec80 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x70bfb)[0x7ff7e6f70bfb]
/lib/x86_64-linux-gnu/libc.so.6(+0x76fc6)[0x7ff7e6f76fc6]
/lib/x86_64-linux-gnu/libc.so.6(+0x7780e)[0x7ff7e6f7780e]
./main[0x400fa1]
./main[0x400fe2]
./main[0x400aa2]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf1)[0x7ff7e6f202e1]
./main[0x40097a]
======= Memory map: ========
00400000-00402000 r-xp 00000000 08:01 11008976                           /home/runner/main

我不会全部发布，因为它很长。是我的析构函数造成的吗？我不知道如何解决这个问题，我以前也没有遇到过这个错误。

【问题讨论】：

在调试器中运行它。 “双重释放”错误意味着您在同一个指针上多次调用delete。你能找到这样做的delete吗？ 【参考方案1】：

复制构造函数中的这一行可能是问题：