Opened 5 years ago

Closed 5 years ago

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

WW #3

Reported by: lebedev.egor Owned by: Vasily Alferov
Component: WW_intrusive_list Version: 3.0
Keywords: Cc:

Description


Change History (7)

comment:1 Changed 5 years ago by Vasily Alferov

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

Стиль: 2/3, за сканф и принтф.
Корректность: 7/9, исправь, пожалуйста, крайние случаи и работу с памятью (см. ниже).

  • В remove_point и remove_all_points ты делаешь сначала free, а потом remove_point, тем самым обращаясь к освобождённой памяти.

Падает на

add 1 2
add 1 2
rm 1 2
  • Боже мой, чувак! Числа читаются сканфом так: scanf('%d', &x), до пробела. Зачем ты их парсишь руками?

Кстати, ещё и неправильно: у тебя не работают отрицательные числа.

  • printf тоже используется очень сложно. Можно так: printf("(%d %d) ", x, y).
  • remove_all_points падает на пустом списке

comment:2 Changed 5 years ago by lebedev.egor

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

comment:3 Changed 5 years ago by Vasily Alferov

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

Теперь 10/12.

Стиль: 2/3, в одном файле табы, в другом — почти везде пробелы, но табы тоже есть.
Корректность: 8/9, node = node->next после free — обращение к освобождённой памяти.

Исправишь?

comment:4 Changed 5 years ago by lebedev.egor

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

comment:5 Changed 5 years ago by lebedev.egor

Last edited 5 years ago by lebedev.egor (previous) (diff)

comment:6 Changed 5 years ago by lebedev.egor

Last edited 5 years ago by lebedev.egor (previous) (diff)

comment:7 Changed 5 years ago by Vasily Alferov

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

Зачтено.

Note: See TracTickets for help on using tickets.