我的 CS50 mario 不太舒服的代码有什么问题？

Question

这是我的代码的副本：

#include <cs50.h>
#include <stdio.h>

int main(void)
{
    int height = get_int("Please enter a height between 1 and 8 (inclusive)\nHeight: ");

    if(0 < height && height < 9)
    for(int i = 0; i <= height; i++)
    {
        for(int s = (height - 1); s >= 1; s--)
        {
            printf("a");
        }

        for(int h = 1; h <= i; h++)
        {
            printf("#");
        }

        printf("\n");
    }

    else
    printf("Please try again.\n"),
    main();
}

我似乎无法正确打印的部分是打印正确数量的空格，因为它似乎没有在每个循环之后减少变量“s”。 （我已经用字母“a”替换了空格，这样我就可以看到代码哪里出错了。）代码的目的是打印一个右对齐的#金字塔。 我在这里哪里出错了？

例如，如果用户输入高度 3，则 output 将是：

aa

aa#

aa##

aa###

但是，我希望 output 是：

aa#

a##

###

Answer 1

主要问题是打印“a”时代码中的第二个 for 循环。 每次你在这个特定的循环中，你基本上打印（高度 - 1）乘以“a”。 事实上，你应该每次少一个“a”（多一个哈希）就打印出来。 为此，我们需要调整 s 的条件。 而不是这个

int s = height - 1; s >= 1

用这个

int s = height - i; s > 1

由于您的主循环有迭代器i ，我们可以在这里使用它。

第二个问题是，根据 CS50 问题集描述，您实际上应该打印" " （空格），而不是"a" 。 为此，您应该将"a"替换为" " 。

第三个问题是您的代码在开头打印了一个充满空间的额外行。 您需要使用此条件调整主 for 循环以删除额外的一行。

i < height

在打印#符号的 for 循环中，您需要从零开始迭代器 h，以便它立即打印第一行。

int h = 0

Answer 2

更好的解决方案是使用已经存在的printf的内置功能，将问题简化为一个简单的循环。

#include <stdio.h>

int main(void) {
    int height = get_int("Please enter a height between 1 and 8 (inclusive)\nHeight: ");

    unsigned char blocks[height];
    memset(blocks, '#', height);

    for(int i=0; i<height; ++i)
    {
        printf("%*.*s\n", height, i+1, blocks);
    }
    return 0;
}

Output

Success #stdin #stdout 0s 4212KB
    #
   ##
  ###
 ####
#####