Question

我从一个文件（它只是一个小的（100个元素）随机整数列表）读入一个向量并尝试使用快速排序对它进行排序，但它会挂断。快速排序函数最终在我在代码中注释的位置无限重复i = 0，j = 30，left = 31和right = 30.

#include <iostream>
#include <limits.h>
#include <stdio.h>
#include <stdlib.h>
#include <fstream>
#include <vector>

using namespace std;

void quicksort(vector<size_t> &fileV, size_t left, size_t right);
void swap(size_t &a, size_t &b);

int main(int argc, char* argv[]){
    if (argc != 2){
        cout << "error: quicks <file name> "<< endl;
        return 1;
    }

    fstream file;
    file.open(argv[1]);
    if (!file.is_open()){
        cout << "error: failed to open file " << argv[1] << endl;
        return 1;
    }

    vector<size_t> fileV;
    size_t ranNum;
    size_t i = 0;
    while(file >> ranNum)
        fileV.push_back(ranNum);

    quicksort(fileV, 0, fileV.size());

    file.close();

    return 0;
}

void quicksort(vector<size_t> &fileV, size_t left, size_t right){
    size_t i = left, j = right, center = (left + right) / 2;
    size_t pivot = fileV[center];

    while (i <= j){
        while (fileV[i] <= pivot)
            i++;
        while (fileV[j] > pivot)
            j--;
        if (i <= j){
            swap (fileV[i], fileV[j]);
            i++;
            j--;
        }
    }

    //repeats infinitely with i = 0, j = 30, left = 31 and right = 30

    if (left < j) 
        quicksort(fileV, left, j);
    if (i < right) 
        quicksort(fileV, i, right);
}

void swap(size_t &a, size_t &b){
    size_t t = a;
    a = b;
    b = t;
}

Answer 1

您的分区算法步骤错误。例如，我们假设我们从：

开始

[1, 2, 3, 2, 8, 9]
 ^i    ^center  ^j

我们进入while循环，首先将i向上移动大于3的第一个元素（8），然后j向下移动3到小于或等于2的第一个元素（最后[1, 2, 3, 2, 8, 9] ^j ^i）：

此时，没有发生交换，我们认为我们的分区已完成。这是错误的：我们需要交换3和{{1}}。无论为什么它最终在某个地方无限循环，这个实现都无法产生正确的答案。

为什么我的快速挂断？

1 个答案: