Graphics.FillRectangle参数无效，如何用方向键在Panel上移动玩家？

嘿，我帮你梳理下问题根源，咱们一步步搞定：

1. 先解决重复绑定事件的坑 #

你现在每次触发panel1_Paint都新建GameManager实例，这会导致KeyUp事件被重复绑定N次——每次按键都会触发多轮MovePlayer，不仅浪费资源，还会让玩家位置计算混乱，甚至间接引发绘制参数错误。

修正方案：
把GameManager改成Form的成员变量，只在初始化时创建一次，避免重复绑定事件：

// Form1类里定义成员变量
private GameManager _gameManager;

// 在Form1构造函数里初始化（只执行一次）
public Form1()
{
    InitializeComponent();
    _gameManager = new GameManager(this);
    // 记得把Form的KeyPreview设为true，确保能捕获键盘事件
    this.KeyPreview = true;
}

// 修改Panel的Paint事件，不再每次新建GameManager
private void panel1_Paint(object sender, PaintEventArgs e)
{
    _gameManager.Draw(e.Graphics); // 把当前Paint事件的临时Graphics传进去绘制
}

2. 修复Graphics对象生命周期问题 #

你之前把PaintEventArgs里的Graphics保存到GameManager里是错误的——这个Graphics是临时对象，只在Paint事件执行期间有效，后续复用必然引发各种异常（包括你遇到的参数无效）。正确做法是每次Paint时传递当前有效的Graphics给绘制方法，不长期持有。

重构GameManager和Player的逻辑：

// GameManager.cs
public class GameManager
{
    private Form1 _gameForm;
    private Player _player;

    public GameManager(Form1 gameForm)
    {
        _gameForm = gameForm;
        // 初始化玩家位置和尺寸，不要传Graphics
        _player = new Player(100, 100, 50, 50);
        // 只绑定一次KeyUp事件
        _gameForm.KeyUp += MovePlayer;
    }

    // 提供Draw方法，接收当前的Graphics实例
    public void Draw(Graphics graphic)
    {
        _player.Draw(graphic); // 让Player用当前有效Graphics绘制自己
    }

    private void MovePlayer(object sender, KeyEventArgs e)
    {
        int moveStep = 10;
        switch (e.KeyCode)
        {
            case Keys.Up:
                _player.Y -= moveStep;
                break;
            case Keys.Down:
                _player.Y += moveStep;
                break;
            case Keys.Left:
                _player.X -= moveStep;
                break;
            case Keys.Right:
                _player.X += moveStep;
                break;
        }
        // 移动后触发Panel重绘，更新画面
        _gameForm.panel1.Invalidate();
    }
}

// Player.cs
public class Player
{
    public int X { get; set; }
    public int Y { get; set; }
    public int Width { get; set; }
    public int Height { get; set; }
    private readonly Brush _drawBrush;

    public Player(int x, int y, int width, int height)
    {
        X = x;
        Y = y;
        Width = width;
        Height = height;
        _drawBrush = Brushes.DodgerBlue; // 提前定义画笔，避免重复创建
    }

    public void Draw(Graphics graphic)
    {
        // 先检查参数合法性，避免FillRectangle报错
        if (Width <= 0 || Height <= 0)
            return;
        // 可选：限制玩家不要移出Panel边界
        // X = Math.Max(0, Math.Min(X, _gameForm.panel1.Width - Width));
        // Y = Math.Max(0, Math.Min(Y, _gameForm.panel1.Height - Height));
        graphic.FillRectangle(_drawBrush, X, Y, Width, Height);
    }
}

3. 彻底解决Graphics.FillRectangle参数无效问题 #

这个错误的核心原因通常是这几点：

• 使用了已失效的Graphics对象（就是你之前保存的临时实例）
• 矩形的Width/Height小于等于0
• 玩家的X/Y出现负数且未做边界限制

上面的代码已经通过以下方式规避：

• 不再保存Graphics对象，每次用Paint事件里的有效实例
• 在Player的Draw方法里先检查宽高是否合法
• 可以添加边界限制代码（注释部分），避免玩家移出Panel导致坐标异常

最后补充：如果Panel还是捕获不到键盘事件，除了设置Form的KeyPreview=true，还可以在Form加载时调用panel1.Focus()让Panel获取焦点。

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