多线程光线追踪器（MT Ray-Tracer）执行耗时超预期达单线程两倍

哦，这问题我太熟了——你现在相当于让两个线程各自把整个渲染任务从头干一遍，等于做了两倍的工作量，耗时翻倍一点都不奇怪！毕竟哪怕线程并行跑，总工作量翻了倍，时间自然也会上去，完全和你要的“减半”目标背道而驰。

核心问题出在你的任务分配逻辑上，正确的多线程渲染应该是拆分任务，让每个线程只处理一部分工作，而不是重复执行全量任务。给你几个具体的解决步骤：

• 拆分渲染任务：把图像拆成若干独立的小块，每个线程只负责自己的区块。最简单的方式是按屏幕高度拆分：比如两个线程的话，线程1处理上半部分的所有扫描线，线程2处理下半部分。也可以用交替扫描线的方式（线程1处理奇数行，线程2处理偶数行），这种方式能避免某些区块复杂度不均导致的负载不均衡问题。
• 避免不必要的资源竞争：如果你的场景数据、相机参数是只读的，直接让线程读取就行，不用加锁；最终的图像缓冲区，每个线程只写自己负责的区域，也不会有竞争，完全不需要额外的同步开销。
• 控制线程数量：线程数不要超过你的CPU核心数，不然会频繁触发上下文切换，反而拖慢速度。

给你一段伪代码参考（以C++为例），直观展示正确的任务拆分方式：

// 全局图像缓冲区，每个线程只操作自己负责的区域
Color image[WIDTH][HEIGHT];

// 线程函数：处理从start_y到end_y的所有扫描线
void render_thread(int start_y, int end_y, const Camera& camera, const Scene& scene) {
    for (int y = start_y; y < end_y; ++y) {
        for (int x = 0; x < WIDTH; ++x) {
            // 生成光线、追踪计算颜色
            Ray ray = camera.generate_ray(x, y);
            Color pixel_color = trace_ray(ray, scene);
            // 写入自己负责的像素，无竞争
            image[x][y] = pixel_color;
        }
    }
}

int main() {
    Camera my_camera;
    Scene my_scene;

    // 拆分任务：两个线程各处理一半高度的图像
    int half_height = HEIGHT / 2;
    std::thread t1(render_thread, 0, half_height, std::cref(my_camera), std::cref(my_scene));
    std::thread t2(render_thread, half_height, HEIGHT, std::cref(my_camera), std::cref(my_scene));

    // 等待线程完成
    t1.join();
    t2.join();

    // 保存最终图像
    save_image(image, "output.png");
}

按照这个逻辑改完后，你应该就能看到渲染时间接近单线程的一半（考虑到少量线程同步开销，可能不会完全减半，但肯定会比现在快很多）。

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