なぜout of rangeになるのか分からない

Question

実現したいこと

カードの数字の積の最大値を求めたい

前提

Nはカードの枚数(2 <= N <= 10)
CにN個の数字がスペースで区切られて入力されます。(-5 <= c <= 5)

そのカードの中から2枚取り出し(card_a, card_b)、
片方の数から1を引いて、もう片方の数に1を加えます。
(card_a-1, card_b+1)

そしてcard_a,とcard_bと残りのカードの数の積を求めます。
この積の最大値を求めたいです。

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

Traceback (most recent call last):
  File "Main.py", line 12, in <module>
    l.append(card_a* card_b* copylist[0]* copylist[1]* copylist[2])
IndexError: list index out of range

l.append(数式)の部分がエラーになっているようです

C = [-3,-1,-5,1,2]の場合に正しい結果(０)を導き出せない問題があります。

該当のソースコード

# coding: utf-8
import math
N = int(input())
C = [c for c in map(int,input().split(" ")) if 2 <= N <= 10 and -5 <= c <= 5]
l = []
for card_a in C:
    copylist = C.copy()
    copylist.remove(card_a)
    card_a -= 1
    for card_b in copylist:
        copylist.remove(card_b)
        card_b += 1
        l.append(card_a* card_b* math.plod(copylist)) #変更
print(max(l))

試したこと

元のリストCをコピーして、card_a, card_bをremoveメソッドで削除しました。
コピーしたリストには3つの数字が残っているはずなのですが、
なぜかcopyist[2]と書くとout of rangeになってしまいます。

Comments

[Plasma]

12行目にprint(copylist)を追加すると

[5, 1, -4]
[5, -4]

このように二つリストが表示されます。
朝から色々試行錯誤していますがこの原因が全く分からないので困っています。

[TakaiY]

> l.append(card_a* card_b* copylist[0]* copylist[1]* copylist[2])

ここで、なぜ、「残りのカード」を3枚(0～2)固定としているのでしょうか？
N は 5 と決まっているのですか？

[Plasma]

確かにそうですね！入力値を固定したまま試行錯誤していました！

import math
l.append(card_a* card_b* math.prod(copyist))

↑このように変更したらC = [1,2,3,4,5]の場合に正しい答え(⭕️192)を出せたのですが、
C = [-3,-1,-5,1,2] のような場合に間違えてしまいます。(❌80→⭕️0)

他にもおかしいところがあるみたいです...

[melian]

こちらでは、C = [-3,-1,-5,1,2] の場合を変更後のコードで実行すると正しく 80 という結果が返ります。

[Plasma]

失礼しました。正答が「0」になるようなのですが、
80が出てしまうという事ですね。

マイナスが3つあるので単純に積はマイナスになるのですが、
-1に対して1を加えて0にすることで
最大値が「0」になるという事だと思うのですが、
このプログラムだとその結果が出せないようです...

[melian]

お手数ですみません、読み間違いをしていました。

[TakaiY]

デバッグしてどのような時に80となるのか調べてみたらいいと思うのですが。

また、can110さんの回答は確認されましたか？

Answer 1

エラーの原因はおいておいて、そもそも提示された問題を解く場合にremoveを使うのは止めた方がよいです。
removeは同じ値があれば、つねにその先頭のものが削除されます。
つまり[1,2,1].remove(1)の結果はつねに[2,1]になります
そのかわりに配列の位置ベースで考えたほうが分かりやすいです。

以下のコードがどのような動きをするかを確認し
問題を解くためにどのように修正して動かせばよいかを考えてみてください。

N = 3
C = [1,2,3]
for na in range(N):
    for nb in range(N):
        if True: # bはa以外を選択したい…
            print(f'a={C[na]}, b={C[nb]} 他=', end='')
            for n in range(N):
                if True: # 他はa,b以外を選択したい…
                    print(C[n], end=',')
            print()
"""
a=1, b=1 他=1,2,3,
a=1, b=2 他=1,2,3,
a=1, b=3 他=1,2,3,
a=2, b=1 他=1,2,3,
a=2, b=2 他=1,2,3,
a=2, b=3 他=1,2,3,
a=3, b=1 他=1,2,3,
a=3, b=2 他=1,2,3,
a=3, b=3 他=1,2,3,
"""

Answer 2

例えばN=3であれば最初のappendまでに
copylist.remove(card_a)
copylist.remove(card_b)
で2要素が消されますのでcopyistの要素数は1になります。
明らかにcopyist[2]は範囲外です。

Nが大きくてもcopylist.remove(card_b)が繰り返されてcopyistの要素数はどんどん減るので
いつかはcopyist[2]は範囲外になります。

Answer 3

なぜかcopyist[2]と書くとout of rangeになってしまいます。

エラーが出たからとそこで思考停止するんではなく、なぜそうなったのかを調査しましょう。
まずは、copyistになにが入ってるのか見てみよう