有人可以告诉我为什么我在这个简单的 C 程序中出现段错误吗？

Question

在我结束第一个 for 循环后，我不断出现 seg 错误，而且我一生都不知道为什么。 我正在扫描的文件只有 18 行 18 个字符串。 我认为问题在于我分配称为 picks 的双指针的方式，但我不知道确切的原因。 我只是尝试扫描长度小于 15 个字符的字符串，所以我没有看到问题。有人可以帮忙吗？

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAX_LENGTH 100

int main( int argc,char *argv[] )
{

   char* string = malloc( 15*sizeof(char) );
   char** picks = malloc(15*sizeof(char*));
   FILE* pick_file = fopen( argv[l], "r" );
   int num_picks;


   for( num_picks=0 ; fgets( string, MAX_LENGTH, pick_file ) != NULL ; num_picks++ )
     {
       scanf( "%s", picks+num_picks );
     }
   //this is where i seg fault
   int x;
   for(x=0; x<num_picks;x++)
     printf("s\n", picks+x);
}

Answer 1

picks 是一个指向指针的指针：这意味着它指向的东西本身就是指针。

当你这样做时：

char** picks = malloc(15*sizeof(char*));

你正在使 picks 指向一个由 15 个指针组成的块 - 就其本身而言，这很好（尽管如此，因为你想读取 18 行，这里确实需要 18，而不是 15)。这意味着 picks 指向内存中的一个变量块，如下所示：

| picks (char **) | --------> | picks[0] (char *)  | ----> ?
                              | picks[1] (char *)  | ----> ?
                              | picks[2] (char *)  | ----> ?
                              | ...                |
                              | picks[14] (char *) | ----> ?

如您所见，这 15 个 char * 指针现在未初始化 - 它们不指向任何东西。在为它们分配一些内存之前，您无法开始使用它们 - 您需要执行以下操作：

int i;
for (i = 0; i < 15; i++)
{
    picks[i] = malloc(15);
}

现在，之后，内存布局如下所示：

| picks (char **) | --------> | picks[0] (char *)  | ----> | picks[0][0] (char)  |
                                                           | picks[0][1] (char)  |
                                                           | ...                 |
                                                           | picks[0][14] (char) |

                              | picks[1] (char *)  | ----> | picks[1][0] (char)  |
                                                           | ...                 |

                              | ...                |

...现在您有地方可以存储您想要读取的所有字符。

Answer 2

1. string 仅分配足够的内存来存储一个字符 (sizeof(char))。如果要存储更多字符，则需要将 sizeof(char) 乘以要存储的字符串的大小，并为末尾的 null 值加一。

2. 而不是：

   char** picks = malloc(15*sizeof(char));
   

   您想要执行此操作：

   char** picks = malloc(15*sizeof(char*));
   

   picks 数组的每个元素都需要足够大以容纳一个指针。

Answer 3

首先，在 C 中，字符串存储为字节（char）数组，并且必须进行分配。在这种情况下，用于读取文件的字符串应分配 MAX_LENGTH+1（+1 表示字符串终止符，\0）个字符：

char* string = malloc( (MAX_LENGTH+1) * sizeof(char) );

这将为最大长度的字符串分配足够的内存：MAX_LENGTH。

另一个问题是指针数组 char **picks 没有分配来存储您期望读取的 18 个字符串：

它必须分配给 15 个字符指针 (char *)，这也必须在第一个循环中分配。

int main( int argc,char *argv[] )
{
   ...
   char* string = malloc( (MAX_LENGTH+1) * sizeof(char) );
   char** picks = malloc(15*sizeof(char *));
   FILE* pick_file = fopen( argv[l], "r" );
   int num_picks;

   for( num_picks=0 ; fgets( string, MAX_LENGTH, pick_file ) != NULL ; num_picks++ )
     {
       printf("pick a/an %s ", string );
       //--- allocate the char array to store the current string
       picks[num_picks] = malloc (15 * sizeof(char));
       sscanf(string, "%s", picks[num_picks] );
     }

   for(int x=0; x<num_picks;x++)
     printf("%s\n", picks[x]);
}

您还必须测试 malloc() 返回值，并且您可能想测试文件内容是否确实符合预期并且不包含更多行或长度超过 15 个字符的行。

---

另外 scanf() 读取标准输入，我将其替换为 sscanf()，并在第二个 printf() 中添加了缺少的“%”符号。

Answer 4

• string 指向的位置仅分配了 一个 个字符的空间，但您尝试最多读取 MAX_LENGTH 个字符；
• picks 指向的位置只为 15 个 char * 指针分配了空间，但您显然希望在那里存储指向 18 个字符串的指针；
• picks 指向的位置已分配但从未初始化。在将这 15 个（或 18 个）指针交给 scanf 之前，您需要使它们实际指向某些东西本身。

在这种情况下，实际上根本不需要动态分配 - 您可以使用数组执行您想要的操作：

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

#define MAX_LENGTH 100
#define MAX_LINES 18

int main(int argc, char *argv[])
{
    char string[MAX_LENGTH];
    char picks[MAX_LINES][MAX_LENGTH];
    FILE *pick_file = NULL;
    int num_picks;

    if (argc > 1)
        pick_file = fopen(argv[1], "r");

    if (pick_file == NULL)
        return 1;

    for (num_picks = 0; num_picks < MAX_LINES && fgets(string, MAX_LENGTH, pick_file) != NULL; num_picks++)
    {
        printf("pick a/an %s ", string);
        scanf("%s", picks[num_picks]);
    }

    int x;
    for (x = 0; x < num_picks; x++)
        printf("%s\n", picks[x]);

    return 0;
}

Answer 5

是因为在该行：
for( num_picks=0 ; fgets( string, MAX_LENGTH, pick_file ) != NULL ; num_picks++ )
MAX_LENGTH 是 100，但您正在读入变量 string，它只能保存单个字符？

Answer 6

您正在为 picks 分配一个包含 15 个 char* 的数组：

char** picks = malloc(15*sizeof(char*));

现在 picks 有足够的空间来存储 15 个 char* ，但您实际上从未在其中放置任何指向应包含读取字符的内存的指针。

您需要为将使用 scanf 读取的这些字符串分配内存。目前，scanf 只是将数据写入 picks 中的指针（尚未初始化为指向任何有用的位置）所指向的位置。

Answer 7

你在这里遇到了段错误。

scanf( "%s", picks+num_picks );

而是这样做，

for( num_picks=0 ; fgets( string, MAX_LENGTH, pick_file ) != NULL ; num_picks++ )
 {
   pics[num_picks] = (char* )malloc(100);
   scanf( "%s", picks+num_picks );
 }

问题是，你分配了 **pics 来容纳 15 个字符串，但你读取的字符串没有为这些字符串分配空间。确保您始终读取分配的指针。