Ruby线程池因ThreadError(队列空)卡顿，求解决方法

你的问题出在使用jobs.pop(true)（非阻塞弹出）时，当队列任务耗尽会抛出ThreadError，即便捕获了异常，这种频繁触发异常的方式也会引发不必要的性能开销，导致程序卡顿。以下是两种更可靠的解决方案：

方案一：使用终止信号（推荐） #

在所有任务加入队列后，添加与线程数量相等的nil作为终止标记，线程获取到nil时主动退出循环，完全规避异常：

children = Dir.children(MEDIA_FOLDER)

jobs = Queue.new
children.each { |i| jobs.push i }
# 给每个线程发送终止信号
8.times { jobs.push nil }

workers = 8.times.map do
  Thread.new do
    # 阻塞式pop，线程会在队列空时休眠，直到获取元素
    while (x = jobs.pop)
      break if x.nil? # 收到终止信号就退出线程
      check_subdirectory(x)
    end
  end
end

workers.map(&:join)

方案二：精准处理ThreadError并退出 #

如果坚持用非阻塞pop，需捕获特定的ThreadError（队列为空的异常）后直接退出线程，避免无效重试：

children = Dir.children(MEDIA_FOLDER)

jobs = Queue.new
children.each { |i| jobs.push i }

workers = 8.times.map do
  Thread.new do
    loop do
      begin
        x = jobs.pop(true)
        check_subdirectory(x)
      rescue ThreadError
        # 队列为空，直接终止线程
        break
      rescue Exception => error
        # 处理业务逻辑中的其他异常
        puts "处理任务出错: #{error}"
      end
    end
  end
end

workers.map(&:join)

关键优化点 #

• 移除了Thread.report_on_exception = false和Thread.abort_on_exception = false，这两个配置会隐藏线程异常，不利于调试；若生产环境需抑制异常报告，建议仅在确认异常被完全处理后开启。
• 方案一的阻塞式pop更高效，线程在队列空时会进入休眠状态，不会空转消耗CPU资源。
• 方案二通过捕获特定异常替代全局Exception捕获，减少潜在的问题遗漏。

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