Python

【Python】コンストラクタの引数のデフォルト値にミュータブルな変数を指定してはいけない理由

MAX

クラスを作成する場合、通常は__init__()でインスタンス変数などの初期設定を行う。

__init__()には引数を指定できるが、その引数のデフォルト値にミュータブルな型の変数を指定するとバグの温床となる。

多分きっと、初心者は必ず通る道であると思う。(そして、ミュータブル、イミュータブルという考え方を意識し始める(はず))

スポンサーリンク

ミュータブルなデフォルト値を設定した場合

リストをデフォルト値に持つクラス作成

ミュータブルな型の代表例としてリストを使ってみる。

1class ClsA:
2    # valsのデフォルト値に空のリストを設定
3    def __init__(self, a, vals=[]):
4        self.a = a
5        self.vals = vals
6
7# ClsAのインスタンスを2種類作成する。
8# valsにはデフォルト値が設定されるようにする。
9cls_a_1 = ClsA(1)
10cls_a_2 = ClsA(2)
11
12# cls_a_1のvalsのidと内容確認
13print(f"{id(cls_a_1.vals)=}")
14print(cls_a_1.vals)
15
16# cls_a_2のvalsのidと内容確認
17print(f"{id(cls_a_2.vals)=}")
18print(cls_a_2.vals)
19
20# id(cls_a_1.vals)=2360930307456
21# []
22# id(cls_a_2.vals)=2360930307456
23# []

cls_a_1のvalsとcls_a_2のvalsのidが同じ。

リストはミュータブルなので、cls_a_1またはcls_a_2のvalsにオブジェクトidが変わらない操作を行うと、両方のインスタンスのvalsが更新される。

デフォルト値が設定されたインスタンス変数の更新(破壊的)

リストは大抵の操作がオブジェクトのidを変えずにデータを更新可能。

例として、よく使われるappendをしてみる。

1# cls_a_1のvalsのidが変わらない操作を行う。
2cls_a_1.vals.append(1)
3
4print(f"{id(cls_a_1.vals)=}")
5print(cls_a_1.vals)
6
7print(f"{id(cls_a_2.vals)=}")
8print(cls_a_2.vals)
9
10# id(cls_a_1.vals)=2360930307456
11# [1]
12# id(cls_a_2.vals)=2360930307456
13# [1]

cls_a_1のvalsを変更すると、同じオブジェクトを参照しているcls_a_2のvalsの値も更新される(更新というよりも同じオブジェクトを指しているだけ)。

通常、特定のインスタンスに対して行った変更が、別のインスタンスにまで反映されることを想定して開発することはなく、バグの温床となる。

デフォルト値が更新された状態でインスタンス生成

上記のような操作でデフォルト値が更新された状態でデフォルト値を使うようなインスタンス生成を行うと、当然、更新された状態のデフォルト値でインスタンスが生成される。

1cls_a_3 = ClsA(3)
2print(f"{id(cls_a_3.vals)=}")
3print(cls_a_3.vals)
4
5# id(cls_a_3.vals)=2360930307456
6# [1]

valsの値を指定しない場合は空のリストとなることを想定しているので、確実にバグの温床となる。

イミュータブルなデフォルト値を設定した場合

Noneをデフォルト値に持つクラス作成

困った時のNone。

1class ClsB:
2    # valsのデフォルト値にNoneを設定する
3    def __init__(self, a, vals=None):
4        self.a = a
5        # valsの値がNoneの場合、空のリストを設定する
6        self.vals = vals if vals is not None else []
7
8# 2個のインスタンス生成
9# valsはデフォルト値を使って生成
10cls_b_1 = ClsB(1)
11cls_b_2 = ClsB(2)
12# 各インスタンスのvalsのオブジェクトidと値の確認
13print(f"{id(cls_b_1.vals)=}")
14print(cls_b_1.vals)
15
16print(f"{id(cls_b_2.vals)=}")
17print(cls_b_2.vals)
18
19# id(cls_b_1.vals)=2360929857792
20# []
21# id(cls_b_2.vals)=2360929996224
22# []

どちらのvalsも値は空のリストだが、オブジェクトidが異なる。

インスタンス変数の更新

オブジェクトidが異なるので、cls_b_1のvalsを更新しても、他のインスタンスには反映されない。

1cls_b_1.vals.append(1)
2print(f"{id(cls_b_1.vals)=}")
3print(cls_b_1.vals)
4
5print(f"{id(cls_b_2.vals)=}")
6print(cls_b_2.vals)
7
8# id(cls_b_1.vals)=2360929857792
9# [1]
10# id(cls_b_2.vals)=2360929996224
11# []

多分、これが、みんなが想定している動作。

まとめ

破壊的な変更が可能(オブジェクトのidが変わらずに内容を変更可能)なオブジェクトは基本的にはクラスのコンストラクタの引数に設定しない方が良い。

  • リスト
  • DataFrame、Series

DataFrameやSeiresは必ずしも破壊的な変更になるとは限らないが、空のDataFrameやSeriesに何か操作をする場合、破壊的な変更をしてしまう可能性はあると思う。

少々面倒でも、デフォルト値にはNoneなどを設定し、処理の中で空のリストやDataFrameなどを設定するようにした方が良い。

スポンサーリンク
ABOUT ME
MAX
MAX
ITエンジニア、データサイエンティスト
新卒でSIerに入社し、フリーランスになってWEB系へ転向。
その後AIの世界へ足を踏み入れ、正社員に戻る。 テーブルデータの分析がメイン。
スポンサーリンク
記事URLをコピーしました