Python-сообщество

Alex2ndr
Сделал на threading.Event - вроде таких граблей нет.

Отредактировано (Июль 19, 2010 01:26:20)

nerijus
Вот так с самого начала и надо было делать.

Alex2ndr
Делал. Без переопределения join это все равно не работало.

Alex2ndr
т к “Simple is better than complex” - Если бы подощло простое решение, то зачем делать сложно?

Согласен тобой, nerijus. threading.Event тут явно правильнее обычного флага.
Если бы нужно было бы сделать АПИ не для внутреннего использования, я бы использовал property, так как это явно красивее. Но тут это уже лишнее.

В дополнение к этой теме еще 2 статейки(если можно так выразится):
http://habrahabr.ru/blogs/python/84629/
http://community.livejournal.com/ru_python/275027.html
После изучения этого материала и ОСОЗНАНИЯ того факта, что когда мы в Main Thread сделали join, то управление к нему не возвращается никогда (ну или очень долго - дольше чем я ждал), мне стало понятно высказывание:

nerijus
С join все в порядке. Дело просто в том что join как и например socket.accept, блокирует поток и при этом не получает KeyboardInterrupt. А это исключение получает только главный поток. Так что нужно запустить цикл со sleep или join с таймаоутом (что вы практически и пытались неявно сделать переписывая сам join). Главное тут не попасть в блокировку на долго.

Ненужно даже флаг в цикле проверять. Вот набросил маленький пример:

from threading import Thread
from time import sleep

class Worker(Thread):
    def __init__(self):
        Thread.__init__(self)
        self.terminated = False

    def run(self):
        while not self.terminated:
            sleep(1)

    def terminate(self):
        self.terminated = True
        self.join()


if __name__ == '__main__':
    worker = Worker()
    worker.start()
    try:
        while True:
            sleep(1)
    except KeyboardInterrupt:
        print "Ctrl-C"
        worker.terminate()
    print "Done"

Тогда такой вопрос: а если подойти архитектурно(как в соседней теме :) ) то насколько грамотен тот или иной подход? После изучения всего материала пришел к выводу, что результат в принципе получается одинаковым - т е выполнение перехватывается всеми потоками, в том числе и main thread. Разве что переопределять чужие функции, без должной причины, не хорошо…

Alex2ndr
т е выполнение перехватывается всеми потоками

Alex2ndr
Разве что переопределять чужие функции, без должной причины, не хорошо…

nerijus, в твоём подходе есть одно странное недоразумение… Все нити завершаются последовательно, а не параллельно. В общем-то иногда это будет хорошей фишкой, но нам уж лучше, чтобы оно быстрее завершалось.
Архитектурно такой приём мне не нравится. Не люблю:

        while True:
            sleep(1)

ZZZ
в твоём подходе есть одно странное недоразумение… Все нити завершаются последовательно, а не параллельно.

ZZZ
Архитектурно такой приём мне не нравится. Не люблю: