Question

与Hoare分区的Quicksort

canvas

尝试使用Hoare的分区实现Quicksort，但无法弄清楚我做错了什么。它陷入无限循环，总是运行内存不足

// Hoare's partitioning scheme
func PartitionHoare(arr []int, low, high int) int {
    length := len(arr)

    if length == 0 {
        panic("Array size is 0")
    }

    pivot := arr[low]
    i := low - 1
    j := high + 1

    for {
        for {
            j--
            if arr[j] > pivot {
                break
            }
        }

        for {
            i++
            if arr[i] < pivot {
                break
            }
        }

        if i < j {
            Swap(&arr, i, j)
        } else {
            return j
        }
    }
}

// Sort
func SortHoare(arr []int, low, high int) {
    if low < high {
        p := PartitionHoare(arr, low, high)
        SortHoare(arr, low, p)
        SortHoare(arr, p+1, high)
    }
}


// Swap i <--> j
func Swap(arr *[]int, i, j int) {
    (*arr)[i], (*arr)[j] = (*arr)[j], (*arr)[i]
}

Answer 1

在寻找i和j的位置时，你应该使用非严格的不等式来进行交换。而不是

        if arr[j] > pivot {
            break
        }

你应该

        if arr[j] >= pivot {
            break
        }

对我而言也一样。而不是

        if arr[i] < pivot {
            break
        }

使用

        if arr[i] <= pivot {
            break
        }

此外，我不确定它是否有意，但目前您的算法按降序排序。如果要按升序排序，请交换i和j之间的比较。所以：

    if arr[j] <= pivot {
        break
    }

和

    if arr[i] >= pivot {
        break
    }

Hoare的分区方案golang

1 个答案: