C fgets问题

Question

struct DVDInfo  *ReadStruct( void ) {
    struct DVDInfo  *infoPtr;
    int             num;
    char            line[ kMaxLineLength ];
    char            *result;

    infoPtr = malloc( sizeof( struct DVDInfo ) );

    if ( NULL == infoPtr ) {
        printf( "Out of memory!!!  Goodbye!\n" );
        exit( 0 );
    }

    printf( "Enter DVD Title:  " );
    result = fgets( line, kMaxLineLength, stdin );
    line[ strlen( line ) - 1 ] = '\0';
    infoPtr->title = MallocAndCopy( line );

    printf( "Enter DVD comment:  " );
    result = fgets( line, kMaxLineLength, stdin );
    line[ strlen( line ) - 1 ] = '\0';
    infoPtr->comment = MallocAndCopy( line );

    do {
        printf( "Enter DVD Rating (1-10):  " );
        scanf( "%d", &num );
        Flush();
    }
    while ( ( num < 1 ) || ( num > 10 ) );

    infoPtr->rating = num;

    printf( "\n----------\n" );

    return( infoPtr );
}

上面的变量“结果”的目的是什么？没有用它做任何事情。从 fgets 返回的指针被存储到其中，但仅此而已，它没有任何用途。

Answer 1

您应该测试该结果是否为 NULL，以检查 EOF 条件或错误，而不是仅仅忽略它。另外，如果不检查结果，您将在线执行 strlen，这可能包含未初始化的数据，因为 fgets 失败。实际上，您应该在 fgets 之后：

if (!result)
{
  free(infoPtr); // To not leak the object allocated at the start
  return NULL; // Function failed
}

如果第一个 fgets 成功而第二个 fgets 失败，您可能仍然会出现泄漏，因为对结构的指针成员进行了额外的分配。不幸的是，由于该结构未初始化为零，因此您无法检查这些指针是否为 NULL。因此，也许使用 calloc 而不是 malloc 或至少将所有结构指针成员初始化为 NULL，会是一个更好的主意。

Answer 2

似乎有人开始实施错误检查，但最终却搞砸了。返回值应与NULL进行比较，如果相等则报告错误。

Answer 3

最有可能的是，编译器发出了有关被忽略的函数返回值的警告。程序员并不关心fgets的返回值，只需在result=中添加即可让编译器不再纠缠它。正确的解决方案是检查返回值以确保函数成功完成。