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

1 ответ 1

1

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

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

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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