WinForms中累加TextBox值至SubTotal时提示输入格式错误的问题排查

咱们来一步步拆解你遇到的问题哈～你碰到的这个错误，核心原因是直接用Convert.ToInt16转换字符串时，遇到了非数字内容或者空字符串，再加上代码里有个多余的判断逻辑，咱们逐个解决：

错误根源分析 #

1. 空值/无效字符串转换失败：当textBoxSubTotal是空的时候，Convert.ToInt16(textBoxSubTotal.Text)会直接抛出“输入字符串格式不正确”的异常——因为空字符串没法转成短整型；另外如果textBoxTotal里输入了非数字内容（比如空格、字母），转换也会炸。
2. 多余的判断逻辑：你写的if (textBoxSubTotal.Text == "" || textBoxSubTotal.Text != "")其实是个永远为true的条件，完全没必要存在，反而让代码逻辑变乱。

修正后的代码方案 #

用TryParse方法来做安全的数值转换，它会尝试解析字符串，失败时不会抛出异常，还能帮你验证输入是否有效：

// 先处理ListView添加项的逻辑，给下拉框加个空值保护
string[] arr = new string[4]; 
arr[0] = comboBox1.SelectedItem?.ToString() ?? ""; // 防止下拉框未选中时出现空引用
arr[1] = textBox1Price.Text; 
arr[2] = textBox2Qty.Text; 
arr[3] = textBoxTotal.Text; 
ListViewItem item = new ListViewItem(arr); 
listView1.Items.Add(item);

// 核心：安全计算SubTotal总和
short currentTotal;
short currentSubTotal = 0;

// 先验证当前Total输入是否有效
if (short.TryParse(textBoxTotal.Text, out currentTotal))
{
    // 解析现有SubTotal，空值或无效内容时自动用0
    short.TryParse(textBoxSubTotal.Text, out currentSubTotal);
    
    // 计算总和并赋值
    textBoxSubTotal.Text = (currentTotal + currentSubTotal).ToString();
}
else
{
    // 可选：给用户提示输入无效
    MessageBox.Show("请输入有效的Total数值哦！");
}

关键优化点说明 #

• short.TryParse的优势：它会返回布尔值表示解析是否成功，成功就把数值放到out参数里，失败则不会抛出异常，完美解决你遇到的格式错误问题。
• 空值保护：给comboBox1.SelectedItem加了?.ToString() ?? ""，避免下拉框未选中任何项时调用ToString()出现空引用异常。
• 简化逻辑：去掉了多余的if判断，代码更清晰易懂。

额外建议 #

1. 输入限制：可以给textBoxTotal和textBoxSubTotal添加KeyPress事件，只允许输入数字，从源头避免无效输入：

   private void textBoxTotal_KeyPress(object sender, KeyPressEventArgs e)
   {
       // 只允许数字和退格键
       if (!char.IsDigit(e.KeyChar) && e.KeyChar != (char)Keys.Back)
       {
           e.Handled = true;
       }
   }
   

2. 数值类型选择：如果你的Total/SubTotal数值可能超过short的范围（最大32767），或者涉及价格计算，建议改用decimal（更精准）或者int，对应换成decimal.TryParse/int.TryParse即可。

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