Opened 3 years ago

Closed 3 years ago

#188 closed ожидается проверка (задача сдана)

WW #5

Reported by: predelina.anastasiya Owned by: Святослав Власов
Component: WW_c_io Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

Корректность -- 5/7

  1. Кривовато работает загрузка бинарных данных. Попробуй сохранить точку (420 53), а потом загрузить её и распечатать.
  2. Функция apply у тебя суммарно проходит по списку три раза, хотя может сделать всё необходимое за один проход.

Стиль -- 0/3

  1. Чего это point_list.o и main.o не зависят от clist.h в Makefile?
  2. add_one логичнее будет перенести в main.c
  3. Что это за жуткая простыня кода в функции main? Её нужно срочно разбить на отдельные функции.
  4. savetext и savebin лучше сделать через apply
  5. argc неплохо было бы проверить хотя бы assert'ом, а не падать в UB при дефиците аргументов
  6. Бинарные файлы нужно открывать в бинарном режиме через "rb" или "wb". Под линуксом это не выстрелит, но вот под виндой читать бинарные файлы в текстовом режиме небезопасно.

comment:2 Changed 3 years ago by predelina.anastasiya

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

Поправила

comment:3 Changed 3 years ago by Святослав Власов

Resolution: задача сдана
Status: assignedclosed

Макрос container_of лучше поместить в clist.h, это ведь инструмент для работы с абстрактными списками.

В остальном всё ок, бонус тоже работает, 12/12

Note: See TracTickets for help on using tickets.