python tkinter クリックする度に重くなるのを回避するには？

Question

将棋盤を模したボードを再現し、クリックした部分の色が変わるという動作をさせています。

別の場所をクリックすると、以前にクリックした場所の色が元に戻るようにしているのですが、
「上書き描写」という方法でその動作を再現しているので、タイトルにもある通りクリックする度に重くなっているのだと思います。

重くなることを回避できる、良い記述方法はありますでしょうか？

import tkinter as tk
root = tk.Tk()
root.geometry("960x480")
root.title("ボード")

BOADTYPE = 9

size = 40
base_x = 0
base_y = 1
BaseColor          = "#e9967a"
ClickedColore      = "#a52a2a"

def OneSquareDisplay(position, _bg):
    label = tk.Label(root, bg=_bg)
    label.userOwnValue = position
    label.place(x=size*(BOADTYPE-position//BOADTYPE+base_x), y=size*(position%BOADTYPE+base_y), height=size-1, width=size-1)
    label.bind("<1>",Callback)


def BoadGeneration():
    for i in range(BOADTYPE**2):
        OneSquareDisplay(i, BaseColor)


def Callback(event):
    click = event.widget.userOwnValue
    click = int(str(click))
    print(click+1)
    BoadGeneration()
    OneSquareDisplay(click, ClickedColore)


BoadGeneration()
tk.mainloop()

Answer 1

現状のコードではクリックする毎に BoadGeneration() が呼ばれるので、81個のLabelsが生成(＆上書き)されております。

つまり１度クリックされると全部で162個のラベルが、２度クリックされると243個のラベルが、ボード上に配置されていることになりますのでクリック毎にどんどん『重く』なっていくのかと思います。

ということで、クリックする毎にLabelを新たに生成するのではなく、単にクリックされたLabelの背景色を変更するだけの処理を行う、具体的に書くと Callback()の処理を

def Callback(event):
    click = event.widget.userOwnValue
    click = int(str(click))
    print(click+1)
    event.widget.config(bg=ClickedColore)
    # BoadGeneration()
    # OneSquareDisplay(click, ClickedColore)

のように修正すると良いのではないでしょうか

---

【追記】

背景色をトグル動作したい場合は、こんな感じで現在の背景色によって処理を変えればよいかと思います。

def Callback(event):
    click = event.widget.userOwnValue
    click = int(str(click))
    print(click+1)
    if event.widget.cget('bg') == BaseColor:
        event.widget.config(bg=ClickedColore)
    else:
        event.widget.config(bg=BaseColor)

---

【更に追記】

def Callback(event):
    click = event.widget.userOwnValue
    click = int(str(click))
    print(click+1)

    # 全てのLabelの色を戻す
    for child in root.winfo_children():
        if child.winfo_class() == 'Label':
            child.config(bg=BaseColor)
    # クリックされてLabelの色を変更
    event.widget.config(bg=ClickedColore)

Comments

[vibrato]

回答ありがとうございます。

教えていただいた方法を試してみたところ、一度クリックした位置の背景色は変更されたままになるのですが、希望している処理は次回クリック時に、新たにクリックした場所以外の背景色は元に戻る動作を望んでおります。そのように動作させる方法はありますでしょうか？

[magichan]

補足を追加しました

[vibrato]

返信及び追記ありがとうございます。

追記していただいた動作ですと、クリックした場所の背景色のみが変わるという動作ですが、

例
マスAをクリック（マスAの色が変わる）
↓
マスBをクリック（マスBの色が変わる）（マスAの色が戻る）

このようにBをクリックすることで、Bの色の変更及び、Aの色の戻しを行いたいと考えております。
可能でしょうか？

[magichan]

やり方としては、

方法１：色が変わっている（前回クリックした）Labelを覚えておいて、Callback関数内にて戻す処理を入れる
方法２：Callback関数内で 全てのLabelの色を一旦戻した後に、クリックされたLabelの色を変更する

あたりで出来るかと。

[magichan]

方法２での実装例を追加しました

[vibrato]

希望の動作をさせることができました。丁寧にわかりやすく教えていただきありがとうございました！