なぜエラーになるのかわかりません

Question

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
Python3です
配列の各要素を2倍した配列を返す関数all2x(list)を作成したいです。
詳細
all2x(list)は配列を引数にとり、配列を返す関数です。
渡される配列は1つ以上の要素を持ちます。
実行例
all2x([1, 2, 3]) => [2, 4, 6]
all2x([0, 0, 0]) => [0, 0, 0]
###発生している問題・エラーメッセージ

all2x([1, 2, 3])

Traceback (most recent call last):
File "<pyshell#1>", line 1, in <module>
all2x([1, 2, 3])
File "<pyshell#0>", line 3, in all2x
list[i]*=2
IndexError: list index out of range

###該当のソースコード
def all2x(list):
for i in list:
list[i]*=2
return list
###試したこと
def all2x(list):
return [x * 2 for x in list]
で済むことはわかりましたが上のコードで同じ結果は望めないのでしょうか？

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

listが[1, 2, 3]の時に、for i in list:だと、iは、添え字の0,1,2じゃなくて中身の1,2,3になるので、iが3の時に添え字範囲オーバーになります。

Comments

[mimamoru]

ありがとうございます。なぜインデックスが超えてしまうのかが良くわかりました！助かりました！

Answer 2

こんにちは。

def all2x(list):
  for i in list:
    list[i]*=2
  return list

だと、 i には、list の要素そのものが入ってきますが、
その書き方では、i に要素のインデクスが入って
くることを想定したコードになっているから、うまく動かない
のではないでしょうか？

参考になれば幸いです。

---

追記

def all2x(list):
  for i in list:
    list[i]*=2
  return list


all2x([1,2,3])

と書いて、実行してみたときに、[発生している問題・エラーメッセージ] にあるとおり
IndexError: list index out of range
が出るわけですが、このメッセージは、
「リストのインデクスがリストの許容する範囲外になっている。」
という意味です。
これを見たときに、
「あれ？　インデクスがリストの範囲外になってる？そんなはずはないだろう・・・さて？」
と思うことが大事です。
そう思えたら
「では、範囲外になってしまったインデクスの値は何だろう？」と思うでしょう。
すると、list[i]*=2 で、list のインデクスにしている i の値を
ループの毎回表示させるために、直前に print(i)　を入れて

def all2x(list):
  for i in list:
    print(i)
    list[i]*=2
  return list

all2x([1,2,3])

を実行してみたりするわけです。（上記を main.py とします）
すると、以下のように、２回目の print(i) で（なんと） 4 が出力されます。

最初のコードでは、i は リストのインデクスになる想定で書いており、
0, 1, 2
になることを期待しているわけですから、上記の 4 で明らかに
何かがおかしいと思うことができます。

Comments

[mimamoru]

詳しいソースコードまで載せていただきありがとうございました！とてもよくわかりました

[jun68ykt]

解決したようでよかったです！

Answer 3

mkgreiさんの回答のほかに、こういうのもありです。

for i, _ in enumerate(my_list):
    my_list[i] *= 2

---

なぜエラーになるのかわかりません

printデバッグをしてみると、すぐに理由がわかるかと思いますよ。
下記のようにループを書いた場合、iは添え字ではなく各要素の値となります。

>>> for i in [3, 1, 4]:
...     print(i)
...
3
1
4

エラー自体はただの範囲外エラーです。(otnさんが指摘されているとおり)

>>> hoge = [1, 2]
>>> hoge[0]
1
>>> hoge[1]
2
>>> hoge[2]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
IndexError: list index out of range

ついでに

リストのインスタンスの名前として、listは使わないでください。
組み込み型のlistを潰してしまい、具体的には次のような不可解なエラーを引き起こします。

>>> list(range(3))
[0, 1, 2]
>>> list = [4, 5, 6]
>>> list(range(3))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'list' object is not callable

同様に、str、double、max、min、idなどは使ってはいけません。

---

teratailには、上記のようにコードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。
特にPythonの場合、インデントが崩れるとコードの意味が変わってしまいます。

Comments

[mimamoru]

解説から注意点まで教えてくださりありがとうございました！新しいことも知ることができてよかったです！

Answer 4

for i in range(len(list)):

---

追記：
もともとの書き方がうまくいくのはJavascriptやNode.jsだと思います。
以下のスクリプトは4,5,6,7,8ではなく0,1,2,3,4になります。
Pythonではfor...inはインデックスではなく中身が取り出されます。

var a = [4,5,6,7,8];
for (var i in a) {
  console.log(i);
}

Comments

[mimamoru]

素早い回答ありがとうございました　とてもよくわかりました！