請教：這個代碼的segmentation fault 在哪里

mcmay

我做了一個代碼，預(yù)期功能是從兩個文件list1.txt和list2.txt中讀取中文名單，然后從list2中找出list1里不存在的名字。每個名字占一行，由于是從excel拷貝過來到text中的，所以名字后面有個TAB。但這個代碼運行后總是出現(xiàn)segmentation fault的提示。請問哪里出現(xiàn)了問題，謝謝！

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. }

復(fù)制代碼

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. }

復(fù)制代碼

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

回復(fù) 2# mcmay


    我在MacOS上運行了你的程序，自己建了小數(shù)據(jù)list1和list2，沒有你說的錯誤，能執(zhí)行。
你需要把數(shù)據(jù)也貼上來，看看。也許是數(shù)組越界錯誤，大數(shù)據(jù)才顯現(xiàn)出來。

Herowinter

回復(fù) 6# Hadron74
同意樓上的，我在linux也可以編譯運行，
只是結(jié)果也不大對，只找出了第一個在list2不在list1的名字，
我用的是樓主2樓改進(jìn)過的代碼。

shan_ghost

既然是必現(xiàn)，那還不簡單……

windows，用你的vs單步執(zhí)行，甚至只是用ide執(zhí)行都行，它自己會告訴你到那行訪問違例的；達(dá)到及格線的ide都允許你查看當(dāng)時的調(diào)用棧以及變量值。

如果是linux，編譯時加-g參數(shù)，然后ulimit -c unlimited允許吐core；等程序core掉，gdb 程序名 -c core.xxxx，然后bt看一下調(diào)用棧，或者用p看一下變量值。

有這些信息，結(jié)合程序邏輯稍微做一下推斷……就這幾行代碼，不是太奇葩的情況都是秒殺……


先學(xué)會用好你手頭的工具……尤其是在論壇提問時。這些工具的報告，或許你看起來是天書，但在別人眼里，很可能就是赤裸裸的答案……

__slucx__

cnt_1 <= SIZE

pnames_1[cnt_1] = names_1[cnt_1];

等于可以嗎？

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

不能超過15字

調(diào)試core文件