C# 使用Console.ReadKey递归校验字符输入返回异常结果问题排查

• 递归返回值丢失：你在else分支中调用ValidateInputChars递归方法后，没有将该调用的返回值作为当前函数的返回结果，而是固定返回字符'z'。即使深层递归已经匹配到合法输入返回了'c'/'r'，所有上层递归都会忽略这个正确结果，最终外层调用只能拿到最上层返回的'z'，你观察到返回'w'的现象，大概率是外层调用处额外使用了之前缓存的输入变量导致的。
• 实现方案不合理：输入校验场景属于典型的循环重试场景，使用递归实现会产生不必要的栈开销，极端情况下如果用户多次输入错误还会触发栈溢出异常。

方案1：修复现有递归逻辑（不推荐） #

仅需修改else分支的返回逻辑，将递归调用的结果直接返回即可，同时可以优化返回值写法，不要写死返回'c'/'r'，避免后续修改合法入参时逻辑错误：

public char ValidateInputChars(char actualInput, char input1, char input2)
{
    if (char.Equals(actualInput, input1))
    {
        return input1;
    }
    else if (char.Equals(actualInput, input2))
    {
        return input2;
    }
    else
    {
        Console.WriteLine("\n You pressed " + actualInput + "\nYour input is incorrect, kindly press " + input1 + " or " + input2 + " to proceed");
        actualInput = Console.ReadKey().KeyChar;
        // 核心修复：返回递归调用的结果，删除原有的固定return 'z'逻辑
        return ValidateInputChars(actualInput, input1, input2);
    }
}

方案2：改用循环实现（推荐） #

用循环重试替代递归，性能更稳定，无栈溢出风险：

public char ValidateInputChars(char actualInput, char input1, char input2)
{
    // 输入不合法就循环要求重新输入
    while (actualInput != input1 && actualInput != input2)
    {
        Console.WriteLine($"\n You pressed {actualInput}\nYour input is incorrect, kindly press {input1} or {input2} to proceed");
        actualInput = Console.ReadKey().KeyChar;
    }
    return actualInput;
}

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