自作関数内の処理が引数側にも適用されてしまう

Question

前提・実現したいこと

python初心者です
こちらのサイトを参考に心拍を検出する関数を作っていたのですが問題が発生したため、質問をさせていただきます。

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

以下の関数を用いた場合に、
もとのプログラム内の第2引数として用いた配列に対しても関数内で行った処理が適用されてしまいます。
関数内で一度別の変数に受け渡しを行ってみたのですが改善されず、困っております。

該当のソースコード

def peaks(sec, res):  # 時間配列.ECG配列
    res2=res
    f = sec[1]-sec[0]
    pwidth = 0.5 / f
    thres = max(res) * 0.7
    length = len(res)
    res2[res2 < thres] = thres

    dif = res2[1:length-1]-res2[0:length-2]
    ddif = dif[1:(length-2)]*dif[0:(length-3)]
    ppos = np.array([0])
    while 1:
        p = np.argmin(ddif)
        if ddif[p] == 0:
            break
        if len(ppos) > 0:
            q = abs(ppos-p)
            if min(q) > pwidth:
                ppos = np.append(ppos, p)
        else:
            ppos = np.append(ppos, p)
        ddif[p] = 0
    ppos = ppos[1:]
    ppos += 1
    return np.sort(ppos)

Answer 1

リストのコピーでハマった話 - Qiita

ここらへんが参考になるかと。
クラス変数（リストも含めて）なんかは、引数に渡すとアドレスが渡される形となるため、関数内でナカミを操作すると渡し元の方も変更されてしまいます

Comments

[shibainu85]

リストのコピーの仕様は初耳でした。
知っていれば簡単なことだったんですね、ありがとうございました！

[LouiS0616]

@y_waiwai さん
なんとなく言いたいことは分かります。
ただ『クラス変数』という単語はPythonでは全く別の意味を持っているので、この場合は不適です。

そもそもPythonは全てがオブジェクトなので、Javaの 値型／参照型 みたく呼び分けることはありません。

Answer 2

単にres2=resとしてもres2がresと同じ配列を指すようになるだけで新しい配列が作られるわけではありません。

処理内容をちゃんと見ていないので確実なことは言えませんが、

■ 配列の中の値を変更して計算するのが効率がよかったりわかりやすいのであれば、配列をコピーして使うのがいいと思います。
そのとき、配列の複雑さによっては、深い(deep)コピーをする必要があるかもしれません。

■ 配列の中身を変更しなくて済む方法があるのであれば、たぶん、そのほうが効率的でしょう。

Comments

[shibainu85]

配列の仕様、とても勉強になります
deepcopy等についてはもう一人の回答者の方のサイトを見て学んでみたいと思います。
ありがとうございました！

[TakaiY]

これは、配列の仕様ではなく、dictionaryなどでも同じです。

Answer 3

res2=np.array(res)

とすることでもとの配列に影響を及ぼさなくなったのでとりあえずこちらでいこうかと思います。