Discussion :

[piokml]

Bonjour,
Je coince actuellement sur un problème avec l'utilisation de Tkinter et des threads.
Le code ci-après résume mon problème.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
import threading
from Tkinter import *
import time
 
class Th(threading.Thread):
    def __init__(self, label):
        threading.Thread.__init__(self)
        self.label = label
        self.index = 0
        self.lock = threading.Lock()
    def run(self):
        while True:            
            self.lock.acquire()
            time.sleep(0.01)
            self.label.config(text=str(self.index))
            self.index += 1
            self.lock.release()
class App():
    def __init__(self, label):
        self.label = label
    def run(self):
        Th(self.label).start()
        Th(self.label).start()
 
class Gui():
    def __init__(self):
        gui = Tk()
        label = Label(gui)
        label.pack()
        app = App(label)
        run = Button(gui, text="Run", command=app.run)
        run.pack()
        gui.mainloop()
Gui()

Je créé une Gui qui lance une application qui elle-même lance plusieurs threads (ici deux) qui se partagent le même widget tkinter (ici un label) à mettre à jour. Ce code semble fonctionner mais plante au bout d'un certain nombre d'itérations (entre 0 et 10000+).
Les messages d'erreurs sont:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
Exception in thread Thread-1:
Traceback (most recent call last):
  File "C:\Python27\lib\threading.py", line 552, in __bootstrap_inner
    self.run()
  File "C:\Users\Pierre\workspace\Driver-Events\tests\threadTkinter.py", line 22, in run
    self.label.config(text=str(self.index))
  File "C:\Python27\lib\lib-tk\Tkinter.py", line 1202, in configure
    return self._configure('configure', cnf, kw)
  File "C:\Python27\lib\lib-tk\Tkinter.py", line 1193, in _configure
    self.tk.call(_flatten((self._w, cmd)) + self._options(cnf))
TclError: invalid command name "-text"

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
Exception in thread Thread-1:
Traceback (most recent call last):
  File "C:\Python27\lib\threading.py", line 552, in __bootstrap_inner
    self.run()
  File "C:\Users\Pierre\workspace\Driver-Events\tests\threadTkinter.py", line 22, in run
    self.label.config(text=str(self.index))
  File "C:\Python27\lib\lib-tk\Tkinter.py", line 1202, in configure
    return self._configure('configure', cnf, kw)
  File "C:\Python27\lib\lib-tk\Tkinter.py", line 1193, in _configure
    self.tk.call(_flatten((self._w, cmd)) + self._options(cnf))
TclError: bad option "590": must be cget or configure

Je souhaiterais votre avis sur ce qui ne vas pas.

[piokml]

Il semble qu'il y ait un conflit avec la mainloop de tkinter. Comment peut-on s'en passer?

[wiztricks]

Salut,
Si vous avez mis un .lock, c'est pour que l'autre thread attende que la séquence, section critique, soit libre.
Si chaque thread à son .lock, ça ne sert plus à rien.
- W

[pfeuh]

Salut,

Pareil que wiztrick. Il faut aussi penser à arrêter les threads et empêcher le bouton de les relancer.

A+

Pfeuh

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
import threading
import Tkinter as tk
import time
import random
 
class Th(threading.Thread):
    def __init__(self, lock, widget):
        threading.Thread.__init__(self)
        self.counter = random.randint(0, 1000000)
        self.lock = lock # le semaphore doit etre partage
        self.widget = widget
        self.running = True # en prevision de l'arret du thread
 
    def run(self):
        while self.running:            
            self.lock.acquire()
            #section supposee critique
            self.widget.config(text=str(self.counter))
            self.counter += 1
            # fin de section critique
            self.lock.release()
            time.sleep(0.001)
 
class App():
    def __init__(self, win, *widgets):
        self.widgets = widgets
        self.win = win
 
    def run(self, bt):
        # demarre les threads
        threads = []
        lock = threading.Lock()
        for widget in self.widgets:
            item = Th(lock, widget)
            item.start()
            threads.append(item)
        #prepare l'arret de l'application
        command = lambda win=self.win, threads=threads:cleanQuit(win, *threads)
        self.win.protocol("WM_DELETE_WINDOW", command)
        bt['command'] = command
        bt['text'] = 'STOP'
 
def cleanQuit(win, *threads):
    for item in threads:
        item.running = False
    win.quit()
    print "Bye..."
 
class Gui():
    def __init__(self):
        gui = tk.Tk()
        widgets = []
        for x in range(5):
            widget = tk.Label(gui)
            widget.pack()
            widgets.append(widget)
        app = App(gui, *widgets)
        bt = tk.Button(gui, text="START", width=10)
        bt['command'] = lambda bt=bt:app.run(bt)
        bt.pack()
        gui.mainloop()
 
Gui()