删除数组时堆损坏错误

时间:2013-02-19 10:17:57

标签: c++ mergesort dynamic-arrays

我正在尝试读取包含随机数列表的文本文件,并使用mergesort进行排序。数字被读入动态数组。不幸的是,每当我尝试删除未使用的数组时,都会检测到堆损坏错误。

Mergesort功能:

void mergesort(int *arr, int first, int last)
{
if(first < last)
   {
   int middle = ((first + last)/2);
   mergesort(arr, first, middle);
   mergesort(arr, middle+1, last);
   merge(arr, first, last); 
   }
}

删除tempArr时,合并功能出错:

void merge(int *arr, int first, int last)
{
int *tempArr = new int[last];

int mid = (first+last)/2;
int first1 = first;
int last1 = mid;
int first2 = mid + 1;
int last2 = last;

int index = first1;

for(; (first1 <= last1) && (first2 <= last2); ++index)
{
    if (arr[first1] < arr[first2])
    {
        tempArr[index] = arr[first1];
        ++first1;
    }
    else
    {
        tempArr[index] = arr[first2];
        ++first2;
    }
}

for(; first1 <= last1; ++first1, ++index)
    tempArr[index] = arr[first1];

for(; first2 <= last2; ++first2, ++index)
    tempArr[index] = arr[first2];

for(index=first;index<=last;++index)
    arr[index] = tempArr[index];

delete [] tempArr;
}

1 个答案:

答案 0 :(得分:6)

问题似乎是您将数组分配为int *tempArr = new int[last]。其元素数量为last,其索引为01,... last - 1

在功能结束时,您有:

for(; first2 <= last2; ++first2, ++index)
    tempArr[index] = arr[first2];

last2初始化为last的值。这意味着循环中的最终分配将在index == last时,因此您正在访问tempArr[last]。这超出了数组范围。