Opened 3 years ago

Closed 3 years ago

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

WW_c_io Kiselev Igor lab_05

Reported by: kiselev.igor Owned by: Антон Филатов
Component: WW_c_io Version: 2.0
Keywords: Cc:

Description


Change History (3)

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

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

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

  • правильно сработал только один тест, где надо было прочитать бинарные файлы и выполнить count
  • большинство тестов падают из-за fprintf(F, "\n"); - зачем вы его добавили?
  • на пустых файлах падает с сегфолтом

1/7

Стиль:

  • не надо делать в Makefile if [ ! -d "obj" ]; then mkdir obj; else echo "Nothing to be done..."; fi. Он сам умеет понимаеть, надо ли пересобирать проект или нет. Вам для этого нужно правильно поставить зависимости
  • забыли ключи -Wall -Wextra -Werror
  • очень много волшебных констант
  • почему бы вам вместо жути на 118 строчки в main не передавать type целиком в printf?
  • в функции count конвертация к char довольно опрометчивая. а если список будет больше, чем из 255 элементов?. Да и вместо [0] здесь лучше написать разыменование

0,5/3

итого: 1,5/10

comment:2 Changed 3 years ago by kiselev.igor

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

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

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

Принимается 12/10

Note: See TracTickets for help on using tickets.