Question

我正在尝试使用向量和迭代器在C ++中实现Merge Sort。该程序编译良好，但当我尝试运行它，它崩溃了。我试过调试但没有成功。

使用数组的实现是直截了当的，但是当我尝试使用向量实现完全相同的算法时，程序无法运行。

#include <iostream>
#include <vector>
#include <algorithm>

void mergesort(std::vector<int>& A, int l, int r);

void merge(std::vector<int>& A, int l, int m, int r);


void mergesort(std::vector<int>& A, int l, int r) {


    if(l < r) {

        int m = l + (r - l) / 2;

        mergesort(A, l, m);

        mergesort(A, m + 1, r);

        merge(A, l, m, r);

    } 
}

void merge(std::vector<int>& A, int l, int m, int r) {

   std::vector<int> L(A.begin() + l, A.begin() + (m - 1));
   std::vector<int> R(A.begin() + m, A.begin() + r);

   std::vector<int>::iterator i = L.begin();
   std::vector<int>::iterator j = R.begin(); 
   std::vector<int>::iterator k = A.begin() + l;

   while(i != L.end() && j != R.end()) {
       if(*i <= *j) {
           *k = *i;
           i++;
       }
       else {
           *k = *j;
           j++;
       }
       k++;
   }

   while(i != L.end()) {
       *k = *i;
       i++;
       k++;
   }

   while(j != R.end()) {
       *k = *j;
       j++;
       k++;
   }
}

void print_vector(std::vector<int> A) {

    std::vector<int>::iterator it;

    for(it = A.begin(); it != A.end(); ++it) {
        std::cout << *it << " "; 
    }
}


int main() {
    std::vector<int> A = {178, 1156, 136, 5, 6, 7};
    mergesort(A, 0, A.size() - 1);

    print_vector(A);
}

Answer 1

在本节中

if(l < r) {
    int m = l + (r - l) / 2;
    mergesort(A, l, m);
    mergesort(A, m + 1, r);
    merge(A, l, m, r);

}

当r = l + 1时，你得到m = 1，然后你调用mergesort，要求函数对空向量进行排序，而mergesort的代码则假定它不为空。

什么是

 std::vector<int> L(A.begin() + l, A.begin() + (m - 1));

在这种情况下？

这应该是进行调试的良好起点。

Answer 2

问题出在功能vector中L s R和void merge(std::vector<int>& A, int l, int m, int r)的内存分配中。

以下是更正后的代码段，虽然未经过优化（您可以自由优化;））：

std::vector<int> L;
std::vector<int> R;
int n1 = m - l + 1;
int n2 =  r - m;
if(n1 > 0)
{
   for(int i=0; i<n1; i++) L.push_back(0);
   std::copy(A.begin() + l, A.begin() + (l+n1), L.begin());
}
if(n2 > 0)
{
   for(int i=0; i<n2; i++) R.push_back(0);
   std::copy(A.begin() + (l+n1), A.begin() + (l+n1+n2), R.begin());
}

您可以找到完整的工作代码here

C ++中的Merge-Sort代码实现

2 个答案: