当计算左右主元素的外观时,循环会超出范围(0,right)。相反,他们应该越过范围(左,右)。从0开始,将导致在较小的子问题中返回不正确的主要元素。在
此外,除了在for循环覆盖的范围内起始索引不正确的问题之外,递归调用参数似乎也有问题,这可能是由于您的直觉导致您忽略了一些细节。在整个get_mostive_element函数中,将parameterright视为不在列表中的第一个索引,而不是将right作为列表中最右边元素的索引。在
但是,在第一个递归调用中,您将给出第三个参数,就好像它是该列表中包含的最后一个元素。如果right是不在该列表中的第一个元素的索引,那么它实际上应该与您在下一行中进行的第二个递归调用的第二个参数相同。因此,第一个递归调用的第三个参数小于它应该的值,是1,导致每次递归执行时忽略1个元素。在
第三,在for循环后面的if语句中有一个错误,类似于循环范围的问题。您正在为所有len(a)元素划分元素的出现次数,尽管您应该只关心当前正在处理的子问题。因此,您应该用子问题中的元素数来除以它,而不是len(a)。(即(右-左)//2)
您可以在下面找到工作代码,并查看here以观察它的执行情况。在def get_majority_element(a, left, right):
if left == right:
return -1
if left + 1 == right:
return a[left]
left_m = get_majority_element(a, left, (left + right - 1)//2 + 1)
right_m = get_majority_element(a, (left + right - 1)//2 + 1, right)
left_count = 0
for i in range(left, right):
if a[i] == left_m:
left_count += 1
if left_count > (right-left)//2:
return left_m
right_count = 0
for i in range(left, right):
if a[i] == right_m:
right_count += 1
if right_count > (right-left)//2:
return right_m
return -1
if __name__ == '__main__':
input = sys.stdin.read()
n, *a = list(map(int, input.split()))
print("n=" + str(n))
if get_majority_element(a, 0, len(a)) != -1:
print(1)
else:
print(0)