C#中变量类型检查、字符串分割及代码执行异常咨询

我明白你现在遇到的困扰啦，你的这段代码里有几个关键的逻辑问题，导致输入dsf/asdf时没有按预期工作，咱们一步步拆解并修正：

首先先把你写的代码贴出来方便分析：

public void divide(string args) { 
    if (args.Contains("/")) { 
        var a = args.Split('/')[0]; 
        var b = args.Split('/')[1]; 
        if (a is float && b is float) { 
            Console.WriteLine(a / b); 
        } else { 
            Console.WriteLine("Error, input isnt float"); 
        } 
    } 
}

核心问题分析 #

• 错误的类型检查逻辑：这里的a is float完全起不到判断字符串是否为浮点数的作用——因为a是string类型的变量，它永远不可能是float类型，所以这个判断条件永远为false。不管你输入的是有效浮点数还是像dsf这种无效内容，代码都会直接走到else分支，甚至连合法的浮点数输入也无法正确计算。
• 重复调用字符串分割：你连续两次调用args.Split('/')，不仅没必要，还会额外消耗性能，应该把分割结果存成一个数组复用。
• 未处理边界情况：如果输入的字符串包含多个/，或者分割后数组长度不足2，直接访问索引[1]会抛出索引越界异常。

修正后的代码 #

public void Divide(string args)
{
    // 先分割字符串并保存结果，避免重复计算
    string[] parts = args.Split('/');
    
    // 先判断分割后是否正好是两个元素，避免索引越界
    if (parts.Length == 2)
    {
        // 使用float.TryParse尝试转换，同时获取转换后的数值
        if (float.TryParse(parts[0], out float numA) && float.TryParse(parts[1], out float numB))
        {
            // 额外处理除零异常，防止程序崩溃
            if (numB != 0)
            {
                Console.WriteLine(numA / numB);
            }
            else
            {
                Console.WriteLine("Error, cannot divide by zero");
            }
        }
        else
        {
            Console.WriteLine("Error, input isn't float");
        }
    }
    else
    {
        Console.WriteLine("Error, input format should be [number]/[number]");
    }
}

修正点说明 #

1. 复用分割结果：把Split的结果存在parts数组里，减少不必要的计算开销。
2. 正确的类型转换判断：用float.TryParse来检查字符串能否转换为浮点数，这个方法会返回布尔值表示转换是否成功，同时通过out参数拿到转换后的浮点数，这才是判断字符串是否为合法浮点数的正确方式。
3. 完善边界处理：先判断分割后的数组长度是否为2，避免索引越界；同时增加了除零异常的判断，防止程序因为除以零直接崩溃。
4. 精准错误提示：不同错误场景返回不同的提示信息，方便快速定位问题。

现在你测试输入dsf/asdf，代码会正确输出Error, input isn't float；如果输入6.2/2，会输出3.1；输入5/0的话会提示除零错误，完全符合预期啦。

内容来源于stack exchange