Question

我写了一个合并排序程序（我编写了基本算法） - 它运行正常。但是，因为我必须从一个非常大的文件中读取整数，所以我想在递归调用中动态地声明该数组。因此我写了下面的代码，但它给了我一些错误，请你帮我确定一下我犯的错误？

程序实际上是计算数组中的反转次数（如果i＆lt; j和arr [i]＆gt; arr [j]，那么这是一个反转）。我写的程序如下：我不希望每次进行递归调用时都在堆栈上声明一个10000个整数的数组

我得到的错误是：std :: bad_alloc在内存位置0x004dd940 .. 我编辑了这个问题，所以它包含了错误信息。执行中断和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

你不需要动态分配，一切都在一个本地命名空间中完成。

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

2 个答案: