Change History (5)

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

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

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

  1. Бинарные данные читаются неверно, посмотри в задании сколько байт на число должно быть

Стиль -- 1/3

  1. Длинные строки нужно подрезать
  2. Зачем нужно вот это телодвижение?
    void (*func)(intrusive_node *, void *);
    func = op;
    

Чем вызов op(node, data); не угодил?

  1. container_of лучше разместить в clist.h, макрос относится к операциям над абстрактным списком, а не над точками.
  2. Бинарные файлы нужно открывать через rb и wb. Открывать их в текстовом режиме не совсем безопасно.
  3. Количество аргументов неплохо бы проверить assert'ом или if-ом, иначе при недостатке аргументов свалишься в UB
  4. Почему point_list.o не зависит от clist.h в Makefile?

comment:2 Changed 3 years ago by Наталья Марцинковская

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

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

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

9.5/10

Для безупречного кода, давай вынесем функции loadtext и loadbin вынесем в отдельные функции, функцию main хочется разгрузить.
Длинные строки в заголовочниках тоже нужно разбить.

comment:4 Changed 3 years ago by Наталья Марцинковская

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

вот так?

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

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

10/10

Note: See TracTickets for help on using tickets.