0

我需要实现 MergeSort 以使用向量迭代器对整数向量进行排序。我实现了它，但我收到了这个错误：Vector iterator not dereferencable。

作为合并排序函数的参数，我使用 (A.begin(),A.end()) 其中 A 是包含 n 个元素的向量。

#include <iostream>
#include <vector>

using namespace std;

void swap(vector<int>::iterator first,vector<int>::iterator last)
{
    int temp;
    temp=*first;
    *first=*last;
    *last=temp;
    return;
}


void mergesort(vector<int>::iterator first,vector<int>::iterator last)
{
    if(first==last) return; 
    if(first+1==last)  
    {
        if(*last>*first) swap(first,last);
        return;
    }

    vector<int>::iterator middle;
    middle=first+(last-first)/2;

    mergesort(first,middle);
    mergesort(middle+1,last);

    int a,b,pa,pb;
    vector<int> h;
        //the number of elements in a
    pa=middle-first+1; 
        //the number of elements in b
    pb=last-middle;    
         h.resize(pa+pb);
    a=0; b=0;

    while(a<pa && b<pb)
        if(*(first+a)<*(middle+1+b)) 
          {
              h[a+b]=*(first+a); a++;
          }
        else
          {
              h[a+b]=*(middle+1+b); b++;
          }

    while(a<pa) 
       {
           h[a+b]=*(first+a); a++;
       }
    while(b<pb)
       {
           h[a+b]=*(middle+1+b); b++;
       }

    for(int i=0;i<((a+b)-1);i++)
    *(first+i)=h[i];

    return;
}




int main(){

    vector<int>A;

    for(int i=1000;i>0;i--)
    {
        A.push_back(i);               
           //vector of integer: 1000,999,998 ... 3,2,1
    }

    mergesort(A.begin(),A.end());     
 //sort vector elements from smallest to biggest: 1,2,3...998,999,1000


    system("pause");
    return 0;
}
4

2 回答 2

2

迭代器范围应表示为引用第一个元素的迭代器和引用过去的迭代器。如果它们相等，则这是一个空范围。你没有那样做。

为了解决这个问题，该if(first+1==last)子句应该返回。第二个mergesort(middle+1,last)电话应该是mergesort(middle,last). pa应该只是相等middle-last。 pb计算正确。而且我认为-1应该删除最后一个 for 循环。

有一个由迭代器定义的范围到第一个和迭代器到最后是一个坏主意。

于 2012-10-30T19:29:23.807 回答
1

你应该做

mergesort(A.begin(),A.end()-1);

实际上，end()迭代器指向向量末尾之后的一个元素。它不指向向量的最后一个元素。

于 2012-10-30T18:37:31.327 回答