Opened 3 years ago

Closed 3 years ago

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

WW_io Lab_05 Kozyrev Andrei

Reported by: Andrei Kozyrev Owned by: Антон Филатов
Component: WW_c_io Version: 2.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Антон Филатов

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

Тесты пройдены 7/7
Стиль:

  • -Werror забыли. И ворнинги не почистили. Можете вставить ассерты, чтобы убрать ворнинги
  • волшебные константы
  • *((int*)data) = *((int*)data) + 1; - ну ++ же
  • на мой взгляд перемудрили с int_to_bytes. Прямолинейный fread и fwrite без этого бы правильно сработали. Или это задел на бонус?

1,5/3
Итого 8,5/10

comment:2 Changed 3 years ago by Антон Филатов

Очень сильно похожий код на Мартынова. задержитесь вдвоём после пары

comment:3 Changed 3 years ago by Andrei Kozyrev

Добрый вечер
Werror я не забыл, просто в двух функциях есть неиспользованные переменные
В одной - интрузивная нода, в другой - сам список. Их не убрать потому что мы передаем на эти функции указатель и короче там так и так будет неиспользованная переменная
A werror помечает это как ошибку и не собирает исполняемый файл
Ну я решил, что лучше убрать werror, чем получить код, который make не будет собирать

Что значит вставить ассерты?
В коде этих функций фиктивно использовать эти переменные что ли?
Ну это же блин костыль какой то

Задела на доп задание не делал, если честно, не читал его даже
Думал, это единственный способ

Остальное поправлю
После пары задержусь. А можно будет там как раз вас вопросами один на один забросать?

comment:4 Changed 3 years ago by Andrei Kozyrev

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

comment:5 Changed 3 years ago by Антон Филатов

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

10/10

Note: See TracTickets for help on using tickets.