python out of rangeトラブル

pythonを始めたばかりの初心者です。
checki0で学習を進めているのですが、"The Most Frequent”という問題があり、リストの中で最も多く使われている文字をプリントしなければなりません。
以下のコードで行うとアウトプットは正常に機能するのですが、check solutionを行うと7行目のif文がout of rangeとなってしまい、うまくいきません。
試しにdata[1]やdata[0]をプリントしたのですが、ちゃんとそれらの該当の値が出てきました。
どなたかご指摘お願いいたします。

python
1コード
2def most_frequent(data: list) -> str:
3    """
4    determines the most frequently occurring string in the sequence.
5    """
6    # your code here
7    
8    if data.count(data[1])>data.count(data[0]):
9        m=data[1]  
10        
11    else:
12        m=data[0]
13        
14    for i in range(2,len(data)-1):
15        if data.count(data[i])>data.count(m):
16            m=data[i] 
17
18    return m
19
20
21
22if __name__ == "__main__":
23    # These "asserts" using only for self-checking and not necessary for auto-testing
24    print("Example:")
25    print(most_frequent(["a", "b", "c", "a", "b", "a"]))
26
27    assert most_frequent(["a", "b", "c", "a", "b", "a"]) == "a"
28
29    assert most_frequent(["a", "a", "bi", "bi", "bi"]) == "bi"
30    print("Done")
31

melian

2022/03/11 10:44

most_frequent([]) の様に空リストを渡す場合を考慮する必要がありそうです。

行動規範の内容に同意します

回答1件

ベストアンサー

set() で重複を除いたものをベースにカウントしていくのはどうでしょうか。

python
1    m = ''
2    n = 0
3    for x in set(data):
4        if data.count(x) > n:
5            m = x
6            n = data.count(x)
7
8    return m

洗練されていない部分は、まあアレで。
--- 追記 ---
原因は melianさんが指摘されている通りで、無条件に配列の先頭を参照してることですね。
--- 追記 ---
手抜きするならこんな方法も

python
1from collections import Counter
2
3def most_frequent(data: list) -> str:
4    if len(data) == 0:
5        return ''
6    return max(Counter(data),key=Counter(data).get)