下使用strtok_r分割功能創建 - 如Perl和awk

Question

我試圖寫我自己的split功能像perl和awk使用strtok_r功能在C它創建陣列，以及在數組元素的收益多少，我試過的東西，但我沒有正確理解動態內存分配的概念，請別人糾正我，也請注意。

我相信strdup照顧內存分配我是對嗎？

錯誤：段錯誤（核心轉儲）

這也是迄今爲止

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


int split(char * str, char **fields, const char *sep) 
{ 
    char *copy = strdup(str), *tmp, *word;  
    int count=0; 

    word = strtok_r(copy, sep, &tmp); 
    while(word) 
    { 
       word=strtok_r(NULL, sep, &tmp); 
     fields[count] = strdup(word); 
     count++;  
    } 

    return count; 
} 

int main() 
{ 
     char string[80]="a,b,c,d,e,f,g,h,i,j,k,l"; 
     const char *sep = ","; 
     char **cols; int i;   
    cols = malloc(strlen(string)+1*sizeof(char *)); 
    printf("%s\n",string); 
    int n = split(string,cols,sep); 
    printf("%s\t%d\n",string,n); 

    for(i=0; i<n; i++)printf("%s\t%s\n",string,cols[i]); 
    free(cols); 

    return 0; 
} 

感謝我試圖提前爲所有的志願者

Answer 1

你有不少初學者的錯誤：

1）分配內存cols，Eg在main()調用split()前：

cols=malloc(sizeof *cols * 256); 

調整數256，因爲你需要或計算它的基礎上的沒有。的「，」你在string。

2）你的環部分是錯誤的，你是檢查NULL或存儲之前的令牌前再次令牌化：

while(word) 
    { 
     word=strtok_r(NULL, sep, &tmp); 
     fields[count] = strdup(word); 
     count++;  
    } 

把令牌化在循環中的最後一條語句：

while(word) 
    { 
     fields[count] = strdup(word); 
     count++;  
     word=strtok_r(NULL, sep, &tmp); 
    } 

3）您還應該在返回之前在split()中免費獲得copy。否則，你的程序會有內存泄漏。請記住，strdup會爲您分配內存（並且您應該檢查它是否也返回NULL），但是您需要負責free。

free(copy); 

這同樣適用於分配給cols[*]的內存。即打印字符串經過main()，做到：

for(i=0;i<n;i++) 
    free(cols[i]); 

雖然這不是絕對必要的，以釋放內存程序結​​束前右（因爲大多數現代操作系統無論如何都將取消分配它們），這是一個很好的做法，清理自己。

4）使用main()的標準原型，如：int main(int argc, char*argv[])或int main(void)。