Question

I’m trying to build a Python application using pyGTK, treads, and sockets. I’m having this weird error, but given all the modules involved, I’m not entirely sure where the error is. I did a little debugging with some print statements to narrow things down a bit and I think the error is somewhere in this snippet of code:

    self.sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
    self.sock.connect(("localhost", 5005))
    self.collectingThread = threading.Thread(target=self.callCollect)
    self.collectingThread.daemon = True
    self.collectingThread.start()

def callCollect(self):
    gobject.timeout_add(500, self.collectData)

def collectData(self):
    print "hello"
    try:
        print self.sock.recv(1024)
    except:
        print "except"
    print "return"
    return True

So basically what I’m trying to do is setup a socket, connect to a “server” script (which is really just another python script running locally), and create a separate thread to collect all incoming data from the server script. This thread is set to run the collectData method every 500 milliseconds.

After inserting the print statements into the collectData method here is what I notice when running the program:

-Initially the GUI is fully functional

-then the following is printed in the terminal:

hello
**all data received from server script and printed here**
return
hello

-after the text is printed in the terminal, the GUI becomes completely nonfunctional (buttons cant be pressed, etc.) and I have to force quit to close the application

What seems to be happening is that the thread prints “hello”, prints the data from the server, and prints “return”. 500 milliseconds later, it runs the collectData method again, prints “hello”, then tries to print data from the server. However, because there is no data left it raises an exception, but for some unknown reason it doesn’t execute the code in the exception block and everything just hangs from there.

Any idea on what is going wrong?

Answer 1

timeout_add is scheduling the action to happen on the main thread — so the recv just blocks the main thread (when it’s just waiting for data) and therefore the GUI, so, no exception unless you put a timeout or set the socket to non-blocking.

You need to delegate the receiving to the thread from the scheduled action rather than viceversa to get the effect you’re after: have the thread e.g. wait on an event object, and the scheduled action signal that event every 500 milliseconds.