Opened 3 years ago

Closed 3 years ago

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

WW #4

Reported by: Азат Валеев Owned by: Святослав Власов
Component: WW_intrusive_list Version: 3.0
Keywords: Cc:

Description


Change History (5)

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

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

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

  1. Если неизвестная команда слишком длинная, вывод не соответствует ожидаемому.

Стиль -- 3/4

  1. Пробелы вокруг стрелочки -> не нужно ставить.
  2. Добавь функцию is_list_empty, это поможет избежать копипасты в нескольких местах.
  3. Вместо strtok лучше использоваться sscanf, это будет понятнее и удобнее.

comment:2 Changed 3 years ago by Азат Валеев

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

Я добавил проверку на пустоту list'а в clist.c, однако в main'e появляется копипаста, но она на этот раз более читабельная.
По поводу корректности -- поправил проверку input'a, теперь выдаёт корректно, однако стало чуть менее читабельно.

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

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

Не надо так проверять на длинную команду:

if (command[5])

У тебя команда может быть короткая, rm, например, но т.к. у тебя буфер изначально не проинициализирован, в нем лежит мусор, у тебя за командой, в 6-м байте, может оказаться ненулевое значение.

Из-за этого валгринд детектит ошибки, а это формально 0 по корректности.
Обязательно проверяй свои программы валгриндом перед отправкой!

Last edited 3 years ago by Святослав Власов (previous) (diff)

comment:4 Changed 3 years ago by Азат Валеев

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

Надеюсь, сейчас всё будет хорошо... (valgrind'ом просканил, он ничего не обнаружил)

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

Resolution: → задача сдана
Status: assigned → closed

Теперь лучше, 12/12

Note: See TracTickets for help on using tickets.