C结构，字符串和分段错误[重复]答案

【问题标题】：C structs, strings and segmentation faults [duplicate]C结构，字符串和分段错误[重复]
【发布时间】：2017-11-20 08:34:45
【问题描述】：

所以这应该是一个从文本文件中抓取单词的索引程序。我正在尝试使用结构来存储字符串，以及该单词在文本文件中出现的次数。我还想将结构对象放入结构数组中，因为一旦我拥有了所有单词，我就需要按字母顺序对它们进行排序。但是，我的 createStruct 函数内部出现分段错误。我知道问题在于我对指针和通过引用传递的知识有限。几天来我一直在搞乱 createStruct 和 compareStruct ，但它只是没有点击。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>

typedef struct word{
    char *wordArr;
    int wordCount;
}word;


char *makeLowerCase(char word[]);
char *removeFirstChar(char word[]);
char *removeLastChar(char word[]);
void createStruct(struct word wordObj, char word[]);
void structCompare(struct word wordObj, struct word objArr[]);

int main(int argc, char * argv []) {

char buff[] ="@#Hello$$$$$"; //hard coded, will grab words from a .txt file
struct word newWord = {.wordArr = NULL, .wordCount = 0};
struct word structArray[500];      

makeLowerCase(buff);                 
removeFirstChar(buff);               
removeLastChar(buff);                
createStruct(newWord, buff);         
structCompare(newWord, structArray);

//trying to print from the array
printf("%s %d", structArray->wordArr, structArray->wordCount);


return 0;
}

char *makeLowerCase(char grabbedWord[]) {
    int i;
    size_t wordLength = strlen(grabbedWord);

    for(i = 0; i < wordLength; i++) {
        grabbedWord[i] = tolower(grabbedWord[i]);
    }
return grabbedWord;
};

char *removeFirstChar(char inputWord[]) {
    int i = 0;
    size_t length = strlen(inputWord);

    if (!isalnum(inputWord[i])) {
        i++;
        strncpy(inputWord, &inputWord[i], length);
        return removeFirstChar(inputWord);
    }

return inputWord;
};

char *removeLastChar(char inputWord[]) {
    size_t length = strlen(inputWord);

    if (!isalnum(inputWord[length - 1])) {
        inputWord[length - 1] = 0;
        return removeLastChar(inputWord);
    }

return inputWord;
};


void createStruct(struct word wordObj, char string[]) {
    strcpy(wordObj.wordArr, string);
    wordObj.wordCount = 1;
};

void structCompare(struct word obj, struct word structArr[]) {

    int i;

    for(i = 0; i < sizeof(structArr); i++) {

        if(structArr[i].wordCount == 0) {
        strcpy(structArr[i].wordArr, obj.wordArr);
        structArr[i].wordCount = obj.wordCount;
        }
        else if(strcmp(structArr[i].wordArr, obj.wordArr) == 0) {
            structArr->wordCount++;
        }
        else {
            strcpy(structArr[i].wordArr, obj.wordArr);
            structArr[i].wordCount = obj.wordCount;
        }
    }
};

【问题讨论】：

欢迎来到 Stack Overflow！听起来您可能需要学习如何使用debugger 来单步执行您的代码。使用好的调试器，您可以逐行执行您的程序，并查看它与您期望的偏差在哪里。如果您要进行任何编程，这是必不可少的工具。进一步阅读：How to debug small programs.
代码乍一看似乎有很多问题，e.g.一个潜在的严重错误是 sizeof(structArr) 没有按照您的想法执行。

标签： c struct segmentation-fault

【解决方案1】：

由于 NULL 指针，您会得到 segmentation fault。

要复制字符串，请使用strcpy(char *dest, char *src)。但是需要分配dest。在你的情况下，只是 NULL;

所以这就是你需要做的：

// Add a \0 to the end of a string so you know when to stop.
char buff[] ="@#Hello$$$$$\0";

// Allocate the char array so you know where to copy it. I allocate it by default to 500, change this based on your needs.
struct word newWord = {.wordArr = (char *)malloc(sizeof(char) * 500), .wordCount = 0};

如果你将结构体直接传递给函数，你将传递一个copy，因此函数中所做的任何更改都不会在函数外部看到。所以你需要将pointer 传递给struct 而不是实际的struct。

【讨论】：