函数调用的参数过多 "c"

Question

我正在尝试使用一个函数来打印一定数量的字符。字符的数量取决于用户提供的整数和for循环。

int get_height(void);
string printsp(void);
string printhash(void);

int main(void)
{
    //running function for getting height
    int h = get_height();

    //setting up the column numer i
    for (int i = 0; i < h; i++)
    {
        printsp(h - 1 - i);
        printhash(i + 1);
        printsp(2);
        printhash(i + 1);

        printf("\n");
    }
}    

//prompting for integer within parameters
int get_height(void)
{
    int h;
    do
    {
        h = get_int("Height: ");
    }
    //Keeping i betwwen 1 and 8 inclusive
    while (h > 8 || h < 1);
    return h;
}

//printing the spaces
string printsp(void)
{
    printf(" ");
}

//printing the hashes
string printhash(void)
{
    printf("#");
}

我一直了解到函数调用中的参数太多。我是新手。

Answer 1

printsp(void)表示这个函数不接受任何参数。

与 printhash(void) 相同。

for (int i = 0; i < h; i++) {
    printsp();
    printhash();
    printsp();
    printhash();
    printf("\n");
}

将是正确的用法。

您的函数（printsp 和 printhash）也应该 return 一个 string，而不是什么都没有！

Answer 2

Too many arguments to function call “c”

你声明 printsp 和 printhash 没有参数，但你用参数调用它们，这是不一致的。

您需要一个参数并使用它，并且您不使用函数的值 return 所以最好有它们 void

你想要类似（最少的更改）声明

void printsp(int n)
void printhash(int n)

和（最小更改）定义：

//printing the spaces
void printsp(int n)
{
  while (n-- > 0)
    printf(" ");
}

//printing the hashes
void printhash(int n)
{
   while (n-- > 0)
    printf("#");
}

另外 printf 只是为了一个 char 很昂贵，你可以只使用 putchar，还有其他改进打印的可能性。

例如：

int get_height(void);
void printsp(int n);
void printhash(int n);

int main(void)
{
    //running function for getting height
    int h = get_height();

    //setting up the column numer i
    for (int i = 0; i < h; i++)
    {
        printsp(h - 1 - i);
        printhash(i + 1);
        printsp(2);
        printhash(i + 1);

        putchar('\n');
    }
}    

//prompting for integer within parameters
int get_height(void)
{
    int h;
    do
    {
        h = get_int("Height: ");
    }
    //Keeping i betwwen 1 and 8 inclusive
    while (h > 8 || h < 1);
    return h;
}

//printing the spaces
void printsp(int n)
{
  while (n-- > 0)
    putchar(' ');
}

//printing the hashes
void printhash(int n)
{
   while (n-- > 0)
    putchar('#');
}

根据这些定义，输入 5 的结果是：

pi@raspberrypi:/tmp $ ./a.out
    Height: 5
        #  #
       ##  ##
      ###  ###
     ####  ####
    #####  #####
    pi@raspberrypi:/tmp $

但是请注意 printsp 和 printhash 可以是一个独特的函数，也可以在参数中接收您要打印的字符

int get_height(void);
void printch(int n, char c);

int main(void)
{
    //running function for getting height
    int h = get_height();

    //setting up the column numer i
    for (int i = 0; i < h; i++)
    {
        printch(h - 1 - i, ' ');
        printch(i + 1, '#');
        printch(2, ' ');
        printch(i + 1, '#');

        putchar('\n');
    }
}    

//prompting for integer within parameters
int get_height(void)
{
    int h;
    do
    {
        h = get_int("Height: ");
    }
    //Keeping i betwwen 1 and 8 inclusive
    while (h > 8 || h < 1);
    return h;
}

//printing ch n times
void printch(int n, char ch)
{
   while (n-- > 0)
    putchar(ch);
}

函数调用的参数过多 "c"

Too many arguments to function call "c"

c

cs50