0から99までの足し算

Question

0から99までの足し算をpop()を使用して行いたいのですが、模範解答が理解できません。わざわざrangeとlenを使っている意味を教えて頂きたいです。
本当の初心者なのでわかりやすく教えて頂きたいです。

模範解答はこれです

list_hundred = list(range(100))
sum_all = 0
for i in range(len(list_hundred)):
    sum_all += list_hundred.pop()
print(sum_all)

自分の答えはこれです

list_hundred = list(range())
sum_all = 0
for i in list_hundred:
  sum_all += list_hundred.pop()
print(sum_all)

Comments

[Takumiboo]

「POP」タグは今回の質問と無関係なようなので外してください。

[tiitoi]

そもそも pop していく必要があるんですかね
for i in list_hundred:
sum_all += i でいい気がするのですけど

[meg_]

質問者さんのコードは「TypeError: range expected 1 argument, got 0」が出ます。

Answer 1

正しい回答方法などではなく
なぜ今回の様な結果になっているかについて例に挙げて解説させていただきます。

用意したリストを使用してfor文でループさせていますが
その用意したリストの中から1つずつ要素を削除してループさせている為に
ちょうど半分時点でループが終了してしまっています。
この様な動作を反復的、反復処理と言います。

手元に全100巻の漫画があったとします。
1巻読み進める都度、最後の1巻を燃やしたとしたら
49巻に到達した時点で次の巻がなくなってしまいますよね？

手元にある漫画全巻 = list(range(100))

for 現在の巻 in 手元にある漫画全巻:
	print('今読んだ巻 :', 現在の巻)
	print('燃やした最後の巻 :', 手元にある漫画全巻.pop())
	print('残っている巻数一覧 :\n', 手元にある漫画全巻)

	次の巻数 = 現在の巻 + 1

	try:
		次の巻 = 手元にある漫画全巻[次の巻数]
		print('次は' + str(次の巻) + 'を読む予定です')
	except IndexError:
		print('もう次に読む' + str(次の巻数) + '巻が燃やされてしまっていているので処理は終了です')

	print('='*50)

Answer 2

Pythonの公式サイトに、その事が書かれています。

コレクションオブジェクトの値を反復処理をしているときに、そのコレクションオブジェクトを変更するコードは理解するのが面倒になり得ます。 そうするよりも、コレクションオブジェクトのコピーに対して反復処理をするか、新しいコレクションオブジェクトを作成する方が通常は理解しやすいです:

実際、「自分の答え」のコードでは正しい結果を出力しません。

Comments

[maisumakun]

popさせた値も出力しながら見ていくと、半分しか足されていないことがわかります。

https://paiza.io/projects/qd7tDgIVTnTzoiaZqQKwMw?language=python3

[ebal]

正直，これ模範解答もなぜiが必要なのか，なぜforなのかといったところで気持ち悪い気がします．

気持ちいい回答1 再帰
```python3
list_hundred = list(range(100))
def recur_add(acc, l):
if l:
return recur_add(acc+l.pop(), l)
else:
return acc

print(recur_add(0, list_hundred))
```

気持ちいい回答2 while
```python3
list_hundred = list(range(100))
while list_hundred:
sum_all+= list_hundred.pop()
print(sum_all)
```

[toast-uz]

あー、でも、pop()の意味を掘り下げると、ebal様の回答2の方がよいかも。

[ebal]

回答1は!!! なんて冗談はおいておいて，popで周回するのってなかなかないですよね．
コピー作らないしwhile内でリストにappendできるしで回答2はそれなりにわかりやすくなったのではないかと思っています．

[toast-uz]

回答1は、「もっと気持ち悪い回答１」と命名しておけば、Good!

[ebal]

listにおけるpopの使い方としては（見た目上）無駄にループもしないし一番正しいのでは……？（気持ち悪さとコールスタックは無視）

[toast-uz]

再起がは良いとしても、同じ引数の中でlが変化しているところは、改善したいですね
return recur_add(acc+l.pop(), l)

[ebal]

代入してから，というのも考えたのですが，再帰としては代入なんか使わないで一行で収まるほうが正義かなと考えてこうなりました．（よってとても気持ち悪い）

[toast-uz]

popという変数を変更する操作が、再帰という関数型的な構造に入っているから、気持ち悪いのでしょうか・・。popから離れて、再帰を単に書くだけなら、気持ち良く書けます。

list_hundred = list(range(100))
def recur_add(l):
if l:
return l[0] + recur_add(l[1:])
else:
return 0

print(recur_add(list_hundred))

[maisumakun]

言語を問わず、「あるコレクションを基準に回しながら、そのコレクションの中身を書き換える」ような行為は、入念な注意が必要です（その構造を取る積極的な理由がないなら、避けるほうが賢明です）。Javaなどでは例外が起きる場合もあります。

[ebal]

そのあたりはやっぱりLispやRubyの破壊的変更は`!`付きメソッドというのわかりやすいですよね．
toast-uzさんのコードだと（Pythonでは意味ないけど）末尾再帰にならないのでアキュムレータはほしいと思ってしまいます．

[toast-uz]

末尾再帰版です！

list_hundred = list(range(100))
def recur_add(acc, l):
if l:
return recur_add(acc + l[0], l[1:])
else:
return acc

print(recur_add(0, list_hundred))

maisumakun様の言う通り、結局、この問題設定がよくないです、が結論ですね。そこを主張して「この講座テキスト？は使わない方が良い」と断言した回答を書いた人が、ベストアンサー決定!　笑

[ebal]

popが消えてしまった……
この問題設定が謎で，「このテキスト（仮）は使わないほうが良い」という結論には強く同意します．

Answer 3

※ 質問の意図を間違えた回答でしたので、削除しました。

Answer 4

この質問を見て、私も質問者様の気持ちになり、その後になぜ結果が異なるのか、少し悩みました。

ということで、不具合の原因に直接対処する、気持ちの良いコードを作ってみました。

list_hundred = list(range(100))
sum_all = 0
for i in list_hundred.copy():  # イテレーション用に新たなリストを作る
  sum_all += list_hundred.pop()
print(sum_all)