回答編集履歴

コード修正。変数が未定義: count, interval

2020/04/28 16:37

投稿

スコア8760

test CHANGED Viewed

@@ -222,7 +222,7 @@
-    def run(self):
+    def run(self, count=10, interval=1):
         while True:

属性名を変更 self.target -> self.win （Threadのコンストラクタにもtarget引数があるので、混同されるのを避ける為)

2020/04/28 16:37

投稿

スコア8760

test CHANGED Viewed

@@ -46,7 +46,7 @@
         # イベント通知対象のウィジェット。(MainWindow)
-        self.target = win
+        self.win = win
@@ -88,7 +88,7 @@
                 # 通知 (MainLoopのキューにイベントを入れる)
-                wx.PostEvent(self.target, event)
+                wx.PostEvent(self.win, event)
@@ -218,7 +218,7 @@
         # 通知 (MainLoopのキューにイベントを入れる)
-        wx.PostEvent(self.target, event)
+        wx.PostEvent(self.win, event)

分離方法の修正

2020/04/28 11:41

投稿

スコア8760

test CHANGED Viewed

@@ -182,7 +182,7 @@
 Pythonの数値型は多倍長なので、桁溢れの心配はないとはいえ
-気になる部分なので、改善案。
+気になる部分なので、改善案。追記: インデントが深くならないように再度修正。
@@ -202,7 +202,27 @@
-    def countWithInterval(self, count=10, interval=1):
+    def onCountInterval(self):
+        # 次に表示する画像ファイル
+        filepath = self.nextImageFile()
+        # イベント通知する為、引数となるイベントオブジェクトを作成
+        event = UpdateImageEvent(filepath=filepath)
+        # 通知 (MainLoopのキューにイベントを入れる)
+        wx.PostEvent(self.target, event)
+    def run(self):
         while True:
@@ -212,28 +232,8 @@
             else:
-                yield
-    def run(self):
-        for _ in self.countWithInterval(count=10):
+                self.onCountInterval()
-            # 次に表示する画像ファイル
-            filepath = self.nextImageFile()
-            # イベント通知する為、引数となるイベントオブジェクトを作成
-            event = UpdateImageEvent(filepath=filepath)
-            # 通知 (MainLoopのキューにイベントを入れる)
-            wx.PostEvent(self.target, event)
 ```

説明修正

2020/04/28 01:06

投稿

スコア8760

test CHANGED Viewed

@@ -237,9 +237,3 @@
             wx.PostEvent(self.target, event)
 ```
-追記: 自分で実装しなくても、threading.Timer で良かったです。
-ちなみに、wxにも同様のクラスが用意されています。

タイマークラスについて追記

2020/04/28 00:23

投稿

スコア8760

test CHANGED Viewed

@@ -237,3 +237,9 @@
             wx.PostEvent(self.target, event)
 ```
+追記: 自分で実装しなくても、threading.Timer で良かったです。
+ちなみに、wxにも同様のクラスが用意されています。

説明修正

2020/04/28 00:06

投稿

スコア8760

test CHANGED Viewed

@@ -186,7 +186,7 @@
-カウント部分の処理と、ロジックの分離もできます。
+カウント自体の処理と、カウント毎の処理の分離もできます。

カウントタイマーの手法について

2020/04/27 23:25

投稿

スコア8760

test CHANGED Viewed

@@ -171,3 +171,69 @@
     main()
 ```
+----
+数値のカウントの方法について、
+Pythonの数値型は多倍長なので、桁溢れの心配はないとはいえ
+気になる部分なので、改善案。
+カウント部分の処理と、ロジックの分離もできます。
+```python
+class CountTimer(Thread):
+    daemon = True
+    # 省略
+    def countWithInterval(self, count=10, interval=1):
+        while True:
+            for _ in range(count):
+                time.sleep(interval)
+            else:
+                yield
+    def run(self):
+        for _ in self.countWithInterval(count=10):
+            # 次に表示する画像ファイル
+            filepath = self.nextImageFile()
+            # イベント通知する為、引数となるイベントオブジェクトを作成
+            event = UpdateImageEvent(filepath=filepath)
+            # 通知 (MainLoopのキューにイベントを入れる)
+            wx.PostEvent(self.target, event)
+```