请教：这个代码的segmentation fault 在哪里

mcmay

我做了一个代码，预期功能是从两个文件list1.txt和list2.txt中读取中文名单，然后从list2中找出list1里不存在的名字。每个名字占一行，由于是从excel拷贝过来到text中的，所以名字后面有个TAB。但这个代码运行后总是出现segmentation fault的提示。请问哪里出现了问题，谢谢！

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <string.h>
   
4. #include <ctype.h>
   
5. 
   
6. #define SIZE 60
   
7. #define MAX  20
   
8. 
   
9. int main(void)
   
10. {
    
11.     char names_1[SIZE][MAX];
    
12.     char names_2[SIZE][MAX];
    
13.     char * pnames_1[SIZE], * pnames_2[SIZE];
    
14.     char nms_1[SIZE][MAX];
    
15.     char nms_2[SIZE][MAX];
    
16.     char * pnms_1[SIZE], * pnms_2[SIZE];
    
17.     char * fnd[SIZE];
    
18.     int cnt_1 = 0, cnt_2 = 0, cnt_fnd = 0;
    
19.     int i,j;
    
20.     FILE *fp1, *fp2, *fp3;
    
21. 
    
22.     if((fp1 = fopen("list1.txt", "r")) == NULL)
    
23.     {
    
24.         fputs("Error reading \"listing1.txt\".", stderr);
    
25.         exit(1);
    
26.     }
    
27.     if((fp2 = fopen("list2.txt", "r")) == NULL)
    
28.     {
    
29.         fputs("Error reading \"listing2.txt\".", stderr);
    
30.         exit(1);
    
31.     }
    
32.     if((fp3 = fopen("results.txt", "w")) == NULL)
    
33.     {
    
34.         fputs("Error writing \"results.txt\".", stderr);
    
35.         exit(1);
    
36.     }
    
37.     while(cnt_1 <= SIZE && fgets(names_1[cnt_1], MAX, fp1) != NULL)
    
38.     {
    
39.         pnames_1[cnt_1] = names_1[cnt_1];
    
40.         pnms_1[cnt_1] = nms_1[cnt_1];
    
41.         cnt_1++;
    
42.     }
    
43.     while(cnt_2 <= SIZE && fgets(names_2[cnt_2], MAX, fp2) != NULL)
    
44.     {
    
45.         pnames_2[cnt_2] = names_2[cnt_2];
    
46.         pnms_2[cnt_2] = nms_2[cnt_2];
    
47.         cnt_2++;
    
48.     }
    
49. 
    
50.     for(i = 0; i < cnt_1; i++)
    
51.     {
    
52.         while(*(pnames_1[i]) && !isspace(*pnames_1[i]))
    
53.         {
    
54.             *pnms_1[i] = *pnames_1[i];
    
55.             pnames_1[i]++;
    
56.             pnms_1[i]++;
    
57.         }
    
58.         * pnms_1[i] = '\0';
    
59.         puts(pnms_1[i]);
    
60.     }
    
61.     for(i = 0; i < cnt_2; i++)
    
62.     {
    
63.         while(*(pnames_2[i]) && !isspace(*pnames_2[i]))
    
64.         {
    
65.             *pnms_2[i] = *pnames_2[i];
    
66.             pnames_2[i]++;
    
67.             pnms_2[i]++;
    
68.         }
    
69.         * pnms_2[i] = '\0';
    
70.         puts(pnms_2[i]);
    
71.     }
    
72.     for(i = 0; i < cnt_1; i++)
    
73.         for(j = 0; j < cnt_2; j++)
    
74.             if(strcmp(pnms_2[j], pnms_1[i]) != 0)
    
75.             {
    
76.                 fnd[i] = pnms_1[i];
    
77.                 cnt_fnd++;
    
78.             }
    
79.     if(cnt_fnd > 0)
    
80.     {
    
81.         for(i = 0; i < cnt_fnd; i++)
    
82.             fputs(fnd[i], fp3);
    
83.         fprintf(fp3, "\n%d found.", cnt_fnd);
    
84.     }
    
85.     else
    
86.         fputs("\nNo results found.", fp3);
    
87.     printf("\n%d\n", cnt_fnd);
    
88.     puts("\nDone!");
    
89.     fclose(fp1);
    
90.     fclose(fp2);
    
91.     fclose(fp3);
    
92. 
    
93.     return 0;
    
94. }

复制代码

mcmay

我修正了一些错误，但主要错误依然存在：

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <string.h>
   
4. #include <ctype.h>
   
5. 
   
6. #define SIZE 60
   
7. #define MAX  20
   
8. 
   
9. int main(void)
   
10. {
    
11.     char names_1[SIZE][MAX];
    
12.     char names_2[SIZE][MAX];
    
13.     char * pnames_1[SIZE], * pnames_2[SIZE];
    
14.     char nms_1[SIZE][MAX];
    
15.     char nms_2[SIZE][MAX];
    
16.     char * pnms_1[SIZE], * pnms_2[SIZE];
    
17.     char * fnd[SIZE];
    
18.     int cnt_1 = 0, cnt_2 = 0, cnt_fnd = 0;
    
19.     int i,j;
    
20.     FILE *fp1, *fp2, *fp3;
    
21. 
    
22.     if((fp1 = fopen("list1.txt", "r")) == NULL)
    
23.     {
    
24.         fputs("Error reading \"listing1.txt\".", stderr);
    
25.         exit(1);
    
26.     }
    
27.     if((fp2 = fopen("list2.txt", "r")) == NULL)
    
28.     {
    
29.         fputs("Error reading \"listing2.txt\".", stderr);
    
30.         exit(1);
    
31.     }
    
32.     if((fp3 = fopen("results.txt", "w")) == NULL)
    
33.     {
    
34.         fputs("Error writing \"results.txt\".", stderr);
    
35.         exit(1);
    
36.     }
    
37.     while(cnt_1 <= SIZE && fgets(names_1[cnt_1], MAX, fp1) != NULL)
    
38.     {
    
39.         pnames_1[cnt_1] = names_1[cnt_1];
    
40.         pnms_1[cnt_1] = nms_1[cnt_1];
    
41.         cnt_1++;
    
42.     }
    
43.     while(cnt_2 <= SIZE && fgets(names_2[cnt_2], MAX, fp2) != NULL)
    
44.     {
    
45.         pnames_2[cnt_2] = names_2[cnt_2];
    
46.         pnms_2[cnt_2] = nms_2[cnt_2];
    
47.         cnt_2++;
    
48.     }
    
49. 
    
50.     for(i = 0; i < cnt_1; i++)
    
51.     {
    
52.         while(*(pnames_1[i]) && !isspace(*pnames_1[i]))
    
53.         {
    
54.             *pnms_1[i] = *pnames_1[i];
    
55.             pnames_1[i]++;
    
56.             pnms_1[i]++;
    
57.         }
    
58.         * pnms_1[i] = '\0';
    
59.     }
    
60.     for(i = 0; i < cnt_2; i++)
    
61.     {
    
62.         while(*(pnames_2[i]) && !isspace(*pnames_2[i]))
    
63.         {
    
64.             *pnms_2[i] = *pnames_2[i];
    
65.             pnames_2[i]++;
    
66.             pnms_2[i]++;
    
67.         }
    
68.         * pnms_2[i] = '\0';
    
69.     }
    
70.     for(i = 0; i < cnt_1; i++)
    
71.         for(j = 0; j < cnt_2; j++)
    
72.             if(strcmp(nms_1[i], nms_2[j]) == 0)
    
73.             {
    
74.                 fnd[i] = nms_1[i];
    
75.                 cnt_fnd++;
    
76.             }
    
77.     if(cnt_fnd > 0)
    
78.     {
    
79.         for(i = 0; i < cnt_fnd; i++)
    
80.         {
    
81.             fputs(fnd[i], fp3);
    
82.             putchar('\n');
    
83.         }
    
84.         fprintf(fp3, "\n%d found.", cnt_fnd);
    
85.     }
    
86.     else
    
87.         fputs("\nNo results found.", fp3);
    
88.     printf("\n%d\n", cnt_fnd);
    
89.     puts("\nDone!");
    
90.     fclose(fp1);
    
91.     fclose(fp2);
    
92.     fclose(fp3);
    
93. 
    
94.     return 0;
    
95. }

复制代码

huangyukun2012

问一下楼主：在line41：     pnms_1[cnt_1] = nms_1[cnt_1];这句之前，对nms_1[cnt_1]赋初值了吗？

huangyukun2012

问一下楼主：在line41：     pnms_1[cnt_1] = nms_1[cnt_1];这句之前，对nms_1[cnt_1]赋初值了吗？

天地散人

似乎问题出在fputs上

Hadron74

回复 2# mcmay


    我在MacOS上运行了你的程序，自己建了小数据list1和list2，没有你说的错误，能执行。
你需要把数据也贴上来，看看。也许是数组越界错误，大数据才显现出来。

Herowinter

回复 6# Hadron74
同意楼上的，我在linux也可以编译运行，
只是结果也不大对，只找出了第一个在list2不在list1的名字，
我用的是楼主2楼改进过的代码。

shan_ghost

既然是必现，那还不简单……

windows，用你的vs单步执行，甚至只是用ide执行都行，它自己会告诉你到那行访问违例的；达到及格线的ide都允许你查看当时的调用栈以及变量值。

如果是linux，编译时加-g参数，然后ulimit -c unlimited允许吐core；等程序core掉，gdb 程序名 -c core.xxxx，然后bt看一下调用栈，或者用p看一下变量值。

有这些信息，结合程序逻辑稍微做一下推断……就这几行代码，不是太奇葩的情况都是秒杀……


先学会用好你手头的工具……尤其是在论坛提问时。这些工具的报告，或许你看起来是天书，但在别人眼里，很可能就是赤裸裸的答案……

__slucx__

cnt_1 <= SIZE

pnames_1[cnt_1] = names_1[cnt_1];

等于可以吗？

fgets(names_1[cnt_1], MAX, fp1)
MAX可以吗？

不能超过15字

调试core文件