递归排序用C++写，结果不对，求大神改

Question

//这是源代码，求帮忙改出正确结果
#include
#include
#include
#define random(x) (rand()%x)
void merge(std::vector&vec,int p,int q,int r){
std::vectorvec1;
std::vectorvec2;
for(int i=p;i<=q;i++)
vec1.push_back(vec[i]);
for(i=q+1;i<=r;i++)
vec2.push_back(vec[i]);
i=0;
int j=0;
for(int k=p;k<=r;k++){
if(vec1[i]<vec2[j]){
vec[k]=vec1[i];
i++;
}
else{
vec[k]=vec2[j];
j++;
}
}

}
void mergesort(std::vector&vec,int p1,int p2){
int q=(p1+p2)/2;
if(p2==p1) return;
mergesort(vec,p1,q);
mergesort(vec,q+1,p2);
merge(vec,p1,q,p2);
}
int main(){
const int size =10;
const int range=1000;
std::vectorvec;
std::cout<<"Before merge sort:"<<std::endl;
for(int i=0;i<size;i++){
vec.push_back(random(range));
std::cout<<vec[i]<<' ';
}
std::cout<<std::endl<<"After merge sort:"<<std::endl;
mergesort(vec,0,size-1);
for( i=0;i<size;i++){
std::cout<<vec[i]<<' ';
}
std::cout<<std::endl;
return 0;
}

Answer 1

该回答引用ChatGPT

您好，看到您的代码后，我发现其中有几个小错误。下面是我修改后的代码：

#include <iostream>
#include <vector>
#include <cstdlib>

#define random(x) (rand()%x)

void merge(std::vector<int>& vec, int p, int q, int r) {
    std::vector<int> vec1;
    std::vector<int> vec2;
    for (int i = p; i <= q; i++)
        vec1.push_back(vec[i]);
    for (int i = q + 1; i <= r; i++)
        vec2.push_back(vec[i]);
    int i = 0;
    int j = 0;
    for (int k = p; k <= r; k++) {
        if (i >= vec1.size()) {
            vec[k] = vec2[j];
            j++;
        }
        else if (j >= vec2.size()) {
            vec[k] = vec1[i];
            i++;
        }
        else if (vec1[i] < vec2[j]) {
            vec[k] = vec1[i];
            i++;
        }
        else {
            vec[k] = vec2[j];
            j++;
        }
    }
}

void mergesort(std::vector<int>& vec, int p1, int p2) {
    if (p2 == p1) return;
    int q = (p1 + p2) / 2;
    mergesort(vec, p1, q);
    mergesort(vec, q + 1, p2);
    merge(vec, p1, q, p2);
}

int main() {
    const int size = 10;
    const int range = 1000;
    std::vector<int> vec;
    std::cout << "Before merge sort:" << std::endl;
    for (int i = 0; i < size; i++) {
        vec.push_back(random(range));
        std::cout << vec[i] << ' ';
    }
    std::cout << std::endl << "After merge sort:" << std::endl;
    mergesort(vec, 0, size - 1);
    for (int i = 0; i < size; i++) {
        std::cout << vec[i] << ' ';
    }
    std::cout << std::endl;
    return 0;
}

以下是修改的说明：

• 在 merge 函数中，vec1 和 vec2 的类型应该为 std::vector。另外，使用 push_back 将元素添加到向量中。

• 在 merge 函数中，循环时需要加上边界检查，以避免访问不存在的元素。另外，在比较 vec1[i] 和 vec2[j] 之前，应该先判断 i 和 j 是否已经超出了它们所在的向量的范围。

• 在 mergesort 函数中，变量 q 的计算应该在递归之前进行。

• 在主函数中，向量 vec 的类型应该为 std::vector。

我希望这些修改能够帮助您解决问题。