[Решено]Как сделать грамотнее ввод входного/выходного файла программы из командной строки? (C++)

[ffldove]

Участок кода на C++ в котором открываются входной и выходной файл введеные в командной строке.

Код: Выделить всё

int main(int argc, char** argv){
  char* read_file="SPR.000"; //файл открывающийся по умолчанию, если бы небыли заданны мараметры для чтения
  char* write_file="SPR.csv"; //файл открывающийся по умолчанию, если бы небыли заданны мараметры для записи
  char* csv=".csv"; //раширение выходного файла
  if (argc>=2) read_file=argv[1]; //если был введен первый параметр то это имя читаемого файла
  if (argc>=3) write_file=strcat(argv[2], csv); //если был введен второй параметр то это имя записываеиого файла
  fstream spr_read(read_file, ios_base::in); //открываем файл для чтения
  fstream spr_write(write_file, ios_base::out); //открываем файл для записи

/*Работа с файлами*/

}

Понимаю что сделано через, ну в общем не лучшим образом. Как сделать грамотнее?
Плюс еще при компиляции вылазят предупреждения:

Код: Выделить всё

1: In function 'int main(int, char**)':
2: warning: deprecated conversion from string constant to 'char*'
3: warning: deprecated conversion from string constant to 'char*'
4: warning: deprecated conversion from string constant to 'char*'

[watashiwa_daredeska]

Строковые константы имеют тип char const*, а не char*.

write_file=strcat(argv[2], csv);

Это просто ужасно. Память кто выделять будет? strcat уж точно не будет этого делать. Это во-первых.

Во-вторых, в современных шеллах есть такая штука, как автодополнение и (сюрприз, сюрприз!) имя файла дополняется вместе с суффиксом. Так что, принудительное дописывание суффикса к имени — махровые суеверия времён MS-DOS.

А в остальном, ничего так, сойдет.

[ffldove]

Строковые константы имеют тип char const*, а не char*.

Понял.

write_file=strcat(argv[2], csv);

Это просто ужасно. Память кто выделять будет? strcat уж точно не будет этого делать. Это во-первых.

Не понял, а зачем выделать память write_file то уже есть?

Во-вторых, в современных шеллах есть такая штука, как автодополнение и (сюрприз, сюрприз!) имя файла дополняется вместе с суффиксом. Так что, принудительное дописывание суффикса к имени — махровые суеверия времён MS-DOS.

Автодополнение - это которое по клавише Tab? Это не подходит, так как файла для записи может и не быть изначально и его потребуется создать именно csv, а пользователь врятли введет в командной строке именно "имя".csv (проверку ввел или нет сделаю потом, а щас лучше перестрахуюсь).

[watashiwa_daredeska]

Не понял, а зачем выделать память write_file то уже есть?

write_file — указатель. На что он указывает? Откуда strcat знает про write_file? Почитайте man strcat, там написано, как она работает. Подумайте над тем, что и куда на самом деле передается и где что делается в этой строке.

пользователь врятли введет в командной строке именно "имя".csv

В среде *nix не принято додумывать такие вещи за пользователя. Мало ли, как я захочу назвать файл, может .csv мне вовсе не нужен (возможно, я сгенерил имя каким-нибудь mktemp).

[ffldove]

пользователь врятли введет в командной строке именно "имя".csv

В среде *nix не принято додумывать такие вещи за пользователя. Мало ли, как я захочу назвать файл, может .csv мне вовсе не нужен (возможно, я сгенерил имя каким-нибудь mktemp).

Про это я что то не подумал.

[agentprog]

Участок кода на C++

char*

это лично мое имхо, к тому же я не знаю какие именно цели Вы преследуете, но не меньше ли геморроя будет, если воспользоваться std::string?

[watashiwa_daredeska]

не меньше ли геморроя будет, если воспользоваться std::string?

В данном случае — не меньше. Ибо это ведь C++, которого в детстве головой роняли. Всякие там fstream принимают имя файла как char const*, в результате неисправимой родовой черепно-мозговой травмы, а main принимает аргументы как char**, вместо std::vector<std::string>, для совместимости с C.

[Crazy]

Код: Выделить всё

    std::string str = ARGV[1];
    std::cout<< str <<std::endl;
    const char* sss = str.c_str();
        std::ifstream    input(sss);

[/dev/random]

Кстати, если вы не делаете seek(), то в качестве файлов по-умолчанию более принято использовать stdin и stdout. А если делаете, то обязать пользователя вводить их имена и выдавать ошибку в противном случае.

[ffldove]

Код: Выделить всё

    std::string str = ARGV[1];
    std::cout<< str <<std::endl;
    const char* sss = str.c_str();
        std::ifstream    input(sss);

Не совсем понял что вы хотели сказать.

[agentprog]

Не совсем понял что вы хотели сказать.

скорее всего то, что std::string довольно адекватно работает с C-строками (которые char *)

[ffldove]

Кстати, если вы не делаете seek(), то в качестве файлов по-умолчанию более принято использовать stdin и stdout. А если делаете, то обязать пользователя вводить их имена и выдавать ошибку в противном случае.

Первое есть но использовал seekp, до второго еще руки не дошли.