如何正確使用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？

回答

相關問題