C++: cin >> * char

Question

Итак, я сейчас пишу редактор строк в качестве учебного проекта по вводу-выводу, записи файлов и тому подобного. Он написан на C++, и в настоящее время я пытаюсь записать файл по выбору пользователя. У меня есть аргументы CLI, но я в настоящее время не знаю, как реализовать программный способ указания файла для записи.

char *filename; 
if (argc >= 2){ 
     filename = argv[1]; 
} else{ 
     cout << "file>"; 
     cin >> filename; 
     cin.ignore(); 
} 

Это прекрасно работает, когда я использую аргументы командной строки; однако, когда я этого не делаю, как только я запускаю программу, это происходит с ошибками сегментации. Место, где я использую фактическое имя файла, находится в команде сохранения:

void save(char filename[], int textlen, string file[]){ 
     ofstream out(filename); 
     out << filestring(textlen, file); 
     out.close(); 
} 

Которая также работает отлично. Есть ли способ помочь мне? Полный исходный код для обзора: https://github.com/GBGamer/SLED

Answer 1

Проблема в том, что char* filename - это всего лишь указатель на некоторую память, содержащую символы. Он не владеет самой памятью. Когда вы используете аргумент командной строки, программа обрабатывает эту строку где-то, и вы получаете указатель на нее. Когда вы пытаетесь читать с использованием cin >> filename, на самом деле нет места для хранения прочитанных данных.

Решение: Заменить char* filename на std::string filename (и #include <string>).

Затем, чтобы открыть выходной файл, вам понадобится строка c-стиля (нулевой конец char). std::string имеет функцию для этого. Вы бы написать

std::ofstream out(filename.c_str()); 
          ^^^^^ 

Или, на самом деле, если вы можете использовать свежий компилятор с C++ 11 функций, вам не нужно даже использовать c_str(). Новый std::ofstream constructor был добавлен, чтобы принять std::string.

Answer 2

Ваши filename переменная указывает на argv[1], когда обеспечивается аргумент командной строки, он не нуждается в памяти, чтобы быть выделены, но при переходе в else блоке, вы не выделили память filename. Его просто указатель.

Используйте malloc, чтобы присвоить filename некоторую память, а затем взять пользовательский ввод.

filename = (char *)malloc(sizeof(char)*(FILE_NAME_LENGTH+1))