允许用户在数组函数中更改“5”的值

Question

正如我的标题所暗示的，我是一个正在玩数组的初学者。尽管我尽力而为，但我无法正确更改数组中的值？如您所见，数组中只有后5位正确，前3位不正确？为什么会这样？我将在下面发布我的代码，以便大家明白我的意思：

#include <stdio.h>
#include <stdlib.h>
#define MAX_ARRAY 8
void input_array(char anumber[MAX_ARRAY])
{
    printf("\n\nPlease insert new data to the 1st array with value 5: ");
    fgets(&anumber[0], MAX_ARRAY, stdin);     
    long ret = strtol(&anumber[0], NULL, 10); // Converts char to int
    printf("Converting char anumber = %d to int ret = %d\n", anumber[0], ret);
    printf("\n(Array after): ");
    for (int i = 0; i < MAX_ARRAY; ++i)
    {
        printf("(%d) ", anumber[i]);
    }
}
int main(void)
{
    char arr[MAX_ARRAY] = {5, 8, 2, 9, 1, 7, 4, 3};
    printf("(Array before): ");
    for (int i = 0; i < MAX_ARRAY; ++i)
    {
        printf("(%d) ", arr[i]);
    }
    input_array(arr); // Function that lets the user change value of "5" inside the array
    return 0;
}

如果我作为用户输入值“3”，此代码的输出是：

(Array before): (5) (8) (2) (9) (1) (7) (4) (3) 

Please insert new data to the 1st array with value 5: 3
Converting char anumber = 51 to int ret = 3

(Array after): (51) (10) (0) (9) (1) (7) (4) (3)

Answer 1

开场白：

线

long ret = strtol(&anumber[0], NULL, 10); // Converts char to int

错了。特别是// Converts char to int 的评论是错误的。函数strtol 不会将单个字符 转换为int。相反，它会将字符串（即字符序列）转换为long。

为了将单个数字字符转换为int，您可以改写以下行：

int number = anumber[0] - '0';

这是可能的，因为 ISO C 要求字符集按顺序存储数字 0 到 9。

您的程序中实际发生的情况如下：

您正在将数组的 8 个元素初始化为以下值：

(5) (8) (2) (9) (1) (7) (4) (3) 

然后，您使用fgets 输入来自用户的字符串"3\n"。对应的ASCII codes 是数字'3' 的51 和换行符'\n' 的10。您用这两个值覆盖数组的前两个元素，并用字符串的终止空字符覆盖第三个元素。

这就是为什么数组在调用fgets 后具有以下值的原因：

(51) (10) (0) (9) (1) (7) (4) (3)

fgets 函数会覆盖前 3 个元素，但保留其余元素不变。

解决问题的正确方法如下：

您似乎只想覆盖数组的第一个元素。因此，您不应将数组用作fgets 函数调用的直接目标，因为该函数将始终写入一个以上的字节（正确使用该函数时）。

解决问题的一个非常简单的方法如下：

void overwrite_first_element_with_input( char arr[MAX_ARRAY] )
{
    //prompt user for input
    printf( "Please enter new data for the first element of the array: " );

    //write input to first element
    if ( scanf( "%hhd", &arr[0] ) != 1 )
    {
        printf( "input error!\n" );
        exit( EXIT_FAILURE );
    }
}

完整的程序如下所示（有一些小的改进）：

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

#define MAX_ARRAY 8

void overwrite_first_element_with_input( char arr[MAX_ARRAY] )
{
    //prompt user for input
    printf( "Please enter new data for the first element of the array: " );

    //write input to first element
    if ( scanf( "%hhd", &arr[0] ) != 1 )
    {
        printf( "input error!\n" );
        exit( EXIT_FAILURE );
    }
}

void print_array( const char arr[MAX_ARRAY], const char *tag )
{
    printf( "%s", tag );

    for ( int i = 0; i < MAX_ARRAY; i++ )
    {
        printf( "(%d) ", arr[i] );
    }

    printf( "\n" );
}

int main( void )
{
    char arr[MAX_ARRAY] = {5, 8, 2, 9, 1, 7, 4, 3};

    print_array( arr, "(Array before): " );
    overwrite_first_element_with_input( arr );
    print_array( arr, "(Array after): " );

    return 0;
}

这是程序的一些示例输出：

(Array before): (5) (8) (2) (9) (1) (7) (4) (3) 
Please enter new data for the first element of the array: 20
(Array after): (20) (8) (2) (9) (1) (7) (4) (3) 

但是，我不想鼓励您使用scanf 代替fgets 和strtol，因为scanf 有many disadvantages。如果你想解决fgets和strtol的问题，我推荐以下代码：

void overwrite_first_element_with_input ( char arr[MAX_ARRAY] )
{
    char line[100], *p;
    long ret;

    //prompt user for input
    printf( "Please enter new data for the first element of the array: " );

    //attempt to read one line of input and verify success
    if (
        //verify that fgets was successful
        fgets( line, sizeof line, stdin ) == NULL

        ||

        //verify that input buffer was large enough to store entire line
        strchr( line, '\n' ) == NULL
    )
    {
        printf( "input error!\n" );
        exit( EXIT_FAILURE );
    }

    //attempt to convert input to number
    ret = strtol( line, &p, 10 );
    if ( p == line )
    {
        printf( "conversion failure!\n" );
        exit( EXIT_FAILURE );
    }

    //write the successfully converted number to the array
    arr[0] = ret;
}

请注意，您必须添加 #include <string.h> 才能使上述代码正常工作。

Answer 2

你好，我看到你试图将你的数字数组保存在 char 中，所以它可能会导致这种错误。为字符定义两个变量，为数字定义另一个变量。