目次と前回の記事

これまでに作成したモジュール

以下のリンクから、これまでに作成したモジュールを見ることができます。

ルールベースの AI の一覧

ルールベースの AI の一覧については、下記の記事を参照して下さい。

`play` メソッドのバグ

前回の記事で、play メソッドを実行した際に 仮引数 ai に代入されている 値を元 に、Dropdown の項目 を 自動的に設定 するプログラムを記述しましたが、その際に play メソッドには 重大なバグ が いくつか存在 するという説明を行いました。

バグその 1（人間の項目が複数作られる）

下記は、play メソッドで人間 VS AI の対戦を行うプログラムです。実行結果 の 下図左 から、正しく動作 しているように 見えるかも しれませんが、Dropdown の上 で マウスを押す と、右下図 のように、人間の項目 が 複数表示 されるという 問題が発生 することが確認できます。このようなことが起きる原因について少し考えてみて下さい。

%matplotlib widget
from marubatsu import Marubatsu

mb = Marubatsu()
mb.play(ai=[None, None], gui=True)

実行結果（下図は、画像なので操作することはできません）

バグの原因の考察

下記は、play メソッドの中で、Dropdown の項目 を作成する部分のプログラムです。

 1  # ai に代入されている内容を ai_list に追加する
 2  for i in range(2):
 3      # ai[i] が ai_values に登録済かどうかを判定する
 4      if ai[i] not in ai_values:
 5          # ラベルと項目の値を計算する
 6          if ai[i] is None:
 7              label = "人間"
 8              value = "人間"
 9          else:
10              label = ai[i].__name__        
11              value = ai[i]
12          # 項目を登録する
13          ai_list.append((label, value))
14          ai_values.append(value)
15          select_values.append(value)

下記は、上記の中 で Dropdown の項目 に 関する処理 を 箇条書き にしたものです。

4 行目：ai[i] が ai_values に 登録済か どうかを判定する
8 行目：人間の担当 の場合は、value に None を代入する
11 行目：AI の担当 場合は、value に ai[i] を代入する
14 行目：ai_values に value を追加する

上記の 4 行目 で ai_values に 登録済であるか どうかを判定する際に使われている ai[i] と、15 行目 で ai_values に追加する、value に代入 されている値が、人間が 担当する場合 は 異なる点 に注目して下さい。人間の 担当の場合 には ai_values に "人間" が登録 されるため、次の繰り返し処理 が 人間の手番 だった場合に、ai[i] in ai_values が False になるので もう一度 15 行目で ai_values に "人間" が登録 されてしまいます。

言葉だけの説明ではわかりづらいと思いますので、人間 VS 人間の場合で 行われる処理 で、関連する 変数の値 が どのように変化 するかを 下記の表 で 示します。なお、条件式 の列は、その行 の 条件式の値 を表します。表から、上記のプログラムを実行すると、ai_list に 人間の項目 が 2 つ登録 されてしまうことが わかります。

	条件式	`i`	`ai[i]`	`value`	`ai_values`	`ai_list`
2 行目		`0`	`None`		`[]`	`[]`
4 行目	`False`	`0`	`None`		`[]`	`[]`
6 行目	`False`	`0`	`None`		`[]`	`[]`
8 行目		`0`	`None`	`"人間"`	`[]`	`[]`
13 行目		`0`	`None`	`"人間"`	`["人間"]`	`[]`
14 行目		`0`	`None`	`"人間"`	`["人間"]`	`[("人間", "人間)]`
2 行目		`1`	`None`		`["人間"]`	`[("人間", "人間)]`
4 行目	`False`	`1`	`None`		`["人間"]`	`[("人間", "人間)]`
6 行目	`False`	`1`	`None`		`["人間"]`	`[("人間", "人間)]`
8 行目		`1`	`None`	`"人間"`	`["人間"]`	`[("人間", "人間)]`
13 行目		`1`	`None`	`"人間"`	`["人間", "人間"]`	`[("人間", "人間)]`
14 行目		`1`	`None`	`"人間"`	`["人間", "人間"]`	`[("人間", "人間),` `("人間", "人間)]`

プログラムを 確認できる ように、コメントを除いた プログラムを下記に再掲します。

 2  for i in range(2):
 4      if ai[i] not in ai_values:
 6          if ai[i] is None:
 7              label = "人間"
 8              value = "人間"
 9          else:
10              label = ai[i].__name__        
11              value = ai[i]
13          ai_list.append((label, value))
14          ai_values.append(value)
15          select_values.append(value)

本当は、2 回目 の 2、4、6 行目 では、value に "人間" が 代入されたまま になっていますが、8 行目 で 改めて value に "人間" が 代入される ことを 明確するため に空欄に しました。

バグの修正

この問題 は、4 行目 の 条件式 で、ai_values に実際に 登録する値 を 使って、in 演算子 で 判定を行う ことで解決することが できます。そのため には、ai_values に登録する value を計算する 6 ~ 13 行目 にあった処理を、4 行目 の 条件式より も 前に移動 する必要が あります。下記は、そのよう に play メソッドを修正したプログラムです。

9 ~ 14 行目：16 行目 の if 文 の ブロックの中 に記述されていた処理を、if 文 の ブロックの前 に移動する

 1  import matplotlib.pyplot as plt
 2  import ipywidgets as widgets
 3  import math
 4
 5  def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
元と同じなので省略
 6      # ai に代入されている内容を ai_list に追加する
 7      for i in range(2):
 8          # ラベルと項目の値を計算する
 9          if ai[i] is None:
10              label = "人間"
11              value = "人間"
12          else:
13              label = ai[i].__name__        
14              value = ai[i]
15          # value が ai_values に登録済かどうかを判定する
16          if value not in ai_values:
17              # 項目を登録する
18              ai_list.append((label, value))
19              ai_values.append(value)
20              select_values.append(value)
元と同じなので省略    
21
22  Marubatsu.play = play

行番号のないプログラム

import matplotlib.pyplot as plt
import ipywidgets as widgets
import math

def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
    # seed が None でない場合は、seed を乱数の種として設定する
    if seed is not None:
        random.seed(seed)
    
    # ai_list から、項目だけを取り出した list を作成する
    ai_values = [value for label, value in ai_list]
    # それぞれの手番の担当を表す Dropdown の項目の値を記録する list を初期化する
    select_values = []
    # ai に代入されている内容を ai_list に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_values:
            # 項目を登録する
            ai_list.append((label, value))
            ai_values.append(value)
            select_values.append(value)
    
    # 〇 と × の Dropdown を作成する   
    dropdown_circle = widgets.Dropdown(
        options=ai_list,
        description="〇",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[0],
    )
    dropdown_cross = widgets.Dropdown(
        options=ai_list,
        description="×",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[1],
    )       
    
    # リセットボタンを作成する
    button = widgets.Button(
        description="リセット",
        layout=widgets.Layout(width="100px"),
    )
    
    # 〇 と × の dropdown と リセットボタンを横に配置した HBox を作成し、表示する
    hbox = widgets.HBox([dropdown_circle, dropdown_cross, button])
    display(hbox)
    
    # リセットボタンのイベントハンドラを定義する
    def on_button_clicked(b):
        self.restart()
        self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)           
        
    # イベントハンドラをリセットボタンに結びつける
    button.on_click(on_button_clicked)

    # gui が True の場合に、ゲーム盤を描画する画像を作成し、イベントハンドラに結びつける
    if gui:
        fig, ax = plt.subplots(figsize=[size, size])
        fig.canvas.toolbar_visible = False
        fig.canvas.header_visible = False
        fig.canvas.footer_visible = False
        fig.canvas.resizable = False          
        
        # ローカル関数としてイベントハンドラを定義する
        def on_mouse_down(event):
            # Axes の上でマウスを押していた場合のみ処理を行う
            if event.inaxes and self.status == Marubatsu.PLAYING:
                x = math.floor(event.xdata)
                y = math.floor(event.ydata)
                self.move(x, y)                
                self.draw_board(ax)

                # 次の手番の処理を行うメソッドを呼び出す
                self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)
                
        # fig の画像にマウスを押した際のイベントハンドラを結び付ける
        fig.canvas.mpl_connect("button_press_event", on_mouse_down)     

    self.restart()
    return self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)

Marubatsu.play = play

修正箇所

import matplotlib.pyplot as plt
import ipywidgets as widgets
import math

def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
元と同じなので省略
    # ai に代入されている内容を ai_list に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
+       if ai[i] is None:
+           label = "人間"
+           value = "人間"
+       else:
+           label = ai[i].__name__        
+           value = ai[i]
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_values:
-           if ai[i] is None:
-               label = "人間"
-               value = "人間"
-           else:
-               label = ai[i].__name__        
-               value = ai[i]
            # 項目を登録する
            ai_list.append((label, value))
            ai_values.append(value)
            select_values.append(value)
元と同じなので省略    

Marubatsu.play = play

上記の 修正後 に、下記のプログラムで play メソッドを実行すると、実行結果 のような エラーが発生 します。エラーが発生 する原因について少し 考えてみて下さい。

mb.play(ai=[None, None], gui=True)

実行結果

略
     38 dropdown_cross = widgets.Dropdown(
     39     options=ai_list,
     40     description="×",
     41     layout=widgets.Layout(width="100px"),
     42     style={"description_width": "20px"},
---> 43     value=select_values[1],
     44 )       
略
IndexError: list index out of range

エラーの検証

エラーメッセージ から、---> 43 value=select_values[1], の行で、list の インデックス に 範囲外（out of range）の 値が設定 されたことが原因であることが わかります。

先程と同様に、人間 VS 人間の場合で行われる処理で、関連する 変数の値 が どのように変化 するかを下記の表で 示します。

	条件式	`i`	`ai[i]`	`value`	`ai_values`	`select_values`
1 行目		`0`	`None`		`[]`	`[]`
2 行目	`False`	`0`	`None`		`[]`	`[]`
4 行目		`0`	`None`	`"人間"`	`[]`	`[]`
8 行目	`False`	`0`	`None`	`"人間"`	`[]`	`[]`
10 行目		`0`	`None`	`"人間"`	`["人間"]`	`[]`
11 行目		`0`	`None`	`"人間"`	`["人間"]`	`["人間"]`
1 行目		`1`	`None`		`["人間"]`	`["人間"]`
2 行目	`False`	`1`	`None`		`["人間"]`	`["人間"]`
4 行目		`1`	`None`	`"人間"`	`["人間"]`	`["人間"]`
8 行目	`True`	`1`	`None`	`"人間"`	`["人間"]`	`["人間"]`

プログラムを 確認できる ように、コメントを除いた プログラムを下記に再掲します。

 1  for i in range(2):
 2      if ai[i] is None:
 3          label = "人間"
 4          value = "人間"
 5      else:
 6          label = ai[i].__name__        
 7          value = ai[i]
 8      if value not in ai_values:
 9          ai_list.append((label, value))
10          ai_values.append(value)
11          select_values.append(value)

2 回目 の 繰り返し の 8 行目 の 条件式 が ai[i] not in ai_values から、value not in ai_values に 変わった ため True になる ので、上記の表 のように、select_values には、["人間"] という、要素が 1 つ しかない list が代入されます。そのため、select_values[1] を参照しようとすると、IndexError: list index out of range という エラーが発生 します。

select_values は、〇と × の 両方の手番 の 項目の値 を 持つ必要 が あります。上記のプログラムの 問題点 は、11 行目 の select_values に value を 追加する処理 が、8 行目 の if 文 によって value が ai_values に 登録されていない場合 でしか 行われない点 です。従って、下記のプログラムように、select_values に value を追加する処理を 8 行目 の 前に移動 することで、この 問題を解決 することが できます。

12 行目：14 行目 の if 文 の中に記述されていた処理を if 文 の 前に移動 する

 1  def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
元と同じなので省略
 2      # ai に代入されている内容を ai_list に追加する
 3      for i in range(2):
 4          # ラベルと項目の値を計算する
 5          if ai[i] is None:
 6              label = "人間"
 7              value = "人間"
 8          else:
 9              label = ai[i].__name__        
10              value = ai[i]
11          # value を select_values に常に登録する
12          select_values.append(value)
13          # value が ai_values に登録済かどうかを判定する
14          if value not in ai_values:
15              # 項目を登録する
16              ai_list.append((label, value))
17              ai_values.append(value)
元と同じなので省略
18
19  Marubatsu.play = play

行番号のないプログラム

def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
    # seed が None でない場合は、seed を乱数の種として設定する
    if seed is not None:
        random.seed(seed)
    
    # ai_list から、項目だけを取り出した list を作成する
    ai_values = [value for label, value in ai_list]
    # それぞれの手番の担当を表す Dropdown の項目の値を記録する list を初期化する
    select_values = []
    # ai に代入されている内容を ai_list に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
        # value を select_values に常に登録する
        select_values.append(value)
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_values:
            # 項目を登録する
            ai_list.append((label, value))
            ai_values.append(value)
    
    # 〇 と × の Dropdown を作成する   
    dropdown_circle = widgets.Dropdown(
        options=ai_list,
        description="〇",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[0],
    )
    dropdown_cross = widgets.Dropdown(
        options=ai_list,
        description="×",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[1],
    )       
    
    # リセットボタンを作成する
    button = widgets.Button(
        description="リセット",
        layout=widgets.Layout(width="100px"),
    )
    
    # 〇 と × の dropdown と リセットボタンを横に配置した HBox を作成し、表示する
    hbox = widgets.HBox([dropdown_circle, dropdown_cross, button])
    display(hbox)
    
    # リセットボタンのイベントハンドラを定義する
    def on_button_clicked(b):
        self.restart()
        self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)           
        
    # イベントハンドラをリセットボタンに結びつける
    button.on_click(on_button_clicked)

    # gui が True の場合に、ゲーム盤を描画する画像を作成し、イベントハンドラに結びつける
    if gui:
        fig, ax = plt.subplots(figsize=[size, size])
        fig.canvas.toolbar_visible = False
        fig.canvas.header_visible = False
        fig.canvas.footer_visible = False
        fig.canvas.resizable = False          
        
        # ローカル関数としてイベントハンドラを定義する
        def on_mouse_down(event):
            # Axes の上でマウスを押していた場合のみ処理を行う
            if event.inaxes and self.status == Marubatsu.PLAYING:
                x = math.floor(event.xdata)
                y = math.floor(event.ydata)
                self.move(x, y)                
                self.draw_board(ax)

                # 次の手番の処理を行うメソッドを呼び出す
                self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)
                
        # fig の画像にマウスを押した際のイベントハンドラを結び付ける
        fig.canvas.mpl_connect("button_press_event", on_mouse_down)     

    self.restart()
    return self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)

Marubatsu.play = play

修正箇所

def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
元と同じなので省略
    # ai に代入されている内容を ai_list に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
        # value を select_values に常に登録する
+       select_values.append(value)
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_values:
            # 項目を登録する
            ai_list.append((label, value))
            ai_values.append(value)
-　　       select_values.append(value)
元と同じなので省略

Marubatsu.play = play

上記の 修正後 に、下記のプログラムで play メソッドを実行すると、実行結果 のように 正しい処理 が 行われる ことが確認できます。

mb.play(ai=[None, None], gui=True)

実行結果（下図は、画像なので操作することはできません）

バグその 2（前に表示された項目の内容が残る）

play メソッドには もう一つバグ がありますが、そのバグは 初心者 には 気づきづらい バグだと思います。具体的 には、上記で人間 VS 人間の対戦を 行った後 で、下記のプログラムで AI VS AI の対戦を行うと、実行結果 の 左下図 のように、一見すると それぞれ の Dropdown に 正しい AI が選択されているので、うまくいっている ように 見えるかもしれません が、Dropdown の上 で マウスを押す と、右下図 のように、今回は対戦を 行っていない、人間の項目 が表示されるという 問題が発生 します。

from ai import ai1s, ai2s

mb.play(ai=[ai1s, ai2s], gui=True);

実行結果（下図は、画像なので操作することはできません）

バグの原因の検証

この バグの原因 を Python の 初心者が気づく のは 難しい かもしれません。実は筆者も上記のプログラムが 正しい と最初は 勘違い していました。この バグの原因 は、下記のプログラムの 2 行目 のように、play メソッドの最初に、仮引数 ai_list の 値を表示 することで確認することができます。

def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
    print("ai_list =", ai_list)
元と同じなので省略

Marubatsu.play = play

行番号のないプログラム

def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
    print("ai_list =", ai_list)
    # seed が None でない場合は、seed を乱数の種として設定する
    if seed is not None:
        random.seed(seed)
    
    # ai_list から、項目だけを取り出した list を作成する
    ai_values = [value for label, value in ai_list]
    # それぞれの手番の担当を表す Dropdown の項目の値を記録する list を初期化する
    select_values = []
    # ai に代入されている内容を ai_list に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
        # value を select_values に常に登録する
        select_values.append(value)
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_values:
            # 項目を登録する
            ai_list.append((label, value))
            ai_values.append(value)
    
    # 〇 と × の Dropdown を作成する   
    dropdown_circle = widgets.Dropdown(
        options=ai_list,
        description="〇",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[0],
    )
    dropdown_cross = widgets.Dropdown(
        options=ai_list,
        description="×",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[1],
    )       
    
    # リセットボタンを作成する
    button = widgets.Button(
        description="リセット",
        layout=widgets.Layout(width="100px"),
    )
    
    # 〇 と × の dropdown と リセットボタンを横に配置した HBox を作成し、表示する
    hbox = widgets.HBox([dropdown_circle, dropdown_cross, button])
    display(hbox)
    
    # リセットボタンのイベントハンドラを定義する
    def on_button_clicked(b):
        self.restart()
        self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)           
        
    # イベントハンドラをリセットボタンに結びつける
    button.on_click(on_button_clicked)

    # gui が True の場合に、ゲーム盤を描画する画像を作成し、イベントハンドラに結びつける
    if gui:
        fig, ax = plt.subplots(figsize=[size, size])
        fig.canvas.toolbar_visible = False
        fig.canvas.header_visible = False
        fig.canvas.footer_visible = False
        fig.canvas.resizable = False          
        
        # ローカル関数としてイベントハンドラを定義する
        def on_mouse_down(event):
            # Axes の上でマウスを押していた場合のみ処理を行う
            if event.inaxes and self.status == Marubatsu.PLAYING:
                x = math.floor(event.xdata)
                y = math.floor(event.ydata)
                self.move(x, y)                
                self.draw_board(ax)

                # 次の手番の処理を行うメソッドを呼び出す
                self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)
                
        # fig の画像にマウスを押した際のイベントハンドラを結び付ける
        fig.canvas.mpl_connect("button_press_event", on_mouse_down)     

    self.restart()
    return self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)

Marubatsu.play = play

修正箇所

def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
+   print("ai_list =", ai_list)
元と同じなので省略

Marubatsu.play = play

上記の修正 を 行った後 で、先程と 同じ順番 で、人間 VS 人間と AI VS AI の対戦を行います。下記は人間 VS 人間の対戦を 行った場合 です。なお、実行結果 にはこの バグの検証 に必要な print(ai_list) の 表示のみ を表記します。

mb.play(ai=[None, None], gui=True);

実行結果

ai_list = []

上記のプログラムは、仮引数 ai_list に対応する 実引数 を 記述していない ので、ai_list には デフォルト値 である [] が代入され、実行結果 に表示されます。

続けて 下記のプログラムで、AI VS AI の対戦を行います。

mb.play(ai=[ai1s, ai2s], gui=True);

実行結果

ai_list = [('人間', '人間')]

実行結果 から、先程と同様 に 仮引数 ai_list に 対応する実引数 を 記述していない にも かかわらず、ai_list に [('人間', '人間')] が 代入されている ことが確認できます。これが、Dropdown に 人間の項目 が表示されてしまう原因です。

この現象 の原因については、本記事で デフォルト引数 の説明を 最初に行った 以前の記事で説明しましたが、〇×ゲーム を 実装する際 に そのような例 がこれまで 出てこなかったの で、ほとんどの方は忘れてしまったのではないかと思いますので、もう一度説明します。

デフォルト引数 の デフォルト値 を表す オブジェクト は、関数の定義 が実行された際に 1 回だけ作成 され、デフォルト引数 に対応する 実引数 が 記述されなかった 場合は、そのオブジェクト が デフォルト引数 に代入されます。関数呼び出し が 行われた際 に、毎回デフォルト値を表す オブジェクト が 新しく作成 される わけではない 点に注意して下さい。

従って、デフォルト値 に list など の ミュータブル な オブジェクト を設定し、関数の ブロックの中 で、デフォルト値 の要素の 値を変更 したり、要素を追加 するなどの処理を行うと、デフォルト値 の内容が変化します。

実際に play メソッドでは、ai_list.append((label, value)) という、ai_list に デフォルト値 が代入されていた場合に、その 要素を追加 するという処理を 行っています。そのため、ai_list に対応する 実引数 を 記述せず に人間 VS 人間の 対戦を行う と、デフォルト値 が 空の list から、[("人間", "人間")] に変化してしまいます。そして、その値 が、次に AI VS AI の対戦を play メソッドで 行った際 に下記のように表示されます。

ai_list = [('人間', '人間')]

この問題 が 発生しない ようにするためには、下記のプログラムのように、list など の ミュータブルな値 を デフォルト値 に したい場合 は、デフォルト値 に None を設定 し、関数の ブロックの中 で、デフォルト引数の値 が None の場合に、デフォルト引数 に実際に 設定したい値 を代入するという処理を 記述する必要 が あります。このように記述 することで、デフォルト引数 に対応する 実引数 を 記述しなかった 場合に、デフォルト引数 に 新しい値 が 毎回代入 されるので上記のような問題は 発生しなくなります。

1 行目：ai_list の デフォルト値 を None に修正する
3、4 行目：ai_list が None の場合 に、ai_list に 空の list を代入する

 1  def play(self, ai, ai_list=None, params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
 2      # ai_list が None の場合は、空の list で置き換える
 3      if ai_list is None:
 4          ai_list = []
元と同じなので省略
 5
 6  Marubatsu.play = play

行番号のないプログラム

def play(self, ai, ai_list=None, params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
    # ai_list が None の場合は、空の list で置き換える
    if ai_list is None:
        ai_list = []
   
    # seed が None でない場合は、seed を乱数の種として設定する
    if seed is not None:
        random.seed(seed)
    
    # ai_list から、項目だけを取り出した list を作成する
    ai_values = [value for label, value in ai_list]
    # それぞれの手番の担当を表す Dropdown の項目の値を記録する list を初期化する
    select_values = []
    # ai に代入されている内容を ai_list に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
        # value を select_values に常に登録する
        select_values.append(value)
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_values:
            # 項目を登録する
            ai_list.append((label, value))
            ai_values.append(value)
    
    # 〇 と × の Dropdown を作成する   
    dropdown_circle = widgets.Dropdown(
        options=ai_list,
        description="〇",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[0],
    )
    dropdown_cross = widgets.Dropdown(
        options=ai_list,
        description="×",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[1],
    )       
    
    # リセットボタンを作成する
    button = widgets.Button(
        description="リセット",
        layout=widgets.Layout(width="100px"),
    )
    
    # 〇 と × の dropdown と リセットボタンを横に配置した HBox を作成し、表示する
    hbox = widgets.HBox([dropdown_circle, dropdown_cross, button])
    display(hbox)
    
    # リセットボタンのイベントハンドラを定義する
    def on_button_clicked(b):
        self.restart()
        self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)           
        
    # イベントハンドラをリセットボタンに結びつける
    button.on_click(on_button_clicked)

    # gui が True の場合に、ゲーム盤を描画する画像を作成し、イベントハンドラに結びつける
    if gui:
        fig, ax = plt.subplots(figsize=[size, size])
        fig.canvas.toolbar_visible = False
        fig.canvas.header_visible = False
        fig.canvas.footer_visible = False
        fig.canvas.resizable = False          
        
        # ローカル関数としてイベントハンドラを定義する
        def on_mouse_down(event):
            # Axes の上でマウスを押していた場合のみ処理を行う
            if event.inaxes and self.status == Marubatsu.PLAYING:
                x = math.floor(event.xdata)
                y = math.floor(event.ydata)
                self.move(x, y)                
                self.draw_board(ax)

                # 次の手番の処理を行うメソッドを呼び出す
                self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)
                
        # fig の画像にマウスを押した際のイベントハンドラを結び付ける
        fig.canvas.mpl_connect("button_press_event", on_mouse_down)     

    self.restart()
    return self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)

Marubatsu.play = play

修正箇所

-def play(self, ai, ai_list=[], params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
+def play(self, ai, ai_list=None, params=[{}, {}], verbose=True, seed=None, gui=False, size=3):      
    # ai_list が None の場合は、空の list で置き換える
+   if ai_list is None:
+       ai_list = []
元と同じなので省略

Marubatsu.play = play

上記の 修正後 に、下記のプログラムで play メソッドを実行すると、実行結果 のように 正しい処理 が 行われる ことが確認できます。人間 VS 人間の 実行結果 は省略します。

なお、下記のプログラムを 同じセルで実行 すると、2 つ の対戦の Dropdown の表示 が ゲーム盤よりも前 に 並べて表示 されるので、別々のセル で実行して下さい。

mb.play(ai=[None, None], gui=True);

mb.play(ai=[ai1s, ai2s], gui=True);

実行結果（下図は、画像なので操作することはできません）

デフォルト引数 `params` の修正

上記の問題 は、下記のような処理を 行う際 に発生します。

デフォルト値 に list など の ミュータブル な データ を代入する
関数の ブロック内 で、デフォルト引数 の要素を 変更する処理 を行う

逆に言えば、上記のような処理を 行わない 場合は、上記の問題 は 発生しません。例えば、play メソッドには、params=[{}, {}] のように、デフォルト値 に list のような ミュータブル な値が設定された デフォルト引数 が存在しますが、params は、関数の ブロックの中 で、その要素を変更することはないので、このまま でも問題は 発生しません。

ただし、今後 params の要素を変更するような処理を 行う可能性 がないとは 言い切れません し、要素を 変更することがない と 勘違い している場合が あるかもしれない ので、ミュータブル な データ を デフォルト値 に 設定したい 場合は、特別な理由 が ない限り、常にデフォルト値 に None を設定し、None の場合 に 設定したい値 を 代入する という処理を 記述したほうが良い でしょう。

下記は、そのよう に play メソッドを修正した プログラム です。

1 行目：params の デフォルト値 を None に修正する
6、7 行目：params が None の場合に 元のデフォルト値 を代入する

1  def play(self, ai, ai_list=None, params=None, verbose=True, seed=None, gui=False, size=3):      
2      # ai_list が None の場合は、空の list で置き換える
3      if ai_list is None:
4          ai_list = []
5      # params が None の場合のデフォルト値を設定する
6      if params is None:
7          params = [{}, {}]
元と同じなので省略
8
9  Marubatsu.play = play

行番号のないプログラム

def play(self, ai, ai_list=None, params=None, verbose=True, seed=None, gui=False, size=3):      
    # ai_list が None の場合は、空の list で置き換える
    if ai_list is None:
        ai_list = []
    # params が None の場合のデフォルト値を設定する
    if params is None:
        params = [{}, {}]
   
    # seed が None でない場合は、seed を乱数の種として設定する
    if seed is not None:
        random.seed(seed)
    
    # ai_list から、項目だけを取り出した list を作成する
    ai_values = [value for label, value in ai_list]
    # それぞれの手番の担当を表す Dropdown の項目の値を記録する list を初期化する
    select_values = []
    # ai に代入されている内容を ai_list に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
        # value を select_values に常に登録する
        select_values.append(value)
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_values:
            # 項目を登録する
            ai_list.append((label, value))
            ai_values.append(value)
    
    # 〇 と × の Dropdown を作成する   
    dropdown_circle = widgets.Dropdown(
        options=ai_list,
        description="〇",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[0],
    )
    dropdown_cross = widgets.Dropdown(
        options=ai_list,
        description="×",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[1],
    )       
    
    # リセットボタンを作成する
    button = widgets.Button(
        description="リセット",
        layout=widgets.Layout(width="100px"),
    )
    
    # 〇 と × の dropdown と リセットボタンを横に配置した HBox を作成し、表示する
    hbox = widgets.HBox([dropdown_circle, dropdown_cross, button])
    display(hbox)
    
    # リセットボタンのイベントハンドラを定義する
    def on_button_clicked(b):
        self.restart()
        self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)           
        
    # イベントハンドラをリセットボタンに結びつける
    button.on_click(on_button_clicked)

    # gui が True の場合に、ゲーム盤を描画する画像を作成し、イベントハンドラに結びつける
    if gui:
        fig, ax = plt.subplots(figsize=[size, size])
        fig.canvas.toolbar_visible = False
        fig.canvas.header_visible = False
        fig.canvas.footer_visible = False
        fig.canvas.resizable = False          
        
        # ローカル関数としてイベントハンドラを定義する
        def on_mouse_down(event):
            # Axes の上でマウスを押していた場合のみ処理を行う
            if event.inaxes and self.status == Marubatsu.PLAYING:
                x = math.floor(event.xdata)
                y = math.floor(event.ydata)
                self.move(x, y)                
                self.draw_board(ax)

                # 次の手番の処理を行うメソッドを呼び出す
                self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)
                
        # fig の画像にマウスを押した際のイベントハンドラを結び付ける
        fig.canvas.mpl_connect("button_press_event", on_mouse_down)     

    self.restart()
    return self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)

Marubatsu.play = play

修正箇所

def play(self, ai, ai_list=None, params=None, verbose=True, seed=None, gui=False, size=3):      
    # ai_list が None の場合は、空の list で置き換える
    if ai_list is None:
        ai_list = []
    # params が None の場合のデフォルト値を設定する
    if params is None:
        params = [{}, {}]
元と同じなので省略

Marubatsu.play = play

実行結果 は省略しますが、上記の 修正後 に、下記のプログラムを実行することで、正しい処理 が 行われる ことが確認できます。

mb.play(ai=[None, None], gui=True)

上記の記事を記述している際に気づいたのですが、これまでは play_loop の いくつか の 仮引数 に デフォルト値 を設定していました。play_loop は、play メソッドの中から、必ず すべての仮引数 に対応する 実引数を記述 して 呼び出している ので、デフォルト値 を設定する意味が ありません。そこで、下記プログラムのように、play_loop の デフォルト引数 を、通常の 仮引数 に修正することにします。

def play_loop(self, ai, ax, params, verbose, gui):
元と同じなので省略

Marubatsu.play_loop = play_loop

プログラム全体

def play_loop(self, ai, ax, params, verbose, gui):
    # ゲームの決着がついていない間繰り返す
    while self.status == Marubatsu.PLAYING:
        # 現在の手番を表す ai のインデックスを計算する
        index = 0 if self.turn == Marubatsu.CIRCLE else 1
        # ゲーム盤の表示
        if verbose:
            if gui:
                # AI どうしの対戦の場合は画面を描画しない
                if ai[0] is None or ai[1] is None:
                    self.draw_board(ax)
                # 手番を人間が担当する場合は、play メソッドを終了する
                if ai[index] is None:
                    return
            else:
                print(self)
                
        # ai が着手を行うかどうかを判定する
        if ai[index] is not None:
            x, y = ai[index](self, **params[index])
        else:
            # キーボードからの座標の入力
            coord = input("x,y の形式で座標を入力して下さい。exit を入力すると終了します")
            # "exit" が入力されていればメッセージを表示して関数を終了する
            if coord == "exit":
                print("ゲームを終了します")
                return       
            # x 座標と y 座標を要素として持つ list を計算する
            xylist = coord.split(",")
            # xylist の要素の数が 2 ではない場合
            if len(xylist) != 2:
                # エラーメッセージを表示する
                print("x, y の形式ではありません")
                # 残りの while 文のブロックを実行せずに、次の繰り返し処理を行う
                continue
            x, y = xylist
        # (x, y) に着手を行う
        try:
            self.move(int(x), int(y))
        except:
            print("整数の座標を入力して下さい")

    # 決着がついたので、ゲーム盤を表示する
    if verbose:
        if gui:
            self.draw_board(ax)
        else:
            print(self)
            
    return self.status   

Marubatsu.play_loop = play_loop

修正箇所

-def play_loop(self, ai, ax, params=[{}, {}], verbose=True, gui=False):
+def play_loop(self, ai, ax, params, verbose, gui):
元と同じなので省略

Marubatsu.play_loop = play_loop

実行結果 は省略しますが、上記の 修正後 に、下記のプログラムを実行することで、正しい処理 が 行われる ことが確認できます。

mb.play(ai=[None, None], gui=True)

細かい話 になりますが、上記の「関数の ブロック内 で、デフォルト引数 の内容を 変更する処理 を行う」は、その関数の中 で デフォルト値 の内容を 変更しなくても、デフォルト値 を 他の関数呼び出し の 実引数に記述 し、呼び出された関数の中 で デフォルト値の内容 を 変更する 場合も 含みます。

例えば、下記のプログラムは、a から 呼び出した 関数 b の中 で デフォルト値 に 要素を追加 するので、a を 呼び出すたび に、x の デフォルト値 は変化します。

def a(x=[]):
    print(x)
    b(x)

def b(x):
    x.append(1)

a()
a()

実行結果

[]
[1]

GUI でゲームを遊ぶ関数の定義

前回の記事で、循環インポート の エラーを解決 するために、ai1s ~ ai14s を Dropdown に 登録する処理 を play メソッドから削除したため、それらの AI を Dropdown で 選択できる ようにするためには、下記のプログラムのように、ai1s ~ ai14s の データが登録 された ai_list を 作成する処理 を 記述する必要 が あります。

import ai

# Dropdown の作成の際に必要となる、AI のリストを作成する    
ai_list = []
for i in range(1, 15):
    ai_name = f"ai{i}s"  
    ai_list.append((ai_name, getattr(ai, ai_name)))   

mb.play(ai=[None, None], ai_list=ai_list, gui=True)

実行結果（下図は、画像なので操作することはできません）

毎回このような 処理を記述 するのは大変なので、GUI で ai1s ~ ai14s の 14 種類 の AI を選択する Dropdown を表示して 〇×ゲームを遊ぶ ための下記のような 関数を定義 することにします。

名前：GUI で遊ぶ（play）ので、gui_play とする
処理：GUI で ai1s ~ ai14s の AI を選択する Dropdown を表示して 〇×ゲームを遊ぶ
入力：最初の担当 と、Dropdown に 表示する項目 を自由に 設定できる ように、play メソッドと 同じ意味 を持つ 仮引数 ai と ai_list を持つようにする
出力：なし

`gui_play` の定義

下記は、gui_play の定義です。なお、gui_play では、ai に対応する 実引数 を 記述しなかった場合 に 人間どうし の対戦が行われるように工夫してみました。

1 行目：gui_play には ai という 仮引数 が 存在する ので、ai モジュール を再び ai_module という 別の名前 で インポート する 必要がある
3 行目：デフォルト値 に None を設定した デフォルト引数 ai と ai_list を持つ gui_play を定義する
5、6 行目：ai に None が代入 されていた場合は、ai に [None, None] を代入することで、人間どうし の対戦が行われるようにする
8 ~ 12 行目：ai_list に None が代入 されていた場合は、ai_list に ai1s ~ ai14s の項目を持つ Dropdown を 作成するため の データ を計算して代入する
14、15 行目：Marubatsu クラスの インスタンスを作成 し、play メソッドで GUI で 上記の設定 で 〇×ゲームの対戦 を行う

 1  import ai as ai_module
 2
 3  def gui_play(ai=None, ai_list=None):
 4      # ai が None の場合は、人間どうしの対戦を行う
 5      if ai is None:
 6          ai = [None, None]
 7      # ai_list が None の場合は、ai1s ~ ai14s の Dropdown を作成するためのデータを計算する
 8      if ai_list is None:
 9          ai_list = []
10          for i in range(1, 15):
11              ai_name = f"ai{i}s"  
12              ai_list.append((ai_name, getattr(ai_module, ai_name)))    
13
14      mb = Marubatsu()
15      mb.play(ai=ai, ai_list=ai_list, gui=True)

行番号のないプログラム

import ai as ai_module

def gui_play(ai=None, ai_list=None):
    # ai が None の場合は、人間どうしの対戦を行う
    if ai is None:
        ai = [None, None]
    # ai_list が None の場合は、ai1s ~ ai14s の Dropdown を作成するためのデータを計算する
    if ai_list is None:
        ai_list = []
        for i in range(1, 15):
            ai_name = f"ai{i}s"  
            ai_list.append((ai_name, getattr(ai_module, ai_name)))    
    
    mb = Marubatsu()
    mb.play(ai=ai, ai_list=ai_list, gui=True)

`gui_play` の動作の確認

下記のプログラムで、gui_play に 実引数 を 記述せず に呼び出した場合は、GUI で人間 VS 人間の対戦が 行われる ことが確認できます。

gui_play()

実行結果（下図は、画像なので操作することはできません）

また、下記のプログラムで、人間 VS ai1s の対戦が 行われる ことが確認できます。

gui_play(ai=[None, ai1s])

実行結果（下図は、画像なので操作することはできません）

また、下記のプログラムで、ai1 のみ が登録された ai_list のデータを作成して キーワード引数 に記述した場合でも gui_play が 正しく動作 することが確認できます。

gui_play(ai_list=[("ai1s", ai.ai1s)])

実行結果（下図は、画像なので操作することはできません）

実は、gui_play には問題が あります。それが何か少し考えてみて下さい。

人間の項目の登録

gui_play には、下記のプログラムのように、AI どうし の対戦を行う場合、実行結果 の 右下図 のように、Dropdown に 人間の項目 が 表示されない という問題が あります。

gui_play(ai=[ai1s, ai1s])

実行結果（下図は、画像なので操作することはできません）

先程、gui_play() を実行した際に、下図のように 人間の項目 が 最後に表示 されたのは、人間 VS 人間の対戦が行われたため、play メソッドの中で、ai_list に 登録されていない 人間の項目が 自動的 に 登録された からです。

この問題 は、下記のプログラムの 7 行目 のように、gui_play の中の、ai_list の 初期化処理 で、人間の項目 の データ を代入することで解決できます。

 1  def gui_play(ai=None, ai_list=None):
 2      # ai が None の場合は、人間どうしの対戦を行う
 3      if ai is None:
 4          ai = [None, None]
 5      # ai_list が None の場合は、人間と ai1s ~ ai14s の Dropdown を作成するためのデータを計算する  
 6      if ai_list is None:
 7          ai_list = [("人間", "人間")]
 8          for i in range(1, 15):
 9              ai_name = f"ai{i}s"  
10              ai_list.append((ai_name, getattr(ai_module, ai_name)))    
11    
12      mb = Marubatsu()
13      mb.play(ai=ai, ai_list=ai_list, gui=True)

行番号のないプログラム

def gui_play(ai=None, ai_list=None):
    # ai が None の場合は、人間どうしの対戦を行う
    if ai is None:
        ai = [None, None]
    # ai_list が None の場合は、人間と ai1s ~ ai14s の Dropdown を作成するためのデータを計算する  
    if ai_list is None:
        ai_list = [("人間", "人間")]
        for i in range(1, 15):
            ai_name = f"ai{i}s"  
            ai_list.append((ai_name, getattr(ai_module, ai_name)))    
    
    mb = Marubatsu()
    mb.play(ai=ai, ai_list=ai_list, gui=True)

修正箇所

def gui_play(ai=None, ai_list=None):
    # ai が None の場合は、人間どうしの対戦を行う
    if ai is None:
        ai = [None, None]
    # ai_list が None の場合は、人間と ai1s ~ ai14s の Dropdown を作成するためのデータを計算する  
    if ai_list is None:
-       ai_list = []
+       ai_list = [("人間", "人間")]
        for i in range(1, 15):
            ai_name = f"ai{i}s"  
            ai_list.append((ai_name, getattr(ai_module, ai_name)))    
    
    mb = Marubatsu()
    mb.play(ai=ai, ai_list=ai_list, gui=True)

上記の 修正後 に、下記のプログラムで play メソッドを実行すると、実行結果 のように 正しい処理 が 行われる ことが確認できます。なお、人間の項目 を gui_play の中 で 最初に登録 したので、先程と 異なり、人間の項目 が 最初に表示 されるようになります。

gui_play(ai=[ai1s, ai1s])

実行結果（下図は、画像なので操作することはできません）

`ai1s` ~ `ai14s` 以外の AI で対戦を行う場合の確認

下記のプログラムで、gui_play を使って、ai1s ~ ai14s 以外 の AI で 対戦を行う 場合を確認します。実行結果 から、正しい処理 が行われることが確認できます。

from ai import ai1, ai2

gui_play(ai=[ai1, ai2])

実行結果（下図は、画像なので操作することはできません）

`gui_play` の記述場所

gui_play を Marubatsu クラスの メソッド として 定義しなかった 点が 気になってい る人が いるかもしれません が、gui_play の中 で ai モジュール の関数を 利用している ので、gui_play を Marubatsu クラスの メソッド として定義すると、前回の記事で説明したように、インポートの循環 の エラーが発生 してしまいます。

また、gui_play は、AI の 処理を行う 関数 ではない ので、ai モジュール の 中に記述 するのは 少し不自然 です。そこで、本記事では gui_play を util.py という名前の ファイルに保存する ことにします。util.py の中 で marubatsu と ai の 両方のモジュール を インポート しても、インポートの循環 は 起こらない ので エラー は 発生しません。

なお、util とは、小規模 で、補助的 な機能を持つ プログラム を表す ユーティリティ（utility）の略で、比較的 良く使われるプログラミング用語 です。

dict を用いた Dropdown のラベルの設定

以前の記事で、Dropdown の ラベル を 設定する方法 として、options 属性 の値に (ラベル, 項目の値) という tuple を要素として持つ list で設定する方法を紹介しましたが、それ以外 にも { "ラベル": 項目の値 } という dict で設定 することが できます。また、dict で設定 することで、play メソッドの処理を より簡潔に記述 することが できるようになる ことに気が付きましたので、その方法を紹介することにします。

前回の記事では、ai の内容 を Dropdown に 自動登録 する処理を実装しましたが、その際に、ai_list の中 に、ai の要素が 含まれているか を 判定する必要 が あります。

ai_list の要素には、(ラベル, 項目の値) という tuple が代入されているため、in 演算子 を使って その判定 を行うことは できません。そのため、ai_list の中 から 項目の値 を 取り出した list を作成して ai_values に代入するという処理を、下記のプログラムのように リスト内包表記 を用いて行いました。

    # ai_list から、項目だけを取り出した list を作成する
    ai_values = [value for label, value in ai_list]

上記の処理 は、ai_list を、それぞれの項目 を { "ラベル": 項目の値 } というキーと キーの値 を持つ dict で設定 することで、簡潔に記述 することが できます。

具体的には、下記のプログラムのように、dict の キーの値 の一覧を表す list を計算する values という メソッド を利用します。

ai_values = ai_list.values()

ai_list を dict で表現 することで、play メソッドを下記のように修正します。なお、ai_list という 名前のまま では変なので、ai_dict という 名前に変更 しました。

1 行目：仮引数 ai_list を ai_dict に変更する
4 行目：ai_dict が None の場合に 代入する値 を 空の dict に修正する
14 行目：この下にあった ai_values を計算する処理を削除する
18 行目：ai_values の代わりに、ai_dict.values() で 判定を行う ように修正する
20 行目：ai_dict の label の キーの値 に value を代入する
21 行目：この下にあった ai_values に 項目を登録 する処理を削除する
3、25、32 行目：ai_list を ai_dict に変更する

 1  def play(self, ai, ai_dict=None, params=None, verbose=True, seed=None, gui=False, size=3):      
 2      # ai_dict が None の場合は、空の list で置き換える
 3      if ai_dict is None:
 4          ai_dict = {}
元と同じなので省略
 5      # ai に代入されている内容を ai_dict に追加する
 6      for i in range(2):
 7          # ラベルと項目の値を計算する
 8          if ai[i] is None:
 9              label = "人間"
10              value = "人間"
11          else:
12              label = ai[i].__name__        
13              value = ai[i]
14          # この下にあった、ai_values を計算する処理を削除する
15          # value を select_values に常に登録する
16          select_values.append(value)
17          # value が ai_values に登録済かどうかを判定する
18          if value not in ai_dict.values():
19              # 項目を登録する
20              ai_dict[label] = value
21          # この下にあった、ai_values に value を登録する処理を削除する
22
23      # 〇 と × の Dropdown を作成する   
24      dropdown_circle = widgets.Dropdown(
25          options=ai_dict,
26          description="〇",
27          layout=widgets.Layout(width="100px"),
28          style={"description_width": "20px"},
29          value=select_values[0],
30      )
31      dropdown_cross = widgets.Dropdown(
32          options=ai_dict,
33          description="×",
34          layout=widgets.Layout(width="100px"),
35          style={"description_width": "20px"},
36          value=select_values[1],
37      )   
元と同じなので省略
38
39  Marubatsu.play = play

行番号のないプログラム

def play(self, ai, ai_dict=None, params=None, verbose=True, seed=None, gui=False, size=3):      
    # ai_dict が None の場合は、空の list で置き換える
    if ai_dict is None:
        ai_dict = {}
    # params が None の場合のデフォルト値を設定する
    if params is None:
        params = [{}, {}]
   
    # seed が None でない場合は、seed を乱数の種として設定する
    if seed is not None:
        random.seed(seed)
    
    # それぞれの手番の担当を表す Dropdown の項目の値を記録する list を初期化する
    select_values = []
    # ai に代入されている内容を ai_dict に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
        # value を select_values に常に登録する
        select_values.append(value)
        # value が ai_values に登録済かどうかを判定する
        if value not in ai_dict.values():
            # 項目を登録する
            ai_dict[label] = value
    
    # 〇 と × の Dropdown を作成する   
    dropdown_circle = widgets.Dropdown(
        options=ai_dict,
        description="〇",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[0],
    )
    dropdown_cross = widgets.Dropdown(
        options=ai_dict,
        description="×",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[1],
    )       
    
    # リセットボタンを作成する
    button = widgets.Button(
        description="リセット",
        layout=widgets.Layout(width="100px"),
    )
    
    # 〇 と × の dropdown と リセットボタンを横に配置した HBox を作成し、表示する
    hbox = widgets.HBox([dropdown_circle, dropdown_cross, button])
    display(hbox)
    
    # リセットボタンのイベントハンドラを定義する
    def on_button_clicked(b):
        self.restart()
        self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)           
        
    # イベントハンドラをリセットボタンに結びつける
    button.on_click(on_button_clicked)

    # gui が True の場合に、ゲーム盤を描画する画像を作成し、イベントハンドラに結びつける
    if gui:
        fig, ax = plt.subplots(figsize=[size, size])
        fig.canvas.toolbar_visible = False
        fig.canvas.header_visible = False
        fig.canvas.footer_visible = False
        fig.canvas.resizable = False          
        
        # ローカル関数としてイベントハンドラを定義する
        def on_mouse_down(event):
            # Axes の上でマウスを押していた場合のみ処理を行う
            if event.inaxes and self.status == Marubatsu.PLAYING:
                x = math.floor(event.xdata)
                y = math.floor(event.ydata)
                self.move(x, y)                
                self.draw_board(ax)

                # 次の手番の処理を行うメソッドを呼び出す
                self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)
                
        # fig の画像にマウスを押した際のイベントハンドラを結び付ける
        fig.canvas.mpl_connect("button_press_event", on_mouse_down)     

    self.restart()
    return self.play_loop(ai=ai, ax=ax, params=params, verbose=verbose, gui=gui)

Marubatsu.play = play

修正箇所

-def play(self, ai, ai_list=None, params=None, verbose=True, seed=None, gui=False, size=3):      
+def play(self, ai, ai_dict=None, params=None, verbose=True, seed=None, gui=False, size=3):      
    # ai_dict が None の場合は、空の list で置き換える
-   if ai_list is None:
+   if ai_dict is None:
-       ai_list = []
+       ai_dict = {}
元と同じなので省略
    # ai に代入されている内容を ai_dict に追加する
    for i in range(2):
        # ラベルと項目の値を計算する
        if ai[i] is None:
            label = "人間"
            value = "人間"
        else:
            label = ai[i].__name__        
            value = ai[i]
-       ai_values = [value for label, value in ai_list]
        # value を select_values に常に登録する
        select_values.append(value)
        # value が ai_values に登録済かどうかを判定する
-       if value not in ai_values:
+       if value not in ai_dict.values():
            # 項目を登録する
-           ai_list.append((label, value))
+           ai_dict[label] = value
-           ai_values.append(value)

    # 〇 と × の Dropdown を作成する   
    dropdown_circle = widgets.Dropdown(
-       options=ai_list,
+       options=ai_dict,
        description="〇",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[0],
    )
    dropdown_cross = widgets.Dropdown(
-       options=ai_list,
+       options=ai_dict,
        description="×",
        layout=widgets.Layout(width="100px"),
        style={"description_width": "20px"},
        value=select_values[1],
    )     
元と同じなので省略

Marubatsu.play = play

次に、gui_play の中の ai_dict を 作成する処理 を、下記のように修正します。

1 行目：仮引数 ai_list を ai_dict に変更する
7 行目：ai_dict に、人間の項目 を表す dict を代入するように修正する
10 行目：ai_dict の ai_name の キーの値 に、getattr で計算した AI の関数 を代入する
6、13 行目：ai_list を ai_dict に変更する

 1  def gui_play(ai=None, ai_dict=None):
 2      # ai が None の場合は、人間どうしの対戦を行う
 3      if ai is None:
 4          ai = [None, None]
 5      # ai_dict が None の場合は、ai1s ~ ai14s の Dropdown を作成するためのデータを計算する
 6      if ai_dict is None:
 7          ai_dict = { "人間": "人間"}
 8          for i in range(1, 15):
 9              ai_name = f"ai{i}s"  
10              ai_dict[ai_name] = getattr(ai_module, ai_name)
11    
12      mb = Marubatsu()
13      mb.play(ai=ai, ai_dict=ai_dict, gui=True)

行番号のないプログラム

def gui_play(ai=None, ai_dict=None):
    # ai が None の場合は、人間どうしの対戦を行う
    if ai is None:
        ai = [None, None]
    # ai_dict が None の場合は、ai1s ~ ai14s の Dropdown を作成するためのデータを計算する
    if ai_dict is None:
        ai_dict = { "人間": "人間"}
        for i in range(1, 15):
            ai_name = f"ai{i}s"  
            ai_dict[ai_name] = getattr(ai_module, ai_name)
    
    mb = Marubatsu()
    mb.play(ai=ai, ai_dict=ai_dict, gui=True)

修正箇所

-def gui_play(ai=None, ai_list=None):
+def gui_play(ai=None, ai_dict=None):
    # ai が None の場合は、人間どうしの対戦を行う
    if ai is None:
        ai = [None, None]
    # ai_dict が None の場合は、ai1s ~ ai14s の Dropdown を作成するためのデータを計算する
-   if ai_list is None:
+   if ai_dict is None:
-       ai_list = [("人間", "人間")]
+       ai_dict = { "人間": "人間"}
        for i in range(1, 15):
            ai_name = f"ai{i}s"  
-           ai_list.append((ai_name, getattr(ai_module, ai_name)))               
+           ai_dict[ai_name] = getattr(ai_module, ai_name)
    
    mb = Marubatsu()
-   mb.play(ai=ai, ai_list=ai_list, gui=True)
+   mb.play(ai=ai, ai_dict=ai_dict, gui=True)

上記の 修正後 に、下記のプログラムで、gui_play に 実引数 を 記述せず に呼び出した場合は、GUI で人間 VS 人間の対戦が 正しく行われる ことが確認できます。

gui_play()

実行結果（下図は、画像なので操作することはできません）

本記事では省略しますが、余裕がある方 は先程行った gui_play(ai=[ai1s, ai1s]) など を実行して gui_play が 正しく動作する ことを確認してみて下さい。

今回の記事のまとめ

今回の記事では、play メソッドの バグの修正 を行いました。また、その際に、デフォルト引数 の デフォルト値 に list など の ミュータブル な データ を代入した場合の 注意点 と 対処法 について説明しました。

本記事で入力したプログラム

以下のリンクから、本記事で入力して実行した JupyterLab のファイルを見ることができます。

以下のリンクは、今回の記事で更新した marubatsu.py です。

以下のリンクは、今回の記事で作成した util.py です。

次回の記事