Как в C выделить динамическую память для текстового файла?

Question

[Artyom_Kopan]

У меня есть функция:

FILE* openInputFile()
{
    printf("Enter the name of the input file: ");
    char* inputFileName;
    scanf("%s", &inputFileName);
    printf("%s", inputFileName);
    if (strstr(inputFileName, ".txt") == NULL)
        inputFileName = strcat(inputFileName, ".txt");
    printf("%s", inputFileName);
    FILE* inputFile = fopen(inputFileName, "r");
    return inputFile;
}

При её запуске программа падает в ошибкой SIGSEGV. Может быть, для этого файла нужно выделить динамическую память?
(Сейчас проверил, от размера файла это не зависит. В чём тогда может быть ошибка?)

Comments

[Евгений Шатунов]

Artyom_Kopan , на какой точно строке твоя программа обрывается?

Давай вместе внимательно посмотрим на этот код.

char* inputFileName;
scanf("%s", &inputFileName);

Ты видишь что-нибудь странное или неправильное в нем?

[Artyom_Kopan]

Ага, программа падает как раз на scanf. Но я не понимаю, что здесь неправильно(

[Artyom_Kopan]

Понял! Нужно убрать амперсанд

[Tony]

Artyom_Kopan,)))
Нужно выделить память. Например не стеке.
Сейчас у тебя просто указатель в никуда.

[Artyom_Kopan]

Когда я пишу так:

char* inputFileName;
    scanf("%s", inputFileName);
    printf("\n%s", inputFileName);

, то всё отрабатывает правильно.
Но почему-то когда я в другой функции пишу

char* outputFileName;
    scanf("%s", outputFileName);

, то получаю ту же ошибку, хотя код вроде бы идентичный.
А как выделить память для char* на стеке? Присвоить ему строковый литерал?

[Василий Дёмин]

Artyom_Kopan, Оно может и не отработать правильно при таком варианте. В scanf передаётся указатель, в момент запуска программы inputFileName имеет неопределённое значение, если оно будет указывать на недоступную область памяти, программа упадёт

[Василий Дёмин]

Artyom_Kopan, Читайте учебник(и), а не задавайте вопросы на тостере. В любом учебнике про выделение памяти говорится

[Tony]

Artyom_Kopan,
char inputFileName[64];

[Artyom_Kopan]

Tony, спасибо! А как в таком случае выделить динамическую память для самого файла? (он весит аж 160 кБ)

[Artyom_Kopan]

Tony, при таком выделении не работает строчка
inputFileName = strcat(inputFileName, ".txt");
а вот если выделить память в куче, то всё работает

[Василий Дёмин]

Artyom_Kopan, Узнать размер файла и выделить соответствующее количество

[Tony]

Artyom_Kopan,

А как в таком случае выделить динамическую память для самого файла? (он весит аж 160 кБ)

malloc, free/new, delete [].
Но вы в своём примере даже не читаете файл.

[Artyom_Kopan]

Для чтения файла у меня такая функция:

void readInputFile(FILE* inputFile, Dictionary* dict)
{
    char* word;
    while (fscanf(inputFile, "%s", &word)) {
        if (!hasKey(dict, word))
            put(dict, word, 1);
        else {
            DictElement* currentElement = dict->head;
            while (currentElement != NULL) {
                if (currentElement->key == word) {
                    currentElement->value++;
                    break;
                }
                currentElement = currentElement->nextElement;
            }
        }
    }
}

Динамическая память выделяется в процедуре put()
А сам файл при открытии (до начала чтения) выгружается в память или хранится только указатель на него?

[Artyom_Kopan]

(Я понял, что должен поправить чтение word)

[Artyom_Kopan]

Artyom_Kopan, интересно, почему так?

[Tony]

Artyom_Kopan,

А сам файл при открытии (до начала чтения) выгружается в память или хранится только указатель на него?

Только указатель (дескриптор).

Я понял, что должен поправить чтение word

Да. И так же выделить память под word.

[Artyom_Kopan]

О боже...Всё-таки не получается открыть входной файл (.txt), а вот выходной (.csv) открывается нормально.
Вот функция для открытия входного файла:

FILE* openInputFile()
{
    printf("Enter the name of the input file: ");
    char* inputFileName = malloc(256 * sizeof(char));
    scanf("%s", inputFileName);
    if (strstr(inputFileName, ".txt") == NULL)
        inputFileName = strcat(inputFileName, ".txt");
    FILE* inputFile = fopen(inputFileName, "r");
    free(inputFileName);
    return inputFile;
}

[Евгений Шатунов]

Artyom_Kopan
char* inputFileName = malloc(256 * sizeof(char));
Чему равен sizeof(char)? Имеет ли смысл произведение в этом месте?

inputFileName = strcat(inputFileName, ".txt");
Какую задачу в том коде решает присвоение? Если принять во внимание документацию, то так ли, на самом деле, тебе нужна динамическая память?

Answer 1

[Кот Абсолютный]

char* inputFileName;

Было бы странно, если бы она не падала, поскольку сейчас inputFileName указывает куда-то в пространство.

Чайники обычно делают так:
char inputFileName[256];
и потом думают - как так, меня сломали?
кто поумнее делают так:

char *inputFileName;
inputFileName = (char *) calloc(256, 1);

(это конечно тоже не звездец - нужно учесть максимально допустимую длину пути и имени файла, но уже куда лучше)

Comments

[res2001]

Чайники обычно делают так:

Никто не мешает указать спецификатор width в scanf для вводимой строки. И надо предусмотреть в массиве место под завершающий нулевой байт.

кто поумнее делают так

Пользователь с клавиатуры может вводить строку какой угодно длинны, ему до фанаря размер массива под вводимую строку, который предусмотрен в программе. Это вызовет выход за пределы массива и тот же SYGSEGV с вылетом программы. Так что это не многим лучше первого варианта. Выход все тот же - спецификатор width.
Оба совета годные, но они не достаточные - в любом случае надо при вводе контролировать размер вводимых данных.
https://en.cppreference.com/w/c/io/fscanf

[Кот Абсолютный]

res2001,

Пользователь с клавиатуры может вводить строку какой угодно длинны

Разумеется, причем это стоит проверять, намеренно вводя лажу. Но длина имени файла и полного пути обычно ограничена и попытка использовать более длинное имя вызовет ошибку.