如何让用户在 C 中输入多个输入？

Question

#include <stdio.h>

int main(){
    int num, counter, numberOfInputs, even, odd, sumEven, sumOdd;
    
    printf("Enter number of inputs:\n ");
    scanf("%d", &numberOfInputs);
    
    printf("Enter %d numbers: ", numberOfInputs);
    for(counter=1; numberOfInputs>counter; counter++){
        scanf("%d", counter);

    return 0;
}

我有一个项目可以让用户输入一些输入。我似乎无法解决它。我完成了程序的其他部分，它将确定奇数和偶数的数量关系，我只是没有包括它。 请帮助和tnx

后续问题：

为什么我用printf显示输入的数字，最后输入的数字总是加1？

#include <stdio.h>

int main(){
    int num, counter, numberOfInputs, even, odd, sumEven, sumOdd;
    
    printf("Enter number of inputs:");
    scanf("%d", &numberOfInputs);
    
    printf("Enter %d numbers:\n", numberOfInputs);
    for(counter=0; numberOfInputs>=counter;){
        scanf("%d", &counter);
        counter++;
    }
    /*when I put a printf here, the last number is added by 1, it works fine without the printf though*/

Answer 1

回答原问题：

线

scanf("%d", counter);

错了，有几个原因：

%dscanf 转换格式说明符需要int 的地址才能写入（即指针）。但是，您没有传递这样的地址。相反，您将循环计数器的值传递给它，这没有意义。

如果您改为将行更改为

scanf( "%d", &counter );

然后将满足%d 转换格式说明符的要求。但这仍然没有意义，因为这会导致循环计数器被用户输入覆盖。

另一个问题是行

for(counter=1; numberOfInputs>counter; counter++){

错了，因为这个循环只会进行numberOfInputs - 1 迭代，尽管您可能想要进行numberOfInputs 迭代。

此外，检查scanf 的返回值通常总是一个好主意，如果它无法匹配输入，则提供错误消息。

在您的问题中，您没有指定是否要这样做

• 记住用户输入的所有输入，或者
• 在处理完所有输入后丢弃它们。

但是，根据代码中的变量名称 sumEven、sumOdd、even 和 odd，我假设您只想计算所有偶数和奇数的总和，并且偶数和奇数的个数。在这种情况下，您不必记住所有输入；它们可以在处理后丢弃。在这种情况下，您的代码应如下所示：

#include <stdio.h>
#include <stdlib.h>

int main( void )
{
    int numberOfInputs, numEven = 0, numOdd = 0, sumEven = 0, sumOdd = 0;
    
    printf( "Enter number of inputs: " );
    if ( scanf( "%d", &numberOfInputs ) != 1 )
    {
        printf( "input failure!" );
        exit( EXIT_FAILURE );
    }
    
    printf( "Enter %d numbers: ", numberOfInputs );

    for( int counter = 0; counter < numberOfInputs; counter++ )
    {
        int num;
        if ( scanf( "%d", &num ) != 1 )
        {
            printf( "input failure!" );
            exit( EXIT_FAILURE );
        }

        //process input

        //determine whether number is odd or even
        if ( num % 2 == 0 )
        {
            numEven++;
            sumEven += num;
        }
        else
        {
            numOdd++;
            sumOdd += num;
        }
    }

    printf( "\n" );
    printf( "Number of even numbers: %d\n", numEven );
    printf( "Number of  odd numbers: %d\n", numOdd  );
    printf( "\n" );
    printf( "Sum of even numbers: %d\n", sumEven );
    printf( "Sum of  odd numbers: %d\n", sumOdd );

    return 0;
}

这个程序有以下输出：

Enter number of inputs: 5
Enter 5 numbers: 3 5 6 7 8

Number of even numbers: 2
Number of  odd numbers: 3

Sum of even numbers: 14
Sum of  odd numbers: 15

回答后续问题：

以下循环将执行 numberOfInputs + 1 迭代，尽管您可能只希望它执行 numberOfInput 迭代。

for(counter=0; numberOfInputs>=counter;){
    scanf("%d", &counter);
    counter++;
}

要解决此问题，您可以将其更改为以下内容：

for ( counter = 0; counter < numberOfInputs; counter++ ) {
    scanf("%d", &counter);
}

但是，正如我对原始问题的回答中所述，用用户输入覆盖循环计数器是没有意义的。

Answer 2

您发布的代码有几个问题。

第一个scanf("%d", counter) 是错误的。 %d 格式说明符需要一个指向 int (int*) 的指针，而您提供的是 int。逻辑也是错误的，因为您覆盖了counter。

此外，您的循环将比numberOfInputs 少执行一次，因为您从1 开始计数，所以它应该是numberOfInputs>=counter。

最后，您应该检查scanf 的返回值，看看它是否成功（尝试输入除整数以外的其他内容）。

    for(counter=1; numberOfInputs>=counter; counter++){
        if (scanf("%d", &num) != 1)
             /* handle error */
     }

启用编译器警告也是一个好主意，如果您这样做了，它会检测到scanf 类型冲突。像-Wall 这样的开关将是一个好的开始。