0

Правильно ли писать так?:

void main()
{
    char *path;
    scanf("%s", path);
}

В досовском компиляторе (qc2) работает без проблем, варнингов и ошибок. Да и препод сказал что правильно так писать, чем создавать статический массив.

Не было бы вопросов, если бы не одна незадача. В компиляторе gcc на этом моменте программа прерывается (ошибка сегментации).

Тут стоит уточнить: код выше тоже будет работать в gcc, однако, этот код уже неработоспособный:

void main()
{
    char *path;
    int i;
    i = 0;
    scanf("%s", path);
}

Почему в старом компиляторе всё в норме, а в современном - нет? Или всё-таки так писать неправильно?


UPD 1:

@KoVadim, спасибо, прояснил.

@Janycz, в данном случае вроде особой разницы нет, динамический или статический, ибо длина строки наперёд неизвестна.

А по сути, как в таком случае считывать строку, не зная её длину? Ведь создавать большой массив неразумно.

5
  • 4
    char *path указывает в никуда. Просто под досом компилятор этим не заморачивался, а Вам везло и туда, куда указывал path было доступно по чтению/записи и было безопасно. А ведь могло указывать где нибудь и в опасное место. И тогда бы долго удивлялись, почему системные часы меняются или принтер начинает печатать. Современные компиляторы это дело просекают. Вывод. Преподу подучится, старый компилятор просто недостаточно умный.
    – KoVadim
    Commented 5 дек. 2013 в 17:32
  • В C придется создавать большой массив. В С++ можно не создавать большой массив.
    – Janycz
    Commented 5 дек. 2013 в 18:11
  • @froxxendsg а никак. Выделять буфер и внимательно следить за тем, чтобы scanf не вызвал переполнения (там есть возможность ограничить размер считываемой строки). Или делать самому считывание посимвольно и соответственно выделять память (правда, все равно памяти понадобится 2x длина считанной строки)
    – gecube
    Commented 5 дек. 2013 в 18:51
  • @froxxendsg, в Linux в такой ситуации используйте ssize_t getline(char **lineptr, size_t *n, FILE *stream); см. man 3 getline. Для винды напишите свой аналог (несколько десятков строк, хорошая тренировка).
    – avp
    Commented 5 дек. 2013 в 20:14
  • 1
    Вот небольшой трактат по поводу того, как это делать правильно: stackoverflow.com/a/4874163/276994
    – VladD
    Commented 5 дек. 2013 в 20:36

1 ответ 1

1

Для нового компилятора char *path; не строка, а указатель на один символ. Поэтому делайте статический массив или используйте malloc: char *path = (char*)malloc(100);

4
  • Для старого -- тоже. См. ответ @KoVadim.
    – alexlz
    Commented 5 дек. 2013 в 19:39
  • 1
    Ну, не очень хороший совет. Пользователь введёт больше ста символов, и — бац! — срыв стека. Вы думаете, откуда берутся эксплоиты? Вот из такого кода.
    – VladD
    Commented 5 дек. 2013 в 20:30
  • @VladD, ну, вряд ли вылетит по стеку, а вот структуры в куче попортятся и при следующих malloc/realloc/free (new/delete), и -- бац! -- segmentation fault.
    – avp
    Commented 5 дек. 2013 в 20:55
  • @avp: «бац!» у меня не в смысле «программа обрушится», а «на ровном месте дыра в безопасности».
    – VladD
    Commented 5 дек. 2013 в 21:18

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.