用char指针分配

Question

我是初学者。我有一个包含MONSTER，ERRTU，14,2这样的行的文件。 当我尝试读取这些行并将其存储到动态内存分配的数组中时。 似乎一开始它就工作了，但是后来当我尝试打印它的元素时，它不能正常工作。这是关于使用char *吗？ 我该如何解决这个问题？

这是我的代码；

char *lines_ptr;
line_ptr=(char *)malloc(line*(sizeof(char)*100));   
int i=0;

if (fptr==NULL){

    printf("file could not be opened.");
}
else{ 

//line=number of lines
while(!feof(fptr)){
    for(i=0;i<line;i++){

        fscanf(fptr,"%s",lines_ptr+i);

        printf("%s\n",(lines_ptr+i));
    }
}

printf("%s",(lines_ptr));//this part shows me i did something wrong.

}  

这是我的输出；

   HERO,DRIZZT,8,3
   HERO,CATTIE,6,3
   HERO,BRUENOR,10,1
   HERO,WULFGAR,12,4
   MONSTER,TROLL,4,3
   MONSTER,GOBLIN,1,3
   MONSTER,UNDEAD,1,1
   MONSTER,VERMIN,3,2
   MONSTER,MINDFLAYER,10,2
   MONSTER,ERRTU,14,2
   HHHHMMMMMMONSTER,ERRTU,14,2 

为什么会发生？

Answer 1

 HHHHMMMMMMONSTER,ERRTU,14,2 //why does it happen? 

您要做的如下。

您读到行缓冲区，但是每次将文本开头移动1字符。 因此，当您返回并打印缓冲区的开头时，您将获得所有先前读取的行的所有首字符以及最后读取的行。

Answer 2

您错误地使用了“ line_ptr”（以及许多其他内容……）。

据我了解，您似乎希望将整个文件加载到“ line_ptr”中，每个文件的行都在“ line_ptr [i]”中。

如果是这样，那么我建议您花些时间做真正简单的代码，然后再使其变得更复杂。

例如，不要在当前代码中使用malloc：无论如何，它是无用的。

假设您有一个最大为NB_LINE行的文件，每行最大为NB_MAX_CHAR字符。

#define NB_LINE_MAX 50
#define NB_CHAR_MAX 100

char   lines[NB_LINE_MAX][NB_CHAR_MAX];
size_t nbLines = 0;

for (nbLines = 0; fgets(lines[nbLines], NB_CHAR_MAX, fptr); ++nbLines) {
  // Nothing to do, fgets do it for us
}

if (!feof(fptr)) {
  // problem while reading :/
}


printf("Test :\");
for (size_t i = 0; i < nbLines; ++i) {
  printf("%d : %s", i, lines[i]);
}

此代码有效吗？ 如果是，请尝试先删除NB_LINE_MAX（然后在文件中可以包含很多行，而不仅仅是50）来改善它，然后再删除NB_CHAR_MAX（然后您可以有没有char限制的行）。

备注：

• sizeof（char）始终为1。 因此，您可以将其从malloc中删除。
• fscanf（fptr，“％s”，lines_ptr + i）很危险。 ％s会读取它想要的许多字符，因此，如果他读取500个字符，而lines_ptr只能容纳100个字符，它将写在不好的地方，并且您可能会发生SIGSEV运行时崩溃。
• while（！feof（fptr））已经说过另一句话，但要注意您如何使用feof。

Answer 3

您正在执行一些错误。首先请仔细查看图片中的内容。假设输入hello和world有两行，而第一个输入line_ptr从0开始，因此它将存储从0到4但在第二行中line_ptr增至1，因此它将存储world从1到5 ，如果有更多行，它将继续。 注意：始终检查scanf()或fscanf()的返回值

为了将行作为c中的输入，您应该使用像char *lineptr[MAXLINE];这样的指针数组char *lineptr[MAXLINE];

 int readline(char* lineptr[])
 {
      char line[1000];
      for(i=0;i<MAXLINE;i++){
           if(fscanf(stdin,"%s",line)){
              char *temp=malloc((strlen(line)+1)*sizeof(char));
              strcpy(temp,line);
              lineptr[i]=temp;  
           }
      }
      return i; 
 }

此函数将读取变量行中的line ，然后它将在temp为该行分配内存，而lineptr[i]将指向该新行。它返回读取的行数。

Answer 4

以下建议的代码：

1. 总是从文件中读取整行
2. 正确检查错误
3. 将读取的数据正确地放置在“ malloc”数组中
4. 从代码中消除混乱
5. 消除了在代码中使用“魔术”数字

现在建议的代码

    if ( !fptr )
    {
        perror( "file could not be opened." );
        exit( EXIT_FAILURE );
    }

    // implied else, fopen successful

    #define MAX_LINE_LEN 100

    //size_t line=number of lines
    char *line_ptr = malloc( line * MAX_LINE_LEN ));
    if( !line_ptr )
    {
        perror( "malloc failed" );
        exit( EXIT_FAILURE );
    }

    // implied else, malloc successful

    size_t i = 0;
    // Note: '%[^\n]' reads to end of line, but not the newline char
    //      AND appends a NUL byte to the end of the input
    //      '%*c' reads and discards the newline char
    // Note: using the '%*c' requires the last line in the file ends with a newline char
    //      otherwise the last call to `fscanf()` will 'hang' looking for the final newline
    while( i < lines && 1 == fscanf(fptr,"%[^\n]%*c", line_ptr+(MAX_LINE_LEN*i) ) )
    {
        printf( "%s\n", line_ptr+(MAX_LINE_LEN*i) );
        i++;
    }