python中的合并排序算法[关闭]

Question

关闭。 这个问题需要更加集中。 它目前不接受答案。

---

想改善这个问题吗？ 更新问题，使其仅通过编辑这篇文章来关注一个问题。

4 天前关闭。

改进这个问题

我正在尝试使用 python 执行合并排序算法，但遇到了一些我不知道如何解决的错误。

我正在使用递归方法，似乎错误主要在 mergeSortRec 函数中，另一个是错误状态（'int' 对象不可订阅）。 有一个图像链接来显示回溯。

我想要做的是用整数按升序对一个简单的列表进行排序。

错误：

'int' object is not subscribable
enter code here

def mergeSort(arr):
    mergeSortRec(arr, 0, len(arr) - 1)
    return

def mergeSortRec(arr, left, right):
    if(left >= right):
        return
    else:
        mid = (left+(right-1))//2
        mergeSortRec(arr, left, mid)
        mergeSortRec(arr, mid + 1, right)
        merge(arr, left, mid, right)
    return

def merge(arr, left, mid, right):
    leftArray = [mid - left + 1]
    rightArray = [right - mid]

    leftLength = range(0,len(leftArray))
    for x in leftLength:
        leftArray = arr[left+x]

    rightLength = range(0, len(rightArray))
    for x in rightLength:
        rightArray = arr[math.floor(mid + 1 + x)]

    point1 = 0
    point2 = 0
    mPoint = left
    while(point1 < leftArray and point2 < rightArray):
        if(leftArray[point1] < rightArray[point2]):
            arr[mPoint] = leftArray[point1]
            point1 +=1
        else:
            arr[mPoint] = rightArray[point2]
            point2 += 1

        mPoint +=1

    while(point1< leftArray):
        arr[mPoint] = leftArray[point1]
        point1 +=1
        mPoint +=1

    while(point2 < rightArray):
        arr[mPoint] = rightArray[point2]
        mPoint += 1
        point2 += 1

    return

Answer 1

您的代码中存在以下问题：

• 以下赋值创建包含一个元素的列表，该值与原始列表的内容无关：

   leftArray = [mid - left + 1] rightArray = [right - mid]

• 下面的循环只会迭代一次，但即使迭代多次，它也会覆盖leftArray变量，使之前的任何迭代都没用，并且实际上将leftArray变成单个数字（取自arr ）而不是数组：

   for x in leftLength: leftArray = arr[left+x]

  您在此处尝试执行的操作称为切片，您可以使用以下语法进行操作：

   leftArray = arr[left:mid+1]

  ...无需显式循环。

• 假设您打算将leftArray （和rightArray ）作为列表，则以下条件没有意义：

   while(point1 < leftArray and point2 < rightArray): # ... if(point1 < leftArray): # ... if(point2 < rightArray): # ...

  这些将比较数字与列表。 您想与相关列表的长度进行比较，因此point1 < len(leftArray) ，...等。

不是问题，但是：

• 没有必要使用math.floor 。
• 如果函数的最后一条语句是一个简单的return ，你可以省略它。
• 不需要将while或if条件放在括号中。

无需更改太多其他内容，您可以使用此版本的merge函数更正您的代码：

def merge(arr, left, mid, right):
    leftArray = arr[left:mid+1]
    rightArray = arr[mid+1:right+1]

    point1 = 0
    point2 = 0
    mPoint = left
    while point1 < len(leftArray) and point2 < len(rightArray):
        if leftArray[point1] < rightArray[point2]:
            arr[mPoint] = leftArray[point1]
            point1 +=1
        else:
            arr[mPoint] = rightArray[point2]
            point2 += 1

        mPoint +=1

    while point1 < len(leftArray):
        arr[mPoint] = leftArray[point1]
        point1 +=1
        mPoint +=1

    while point2 < len(rightArray):
        arr[mPoint] = rightArray[point2]
        mPoint += 1
        point2 += 1

最后两个while循环可以进一步缩短为以下切片分配：

arr[mPoint:right+1] = leftArray[point1:] or rightArray[point2:]