C#密码检查器校验失败无法循环重输的问题求解

「反复校验直到输入合法」的通用实现逻辑 #

这类交互的执行逻辑是固定的，三个关键要求缺一个都会导致循环失效：

• 输入获取动作必须落在循环内部：所有需要重复执行的操作（获取用户新输入、校验输入、返回错误提示），都要放在循环体的作用域内，不能把获取输入的动作放在循环外只执行一次。如果只在循环外读一次输入，整个循环过程中传给校验方法的永远是第一次读到的旧值，哪怕循环执行多次，校验用的字符串从来没有更新过。
• 读取的输入必须回存到变量：Console.ReadLine()的作用是把用户输入的字符串作为返回值传回，如果调用这个方法但不把返回值赋值给存储输入的变量，相当于读取了无效内容，用户的输入根本无法传递到校验逻辑中。
• 同一份输入只做一次校验：不能对同一个输入重复调用多次校验方法，不然同样的错误提示会重复打印两次，也会让执行逻辑混乱。

现有代码的具体错位位置 #

对照上面的逻辑要求，当前代码有三个明显的结构问题：

• 第一，把string chosenPass = Console.ReadLine()写在了do循环的外面，这个变量从第一次赋值之后，整个循环执行过程中从来没有被更新过。
• 第二，do循环体里两次调用Console.ReadLine()，但从来没有把读到的内容赋值给chosenPass，相当于用户在循环里输入的所有内容都被直接丢弃了，根本没有进入校验流程。
• 第三，在循环体里调用了一次IsPassCorrect(chosenPass)输出提示，在do-while的循环条件里又对同一个旧值调用了一次同样的校验方法，等于同一份输入的校验结果会计算两次，错误提示会重复打印两遍。

额外的校验逻辑矛盾提醒 #

当前代码的校验规则存在两处完全冲突的逻辑：

• 前面定义的要求是密码“必须包含至少一个数字”，对应判断逻辑是如果没有数字就报错。
• 后面又写了判断“如果密码包含数字就报错”，提示文案是“包含小数”。这两个判断对同一份带合法数字的密码，会前后矛盾触发报错。从你写的提示文案“不允许包含小数”来看，你本来想判断的应该是密码里有没有小数点.，而不是判断有没有数字字符——char.IsDigit的作用就是判断字符是不是0-9的数字，和“小数/小数点”完全不是一回事。

正确的流程结构参考 #

按照之前讲的原理调整代码结构，每一步的位置对应固定逻辑，不需要额外堆砌循环：

1. 最开始只打印一次固定的密码要求提示文本，这类不变的内容不需要放在循环里重复执行。
2. do循环的起点设置在第一次获取用户输入之前，循环内部按顺序执行：
   • 第一步：执行chosenPass = Console.ReadLine()，每次进入循环都重新读取用户的新输入，存到chosenPass变量中。
   • 第二步：只调用一次IsPassCorrect(chosenPass)，把返回的bool值存到临时变量里，比如bool isLegal = IsPassCorrect(chosenPass)。
   • 循环的终止条件设置为：只要isLegal是false，就继续循环重新获取输入；如果是true，就退出循环进入后续流程。
   • 不需要在循环里额外添加单独的Console.ReadLine()，因为读取输入的动作已经在循环第一步完成了。

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