列表第二大数值获取函数异常：测试特定列表时返回错误结果

你的函数在处理[16,5,7,9,12]时返回错误结果16，核心问题出在初始化逻辑：你将largest和second_largest都设为列表第一个元素16，后续遍历的所有元素都小于16，导致elif nums[i] > second_largest的条件始终不触发，second_largest从未更新，最终返回初始值16而非正确的12。

修复方案1：优化初始化与循环判断逻辑 #

调整初始值并完善判断逻辑，同时覆盖边界情况：

def get_second_largest(nums):
    if len(nums) < 2:
        return None  # 处理列表长度不足2的情况
    largest = nums[0]
    second_largest = float('-inf')  # 初始化为负无穷，确保小于所有可能的元素
    for num in nums[1:]:
        if num > largest:
            second_largest = largest
            largest = num
        elif num > second_largest and num != largest:  # 避免重复最大值干扰
            second_largest = num
    # 处理所有元素相同的情况
    return second_largest if second_largest != float('-inf') else None

my_nums = [16,5,7,9,12]
second_largest = get_second_largest(my_nums)
print("Second highest number is : ", second_largest)  # 输出12

改进点说明： #

• 将second_largest初始化为float('-inf')，确保第一个比largest小的元素能正确更新它
• 使用for num in nums[1:]遍历更简洁，替代索引循环
• 添加num != largest的判断，避免列表存在重复最大值时返回错误结果（比如[16,16,12]会正确返回12）
• 增加边界情况处理：列表长度不足2或所有元素相同时返回None（可根据需求调整）

修复方案2：简洁实现（去重+排序） #

如果不需要极致性能，可采用更直观的去重排序方式：

def get_second_largest(nums):
    unique_nums = list(set(nums))
    if len(unique_nums) < 2:
        return None
    unique_nums.sort()
    return unique_nums[-2]

my_nums = [16,5,7,9,12]
second_largest = get_second_largest(my_nums)
print("Second highest number is : ", second_largest)  # 输出12

实现说明： #

• 用set(nums)去重，避免重复最大值干扰排序结果
• 排序后取倒数第二个元素，即为第二大数值
• 同样处理了边界情况：去重后元素不足2个时返回None

内容的提问来源于stack exchange，提问作者Hasibul Hasan