Python新手实现列表重复元素统计：返回出现次数最多的元素及次数

我来帮你排查下代码里的问题，顺便给你调整成符合要求的新手级代码～

原代码的核心问题 #

1. 循环范围漏了最后一个元素：你用了range(0, n-1)，这会让循环只遍历到列表的倒数第二个元素，比如示例里的最后一个1根本没被统计，直接漏掉了真正的高频元素。
2. 返回值错误：函数最后返回的是最后一次循环的count和cval，也就是最后一个被遍历的元素的统计结果（示例里是5，计数2），而不是从你收集的tempcount和tempcval里找出最大值对应的项。
3. 重复统计同一元素：同一个元素会被多次统计（比如示例里的1会被统计两次），虽然不影响最终结果，但会做无用功。

修正后的基础版代码 #

先解决前两个核心问题，保证功能正确：

def counting(B):
    n = len(B)
    tempcount = []
    tempcval = []
    # 修正循环范围，遍历列表所有元素
    for i in range(n):
        cval = B[i]
        count = 0
        for next in B:
            if next == cval:
                count += 1
        tempcount.append(count)
        tempcval.append(cval)
    
    # 遍历收集到的结果，找到计数最大的元素和次数
    max_count = 0
    max_val = B[0]  # 初始值设为列表第一个元素
    for idx in range(len(tempcount)):
        if tempcount[idx] > max_count:
            max_count = tempcount[idx]
            max_val = tempcval[idx]
    return max_val, max_count

def main():
    list = [1, 5, 1, 3, 5, 1]
    a,b = counting(list)
    print(a,b)
main()

这段代码的关键调整：

• 把循环改成range(n)，确保所有元素都被统计
• 新增了一段循环，专门从收集的统计结果里找出最大值对应的元素和次数，而不是直接返回最后一次循环的临时变量

优化版代码（减少重复计算） #

如果想避免重复统计同一个元素，还可以先给列表去重，这样每个元素只需要统计一次，效率更高：

def counting(B):
    n = len(B)
    # 先收集列表里的唯一元素，避免重复统计
    unique_vals = []
    for val in B:
        if val not in unique_vals:
            unique_vals.append(val)
    
    max_count = 0
    max_val = unique_vals[0]  # 初始值设为第一个唯一元素
    for cval in unique_vals:
        count = 0
        for next in B:
            if next == cval:
                count += 1
        # 统计完直接比较，不用存储所有结果，更省空间
        if count > max_count:
            max_count = count
            max_val = cval
    return max_val, max_count

def main():
    list = [1, 5, 1, 3, 5, 1]
    a,b = counting(list)
    print(a,b)
main()

这个版本的优势是：

• 每个元素只统计一次次数，减少了不必要的循环
• 不需要存储所有元素的统计结果，直接在统计过程中更新最大值，逻辑更简洁

测试这两个版本，输入示例[1,5,1,3,5,1]都会输出1 3，符合预期。

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