这只是调用gets的错误吗？

Question

我们接到了这个任务，想知道我们是否在解决方案中忽略了某些东西：

以下代码片段有问题。它会编译，但存在一个重要的语义问题，这将导致代码的行为与预期不同。确定问题和解决此问题的方法。

int main(void) 
{ 
    char * s = malloc(82 * sizeof(char)); 
    if(s==NULL) 
    {  
        perror("failed to allocate memory");  
        return EXIT_FAILURE; 
    } 
    fgets(s, sizeof(s), stdin); 
    if(s[strlen(s)-1] != '\n') 
    { 
        fprintf(stderr, "Error: you entered too much input"); 
        free(s); 
        return EXIT_FAILURE; 
    } 
    free(s); 
    return EXIT_SUCCESS; 
} 

我们认为错误在于fgets 函数，因为它会读取包括空终止字符在内的所有内容。所以我们相信有可能写一个尚未malloc'd 的角色。所以我们相信fget(s,sizeof(s)-1, stdin); 行将是解决办法。
我们是对的吗？还是有更多/其他错误？

Answer 1

sizeof( s ) sizeof( char * )，这不是你想要的。

Answer 2

sizeof(s) 很可能是 4 或 8。无论如何，它不会是 82。

Answer 3

这不是很长的代码，所以让我们全部获取：

1. 没有包含。你需要：

   #include <stdio.h> // perror fgets fprintf
   #include <stdlib.h> // malloc free EXIT_FAILURE EXIT_SUCCESS
   #include <string.h> // strlen
   

2. 避免使用魔法常数。如果你需要一些常量，在一个地方定义它（用一个有说服力的名字），然后总是引用它。

   #define MY_MAX_STRING_SIZE 82
   // Not length, that would be one less due to the terminator
   

3. 风格点：char 永远是一个 char 大。

   char * s = malloc(MY_MAX_STRING_SIZE);
   

4. 始终检查fgets 的返回值。如果失败，则为 NULL。

5. sizeof-operator 返回其操作数的大小，而不是它可能指向的缓冲区的大小。使用上面的常量：

   if(!fgets(s, MY_MAX_STRING_SIZE, stdin)) {
       perror("failed to read from stdin");
       free(s);
       return EXIT_FAILURE; 
   };
   

6. 这三行相当于结束main：

   return EXIT_SUCCESS;
   return 0;
   // C99+
   

7. 为什么要使用动态分配？自动数组会更安全、更简单且性能更高：

   // includes...
   int main() {
       char s[82];
       if(!fgets(s, sizeof(s), stdin)) {
           perror("failed to read from stdin"); 
           return EXIT_FAILURE; 
       };
       if(s[strlen(s)-1] != '\n') 
       { 
           fprintf(stderr, "Error: you entered too much input"); 
           return EXIT_FAILURE; 
       } 
   }