Question

它不完全是合并排序，算法使用合并排序计算数组中的反转数（基本上我只添加了一条简单的行）从文本文件中读取和合并排序100,000个不同的整数需要2.415秒，而解决相同问题的其他人（在coursera.com上）说它花了不到0.5秒

这是我的代码，出了什么问题？文件阅读可能吗？感谢

#include <bits/stdc++.h>

using namespace std;
int a,b,i,j,n,x,k;
int t[100000]={0};
long long int s=0;

void merge(int a, int mid, int b)
     {
         i=a;
        j=mid+1;
        k=a;
        int v[100000]={0};
        while(i<=mid && j<= b)
        {
            if (t[i]<t[j])
                {v[k]=t[i];
                 i++;
                }
            else
            {v[k]=t[j];
             j++;
              s+=mid-i+1; //this line here counts the inversions
             }
           k++;
        }
       while(i<=mid)
        {v[k]=t[i];
         i++; k++;}

        while(j<=b)
        {v[k]=t[j];
         j++; k++;}

      for (i=a;i<k;i++)
         t[i]=v[i];
    }


void mergeSort(int a, int b)
    {
        if(a<b)
        {
           int mid=(a+b)/2;
            mergeSort(a,mid);
            mergeSort(mid+1,b);
            merge(a,mid,b);
        }
    }


int main(){
  ifstream fin("C:\\Users\\ASUS\\Desktop\\coursera.txt");
    n=100000;
    for(i=0;i<n;i++)
        fin>>t[i];

    mergeSort(0,n-1);

     cout<<endl<<s;

}

Answer 1

我可以看到的一个问题是在合并功能中，你分配了太多的空间，并且复制也需要相当的O（N），这使得总复制时间为O（N * N）而不是O（N *日志（N））。简单更改合并功能可能是：

void merge(int a, int mid, int b)
{
    i = a;
    j = mid + 1;
    k = 0;
    int* v = new int[b - a + 1];
    while (i <= mid && j <= b)
    {
        if (t[i]<t[j])
        {
            v[k] = t[i];
            i++;
        }
        else
        {
            v[k] = t[j];
            j++;
            s += mid - i + 1; //this line here counts the inversions
        }
        k++;
    }
    while (i <= mid)
    {
        v[k] = t[i];
        i++; k++;
    }

    while (j <= b)
    {
        v[k] = t[j];
        j++; k++;
    }

    for (i = 0; i<k; i++)
        t[a+i] = v[i];

    delete[] v;
    v = NULL;
}

我的合并排序算法对于大文件输入花费的时间太长了？

1 个答案: