4位数字验证程序重复输入全相同数字时失效问题求助

核心问题分析 #

• 外层循环条件逻辑错误
  原程序外层while循环条件为counter !=4 && counter2 >3，当第一次输入全相同数字（如1111）后，counter会被计算为4，counter2会被设为4。此时用户重新输入新的全相同数字（如2222），回到循环时counter !=4为false，整个逻辑与（&&）条件不成立，循环直接退出，完全跳过了对新输入数字的验证流程。

• 关键变量未初始化/重置

  • counter变量声明时未初始化，初始值为随机内存值，可能导致第一次循环的位数判断异常。
  • 用户重新输入数字后，counter未被重置为0，导致后续位数检查无法正确统计新数字的位数。

• 重复数字检查逻辑冗余易出错
  原代码用两个数组+标记删除的方式判断全相同数字，逻辑复杂且容易引发状态混乱，完全可以用更简洁的方式实现。

修正后的代码 #

#include <iostream>
using namespace std;

int main(){
    int n;
    cout << "enter a 4-digit number with at least one different digit: ";
    
    while(true){
        cin >> n;
        
        // 检查是否为4位数
        if(n < 1000 || n > 9999){
            cout << "the number has less than 4 digits, enter a valid number: ";
            continue;
        }
        
        // 检查是否所有数字相同
        int firstDigit = n / 1000;
        int digit2 = (n / 100) % 10;
        int digit3 = (n / 10) % 10;
        int digit4 = n % 10;
        
        if(firstDigit == digit2 && firstDigit == digit3 && firstDigit == digit4){
            cout << "the number has all its digits the same, enter a valid number: ";
            continue;
        }
        
        // 验证通过，退出循环
        break;
    }
    
    cout << "valid number";
    return 0;
}

修复说明 #

• 简化循环结构：用while(true)作为外层循环，直到输入符合要求才break，逻辑更清晰，彻底避免原条件判断的漏洞。
• 4位数检查优化：直接通过数值范围（1000 ≤ n ≤ 9999）判断，比统计位数的方式更高效准确。
• 全相同数字检查简化：直接提取每一位数字与第一位对比，逻辑简单直观，避免原数组操作引发的错误。
• 变量状态自动重置：每次循环重新处理输入的n，所有判断基于当前输入，无需额外手动重置中间变量。

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