WinForms截图Task运行数分钟崩溃：无效参数异常求助

看起来你的WinForms程序在后台Task持续截图时，遇到了GDI+资源耗尽或者线程安全相关的问题，导致几分钟后抛出「参数无效」的异常。我来帮你分析问题并给出修复方案：

核心问题分析 #

从报错栈和代码来看，主要有几个关键问题：

1. GDI+资源未完全释放：你创建了Graphics对象但没有调用Dispose()，频繁循环后会耗尽系统的GDI资源，导致后续创建Bitmap时失败，抛出参数无效异常。
2. 跨线程访问UI控件不安全：后台Task直接访问boolCheckBoxHP、boolCheckBoxMana这些UI控件的状态，WinForms控件不是线程安全的，这种操作可能导致不可预测的状态异常。
3. 重复截图浪费资源：HP和Mana检查各自单独截图，加倍了资源消耗，加剧了资源耗尽的速度。

修复后的代码示例 #

private void TaskBot()
{
    Task taskHeal = new Task(() =>
    {
        bool checkHP = false;
        bool checkMana = false;

        while (true)
        {
            Thread.Sleep(200);

            // 通过Invoke安全获取UI控件的勾选状态
            this.Invoke((MethodInvoker)delegate
            {
                checkHP = boolCheckBoxHP?.Checked ?? false;
                checkMana = boolCheckBoxMana?.Checked ?? false;
            });

            Bitmap bitmap = null;
            try
            {
                var screenBounds = Screen.PrimaryScreen.Bounds;
                // 先检查屏幕宽高是否有效，避免构造Bitmap时传入非法参数
                if (screenBounds.Width <= 0 || screenBounds.Height <= 0)
                {
                    Thread.Sleep(1000); // 等待屏幕状态稳定后重试
                    continue;
                }

                bitmap = new Bitmap(screenBounds.Width, screenBounds.Height);
                // 使用using语句自动释放Graphics资源，避免泄漏
                using (Graphics graphics = Graphics.FromImage(bitmap))
                {
                    graphics.CopyFromScreen(0, 0, 0, 0, bitmap.Size, CopyPixelOperation.SourceCopy);
                }

                // 复用同一张截图给两个检查方法，减少资源消耗
                if (checkHP)
                {
                    KontrolujPoziomHP(bitmap);
                }
                if (checkMana)
                {
                    Thread.Sleep(50);
                    KontrolujPoziomMany(bitmap);
                }
            }
            catch (Exception ex)
            {
                // 后台线程不能直接弹出MessageBox，需要切换到UI线程执行
                this.Invoke((MethodInvoker)delegate
                {
                    MessageBox.Show($"Error: {ex.Message}\n\nCause: 截图操作失败，可能是资源耗尽或屏幕状态异常。");
                });
            }
            finally
            {
                // 确保Bitmap始终被释放，不管是否发生异常
                bitmap?.Dispose();
            }
        }
    });
    taskHeal.Start();
}

关键修改点说明 #

• 用using管理Graphics：using语句会自动在代码块结束时调用Dispose()，彻底释放GDI资源，从根源避免资源泄漏。
• 线程安全获取UI状态：通过Invoke将获取控件状态的操作切换到UI线程执行，符合WinForms的线程安全规则，避免不可预测的控件状态异常。
• 复用截图资源：只截图一次就同时用于HP和Mana检查，大幅减少资源消耗。
• 增加屏幕边界有效性检查：防止屏幕分辨率变化或显示器状态改变时，传入无效的宽高给Bitmap构造函数。
• 后台线程安全显示提示：弹出MessageBox必须在UI线程执行，所以用Invoke包裹操作。

额外建议 #

• 可以考虑在不需要检查HP/Mana时暂停截图，或者适当降低截图频率，进一步减少资源占用。
• 你可以通过任务管理器的「详细信息」页查看程序的GDI对象数量，验证修复后的资源释放效果。

内容的提问来源于stack exchange，提问作者Paweł Traczyk