listの中の要素の一つが実行されない

Question

IDLEを使用

目的

words.txtの中からそれぞれのアルファベットから始まる言葉の数をカウントしてprintし、最後に一番多いアルファベットをもう一度printする（英語を無理やり翻訳したので間違っていたらすいません）

words = open('words.txt','r')
letters = ['a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z']
x = 0
cnt = 0
num = 0
y = 0
for w in words:
    l = letters[x]
    w = w.lower()

    if w[0] == l:
        cnt += 1
    else:
        print(letters[x],':',cnt)
        if num < cnt:
            num = cnt
            cnt = 0
            y = letters[x]
            x += 1
        else:
            cnt = 0  
            x += 1
print(y,':',num)

試したこと

学校の授業でやったものです。
今まで習ったものを使って作っていて、コードは先生に確認したのであってると思います。
しかし、実行した時にzの結果だけが表示されずに、yで止まってしまい、最後のprintが実行されてしまいます。エラーの表示はありません。先生も原因はわからないと言っていました。友達も同じ状況になっているので、自分だけではないと思います。他のテキストエディタ（Atom, Xcord）も試しました。

解決法と原因を知りたいです。

使ったファイル

words.txt

追記：
’他のユーザからのアドバイスにより追加検証’
listの後ろの部分を抜いてやると、 ’l = letters[x] IndexError: list index out of range’ というエラーが表示され、前の部分を抜いてやると、結果が全て’0’で表示され、前者と同じエラーが出ました。

Comments

[8524ba23]

「a」「b」「b」の３行のみのような単純なファイル例を試してみれば明らかですが
コードに問題があるように思えます。
コード（先生？）があっていることはどのようにして確認されましたか？あるいは問題は正しいですか？

[Kiri3]

そういう検証はやっていませんでした。貴重な情報ありがとうございます。
試したところ、listの後ろの部分を抜いてやると、 ’l = letters[x] IndexError: list index out of range’ というエラーが表示され、前の部分を抜いてやると、結果が全て’0’で表示され、前者と同じエラーが出ました。

[8524ba23]

リストというのがlettersを指しているのであれば、入力データ（a～zすべてが含まれる）においてエラーが出るのは試すまでもなく明らかだと分かるのですが、なぜそれを試そうと思われたのでしょうか？

[Kiri3]

26字の中でひとつでもかけていればエラーが出るけど、Lettersにアルファベットa~zが全て含まれる場合、'エラー'は出ないんです。理由は分かりませんが...

Answer 1

異なる文字が出てきたときに、プリントするようになっていますが、zの次は異なる文字が出てこないのでプリントされません。
安易に修正すると、

ords = open('/home/t/lib/words','r')
letters = ['a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z']
x = 0
cnt = 0
num = 0
y = 0
for w in words:
    l = letters[x]
    w = w.lower()

    if w[0] == l:
        cnt += 1
    else:
        print(letters[x],':',cnt)
        if num < cnt:
            num = cnt
            cnt = 0
            y = letters[x]
            x += 1
        else:
            cnt = 0
            x += 1
print(letters[x],':',cnt)
if num < cnt:
    num = cnt
    y = letters[x]
print(y,':',num)

Comments

[Kiri3]

うまくいきました！ありがとうございます。

Answer 2

変数名前を変えて すこしわかりやすくしてみました。
wc.py

import string

LETTERS = list(string.ascii_lowercase[0:26])

words = open('words.txt','r')
count = 0
ch_idx = 0
max_count = 0
max_ch = ' '

ch = LETTERS[ch_idx]
for w in words:
    if w.lower()[0] == ch:
        count += 1
        continue
    
    print(ch, ':', count)
    if max_count < count:
      max_count = count
      max_ch = ch

    count = 0
    ch_idx += 1
    ch = LETTERS[ch_idx]

print(ch, ':', count)
if max_count < count:
  max_count = count
  max_ch = ch

print(max_ch, ':', max_count)

実行例

Comments

[Kiri3]

import stringバージョンも真似して作らせてもらいました。ありがとうございます！