如何正确使用malloc和strtok？

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

#define MAX_HEIGHT 5 
#define MAX_WIDTH 9 
#define MAX_DIRECT 30 

typedef struct position_t position_t; 

struct position_t { 
     char *position; 
     char *ptr; 
}; 

int main(int argc, char *argv[]) 
{ 
     int i; 
     FILE *fp; 
     char a[50], b[50], c[50], d[50]; 
     position_t pos; 

     pos.position = malloc(sizeof(char) * 20); 

     for (i = 1; i < argc; i++) { 
       fp = fopen(argv[i], "r"); 
       if (fp == NULL) { 
         fprintf(stderr, "cat: can't open %s\n", argv[i]); 
         continue; 
       } 

       fgets(a, 50, fp); 
       fgets(b, 50, fp); 
       fgets(c, 50, fp); 
       fgets(d, 50, fp); 

       fclose(fp); 

       while (1) { 
         int j = 0; 

         pos.position = 0; 
         pos.ptr = strtok(a, ",.; "); 

         while (pos.ptr != NULL) { 
           pos.position[j] = *pos.ptr; 
           j++; 
           pos.ptr = strtok(NULL, ",.; "); 
         } 

         printf("%c", pos.position[j]); 
       } 

     } 

     free(pos.position); 
     return 0; 
}

我想要做的是从文件中读取第一行（它的内容是：START FOYER ELEVATOR）并通过strtok将它们分隔空间，然后将每个字符串存储在malloc pos中。定位并在以后使用时使用。有人可以修复此代码吗？如何正确使用malloc和strtok？

来源

2017-04-20 유승기

我建议你阅读[关于铸造'malloc'返回的这个问题]（http://stackoverflow.com/questions/605845/do-i-cast-the-result-of-malloc）。 –

随着

pos.position = (char*)malloc(sizeof(char)*20);

你分配20个字节的内存，并使得pos.position指向该内存。

但后来你做

pos.position = 0;

这使得pos.position成为一个空指针。

指针复位这将导致两个内存泄漏，你失去的malloc返回的指针，这也将导致未定义行为当你提领与例如指针pos.position[j]。

此外，当你内在的while循环后做

printf("%c", pos.position[j]);

您打印尚未初始化内存的元素。

要使当前代码正常工作，请不要重置循环中的指针。在内部while循环后添加终止符。并将该数组作为字符串打印。哦，并添加一个条件，所以你不写出分配的内存的界限。

实际上，由于您总是分配固定数量的内存，我宁愿建议您不要动态分配内存。而是使pos.position为阵列。

来源

2017-04-20 14:22:13

如何正确使用malloc和strtok？

回答

相关问题