调用数组的特定索引时，打印所有值答案

【问题标题】：When calling a specific index of array, all values are printed调用数组的特定索引时，打印所有值
【发布时间】：2015-04-14 12:24:52
【问题描述】：

我已使用此处找到的一段代码逐行读取.txt 文件，我认为这应该将所有行添加到名为words 的数组中。每当我尝试从数组中返回一个值时，例如printf(words[7]);，文本文档中的所有行都会返回，而不仅仅是数组中的第 7 个值。是不是所有的行都没有被正确地拆分成一个数组，或者数组的所有值都被返回了？

void readFile() {
    FILE* fp; // Declare the file pointer

    int lines_allocated = 128;
    int max_line_len = 100;

    /* Allocate lines of text */
    char** words = (char**)malloc(sizeof(char*) * lines_allocated);
    if (words == NULL) {
        fprintf(stderr, "Out of memory (1).\n");
        exit(1);
    }

    switch (difficulty) // Open the file for read
    {
        case 'e':
            fp = fopen("words_easy.txt", "r");
            break;
        case 'm':
            fp = fopen("words_medium.txt", "r");
            break;
        case 'h':
            fp = fopen("words_hard.txt", "r");
            break;
        default:
            printf("Cannot open file");
    }
    if (fp == NULL) {
        fprintf(stderr, "Error opening file.\n");
        exit(2);
    }

    int i;
    for (i = 0; 1; i++) {
        int j;

        /* Have we gone over our line allocation? */
        if (i >= lines_allocated) {
            int new_size;

            /* Double our allocation and re-allocate */
            new_size = lines_allocated * 2;
            words = (char**)realloc(words, sizeof(char*) * new_size);
            if (words == NULL) {
                fprintf(stderr, "Out of memory.\n");
                exit(3);
            }
            lines_allocated = new_size;
        }
        /* Allocate space for the next line */
        words[i] = malloc(max_line_len);
        if (words[i] == NULL) {
            fprintf(stderr, "Out of memory (3).\n");
            exit(4);
        }
        if (fgets(words[i], max_line_len - 1, fp) == NULL)
            break;

        /* Get rid of CR or LF at end of line */
        for (j = strlen(words[i]) - 1;
             j >= 0 && (words[i][j] == '\n' || words[i][j] == '\r'); j--)
            ;
        words[i][j + 1] = '\0';
    }

    /* Close file */
    fclose(fp);

    int j;
    for (j = 0; j < i; j++)
        printf("%s\n", words[j]);

    /* Good practice to free memory */
    for (; i >= 0; i--)
        free(words[i]);
    free(words);
    printf(words[7]);
    return 0;
}

文档每行一个单词，我只是尝试打印一个单词作为测试，但是当我尝试从数组中调用一个值时，我将所有行输出到控制台。

【问题讨论】：

/* Good practice to free memory */..不，它几乎是强制性的。 :-)
标准警告：请do not castmalloc() 和C 中的家人的返回值。
words = (char **)realloc(words,sizeof(char*)*new_size);...如果realloc() 失败，想想恐怖。
如果您不想打印所有单词，您最后的j 循环有什么特别的原因吗？此外，您刚刚释放了您所做的所有分配，因此最终的 printf(words[7]); 似乎充其量只是一厢情愿，而且肯定是未定义的行为。
@WhozCraig 干得好，从测试阶段就离开了这个循环，疲倦和分心会产生草率的代码，感谢您指出那个愚蠢的错误！

标签： c arrays pointers dynamic-memory-allocation

【解决方案1】：

为什么在你的程序中需要这个 for 循环...???

int j;
for(j = 0; j < i; j++)
    printf("%s\n", words[j]);

【讨论】：

【解决方案2】：

第 1 点：在您的代码中，您已经编写了

free(words);
printf(words[7]);

在指针为free()d 之后访问它会导致undefined behaviour。

第 2 点：

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

无论如何，这个for 循环打印所有行。

【讨论】：