WW_io, garaev.timur, lab_05

[Garaev Timur]

Первое, что показалось неоднозначным -- если использовать реализацию интрузивного списка из прошлой лабы -- список точек будет переворачиваться относительно порядка во входном файле. А мы так наверное не хотим. Мне кажется, порядок важен. Я исправил add так, чтобы он доавлял в конец (за линию, каюсь). Можно было просто переписывать вывод, но он стал бы громоздским и менее читаемым. Чую, заставят переписать, но как говорили люди, которые учили меня программировать: "Работает -- не трогай".

Возникло куча философских вопросов вида: "А где правильнее разместить тот или иной код?". Все-таки, когда появляется столько файлов с несколькими зависимостями, поиск однозначных ответов становится тяжелой задачей. Но в целом и с валгриндом поработал, и над красотой, кажется, тоже (хотя все еще не научился не ставить пробелы перед/после стрелочек. Душа болит их не ставить, честно).

Еще до конца не уверен, стоит ли добавлять в clean .bin и .txt файлы. Пока не написал (потому что так точно не поломаются автотесты). Но вдруг нужно, вы скажите, сделаем.

[Антон Филатов]

Тесты проходят, 7/7 за корректность.

Стиль:

• apply - это всё же должна быть фунция clist, а не pointlist
• я бы сказал, что операторам, которые будут передаваться в apply место в main, поскольку их суть зависит сильно от интерфейса. Обычному пользователю point_list без подготовки вряд ли придёт в голову пользоваться вот так объявленной функцией print (54я строка из point_list.c)
• запись в файл тоже очень хочется сделать через apply. А то в main нездорово трогать head->next. Ещё раз призываю вас не ставить пробелы вокруг ->. Вокруг точки же вы пробелы не ставите. Общепринято писать без пробелов

1,5/3 за стиль

Файлы .bin и .txt, которые генерируются вами по цели clean надо удалять, да. clean должна удалять всё, что генерируется, оставлять то состояние файлов, которое было да вызова первого make. Вообще, если пускаться в рассуждения, то все выходные файлы должны класться в отдельную папку с артефактами, и по хорошему при clean надо спрашивать у пользователя, хочет ли он удалить эти артефакты. Но в простом случае clean - это reset

Итого 8,5/10

[Garaev Timur]

Обещаю, что это последняя лаба, где я пишу -> неправильно!

[Антон Филатов]

Принимается
Эх, пропустил, что вы -Werror и ко не поставили, но ладно, сейчас уже поздно снижать баллы за это. В следующий раз не забудьте
10/10