合并排序中的动态内存分配

Question

我编写了一个用于合并排序的程序（我编写了基本算法），并且运行良好。 但是，由于必须从一个很大的文件中读取整数，因此我想在递归调用中动态声明该数组。 因此，我编写了以下代码，但是它给了我一些错误，请您帮助我确定我在哪里出错了？

该程序实际上是在计算数组中的反转次数（如果i <j并且arr [i]> arr [j]，那么这是一个反转）。 我编写的程序如下：我不想每次递归调用时都在堆栈上声明10000个整数数组

我得到的错误是：在内存位置0x004dd940上的std :: bad_alloc。我已经编辑了问题，因此它包含错误消息。 执行中断，Visual Studio进入调试模式并打开文件osfinfo.c

    #include<stdio.h>
    #include <iostream>
    using namespace std;
    unsigned int mixAndCount(int * arr,int low, int mid,int high) {
        int *num = new int[high-low+1];// THIS IS WHERE THE ERROR OCCURS
        int l = low ;
        int r = mid+1;
        unsigned int count=0;
        int i =low;

        while((l<=mid)&&(r<=high))
     {
      if(arr[l]<=arr[r])
      {
       num[i]=arr[l];
       l++;

      }
      else
      {
      num[i]=arr[r];
       r++;
       count=count + (mid-l+1);

      }
      i++;
         }
     if(l>mid)
     {
      for(int k=r;k<=high;k++)
      {
       num[i]=arr[k];
       i++;
        }
     }
     else
     {
      for(int k=l;k<=mid;k++)
      {
       num[i]=arr[k];
       i++;
        }
     }
     for(int k=low;k<=high;k++) arr[k]=num[k];
delete[] num;    
return count;
    }

    unsigned int mergeAndCount(int * arr, int low , int high ) {
        if(low>=high) {
        return 0;
        }
        else {
            int mid = (low+high)/2;

            unsigned int left = mergeAndCount(arr, low , mid);
            unsigned right = mergeAndCount(arr, mid+1, high);
            unsigned int split = mixAndCount(arr, low , mid , high);
            return left+right+split;

        }

    }
    int main ()
    {
        int  numArr[100000];
        FILE * input = fopen("IntegerArray.txt", "r");
        int i =0;
        while(!feof(input)) {
            int num;
        fscanf(input, "%d", &num);
        numArr[i] = num;
        i++;
        }
        fclose(input);
        unsigned int count = mergeAndCount(numArr,0, i-1  );
        cout<<count<<endl;
        return 0;
    }

Answer 1

 std::bad_alloc at memory location 0x004dd940..

当new无法成功分配请求的内存时抛出的异常。

int *num = new int[high-low+1];  

似乎请求的内存大小太大，这意味着您需要跟踪high ， low值。

Answer 2

注意动态内存分配。 它真的很慢。 在将代码保留为这种形式之前，请三思而后行。 您可以使用std::chrono创建一个简单的测试用例

http://en.cppreference.com/w/cpp/chrono/duration

您不需要动态分配，所有操作都在一个本地名称空间中完成。