SDL_MOUSEBUTTONUP未释放鼠标即触发？视频播放器异常问题咨询

看来你在SDL事件处理上踩了个常见的小坑——鼠标事件的触发逻辑没捋清楚，而且多线程环境下的状态同步也可能导致行为不可控。结合你的需求，我给你梳理一套可行的解决方案：

核心问题分析 #

SDL的鼠标事件是分开触发的：SDL_MOUSEBUTTONDOWN对应按下动作，SDL_MOUSEBUTTONUP对应释放动作。你的问题大概率是：

1. 没有严格过滤事件类型，误把按下事件当成了释放事件；
2. 未校验鼠标操作的区域（比如点击了窗口边框而非视频区域）；
3. 多线程下播放状态变量未做同步保护，导致多次切换；
4. 事件循环逻辑不合理，导致重复处理事件。

具体解决步骤与代码示例 #

1. 定义线程安全的播放状态 #

首先需要一个线程共享的播放状态变量，并用互斥锁保护，避免多线程竞争：

#include <SDL2/SDL.h>
#include <mutex>
#include <iostream>

// 全局/线程共享的播放状态，默认播放中
bool g_is_playing = true;
// 保护播放状态的互斥锁
std::mutex g_play_state_mutex;

2. 修正事件处理线程逻辑 #

事件处理线程要唯一负责SDL事件处理（SDL事件系统不是线程安全的，别在渲染线程里碰事件），严格过滤SDL_MOUSEBUTTONUP事件，同时校验点击区域：

void EventHandlerThread(SDL_Window* window, SDL_Renderer* renderer) {
    SDL_Event event;
    bool is_running = true;
    // 记录是否在视频区域按下了鼠标，防止误触窗口其他区域的释放事件
    bool is_mouse_pressed_in_video = false;

    // 获取视频渲染区域（这里假设视频占满整个窗口，可根据实际情况调整）
    int window_w, window_h;
    SDL_GetWindowSize(window, &window_w, &window_h);
    SDL_Rect video_render_rect = {0, 0, window_w, window_h};

    while (is_running) {
        // 用SDL_WaitEvent阻塞等待事件，减少空转CPU占用
        while (SDL_WaitEvent(&event)) {
            switch (event.type) {
                case SDL_QUIT:
                    is_running = false;
                    break;

                case SDL_MOUSEBUTTONDOWN:
                    // 仅处理左键按下，且校验是否在视频区域内
                    if (event.button.button == SDL_BUTTON_LEFT) {
                        SDL_Point mouse_pos = {event.button.x, event.button.y};
                        if (SDL_PointInRect(&mouse_pos, &video_render_rect)) {
                            is_mouse_pressed_in_video = true;
                        } else {
                            is_mouse_pressed_in_video = false;
                        }
                    }
                    break;

                case SDL_MOUSEBUTTONUP:
                    // 仅处理左键释放，且之前确实在视频区域按下过鼠标
                    if (event.button.button == SDL_BUTTON_LEFT && is_mouse_pressed_in_video) {
                        SDL_Point mouse_pos = {event.button.x, event.button.y};
                        if (SDL_PointInRect(&mouse_pos, &video_render_rect)) {
                            // 加锁切换播放状态，保证线程安全
                            std::lock_guard<std::mutex> lock(g_play_state_mutex);
                            g_is_playing = !g_is_playing;
                            std::cout << "Mouse released - " << (g_is_playing ? "Resumed playing" : "Paused") << std::endl;
                        }
                        // 重置状态，避免下次误触发
                        is_mouse_pressed_in_video = false;
                    }
                    break;

                // 可选：处理窗口大小变化，更新视频渲染区域
                case SDL_WINDOWEVENT:
                    if (event.window.event == SDL_WINDOWEVENT_RESIZED) {
                        window_w = event.window.data1;
                        window_h = event.window.data2;
                        video_render_rect = {0, 0, window_w, window_h};
                    }
                    break;

                default:
                    break;
            }
        }
    }
}

3. 渲染线程同步播放状态 #

渲染线程在处理帧渲染时，需要加锁读取播放状态，避免竞争：

void VideoRenderThread(SDL_Renderer* renderer) {
    while (true) {
        // 加锁读取播放状态
        std::lock_guard<std::mutex> lock(g_play_state_mutex);
        if (!g_is_playing) {
            // 暂停时短暂休眠，降低CPU占用
            SDL_Delay(10);
            continue;
        }

        // 这里替换成你的FFmpeg解码+SDL渲染逻辑
        // 示例：清空渲染器并绘制一个纯色帧（模拟视频帧）
        SDL_SetRenderDrawColor(renderer, 0, 0, 0, 255);
        SDL_RenderClear(renderer);
        SDL_SetRenderDrawColor(renderer, 255, 255, 255, 255);
        SDL_RenderFillRect(renderer, &SDL_Rect{100, 100, 440, 280});
        SDL_RenderPresent(renderer);

        // 控制帧率（约60fps）
        SDL_Delay(16);
    }
}

额外注意事项 #

• SDL事件线程唯一性：所有SDL事件的获取和处理必须在同一个线程中完成，SDL事件系统不支持多线程并发操作，否则会导致事件丢失或误触发。
• 区域校验必要性：如果窗口有控件、边框等非视频区域，一定要校验鼠标坐标是否在视频渲染矩形内，避免误操作。
• 互斥锁范围最小化：加锁的代码块要尽可能小，只保护播放状态的读写，避免影响渲染性能。
• 避免空转：事件线程用SDL_WaitEvent而非SDL_PollEvent，没有事件时会阻塞，减少CPU占用；暂停时渲染线程也要短暂休眠。

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