threading.Timer未按30秒间隔执行问题排查求助

哥们，你的定时函数间隔从30秒变成5秒，核心问题大概率是定时器线程叠加导致的，下面详细拆解原因和解决办法：

核心原因 #

1. threading.Timer的叠加运行 #

你当前的代码逻辑是每次执行printit()时，都会启动一个新的30秒定时器。如果程序重启（restarting()）时没有彻底终止旧的定时器线程，就会出现多个定时器同时在后台运行的情况。比如旧的定时器还在每隔30秒触发printit()，新的程序实例又启动了新的定时器，多个定时器的触发请求挤在一起，就会让打印间隔被压缩到约5秒（比如6个并行定时器的话，30/6=5）。

2. 定时器的触发逻辑不是固定间隔 #

threading.Timer是一次性定时器，它的逻辑是“延迟N秒后执行目标函数”，而非“每隔N秒执行一次”。你在printit()开头就启动下一个定时器，理论上间隔是30秒加上函数执行时间，但如果没有管理好定时器的生命周期，很容易出现多个定时器同时触发的情况。

3. 全局变量的潜在问题 #

你的代码中使用了多个全局变量，但没有显式初始化（比如stuck_counter、pos_x_one等），如果这些变量在程序重启或多次调用中没有被正确重置，可能会导致stuck_counter增长异常，间接让打印频率变高，但这个是次要原因，核心还是定时器叠加。

解决方案 #

方案一：用循环线程替代threading.Timer（推荐） #

这种方式可以保证严格的30秒间隔，且不会出现线程叠加问题：

import threading
import time
import pyautogui

# 初始化全局变量
stuck_counter = 0
check_stuck = 0
pos_x_one = 0
pos_x_two = 0
stop_flag = False  # 用于优雅终止线程

def check_mouse_position():
    global stuck_counter, check_stuck, pos_x_one, pos_x_two, stop_flag
    while not stop_flag:
        pos = pyautogui.position()
        stuck_counter += 1
        if stuck_counter % 2 == 0:
            pos_x_one = pos.x
        else:
            pos_x_two = pos.x
        print(pos_x_one, pos_x_two)
        
        if pos_x_one == pos_x_two:
            check_stuck += 1
            if check_stuck == 10:
                print("STUCK!")
                check_stuck = 0
                stop_flag = True  # 设置终止标志
                restarting()
        else:
            check_stuck = 0
        
        time.sleep(30)  # 固定间隔30秒

# 启动守护线程，程序退出时自动终止
threading.Thread(target=check_mouse_position, daemon=True).start()

方案二：管理threading.Timer的生命周期 #

如果一定要用threading.Timer，需要保存定时器实例，确保重启时取消旧定时器：

import threading
import pyautogui

# 初始化全局变量
stuck_counter = 0
check_stuck = 0
pos_x_one = 0
pos_x_two = 0
current_timer = None

def printit():
    global stuck_counter, check_stuck, pos_x_one, pos_x_two, current_timer
    pos = pyautogui.position()
    stuck_counter += 1
    if stuck_counter %2 == 0:
        pos_x_one = pos.x
    else:
        pos_x_two = pos.x
    print(pos_x_one, pos_x_two)
    
    if pos_x_one == pos_x_two:
        check_stuck += 1
        if check_stuck == 10:
            print("STUCK!")
            check_stuck = 0
            # 取消当前定时器，避免旧线程继续运行
            if current_timer:
                current_timer.cancel()
            restarting()
            return  # 重启后不再启动新定时器
    else:
        check_stuck = 0
    
    # 启动下一个定时器并保存实例
    current_timer = threading.Timer(30, printit)
    current_timer.start()

# 初始启动定时器
current_timer = threading.Timer(30, printit)
current_timer.start()

额外注意事项 #

• 在restarting()函数中，要确保彻底终止所有旧的线程或定时器，避免残留；
• 所有全局变量必须显式初始化，防止运行时出现未定义的异常；
• 如果是GUI程序，要注意线程安全，避免在非主线程中操作GUI组件（如果你的restarting()涉及GUI操作的话）。

内容的提问来源于stack exchange，提问作者ScoobyDuh