我有一个分段错误并且不确定我的代码有什么问题

Question

如有任何指导，我们将不胜感激。我个人认为问题出在加载方法上。另外，每个方法的基本功能都写在注释中。什么可能是我的分段错误的原因？一切都按预期工作吗？谢谢你的时间。

任何可能为我指明正确方向的资源也将不胜感激。

 /**
 * Implements a dictionary's functionality.
 */

#include <stdbool.h>
#include "dictionary.h"
#include <string.h>
#include <stdio.h>
#include <ctype.h>
#include <cs50.h>

//Defining node:
 typedef struct node
 { //Inner workings of each "element" in the linked lists
    char word[LENGTH + 1]; //the word within the node is +1'd due to the memory after the word containing /0
    struct node *next; //linked list
}node;



  node *alphabetList[27]; //26 buckets that can contain variables of type node(of dynamic size)
//one bucket for each letter of the alphabet
  node *cursor = NULL;
  node *head = NULL;

/**
 * Returns true if word is in dictionary else false.
 */
bool check(const char *word)
{
    int bucketIndex ;
    //no need to malloc information b/c we are simply pointing to previously established nodes.
    if(word[0] >= 65 && word[0] < 97){
        bucketIndex = word[0] - 65;
    }
    else{
        bucketIndex = word[0] - 97;
    }


    node *head = alphabetList[bucketIndex];

    node *cursor = head;

    while(cursor != NULL)
    {
        cursor = cursor -> next;
        if(strcmp(cursor -> word, word) != 0)
        {
            return true;
        }

    }


    return false;
}

/**
 * Loads dictionary into memory. Returns true if successful else false.
 */
bool load(const char *dictionary)
{
        char *word = NULL;
        int i = 0; //index
        FILE *dictionaryTextFile;
        dictionaryTextFile = fopen(dictionary, "r");

        //scan for word
        while(fscanf(dictionaryTextFile, "%s", word) != EOF)
        {
            //for every word we scan we want to malloc a node to ascertain we have sufficent memory
            node *new_node = malloc(sizeof(node));
            if(new_node == NULL) //error check(if you run out of memory malloc will return null)
            {
                unload();
                return false;
            }
            //error check complete.
            else{
                strcpy(new_node -> word, word);
            }


            //not sure from here on 
            char first_letter = new_node[i].word[0]; //first letter of node word (confused on how to execute this properly)

            first_letter = tolower(first_letter);
            int index = first_letter - 97;

            if(word){

                for(node *ptr = alphabetList[index]; ptr!= NULL; ptr = ptr->next)
                {
                    if(!ptr-> next){
                        ptr->next = new_node;
                    }
                }

            }
            else
            {
                alphabetList[index] = new_node;
            }
            i++;
            }


     return true;

    }





/**
 * Returns number of words in dictionary if loaded else 0 if not yet loaded.
 */
unsigned int size(void)
{
    return 0;
}

/**
 * Unloads dictionary from memory. Returns true if successful else false.
 */
bool unload(void)
{
    for(int i = 0; i <= 26; i++)
    {

    node *head = alphabetList[i];
    node *cursor = head;

    while(cursor != NULL)
    {
        node *temp = cursor;
        cursor = cursor -> next;
        free(temp);
    }

    }
return true;

}

Answer 1

既然你已经说过代码在哪一行崩溃了，问题就很明显了。考虑这些行...

    char *word = NULL;
    int i = 0; //index
    FILE *dictionaryTextFile;
    dictionaryTextFile = fopen(dictionary, "r");

    //scan for word
    while(fscanf(dictionaryTextFile, "%s", word) != EOF)

你有两个问题。首先，您没有检查对 fopen 的调用是否有效。您应该始终检查返回的值是否不是 NULL.

其次，崩溃的原因是 word 仍然是 NULL - 您没有分配任何 space 来保存其中的字符串。您不妨将其声明为与在 node 中声明的相同，因此替换

    char *word = NULL;

和

    char word[LENGTH+1];

说到 node 并且为了避免您稍后再次崩溃，您应该始终确保初始化结构的所有属性。在这种情况下 new_node->next 应该设置为 NULL 否则你稍后会在你的 for 循环中检查它（顺便说一句，看起来不错）它可能看起来指向一个节点，但它指向内存中的某个随机位置，代码会崩溃。

我有一个分段错误并且不确定我的代码有什么问题

I have a segmentation fault and am unsure about what is wrong with my code

c

pointers

list

nodes

cs50