Передача матрицы с использованием указателей в C

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]; просто объявляет двумерный массив указателей на указатели на символы.И поскольку это автоматический массив (статически или динамически не размещаемый), его указатели просто неинициализированы.

Когда вы поздно используете **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 трудным и попросить помощи.Но создайте небольшой код, содержащий только то, над чем вы хотите работать.И только когда эти маленькие кусочки работают правильно, попробуйте собрать их в большую программу.