[已解决]课后写的一个挖字符函数，但是内存释放出现了问题，一释放就报错

臭屁含

1. int spitString(const char *mytmp, char c, char ***mybuf, int *num)
   
2. {
   
3.         int cls = 0;
   
4.         if (mytmp == NULL || num == NULL)
   
5.         {
   
6.                 cls = -1;
   
7.                 printf("func spitString() err:%d (mytmp == NULL || num == NULL)  \n", cls);
   
8.                 goto END;
   
9.         }
   
10.         char *p1 = mytmp;
    
11.         int count = 0;
    
12.         while (p1 = strchr(p1, ','))
    
13.         {
    
14.                 count++;
    
15.                 p1++;
    
16.         }
    
17.         char **pp = NULL;
    
18.         pp = (char **)malloc(count * sizeof(char *));
    
19.         if (pp == NULL)
    
20.         {
    
21.                 cls = -2;
    
22.                 printf("func spitString() err:%d count * sizeof(char *)\n", cls);
    
23.                 goto END;
    
24.         }
    
25.         char *p = NULL, *ptmp = NULL;
    
26.         count = 0;
    
27.         p = mytmp;
    
28.         ptmp = mytmp;
    
29.         int len = p - ptmp + 1;
    
30.         do
    
31.         {
    
32.                 p = strchr(p, c);
    
33.                 if (p != NULL)
    
34.                 {
    
35.                         if (p - ptmp > 0)
    
36.                         {
    
37.                                 pp[count] = (char *)malloc(len * sizeof(char));
    
38.                                 if (pp[count] == NULL)
    
39.                                 {
    
40.                                         cls = -3;
    
41.                                         printf("func spitString() err:%d (char *)malloc(len * sizeof(char))", cls);
    
42.                                         goto END;
    
43.                                 }
    
44.                                 strncpy(pp[count], ptmp, p - ptmp);
    
45.                                 pp[count][p - ptmp] = '\0';
    
46.                                 count++;
    
47.                                 ptmp = p = p + 1;
    
48.                         }
    
49.                 }
    
50.                 else
    
51.                 {
    
52.                         break;
    
53.                 }
    
54.         } while (*p!='\0');
    
55. END:
    
56.         if (cls != 0)
    
57.         {
    
58.                 if (pp == NULL)
    
59.                 {
    
60.                         return -4;
    
61.                 }
    
62.                 Free_String(&pp, count);
    
63.         }
    
64.         else
    
65.         {
    
66.                 *num = count;
    
67.                 *mybuf = pp;
    
68.         }
    
69.         return cls;
    
70. }
    
71. 
    
72. void Free_String(char ***mybuf, int num)
    
73. {
    
74.         if (mybuf == NULL)
    
75.         {
    
76.                 return;
    
77.         }
    
78.         int i;
    
79.         char **tmp = NULL;
    
80.         tmp = *mybuf;
    
81.         if (tmp == NULL)
    
82.         {
    
83.                 return;
    
84.         }
    
85.         for (i = 0; i < num; i++)
    
86.         {
    
87.                 if (tmp[i] != NULL)
    
88.                 {
    
89.                         free(tmp[i]);
    
90.                 }
    
91.         }
    
92.         if (tmp != NULL)
    
93.         {
    
94.                 free(tmp);
    
95.         }
    
96.         *mybuf = NULL;
    
97. }
    
98. 
    
99. int main()
    
100. {
     
101.         int cls = 0;
     
102.         char tmp[100] = { "abcdef,acccd,eeee,aaaa,e3eeee,ssss," };
     
103.         char c = ',';
     
104.         int num = 0, i;
     
105.         char **buf = NULL;
     
106.         cls = spitString(tmp, c, &buf, &num);
     
107.         if (cls != 0)
     
108.         {
     
109.                 printf("fucn spitString() err:%d \n", cls);
     
110.                 return cls;
     
111.         }
     
112.         for (i = 0; i < num; i++)
     
113.         printf("%s \n", buf[i]);
     
114.         printf("%d \n", num);
     
115.         for (i = 0; i < num; i++)
     
116.         {
     
117.                 if (buf[i] != NULL)
     
118.                 {
     
119.                         free(buf[i]);
     
120.                         buf[i] = NULL;
     
121.                 }
     
122.         }
     
123.         if (buf != NULL)
     
124.         {
     
125.                 free(buf);
     
126.                 buf = NULL;
     
127.         }
     
128.         //Free_String(&buf, num);
     
129.         return cls;
     
130. }

复制代码

最佳答案

月排行榜 / 总排行榜

人造人

2020-3-24 17:21:45

改了好多地方

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <string.h>
   
4. 
   
5. void Free_String(char **mybuf, int num)
   
6. {
   
7.     if(mybuf == NULL)
   
8.         return;
   
9.     for(int i = 0; i < num; ++i) {
   
10.         free(mybuf[i]);
    
11.     }
    
12.     free(mybuf);
    
13. }
    
14. 
    
15. int spitString(const char *mytmp, char c, char ***mybuf, int *num)
    
16. {
    
17.     if(mytmp == NULL || num == NULL)
    
18.         return -1;
    
19.     const char *p1 = mytmp;
    
20.     int count = 0;
    
21.     while(p1 = strchr(p1, ','))
    
22.     {
    
23.         count++;
    
24.         p1++;
    
25.     }
    
26.     ++count;
    
27.     char **pp = NULL;
    
28.     pp = malloc(count * sizeof(char *));
    
29.     if(pp == NULL)
    
30.     {
    
31.         *mybuf = pp;
    
32.         return -2;
    
33.     }
    
34.     const char *p = mytmp, *q;
    
35.     for(int i = 0; i < count; ++i)
    
36.         pp[i] = NULL;
    
37.     for(int i = 0; i < count; ++i) {
    
38.         q = strchr(p, c);
    
39.         if(!q) q = p + strlen(p);
    
40.         pp[i] = malloc(q - p + 1);
    
41.         if(!pp[i]) return -3;
    
42.         strncpy(pp[i], p, q - p);
    
43.         pp[i][q - p] = '\0';
    
44.         p = q + 1;
    
45.     }
    
46.     *mybuf = pp;
    
47.     *num = count;
    
48.     return 0;
    
49. }
    
50. 
    
51. int main(void)
    
52. {
    
53.     int cls = 0;
    
54.     char tmp[100] = {"abcdef,acccd,eeee,aaaa,e3eeee,ssssa,"};
    
55.     char c = ',';
    
56.     int num = 0, i;
    
57.     char **buf = NULL;
    
58.     cls = spitString(tmp, c, &buf, &num);
    
59.     if(cls != 0)
    
60.     {
    
61.         printf("fucn spitString() err: %d\n", cls);
    
62.         return cls;
    
63.     }
    
64.     for(i = 0; i < num; i++)
    
65.         printf("%s\n", buf[i]);
    
66.     printf("%d\n", num);
    
67.     Free_String(buf, num);
    
68.     return cls;
    
69. }
    

复制代码

跳转到最佳答案楼层

臭屁含

找了很久都没找到错误出在什么地方，好打击信心啊

人造人

改了好多地方

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <string.h>
   
4. 
   
5. void Free_String(char **mybuf, int num)
   
6. {
   
7.     if(mybuf == NULL)
   
8.         return;
   
9.     for(int i = 0; i < num; ++i) {
   
10.         free(mybuf[i]);
    
11.     }
    
12.     free(mybuf);
    
13. }
    
14. 
    
15. int spitString(const char *mytmp, char c, char ***mybuf, int *num)
    
16. {
    
17.     if(mytmp == NULL || num == NULL)
    
18.         return -1;
    
19.     const char *p1 = mytmp;
    
20.     int count = 0;
    
21.     while(p1 = strchr(p1, ','))
    
22.     {
    
23.         count++;
    
24.         p1++;
    
25.     }
    
26.     ++count;
    
27.     char **pp = NULL;
    
28.     pp = malloc(count * sizeof(char *));
    
29.     if(pp == NULL)
    
30.     {
    
31.         *mybuf = pp;
    
32.         return -2;
    
33.     }
    
34.     const char *p = mytmp, *q;
    
35.     for(int i = 0; i < count; ++i)
    
36.         pp[i] = NULL;
    
37.     for(int i = 0; i < count; ++i) {
    
38.         q = strchr(p, c);
    
39.         if(!q) q = p + strlen(p);
    
40.         pp[i] = malloc(q - p + 1);
    
41.         if(!pp[i]) return -3;
    
42.         strncpy(pp[i], p, q - p);
    
43.         pp[i][q - p] = '\0';
    
44.         p = q + 1;
    
45.     }
    
46.     *mybuf = pp;
    
47.     *num = count;
    
48.     return 0;
    
49. }
    
50. 
    
51. int main(void)
    
52. {
    
53.     int cls = 0;
    
54.     char tmp[100] = {"abcdef,acccd,eeee,aaaa,e3eeee,ssssa,"};
    
55.     char c = ',';
    
56.     int num = 0, i;
    
57.     char **buf = NULL;
    
58.     cls = spitString(tmp, c, &buf, &num);
    
59.     if(cls != 0)
    
60.     {
    
61.         printf("fucn spitString() err: %d\n", cls);
    
62.         return cls;
    
63.     }
    
64.     for(i = 0; i < num; i++)
    
65.         printf("%s\n", buf[i]);
    
66.     printf("%d\n", num);
    
67.     Free_String(buf, num);
    
68.     return cls;
    
69. }
    

复制代码

臭屁含

谢谢大神，不过我编译器用的VS2013，直接malloc会报错，我将你的答案又处理了下，内存完美释放，感谢

人造人

臭屁含 发表于 2020-3-24 22:17
谢谢大神，不过我编译器用的VS2013，直接malloc会报错，我将你的答案又处理了下，内存完美释放，感谢{:9_22 ...

后缀是 .cpp  ?

臭屁含

人造人 发表于 2020-3-24 22:55
后缀是 .cpp  ?

是的呀，我感觉.cpp跟.c没啥区别就懒得改了

臭屁含

臭屁含 发表于 2020-3-24 23:42
是的呀，我感觉.cpp跟.c没啥区别就懒得改了

刚才我仔细对照了下我之前写的代码，终于发现了问题，函数内定义的那个int len值是错的，造成malloc内存出错，不能释放内存，我将int len放到了指针p赋值之后，问题解决了

人造人

臭屁含 发表于 2020-3-24 23:42
是的呀，我感觉.cpp跟.c没啥区别就懒得改了

就用我的代码，不要修改
你换成 .c 看看能不能编译过

人造人

臭屁含 发表于 2020-3-24 23:46
刚才我仔细对照了下我之前写的代码，终于发现了问题，函数内定义的那个int len值是错的，造成malloc内存 ...

嗯