为何MouseEventArgs e与MouseButtons的判断无法生效？

错误原因 #

你代码里的if (e == MouseButtons.Left)完全不成立，因为：

• e是MouseEventArgs类型的实例对象，它封装了鼠标事件的所有上下文数据（按钮状态、坐标、点击次数等）；
• MouseButtons.Left是MouseButtons枚举的成员，本质是一个整数常量。

两者类型完全不同，直接用==比较永远会返回false，这就是判断不生效的核心原因。要获取当前按下的鼠标按钮，必须访问MouseEventArgs的Button属性。

正确解决方案 #

把判断条件替换为e.Button == MouseButtons.Left和e.Button == MouseButtons.Right，同时可以提取重复逻辑到单独方法，减少代码冗余：

private void BtnSize_MouseDown(object sender, MouseEventArgs e)
{
    if (e.Button == MouseButtons.Left)
    {
        GenerateBalls(shouldBeDistinct: true);
    }
    else if (e.Button == MouseButtons.Right)
    {
        GenerateBalls(shouldBeDistinct: false);
    }
}

// 提取重复逻辑，避免代码冗余
private void GenerateBalls(bool shouldBeDistinct)
{
    Ball.Loading = true;
    int ballsAdded = 0;
    int ballsRemoved = 0;

    while (ballsAdded < 25 && ballsRemoved < 1000)
    {
        Ball newBall = new Ball(ballRadiusSet);
        // 根据参数判断是否添加球体
        bool shouldAdd = shouldBeDistinct ? !soManyBalls.Contains(newBall) : soManyBalls.Contains(newBall);
        
        if (shouldAdd)
        {
            soManyBalls.Add(newBall);
            ballsAdded++;
        }
        else
        {
            ballsRemoved++;
        }
    }

    foreach (Ball listBall in soManyBalls)
    {
        listBall.AddBall();
    }

    Ball.Loading = false;
    Text = $"Loaded {ballsAdded} distinct balls with {ballsRemoved} discards";
}

避免类似问题的建议 #

• 先看类型说明：在VS中把鼠标悬停在MouseEventArgs上，就能看到它的属性列表；或者查看官方文档，明确类的用途和可用成员，不要想当然地直接比较对象和枚举。
• 检查类型匹配：写判断条件时，确认两边的类型一致。枚举值只能和同类型的枚举值或兼容的整数比较，不能直接和对象实例比较。
• 提取重复代码：当多个分支逻辑高度重合时，把公共逻辑抽成单独方法，既减少重复代码，也能避免在多个地方犯同样的错误。
• 用调试工具：遇到判断不生效的情况，给代码加断点，查看e的具体属性值，就能快速定位到问题所在（比如你会看到e.Button才是存储鼠标按钮状态的字段）。

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