如何防止弹出窗体超出屏幕范围？自定义双列ContextMenu同时超出屏幕宽高时位置修正逻辑失效问题排查

我正在用WinForms构建一个双列布局的自定义上下文菜单，窗体类命名为ContextMenu。我写了一个标记函数来检查菜单调用时是否超出设备屏幕尺寸——单独超出宽度或高度的情况处理都正常，但同时超出宽高时逻辑完全失效。

我已经尝试在控制台打印screenExceedFlag的值（测试时确实是3），但程序就是不会进入case 3分支。有没有可能是我代码里遗漏了什么细节？

相关代码 #

屏幕超出标记处理函数

//屏幕超出标记处理函数
private Point processContextMenuFormLocation(ContextMenu theContextMenu, int screenExceedFlag, Point mouseCoor)
{
    //本地窗体坐标
    int formXCoor;
    int formYCoor;
    switch (screenExceedFlag)
    {
        case 1:
            //超出右边界时
            formXCoor = (mouseCoor.X) - (contextMenuObj.Width); //将上下文菜单向左移动
            formYCoor = mouseCoor.Y; //无需修改
            //根据超出标记设置上下文菜单位置
            theContextMenu.Location = new Point(formXCoor, formYCoor);
            break;
        case 2:
            //超出下边界时
            formXCoor = (mouseCoor.X); //无需修改
            formYCoor = (mouseCoor.Y) - (contextMenuObj.Height); //将上下文菜单向上移动
            theContextMenu.Location = new Point(formXCoor, formYCoor);
            break;
        case 3:
            //同时超出右边界和下边界时
            formXCoor = (mouseCoor.X) - (contextMenuObj.Width); //将上下文菜单向左移动
            formYCoor = (mouseCoor.Y) - (contextMenuObj.Height); //将上下文菜单向上移动
            theContextMenu.Location = new Point(formXCoor, formYCoor);
            break;
        case -1:
            //未超出任何边界时
            theContextMenu.Location = new Point(mouseCoor.X, mouseCoor.Y);
            break;
    }
    //返回上下文菜单的新位置
    return theContextMenu.Location;
}

调用该函数的鼠标抬起事件处理程序

//鼠标抬起事件处理程序
private void richTextBox1_MouseUp(object sender, MouseEventArgs e)
{
    //关闭默认上下文菜单，避免同时打开
    richContextStrip.Visible = false;
    if (e.Button == MouseButtons.Right)
    {
        IsKeyUp((int)MouseButtons.Right); //设置rmbIsUp = true
        //此处设置是因为右键事件处理程序优先级更高
        bool canDisplay = contextMenuDisplayFlag(ctrlIsDown, rmbIsUp); //获取Ctrl键和右键的标记状态
        if (canDisplay)
        {
            //判断自定义上下文菜单是否超出屏幕范围
            exceedScreenFlag = isBeyondScreen(Cursor.Position.X, Cursor.Position.Y, contextMenuObj.Width, contextMenuObj.Height);
            //获取鼠标坐标
            Point theMouseCoor = new Point(Cursor.Position.X, Cursor.Position.Y);
            //调用屏幕超出标记处理函数
            Point formLocation = processContextMenuFormLocation(contextMenuObj, exceedScreenFlag, theMouseCoor);
            //将光标定位到上下文菜单中心
            Point cursorLocation = processContextMenuCursorLocation(contextMenuObj, formLocation);
            displayCustomContextMenu(contextMenuObj, formLocation, cursorLocation);
            toolStripStatusLabel1.Text = "Custom context menu opened!";
        }
        else //未按下Ctrl键时
        {
            richContextStrip.Visible = true;
            toolStripStatusLabel1.Text = "Default context menu opened!";
        }
    }
}

问题排查与解决方案 #

咱们一步步拆解可能的问题点：

1. 确认screenExceedFlag是否真的传入了3

你说打印的值是正确的，但建议在processContextMenuFormLocation函数最开头加一行调试输出，彻底确认传入的值：

private Point processContextMenuFormLocation(ContextMenu theContextMenu, int screenExceedFlag, Point mouseCoor)
{
    // 新增调试输出
    Console.WriteLine($"Received screenExceedFlag: {screenExceedFlag}");
    // 原代码...
}

同时在case 3里也加一行输出：

case 3:
    Console.WriteLine("Entered case 3: adjusting both X and Y");
    // 原代码...
    break;

如果控制台没打印Entered case 3，那说明确实没进入分支；如果打印了但位置还是不对，那问题可能出在后续代码。

2. 检查isBeyondScreen函数的逻辑（核心坑点）

很大概率是这个函数的返回值逻辑有问题！比如如果你的判断逻辑是用else if串联的，那同时超出宽高的情况会被第一个条件拦截，永远返回1或2，而不是3。

错误示例（会导致同时超出时返回1）：

private int isBeyondScreen(int x, int y, int width, int height)
{
    var screen = Screen.PrimaryScreen.WorkingArea;
    if (x + width > screen.Width)
        return 1;
    else if (y + height > screen.Height)
        return 2;
    else if (x + width > screen.Width && y + height > screen.Height)
        return 3; // 永远不会走到这里！
    return -1;
}

正确逻辑：先判断同时超出的情况，或者用独立的if来判断：

private int isBeyondScreen(int x, int y, int width, int height)
{
    var screen = Screen.PrimaryScreen.WorkingArea;
    bool exceedWidth = x + width > screen.Width;
    bool exceedHeight = y + height > screen.Height;

    if (exceedWidth && exceedHeight)
        return 3;
    else if (exceedWidth)
        return 1;
    else if (exceedHeight)
        return 2;
    return -1;
}

这样才能确保同时超出时正确返回3。

3. 排查后续代码是否覆盖了位置设置

检查displayCustomContextMenu函数，看看它里面有没有重新设置contextMenuObj.Location的逻辑——如果有，那你在processContextMenuFormLocation里设置的位置会被覆盖，导致看起来case 3没生效。

4. 给switch加default分支捕获意外值

在switch最后加一个default分支，避免遗漏未定义的标记值：

default:
    Console.WriteLine($"Unexpected screenExceedFlag: {screenExceedFlag}");
    // 可以 fallback 到默认位置
    theContextMenu.Location = mouseCoor;
    break;

按照上面的步骤排查，应该能定位到问题所在——最常见的就是isBeyondScreen的逻辑错误导致返回值不对，或者后续代码覆盖了位置设置。

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