在我的C程序需要一些幫助，我有一個分段錯誤，但我不知道爲什麼

-2

我試圖創建一個單一的鏈接列表，其中包含我的桌面上的.txt文件的一些單詞，但是當我在終端運行它，我得到了分段錯誤。後來我編譯Xcode的代碼並運行它，我得到了這個錯誤信息： 線程1：EXC_BAD_ACCESS（代碼= 1，地址= 0x7fff5fc00000） 我是一個初學者，現在我真的需要一些幫助。謝謝！在我的C程序需要一些幫助，我有一個分段錯誤，但我不知道爲什麼

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
typedef struct word{ 
    int num; 
    char word[50]; 
    struct word *next; 
} Word; 
#define len sizeof(Word) 

Word *create(); 
void print(Word *head); 

int main() 
{ 
    Word *head; 
    head = create(); 
    print(head); 
    return 0; 
} 

Word *create() 
{ 
    Word *head, *p1, *p2; 
    char word[50], c; 
    int i = 0; 
    FILE *fp = fopen("/Users/apple/Desktop/words", "r"); 

    head = p1 = p2 = NULL; 

    while(fp != NULL){ 
     i = 0; 
     p1 = (Word *)malloc(len); 

    //get the English word 
    while((c = fgetc(fp)) != '\n'){ 
     word[i++] = c; 
    } 
    word[i] = '\0'; 
    strcpy(p1->word, word); 
    p1->next = NULL; 

     if(head == NULL){ 
      head = p1; 
      p2 = p1; 
     } else { 
      p2->next = p1; 
      p2 = p1; 
     } 
     p1 = p1->next; 
    } 

    return head; 
} 

void print(Word *head) 
{ 
    Word *p = head; 
    while(p != NULL){ 
     printf("%s\n", p->word); 
     p = p->next; 
    } 
}

這是.txt文件的內容：

enter image description here

來源

2017-02-18 Yan Chen

歡迎來到SO。我強烈建議花一些時間學習如何使用調試器。這是弄清楚你的問題的第一步。 – OldProgrammer

請閱讀[如何調試小程序（由Eric Lippert撰寫）]（https://ericlippert.com/2014/03/05/how-to-debug-small-programs/）。 SO不是調試服務。一旦你找出問題所在，如果你不知道爲什麼*這是一個問題，那麼一定要問一個問題。 – StoryTeller

通過使用調試器來捕捉動作中的崩潰，因此您可以在您的代碼中找到* where *發生的位置。然後當你知道，並且仍然在調試器中時，檢查所有涉及變量的值，以確保它們看起來沒問題。至少請給我們*這些信息（位置和價值）。 –

你的問題被刪除之前，這裏有一些你在你的代碼做不正確的事，你們應該採取甚至在嘗試修復您的段錯誤之前查看它。

while(fp != NULL){ 
    i = 0; 
    p1 = (Word *)malloc(len);

這不是你如何從文件中讀取。文件指針fp預計不會更改。如果你想知道你的文件是否已經完成，你必須檢查讀取函數的輸出。 fscanf()和fgetc()返回EOF當您嘗試讀取文件的最後一個字符。

另外，不要使用feof()。 It's almost always wrong。

while((c = fgetc(fp)) != '\n'){ 
    word[i++] = c; 
}

fgetc()返回一個(int)，所以你應該申報int c。這就是你如何檢查你是否讀過所有東西。當你讀完最後一句話後，fgetc()將返回EOF。它通常是-1，但不管它的價值如何，它都不適合(char)。您需要(int)。

此外，您應該使用fscanf()而不是逐字符讀取它（除非它是您的任務）。 fscanf()會一直讀到該單詞的結尾並自動添加0。

這

p1 = (Word *)malloc(len);

不會幫你。那len不會對你有任何好處。 (Word*)也不是。這是你應該怎麼做：

p1 = malloc(sizeof (Word));

甚至更好

p1 = malloc(sizeof *p1);

最後一個甚至不需要你知道的p1類型。

來源

2017-02-18 18:45:20 giusti

非常感謝您的建議！我會記住這些，並重寫我的代碼。 –

在我的C程序需要一些幫助，我有一個分段錯誤，但我不知道爲什麼

回答

相關問題