星形金字塔生成异常排查：请求修复输入验证与循环逻辑错误

问题现象 #

• 输入不符合要求的正整数4（需输入正奇数）时，程序提示"Try again"，此逻辑正确；
• 重试输入正确的正奇数5后，生成的金字塔底部为4个星号，而非预期的5个。

原始代码 #

static void Main(string[] args)
{
    bool loop = true;
    do
    {
        Console.WriteLine("Enter the number which is an integer and a odd number");
        int userinput = Convert.ToInt32(Console.ReadLine());//converts the string into an integer 
        validation(userinput);//checks if the userinput conditions are met
        pyramid(userinput);//if conditions are meet userinput will be inserted into the PYRAMID parameters
        loop = false;
       
    } while (loop == true);

    Console.WriteLine("Enter the number which is an integer and a odd number again");
    int newuserinput = Convert.ToInt32(Console.ReadLine());//Asks the user to input another number
    validation(newuserinput);//checks if conditions are meet
    pyramid(newuserinput);//if meet put the new userinput into the parameters 
    static int validation(int a)
    {
        
        int counter = 0;
        while (counter == 0) //counter is used to loop to keep and asking and checking 
        {
            if (a > 0 && a % 2 != 0)//checks if the userinput is an odd number integer
            {
                return a;//returns the BOOL value true if it meets the conditions
                counter += 1;
            }
            else
            {
                Console.WriteLine("Try again ");//If it doesnt meet the conditions it will ask again
                int again = Convert.ToInt32(Console.ReadLine());
                if (again > 0 && again % 2 != 0)//checks if the condition is met again
                {
                   
                    return again;//returns true if conditions met 
                    counter += 1;
                }

            }
        }
        return 0;

    }

    static void pyramid(int number)
    {
        for (int i = 0; i < number; i++) //For loop to make sure the other for lopps run inside of it n number of times
        {
            for (int j = 0; j < number - i; j++) //the loop is used to fill in the blanks of the console and as the varible j keeps increaseing the number will also decrease
            {
                Console.Write(" ");
            }
            for (int k = 0; k <= i; k++)//outputs the astrix n amount of times the user inputs 
            {
                Console.Write("* ");

            }
            Console.ReadLine();//makes a space for each for loop looped
        }
    }
}

错误原因分析 #

1. Validation函数返回值未被接收 #

在Main函数中，调用validation(userinput)时，没有将函数返回的正确输入值赋值给userinput变量。例如：

• 第一次输入错误值4，validation函数提示重试后接收了正确值5，但这个5没有覆盖原来的userinput（仍然是4）；
• 后续调用pyramid(userinput)时，使用的还是最初的错误值4，导致金字塔底部星号数量为4，而非预期的5。

另外，validation函数内部逻辑存在冗余和缺陷：

• counter += 1语句写在return之后，永远不会被执行，counter变量完全无用；
• 如果重试输入的again仍然不符合要求，函数会陷入无限循环（因为a还是最初的错误值，下一次循环会再次进入else分支）。

2. 金字塔生成逻辑不符合预期 #

当前pyramid函数的星号生成逻辑会导致每行星号数为i+1（带空格），且空格数量计算不合理：

• 输入奇数5时，会生成5行，最后一行输出5个* （即5个带空格的星号），但视觉上的金字塔结构不标准；
• 若要生成底部为number个星号的标准金字塔，需要调整行数和每行星号数的逻辑。

修复后的代码 #

static void Main(string[] args)
{
    bool loop = true;
    do
    {
        Console.WriteLine("Enter the number which is an integer and a odd number");
        int userinput = Convert.ToInt32(Console.ReadLine());
        // 接收validation返回的正确值
        userinput = validation(userinput);
        pyramid(userinput);
        loop = false;
    } while (loop == true);

    Console.WriteLine("Enter the number which is an integer and a odd number again");
    int newuserinput = Convert.ToInt32(Console.ReadLine());
    // 接收validation返回的正确值
    newuserinput = validation(newuserinput);
    pyramid(newuserinput);
}

static int validation(int a)
{
    // 无限循环直到输入符合要求
    while (true)
    {
        if (a > 0 && a % 2 != 0)
        {
            return a;
        }
        Console.WriteLine("Try again ");
        // 将新输入赋值给a，下次循环验证新值
        a = Convert.ToInt32(Console.ReadLine());
    }
}

static void pyramid(int number)
{
    // 生成标准金字塔：行数为(number + 1)/2，每行星号数为2i+1
    for (int i = 0; i < (number + 1) / 2; i++)
    {
        // 计算左侧空格数，使金字塔居中
        int spaceCount = (number - (2 * i + 1)) / 2;
        for (int j = 0; j < spaceCount; j++)
        {
            Console.Write(" ");
        }
        // 输出当前行的星号
        for (int k = 0; k < 2 * i + 1; k++)
        {
            Console.Write("*");
        }
        Console.WriteLine();
    }
    // 等待用户输入后继续
    Console.WriteLine("\nPress Enter to continue...");
    Console.ReadLine();
}

修复说明 #

1. 接收Validation返回值：在Main中调用validation时，将返回值重新赋值给输入变量，确保后续使用的是经过验证的正确值；
2. 简化Validation逻辑：去掉无用的counter变量，改用while(true)循环，每次重试时将新输入赋值给a，确保循环验证的是最新输入；
3. 优化金字塔生成：调整为标准金字塔结构，输入奇数n时，生成(n+1)/2行，底部恰好有n个星号，且通过计算空格数使金字塔居中显示。

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