WW #5 chistyakova.polina

[Дмитрий Лапшин (lapshin)]

А оригинал был вовремя. Проверяем!

Ну ведь предупреждение просто феноменального масштаба лажи!

И чтение до feof это плохо. EOF это значит файл кончился и ты уже пару раз постучала по нему лбом.

Так что плохо работает, надо чинить.

[chistyakova.polina]

Исправила)
Извини, что так поздно...(не бей >.<)
Да, нам что-то такое на лекции говорили, но как читать было только так...
Но сейчас вроде как всё хорошо
Надеюсь)

[chistyakova.polina]

Запуталась в тикетах....
Надеюсь, он до тебя дошёл...

[Дмитрий Лапшин (lapshin)]

Ну эх)

Корректность:

1. Читаешь ты всё ещё плохо. Смотри, давай возьмём твой код:

   while ((fscanf(fLoad, "%d %d", &a, &b) != EOF) && (!feof(fLoad))) {
       add_point(l, a, b);
   }
   add_point(l, a, b);
   

   Мне кажется, что он очень много где некорректен. У него сложные инварианты проверки, что чтение не удалось. И зачем-то ещё раз добавляется точка (видимо, в попытках починить инвариант). Сходу возникает вопрос, а что если fscanf вернёт, ну, 0? Как следствие, ни один текстовый тест не прошёл.

Но смотри, я в стиле Чилинтано чиню машину, выкидывая детали, и о чудо:

while (fscanf(fLoad, "%d %d", &a, &b) == 2) {
    add_point(l, a, b);
}

Вот это проходит все тесты и очевидно, что работает.

1. Аналогично с бинарными файлами. Читаем, пока говорят, что читается.

Стиль:

1. Выделять структурки на кучи ради кучи вредно. Если она переживёт код использования прямо на стеке и маленькая, пусть и будет маленькой на стеке.
2. Я тебе уже много раз рассказывал, но полезнее по стилю ответвлять ошибки, а не «прямое» исполнение.

   if ((strcmp(argv[1], "loadtext")) != 0 && (strcmp(argv[1], "loadbin") != 0)) {
       printf(…);
       return 1;
   }
   
   // continue here!
   

   }}}
3. Структурка объявлена дважды. Может её в заголовок?

Суммарно это 7, к сожалению.