wxPythonのボタンの処理について

ボタン一つで時系列的に色を表示したい

wxPythonのボタンの処理内に複数の処理を書くと実行はされますが、時系列的でなく途中結果が見れません。

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

例として信号機のプログラムを書きました。期待する動作は
青→黄色→（2秒）→赤→（2秒）→青と順次表示してほしいのですが。
実際は
青→（4秒）→青となってしまいます。
強制的に更新？するプログラムなどはあるのでしょうか。
それとも.SetBackgroudColour()がいけないのでしょうか。
はたまたこのようなことは仕様上できないのでしょうか。
よろしくお願いします。

該当のソースコード

python
1import wx
2from time import sleep
3
4
5class TrafficLight(wx.Frame):
6    def __init__(self):
7        wx.Frame.__init__(self, None, wx.ID_ANY, size=(330, 200))
8
9        blue = wx.Panel(self, wx.ID_ANY, pos=(0, 0), size=(100, 100))
10        self.blue = blue
11        yellow = wx.Panel(self, wx.ID_ANY, pos=(102, 0), size=(100, 100))
12        self.yellow = yellow
13        red = wx.Panel(self, wx.ID_ANY, pos=(204, 0), size=(100, 100))
14        self.red = red
15        btm = wx.Button(self, wx.ID_ANY, pos=(102, 105), size=(100, 100))
16
17        blue.SetBackgroundColour('green')
18
19        btm.Bind(wx.EVT_BUTTON, self.OnTraffic)
20
21    def OnTraffic(self, event):
22        self.blue.SetBackgroundColour(None)
23        self.yellow.SetBackgroundColour('yellow')
24        sleep(2)
25        self.yellow.SetBackgroundColour(None)
26        self.red.SetBackgroundColour('red')
27        sleep(2)
28        self.red.SetBackgroundColour(None)
29        self.blue.SetBackgroundColour('green')
30        self.Refresh()
31
32
33if __name__ == '__main__':
34    app = wx.App()
35    frame = TrafficLight().Show()
36    app.MainLoop()
37

行動規範の内容に同意します

回答2件

ベストアンサー

質問に挙げたコードのように、イベントハンドラの処理にて長い時間処理を行うと(今回は４秒間)、他のイベント処理が一切行うことが出来ません。
その結果アプリケーションが固まったようになってしまい、今回のように背景色の更新も行うことができなくなります。
ですので、今回のように時間のかかる処理をイベント起動にて実行するときは別スレッドにて処理を行うことが一般的となります。

今回の場合はボタンを押してから２秒間隔に処理を行えばよいだけので、wx.Timer()にて２秒間隔でコールバックイベントを受け取り、その中で処理するようにすると解決するのではないかと思います。

以下にwx.Timer()を使って書き直して見ましたので参考にしてください

Python
1import wx
2from time import sleep
3
4class TrafficLight(wx.Frame):
5    def __init__(self):
6        wx.Frame.__init__(self, None, wx.ID_ANY, size=(330, 200))
7
8        blue = wx.Panel(self, wx.ID_ANY, pos=(0, 0), size=(100, 100))
9        self.blue = blue
10        yellow = wx.Panel(self, wx.ID_ANY, pos=(102, 0), size=(100, 100))
11        self.yellow = yellow
12        red = wx.Panel(self, wx.ID_ANY, pos=(204, 0), size=(100, 100))
13        self.red = red
14        btm = wx.Button(self, wx.ID_ANY, pos=(102, 105), size=(100, 100))
15        self.btm = btm
16        timer = wx.Timer(self)
17        self.timer = timer
18
19        blue.SetBackgroundColour('green')
20        self.current_color = 'green'
21
22        btm.Bind(wx.EVT_BUTTON, self.OnTraffic)
23        self.Bind(wx.EVT_TIMER, self.OnTimer)
24
25    def OnTraffic(self, event):
26        # 処理中はボタンを押せないようにする
27        self.btm.Disable()
28
29        # とりあえず黄色にする
30        self.blue.SetBackgroundColour(None)
31        self.yellow.SetBackgroundColour('yellow')
32        self.current_color = 'yellow'
33        self.Refresh()
34
35        # 2秒後にタイマーイベントを受け取る
36        self.timer.Start(2000)
37
38    def OnTimer(self, event):
39        if self.current_color == 'yellow':
40            # 黄色だったら赤に変える
41            self.yellow.SetBackgroundColour(None)
42            self.red.SetBackgroundColour('red')
43            self.current_color = 'red'
44        else:
45            # その他（赤）だったら緑に戻して処理を終える
46            self.red.SetBackgroundColour(None)
47            self.blue.SetBackgroundColour('green')
48            self.current_color = 'green'
49
50            # タイマーを止める
51            self.timer.Stop()
52            # 再度ボタンを有効化
53            self.btm.Enable()
54
55        self.Refresh()
56
57if __name__ == '__main__':
58    app = wx.App()
59    frame = TrafficLight().Show()
60    app.MainLoop()

投稿2018/02/13 01:34

magichan

総合スコア15898

adoth

2018/02/13 07:30

ありがとうございます。Timer()を知らなかったので勉強になり、また問題解決することができました。

行動規範の内容に同意します

Refresh()のタイミングの問題だと思います。Refresh関数はフレームの再描画関数なので、自動で呼び出される場合を除いて、フォームを変更するたびに呼び出す必要があります。

python
1#TrafficLight内
2def OnTraffic(self, event):
3    self.blue.SetBackgroundColour(None)
4    self.yellow.SetBackgroundColour('yellow')
5    self.Refresh()
6    sleep(2)
7
8    self.yellow.SetBackgroundColour(None)
9    self.red.SetBackgroundColour('red')
10    self.Refresh()
11    sleep(2)
12
13    self.red.SetBackgroundColour(None)
14    self.blue.SetBackgroundColour('green')
15    self.Refresh()