TextBox的KeyPress事件maxBroj限制逻辑失效问题求助

嘿，我一眼就看出问题出在哪了——你在KeyPress事件里判断maxBroj的逻辑有个关键漏洞：

当你输入12的时候，第一次敲1，文本框里是"1"，小于10，没问题；第二次敲2的时候，KeyPress事件触发时，文本框里的内容还是"1"（因为这个事件是在字符被插入到文本框之前执行的），所以Convert.ToDouble(textBox1.Text)得到的是1，根本不会触发>maxBroj的判断，于是2就被顺利插入，最终文本框变成"12"，这才超过了限制，但已经晚了。

简单说：你判断的是输入前的文本值，而不是输入后的完整值，所以拦不住那些会让总和超标的后续字符。

下面给你两种解决思路：

方案1：修改KeyPress逻辑，计算输入后的完整文本 #

直接在KeyPress里构建出输入后的完整字符串，再判断是否超过maxBroj，同时用TryParse避免异常：

把你原来的maxBroj判断代码替换成下面这段：

// 先处理空文本的情况：如果输入的是数字，先放行，后续判断完整值
if (string.IsNullOrWhiteSpace(textBox1.Text))
{
    if (char.IsDigit(e.KeyChar))
        return;
}

// 构建输入后的完整文本
string newInput = textBox1.Text + e.KeyChar;
// 用TryParse避免非数字输入的异常（虽然你已经拦截了非数字，但保险起见）
if (maxBroj != -1 && double.TryParse(newInput, out double newValue) && newValue > maxBroj)
{
    e.Handled = true;
    return;
}

另外，你原来的标点符号判断逻辑有个优先级问题，应该给||的部分加括号，避免逻辑错误：

// 原来的代码
// if (!znakovi && char.IsPunctuation(e.KeyChar) || !znakovi && char.IsSymbol(e.KeyChar))
// 修改后
if (!znakovi && (char.IsPunctuation(e.KeyChar) || char.IsSymbol(e.KeyChar)))

方案2：使用Validating事件（更推荐） #

KeyPress适合拦截单个字符，但数值范围这种需要验证完整输入的场景，用Validating事件更靠谱——它在控件失去焦点时触发，能直接验证完整的输入值，还能给用户友好提示：

第一步：添加Validating事件处理方法 #

private void textBox1_Validating(object sender, CancelEventArgs e)
{
    // 空文本的话可以根据需求处理，这里允许空（如果不允许可以加判断）
    if (string.IsNullOrWhiteSpace(textBox1.Text))
        return;

    if (maxBroj != -1 && double.TryParse(textBox1.Text, out double inputValue))
    {
        if (inputValue > maxBroj)
        {
            MessageBox.Show($"输入值不能超过 {maxBroj}", "输入错误");
            e.Cancel = true; // 阻止文本框失去焦点，强制用户修改
            textBox1.SelectAll(); // 选中全部文本，方便用户直接修改
        }
    }
    else
    {
        MessageBox.Show("请输入有效的数字", "输入错误");
        e.Cancel = true;
        textBox1.SelectAll();
    }
}

第二步：绑定事件 #

在窗体的构造函数里添加事件绑定（或者直接在设计器里双击Validating事件）：

public MessageBoxWithValue(string naslov, string opis)
{
    InitializeComponent();
    this.Text = naslov;
    label1.Text = opis;
    // 绑定Validating事件
    textBox1.Validating += textBox1_Validating;
}

这种方法的好处是，用户可以自由输入，最后再验证，体验更流畅，而且能处理各种边缘情况（比如粘贴进来的数值）。

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