tkinterで動くボタンを押すゲームを作りたいけれど、threadがうまく動かない

Question

前提・実現したいこと

tkinterで動くボタンを押すゲームを作りたい

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

Traceback (most recent call last):
  File "C:\Users", line 47, in <module>
    th_1.start()
AttributeError: 'NoneType' object has no attribute 'start'

該当のソースコード

from tkinter import ttk
import tkinter
import threading
import random

class Move(threading.Thread):
    def __init__(self,text,flag_x=True,flag_y=True,x=random.randint(0,400),y=random.randint(0,300)):
        self.flag_x,self.flag_y = flag_x,flag_y
        self.x,self.y = x,y
        self.text = text
        self.button = tkinter.Button(master=root,text=text,foreground="red",command=self.button_press)
    # ボタンを押したときの処理
    def button_press(self):
        self.button.pack_forget()
        lb = tkinter.Label(master=root,text=self.text,font=("MS ゴシック",50),fg="blue",bg="red")
        lb.place(x=100,y=100)
    # 実行
    def run(self):
        # 画面外に出ないように調節
        if(self.flag_y==True):
            self.y-=1
        else:
            self.y+=1
        if(self.y==0):
            self.flag_y=False
        elif(self.y+self.button.winfo_reqheight()==300):
            self.flag_y=True
    
        if(self.flag_x==True):
            self.x-=1
        else:
            self.x+=1
        if(self.x==0):
            self.flag_x=False
        elif(self.x+self.button.winfo_reqwidth()==400):
            self.flag_x=True
        # ボタンをコンマ4秒ごとに配置
        self.button.place(x=self.x,y=self.y)
        self.button.after(4,lambda: self.run())

# ウィンドウの情報
w,h="400","300"
root=tkinter.Tk()
ttk.Style().configure("TP.TFrame", background="snow")
f=ttk.Frame(master=root,style="TP.TFrame",width=w,height=h)
f.pack()
# TODO/動くボタンをいくつか作成
if __name__ == "__main__":
    th_1 = Move("a").run();
    th_2 = Move("b").run();
    th_1.start()
    th_2.start()
root.mainloop()

試したこと

threadについてインターネットで学んだ。

補足情報（FW/ツールのバージョンなど）

python3のIDLEです。

Answer 1

すでに回答が出ていますが、ちょっとだけ補足

threading.Threadから派生しているので、startを行なった時点でスレッドが開始されます。明示的にrunを呼ぶ必要がありません。
ですので、すでに回答があるように、オブジェクト生成とstartを分け、以下のようにするのが良いかと思います。

    th_1 = Move("a")
    th_2 = Move("b")
    th_1.start()
    th_2.start()

    root.mainloop()
    # 終了処理もきちんと
    th_1.join()
    th_2.join()

ただ、startを呼ぶと、RuntimeError: thread.__init__() not calledとエラーが出るので、エラーに従い、オーバーライドしたコンストラクタ内で、呼び出してあげる必要があります。（これも回答済みですが）

また、上記を修正して動かすと、期待した動作としては、2つのオブジェクトaとbが表示され、ランダムにウィンドウ内を動くはずであるが、実際に動作させると、一つのオブジェクトのみが表示されることになります。

これは、コンストラクタのデフォルト引数に問題があるからです。

def __init__(self,text,flag_x=True,flag_y=True,x=random.randint(0,400),y=random.randint(0,300)):

上記のようなコードを記載すると、xやyに全く同じ値が設定されます。
これは、実行時にrandom.randintが呼び出されるのではなく、関数の定義時に呼び出され、実行時にはその際の値が設定されて呼び出されるpythonの仕様だからです。

本来の動作をしたいのであれば、xやyにデフォルトでNoneを設定し、関数内部でNoneの場合に値を設定する必要があります。

def __init__(self,text,flag_x=True,flag_y=True,x=None,y=None):

  threading.Thread.__init__(self)   # 追加
  if x==None:
    x=random.randint(0,300)
  if y==None:
    y=random.randint(0,300)

Comments

[usagitobi]

こんなに丁寧に教えていただきありがとうございます！
無事正常に動きました。

Answer 2

以下のコードで正常動作を確認しました。

from tkinter import ttk
import tkinter
import threading
import random

class Move(threading.Thread):
    def __init__(self,text,flag_x=True,flag_y=True,x=random.randint(0,400),y=random.randint(0,300)):
        threading.Thread.__init__(self)
        self.flag_x,self.flag_y = flag_x,flag_y
        self.x,self.y = x,y
        self.text = text
        self.button = tkinter.Button(master=root,text=text,foreground="red",command=self.button_press)
    # ボタンを押したときの処理
    def button_press(self):
        self.button.pack_forget()
        lb = tkinter.Label(master=root,text=self.text,font=("MS ゴシック",50),fg="blue",bg="red")
        lb.place(x=100,y=100)
    # 実行
    def run(self):
        # 画面外に出ないように調節
        if(self.flag_y==True):
            self.y-=1
        else:
            self.y+=1
        if(self.y==0):
            self.flag_y=False
        elif(self.y+self.button.winfo_reqheight()==300):
            self.flag_y=True

        if(self.flag_x==True):
            self.x-=1
        else:
            self.x+=1
        if(self.x==0):
            self.flag_x=False
        elif(self.x+self.button.winfo_reqwidth()==400):
            self.flag_x=True
        # ボタンをコンマ4秒ごとに配置
        self.button.place(x=self.x,y=self.y)
        self.button.after(4,lambda: self.run())

# ウィンドウの情報
w,h="400","300"
root=tkinter.Tk()
ttk.Style().configure("TP.TFrame", background="snow")
f=ttk.Frame(master=root,style="TP.TFrame",width=w,height=h)
f.pack()
# TODO/動くボタンをいくつか作成
if __name__ == "__main__":
    th_1 = Move("a")
    th_2 = Move("b")
    th_1.run()
    th_2.run()
    th_1.start()
    th_2.start()
root.mainloop()

まず、run関数は戻り値がないため、そのままだとNoneを返します。よって、AttributeError: 'NoneType' object has no attribute 'start'が発生します。

これを回避するには、一度Moveクラスでth_1を定義してからrun関数を実行する必要があります。

また、th_1.run()が正常に実行されても、このままだとth_1.start()で以下のようなエラーが発生します。

RuntimeError: thread.__init__() not called

このエラーは、クラスの__init__()にthreading.Thread.__init__(self)を入れることでなくせます。

Comments

[usagitobi]

回答ありがとうございます！
すごくわかりやすかったです。