在 C 非常基本的程序中编译问题

Question

以下是错误：

mario.c:23:25: error: expected identifier or ‘(’ before ‘int’
for (int levelCounter, int usrHeight, int paddIt = usrHeight - 1, char hashMaker, int hashCounter;(levelCounter <= usrHeight);)

                      ^


mario.c:25:39: error: expected ‘)’ before ‘paddIt’
printf("%'' * paddIt %c*hashCounter"paddIt,hashMaker,hashCounter);
                                    ^

^ 标识编译器指出错误发生的位置。

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

int main(void)
{
int usrHeight = 0;
int levelCounter = 0;
int paddIt = 0;
int hashCounter = 2;
char hashMaker = "#";

do 
{
    printf("How high?\n");
    int usrHeight = GetInt();
}
while ( usrHeight > 23 || usrHeight < 0);
if ( usrHeight >= 0 && usrHeight <= 23);
{
    printf("constructing...\n");

}
for (int levelCounter, int usrHeight, int paddIt = usrHeight - 1, char hashMaker, int hashCounter;(levelCounter <= usrHeight);)
                 *^ This is where the first error is occuring*
{
    printf("%'' * paddIt %c*hashCounter"paddIt,hashMaker,hashCounter);
                                                *^ And here is where the second is* 
    paddIt = paddIt - 1;
    levelCounter = levelCounter + 1;
    hashCounter = hashCounter + 1;
}
}

我到底做错了什么？

Answer 1

在 C99 及更高版本中，for 循环的初始位置只能有一个声明：

for (int levelCounter, usrHeight, paddIt = usrHeight - 1, hashMaker, hashCounter;
     ...condition...; ...increment...
    )

但是，您有问题； usrHeight 未初始化，因此 paddIt 是一个不确定的值。我不得不将hashMaker 的类型更改为int。您只能使用一种类型。

坦率地说，在循环控制中声明许多变量是错误的。如果你这样做，它们都应该被初始化。否则，您可以使用：

for (int levelCounter; ...condition...; ...increment...)
{
    int usrHeight = -1, paddIt = usrHeight - 1, hashCounter = 0;
    char hashMaker;
    ...loop body...
}

这解决了一些问题。如果您需要在循环的迭代中保留变量的值（这会在每次迭代中定义并初始化它们），那么应该在循环之外定义额外的变量。您很少会在for 循环中声明多个变量，甚至很少声明两个以上的变量。我几乎可以证明：

for (int i = min, j = max; i < j; i++, j--)

其中一个变量向上计数，而另一个变量向下计数，尽管您可以在循环体中计算 j，尤其是当 min 为 0 时。

向我提供（固定版本）原始代码以供代码审查的人会受到我的冷遇——更清楚地重写代码。

---

第二个问题是缺少逗号，编译器说它需要)：

printf("%'' * paddIt %c*hashCounter"paddIt,hashMaker,hashCounter);

应该是：

printf("%'' * paddIt %c*hashCounter", paddIt, hashMaker, hashCounter);

第一部分实际上是格式错误的转换规范；您可能指的是"%%''..."，或者您可能指的是完全不同的东西，例如：

printf("%' *.*c", paddIt, hashMaker, hashCounter);

单引号仍然与 %c 转换说明符无关（因此行为未定义），但它是 printf() 的 POSIX 2008 的一部分——它在必要时为大数字添加了数千个分隔符。

---

我没有注意到您已经声明了您在for 循环中提到的所有变量。看起来您可以将该循环简化为：

for (levelCounter = 0; levelCounter <= usrHeight; levelCounter++)
{
    printf("%.*s", paddIt, "");
    for (int i = 0; i < hashCounter; i++)
        putchar('#'); 
    paddIt = paddIt - 1;
    hashCounter = hashCounter + 1;
}

这使用来自原始问题中不清楚的预期格式的评论信息。

Answer 2

为什么不能将此 for 循环转换为 while 循环？这会做同样的事情

while(levelCounter <= usrHeight)

{
 printf("%'' * paddIt %c*hashCounter",paddIt,hashMaker,hashCounter);

 paddIt = paddIt - 1;
 levelCounter = levelCounter + 1;
 hashCounter = hashCounter + 1;
}