Pythonにおける「ist ndex out of range」への対処

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-82-ae76bf4842a6> in <module>()
----> 1 print(remove_evenindex(["a", "b", "c", "d", "e", "f", "g"]) == ['b', 'd', 'f'] )
      2 print(remove_evenindex([1, 2, 3, 4, 5]) == [2, 4])

<ipython-input-81-73cce9575198> in remove_evenindex(ln)
      1 def remove_evenindex(ln):
      2     for j in range(0,len(ln),2):
----> 3         ln.remove(ln[j])
      4     return ln

IndexError: list index out of range

def remove_evenindex(ln):

for j in range(0,len(ln),2):
    ln.remove(ln[j])
return ln

Python 

試したこと

for j in range(0,len(ln),2)
を
for j in range(1,len(ln),2)
にして、

def remove_evenindex(ln):
ln2 = []
for j in range(1, len(ln), 2):
ln2.append(ln[j])
return ln2

でコードを書けば、問題なくできます。
なぜ「list ndex out of range」となってしまうのでしょう。

補足情報（FW/ツールのバージョンなど）

プログラミング自体、初めて3日目の初心者です。
どうか、皆様のお知恵で、初心者でも理解できるレベルでご説明頂ければ幸いです。

Answer 1

なぜ「list ndex out of range」となってしまうのでしょう。

実際に値を覗いてみると良いです。

def remove_evenindex(ln):
    for j in range(0,len(ln),2):
        print(f'I will remove {j}th elem from {ln}')
        ln.remove(ln[j])
    return ln

remove_evenindex(["a", "b", "c", "d", "e", "f", "g"])

実行結果 Wandbox

I will remove 0th elem from ['a', 'b', 'c', 'd', 'e', 'f', 'g']
I will remove 2th elem from ['b', 'c', 'd', 'e', 'f', 'g']
I will remove 4th elem from ['b', 'c', 'e', 'f', 'g']
I will remove 6th elem from ['b', 'c', 'e', 'f']
Traceback (most recent call last):
  File "prog.py", line 7, in <module>
    remove_evenindex(["a", "b", "c", "d", "e", "f", "g"])
  File "prog.py", line 4, in remove_evenindex
    ln.remove(ln[j])
IndexError: list index out of range

リスト長がどんどん短くなりますが、その都度 len(ln) を再計算してはくれないのです。
ループ対象のデータをいじると、しばしばこのような問題を引き起こします。

解決策

NaokiHiroseさんの書かれたコードのように、新たにリストを作るのが簡単です。
これならば元のリスト長は変化しません。

---

なお、スライスを使うと次のように簡潔に書けます。

def remove_evenindex(ln):
    return ln[1::2]

Comments

[NaokiHirose]

物凄く分かりやすい回答を、しかも迅速にくださりありがとうございました。
駆け出しの私でも一瞬で理解できるほど、分かりやすいものでした。