関数を使ってリストの値を変えたい

Question

recoverを関数としてリストの要素が１かつrandom.random()で生成した値が0.5以下の場合にその値を２にする
プログラムを作りたいです。
自分ではこうしたのですが、何も変化がありませんでした。
import random
def recover(arglist,prob):
for val in arglist:
rnd1 = random.random()
if val == 1 and rnd1 <= prob:
val += 1
else:
val += 0
return arglist

print(recover([0,1,0,1],0.5))

Answer 1

関数定義していることは、今回の問題には関係無いです。

---

valは確かにarglistの一要素ではありますが、valへの代入がarglistに影響することはありません。※
次のコードでarrが[42, 1, 4]にならないのと同じ理由です。

arr = [3, 1, 4]
e = arr[0]
e = 42

print(arr)  # => [3, 1, 4]

次のように添え字で直接アクセスすれば変更可能です。

arr[0] = 42
print(arr)  # => [42, 1, 4]

添え字アクセスをfor文で書こうとする場合、rangeやenumerateを使うことになります。

for i in range(len(arr)):
    if arr[i] == ???:
        arr[i] = ???

for i, e in enumerate(arr):
    if e == ???:
        arr[i] = ???

※ ちょっとややこしい話

a += b と書いたとき、
aの型に依ってはインプレースに操作が行われ、当然元のリストにも影響します。

Pythonにおいては、a += b と a = a + b の意味が異なることがあります。

a = [1, 2, 3]
b = a
b = b + [4, 5, 6]

print(a, b)  # => [1, 2, 3] [1, 2, 3, 4, 5, 6]

a = [1, 2, 3]
b = a
b += [4, 5, 6]

print(a, b)  # => [1, 2, 3, 4, 5, 6] [1, 2, 3, 4, 5, 6]

コードの貼り方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

特にPythonの場合、インデントが崩れるとコードの意味が変わってしまいます。