0

当我编译我的代码时，我没有收到任何错误。但是，当我尝试运行它时，我得到一个分段错误（核心转储）。这是我的主要内容：

原始代码

void main(int argc, char *argv[]){
    if(argc < 3){
          return;
    }

    char *stop_list_name = argv[1];
    char *doc_names[argc - 2];

    int i;
    for(i = 0; i < argc; i++){
            doc_names[i] = argv[i];
    }

//create the array of stop words
    char *stopWords[50];
    char *word;
    int word_counter = 0;
    FILE *fp;
    fp = fopen(stop_list_name, "r");
    if(fp != NULL){
            while(!feof(fp)){
                    fscanf(fp, "%s", word);
                    stopWords[word_counter] = word;
                    word_counter++;
            }
    }

    fclose(fp);

    for(i = 0; stopWords[i] != '\0'; i++){
            printf("%s", stopWords[i]);
    }
}

我很确定我的while循环中有问题，但我不完全知道是什么，或者如何解决它。

修改后的代码

看到答案后，我修改了我的代码，使其看起来像这样，但它仍然崩溃。现在怎么了？

int main(int argc, char *argv[]){
    if(argc < 3){
            return;
    }

    char *stop_list_name = argv[1];
    char *doc_names[argc - 2];

    int i;
    for(i = 2; i < argc; i++){
            doc_names[i-2] = argv[i];
    }

//create the array of stop words
    enum {MAX_STOP_WORDS = 50};
    char *stopWords[MAX_STOP_WORDS];
    int word_counter = 0;
    FILE *fp = fopen(stop_list_name, "r");
    if(fp != NULL){
            char word[64];
            int i;
            for(i = 0; i < MAX_STOP_WORDS && fscanf(fp, "%63s", word) == 1; i++){
                    stopWords[i] = strdup(word);
            }

            word_counter = i;
            fclose(fp);
    }

    for(i = 0; stopWords[i] != '\0'; i++){
            printf("%s", stopWords[i]);
    }
}
4

2 回答 2

3

原代码中的问题

一种可能的问题来源是：

char *doc_names[argc - 2];

int i;
for(i = 0; i < argc; i++){
        doc_names[i] = argv[i];
}

您为指针分配空间argc-2并继续将argc指针复制到该空间中。那是缓冲区溢出（在这种情况下，也是堆栈溢出）。它很容易引起麻烦。一个合理的解决方法是：

for (i = 2; i < argv; i++)
    doc_names[i-2] = argv[i];

但是，您真的不需要复制参数列表；您可以只处理从索引 2 到末尾的参数。我注意到显示的代码实际上并没有使用doc_names，但是越界赋值仍然会引起麻烦。

您没有分配空间来读取单词，也没有为每个停用词分配新空间，也没有确保不会溢出存储单词的数组的边界。

考虑使用：

enum { MAX_STOP_WORDS = 50 };
char *stopWords[MAX_STOP_WORDS];
int word_counter = 0;
FILE *fp = fopen(stop_list_name, "r");
if (fp != NULL)
{
    char word[64];
    for (i = 0; i < MAX_STOP_WORDS && fscanf(fp, "%63s", word) == 1; i++)
        stopWords[i] = strdup(word);
    word_counter = i;
    fclose(fp);
}

这个诊断出的问题绝对是您崩溃的可能原因。我i在循环中使用（在代码前面声明）因为word_counter使循环控制线对于 SO 来说太长了。

严格来说，strdup()它不是标准 C 的一部分，但它是 POSIX 的一部分。如果您没有 POSIX，您可以编写自己的：

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

char *strdup(const char *str)
{
    size_t len = strlen(str) + 1;
    char *result = malloc(len);
    if (result != 0)
        memmove(result, str, len);
    return result;
}

您还展示了其他一些不良做法：

修改代码中的问题

修改后的代码中有一个重要的和几个非常小的问题：

  • 打印停用词的循环取决于空指针（奇怪地拼写为'\0'- 它是空指针的有效但非常规拼写），但初始化代码没有设置空指针。

    有（至少）两个选项可以解决这个问题：

    1. 添加一个空指针：

         for (i = 0; i < MAX_STOP_WORDS-1 && fscanf(fp, "%63s", word) == 1; i++)
       stopWords[i] = strdup(word);

   stopWords[i] = 0;
   fclose(fp);
}

for (i = 0; stopWords[i] != '\0'; i++)
    printf("%s\n", stopWords[i]);

      请注意，上限现在是MAX_STOP_WORDS - 1。

    2. 或者您可以使用wordCount而不是条件：

      for (i = 0; i < wordCount; i++)
    printf("%s\n", stopWords[i]);

    我会选择第二个选项。

  • 这样做的一个原因是它避免了关于wordCount被设置和不使用的警告——这是一个小问题。

  • 并且doc_names也设置但未使用。

我担心这些，因为我的默认编译器选项会为未使用的变量生成错误——所以在我修复它之前代码不会编译。这导致：

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

int main(int argc, char *argv[])
{
    if (argc < 3)
    {
        fprintf(stderr, "Usage: %s stop-words docfile ...\n", argv[0]);
        return 1;
    }

    char *stop_list_name = argv[1];
    char *doc_names[argc - 2];

    int i;
    for (i = 2; i < argc; i++)
    {
        doc_names[i - 2] = argv[i];
    }
    int doc_count = argc - 2;

    // create the array of stop words
    enum { MAX_STOP_WORDS = 50 };
    char *stopWords[MAX_STOP_WORDS];
    int word_counter = 0;
    FILE *fp = fopen(stop_list_name, "r");
    if (fp != NULL)
    {
        char word[64];
        int i;
        for (i = 0; i < MAX_STOP_WORDS && fscanf(fp, "%63s", word) == 1; i++)
            stopWords[i] = strdup(word);

        word_counter = i;
        fclose(fp);
    }

    for (i = 0; i < word_counter; i++)
        printf("stop word %d: %s\n", i, stopWords[i]);

    for (i = 0; i < doc_count; i++)
        printf("document %d: %s\n", i, doc_names[i]);

    return 0;
}

并且，给定一个停用词文件，其中包含：

help
able
may
can
it
should
do
antonym
prozac

并编译它（源文件sw19.c，程序sw19）：

$ gcc -O3 -g -std=c11 -Wall -Wextra -Wmissing-prototypes -Wstrict-prototypes \
>     -Wold-style-definition -Werror sw19.c -o sw19

并将其运行为：

$ ./sw19 stopwords /dev/null
stop word 0: help
stop word 1: able
stop word 2: may
stop word 3: can
stop word 4: it
stop word 5: should
stop word 6: do
stop word 7: antonym
stop word 8: prozac
document 0: /dev/null
$
于 2015-04-28T01:08:26.317 回答
1

您正在尝试将扫描的字符串存储到未初始化的指针，

fscanf(fp, "%s", word);

并且word, 甚至没有被初始化。

您可以为此使用静态缓冲区，就像这样

char word[100];

if (fscanf(fp, "%99s", word) != 1)
    word[0] = '\0'; /* ensure that `word' is nul terminated on input error */

另外，while (!feof(fp))这是错误的，因为在尝试读取文件末尾EOF之前不会设置标记，因此代码将重复一次额外的时间。fscanf()在这种情况下，您将存储相同的word两次。

请注意，您还需要为指针数组分配空间，也许您可​​以使用malloc().

于 2015-04-28T01:09:27.360 回答