Question

编译时没有警告或错误，当我尝试读取或写入矩阵constValues时，仅以下代码崩溃。

它还需要传递给另一个函数以进行读取；函数createOutputLine。

我如何指向正确保存的数据，以便可以对其进行修改和读取？谢谢。

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

void createOutputFile(FILE*, int, char**);
char createOutputLine(int, int, char*, char**);

int main(int argc, char **argv) {
    int removeComments = 0;
    FILE *file;
    if (argc > 1 && strcmp(argv[1], "-i") == 0) {
        if (argc > 2) {
            if (!(file = fopen(argv[2], "r"))) {
                printf("Error: file not found");
                return -1;
            }
        }
        else {
            printf("Error: no file specified");
            return -1;
        }
    }
    else {
        printf("Error: command requires -i");
        return -2;
    }
    createOutputFile(file, argc, argv);
    fclose(file);
}

void createOutputFile(FILE *file, int argc, char **argv) {
    fseek(file, 0, SEEK_SET);
    char *data = (char*)malloc(2000);
    FILE *header;
    char name[20];
    char *token = strtok(argv[2], ".");
    strcpy(name, strcat(token, ".o"));
    FILE *output = fopen(name, "w");
    char constNames[10][15];
    char **constValues[10][10];
    int constsStored = 0;
    while (fgets(data, 2000, file) != NULL) {
        for (int i = 0; i < strlen(data); i++) {
            int c = i;
            bool linePrinted = false;
            if (data[i] == '#' && data[i + 1] == 'd') {
                for (c = i; c <= i + 7; c++) {
                    data[c] = '\0';
                } int ch = 0;
                while (data[c] != ' ') {
                    constNames[constsStored][ch] = data[c];
                    data[c] = '\0';
                    ch++;
                    c++;
                } ch = 0;
                while (data[c] != '\n') {
                    **constValues[constsStored][ch] = data[c]; //this line crashes
                    data[c] = '\0';
                    ch++;
                    c++;
                }
                if (data[c] == '\n') data[c] = '\0';
                constsStored++;
            }
            for (int ch = 0; ch <= constsStored; ch++) {
                if (data[i] == constNames[ch][0]) {
                    int ch2 = i + 1;
                    int ch3 = 1;
                    bool isConst = false;
                    while (data[ch2] != ' ') {
                        if (data[ch2] == constNames[ch][ch3] && isConst == false) isConst = true;
                        ch2++;
                        ch3++;
                    }
                    if (isConst || data[i + 1] == ' ') {
                        char line[200];
                        line[200] = createOutputLine(i, ch, data, **constValues);
                        fprintf(output, "%c", line[200]);
                        linePrinted = true;
                    }
                }
            }
            if (!linePrinted)
                fprintf(output, "%c", data[i]);
        }
    }
    fclose(output);
    free(data);
}

char createOutputLine(int i, int constElem, char *data, char **constValues) {
    int ch = i;
    int ch2 = 0;
    char temp[200];
    while (data[ch] != '\n' && data[ch] != ' ' && data[ch] != ';') {
        temp[ch2] = data[ch];
        printf("%c", data[ch]);
        ch++;
        ch2++;
    }
    char line[200];
    ch2 = 0;
    for (ch = i; ch <= sizeof(data); ch++) {
        line[ch2] = data[ch];
        ch2++;
    }
    for (ch = 0; ch <= 10; ch++) {
        line[ch2] = constValues[constElem][ch];
        ch2++;
    }
    for (ch = 0; ch <= sizeof(temp); ch++) {
        line[ch2] = temp[ch];
        ch2++;
    }
    line[ch2 + 1] = '\n';
    return line[200];
}

Answer 1

在可以取消引用之前，指针应指向一个对象。句号。

char **constValues[10][10];仅声明一个2D指针数组，该数组指向字符。并且由于它是一个自动数组（既不是静态分配的也不是动态分配的），因此其指针只是未初始化。

当您晚使用**constValues[constsStored][ch] = data[c];时，您尝试取消引用未初始化的指针，这显然是未定义的行为。您很幸运能立即崩溃，因为UB造成的后果显然是无关的问题。

通常的方法是声明对象数组，并将这些对象的地址用作指针。

那还不是全部：C数组不是一等公民。您不能分配给数组，也不能从函数返回它。因此，这是完全错误的：

    char line[200];
    line[200] = createOutputLine(i, ch, data, **constValues);

它只是分配函数返回的唯一字符，而不是数组的结尾！

这是

char line[200];
...
return line[200];

它不返回数组（C不允许它），而是恰好超过数组的字节值。

很抱歉，但程序太长，我无法解决这些错误。

您可能会觉得C很困难并寻求帮助。但是构建仅包含您要处理的内容的小代码。并且只有当这些小片段正常工作时，才尝试将它们组装到更大的程序中。

在C中使用指针传递矩阵

1 个答案: