Opened 5 years ago

Closed 5 years ago

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

WW #3

Reported by: fomenko.ivan Owned by: Дмитрий Лапшин (lapshin)
Component: WW_intrusive_list Version:
Keywords: Cc:

Description


Change History (1)

comment:1 Changed 5 years ago by Дмитрий Лапшин (lapshin)

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

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

  1. Твой список не всегда добавляет элементы.
    1. add_node кажется вставляет куда-то в левое место списка. Как бы вторым элементом всегда? (или нет? я не разобрался)
  2. Нарушен формат вывода (пробел после последнего элемента списка).

Стиль:

  1. Ради бога a->b без пробелов.
  2. remove_point, show_all_points, remove_all_points: а можно попроще? Я могу раза в 2 короче минимум. Какое-то дичайшее дублирование и странные случаи.
  3. Да и в clist.c какое-то дикое число случаев, которые на самом деле не сильно влияют, и даже общие куски скопированы.
  4. Пробелы в управляющих конструкциях while (1) {, if (foo < bar) {.
  5. Святые угодники, сделай scanf("%ЧИСЛОs") и живи. Ну или fgets и sscanf потом.
  6. Имена переменных не помогают понять, что происходит.
  7. typedef struct point … нужен в clist.h?

6/12.

Note: See TracTickets for help on using tickets.