tkinterでボタンの色を更新できない

Question

前提・実現したいこと

tkinterでボタンを押すたびに色が変化するようにしたいのですが、
うまくいきません。

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

Exception in Tkinter callback
Traceback (most recent call last):
  File "C:\Users\User\Anaconda3\lib\tkinter\__init__.py", line 1702, in __call__
    return self.func(*args)
  File "<ipython-input-16-aa2d88e58cf3>", line 49, in startButtonClick
    self.button1["bg"] = "red"
TypeError: 'NoneType' object does not support item assignment

該当のソースコード

class Application(tk.Frame):
    def __init__(self,master):
        super().__init__(master)
        self.pack()

        master.geometry("300x150")
        master.title("ストップウォッチ")
        master.config(bg="black")

        self.playTime=False
        
        #ボタンを設置
        self.button1 = tk.Button(master,text="Start",command=self.startButtonClick,width=10,bg="lightgreen").place(x=110, y=110)
        self.button2 = tk.Button(master,text="Stop",command=self.stopButtonClick,width=10,bg="red").place(x=210, y=110)

        master.after(50,self.update)

    def startButtonClick(self):
        if not self.playTime:
            self.playTime=True
            self.button1["bg"] = "red"
            self.button2["bg"] = "lightgreen"
            
    def stopButtonClick(self):
        if self.playTime:
            #測定終了
            self.playTime=False
            self.button1["bg"]="lightgreen"
            self.button2["bg"] = "red"
                
    def update(self):
        self.master.after(50,self.update)

def main():
    win = tk.Tk()
    #win.resizable(width=False, height=False) #ウィンドウを固定サイズに
    app = Application(master=win)
    app.mainloop()

if __name__ == "__main__":
    main()

試したこと

self.button1["bg"] = "red"の代わりに
self.button1.config(bg = "red)やself.button1.widget["bg"] = "red"としても同じエラーが出たので、
self.button1の書き方が良くない気がします。
ボタンを押すたびにbutton1とbutton2が交互に緑と赤で入れ替わるようにしたいです。

Answer 1

原因：

self.button1の書き方が良くない気がします。

ボタンを押すたびにbutton1とbutton2が交互に緑と赤で入れ替わるようにしたいです。

place() 等のレイアウト関数は値を返さない為、self.button1 が None になってしまいます。

解決策:

#ボタンを設置
self.button1 = tk.Button(master,text="Start",command=self.startButtonClick,width=10,bg="lightgreen")
self.button1.place(x=110, y=110)
self.button2 = tk.Button(master,text="Stop",command=self.stopButtonClick,width=10,bg="red")
self.button2.place(x=210, y=110)

恐らく、このようなコードが混乱の原因だと思うのですが、
戻り値を変数に受け取らない場合、以下の記法自体はよく使われるコードで、
標準ライブラリ内(idlelib)でも見かけられます。
(ボタンやラベル等で後から参照する必要がない場合)

tk.Button(master,text="Button",command=command).place(x=110, y=110)

---

解決策2: 名前を付けて nametowidget() 経由でアクセスする

お勧めという訳ではありませんが、別解として紹介。
nameを付けると、後から参照することもできます。

import tkinter as tk

class Application(tk.Frame):
    def __init__(self,master):
        super().__init__(master)
        self.pack()

        master.geometry("300x150")
        master.title("ストップウォッチ")
        master.config(bg="black")

        self.playTime=False

        #ボタンを設置 (name=を付ける。注意点は第一引数の親ウィジェット)
        tk.Button(master,text="Start",name="button1",command=self.startButtonClick,width=10,bg="lightgreen").place(x=110, y=110)
        tk.Button(master,text="Stop",name="button2",command=self.stopButtonClick,width=10,bg="red").place(x=210, y=110)

    def __getattr__(self, key):
        """
        self.button1 とアクセスされた場合、name="button1" と作成した tk.Button を返します。

        今回は簡単な例なので関係ありませんが、
        nameの部分は、実際には親ウィジェットからのpathになる点に注意。
        例えば、第一引数の親が master ではなく self だった場合は、self.nametowidget(key)
        """
        return self.master.nametowidget(key)

    def startButtonClick(self):
        if not self.playTime:
            self.playTime=True
            self.button1["bg"] = "red"
            self.button2["bg"] = "lightgreen"

    def stopButtonClick(self):
        if self.playTime:
            #測定終了
            self.playTime=False
            self.button1["bg"]="lightgreen"
            self.button2["bg"] = "red"

def main():
    win = tk.Tk()
    app = Application(master=win)
    app.mainloop()

if __name__ == "__main__":
    main()

追記: 解決策2の案を お勧めしない理由は、nametowidgetでは、動的に探索される為
dir(app) としても属性に button1, button2 は列挙されません。=> IDE等で補完候補に出ない
また、静的解析ツール等にとっても型の解らない属性になるので、デメリットになります。
本当に必要な時以外は、記述が楽になるという理由だけでは採用しない方が良いです。

紹介の意図としては、他にこのような(直接 pack/grid/placeを呼んでいる)コードを見かけた時に、
後々参照の必要がないか、nametowidget()を通じて参照できるという点に留意して、
参考にして頂けたらと思います。

Comments

[sonai]

ご回答ありがとうございます。
また、別解とその短所まで教えていただき感謝いたします。
解決策1で動かすことができました。非常に助かりました。