Opened 5 years ago

Closed 5 years ago

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

WW #3 anisimova.karina

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

Description


Change History (5)

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

Owner: changed from Дмитрий Лапшин (lapshin) to anisimova.karina
Summary: WW #3WW #3 anisimova.karina

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

  1. Нарушен формат вывода.
  2. scanf("%s").

Стиль:

  1. strcmp(command, "hello") кто мешает написать?
  2. init_list не должен выделять память.
  3. Кстати, два фиктивных элемента обычно перебор. Можно использовать один элемент, тогда список становится циклом. Я не то чтобы прошу это исправить, но подумать стоит.
  4. Пробелы в управляющих конструкциях (if (2 < 3) {, while (a < b) {).
  5. Разные отступы.
  6. Разный стиль открывающих скобок (на той же строчке / на следующей).
  7. И кстати, лучше уж
    if () {
        foo(1);
    } else if () {
        foo(2);
    } else {
        foo(3);
    }
    

6/12.

comment:2 Changed 5 years ago by anisimova.karina

Owner: changed from anisimova.karina to Дмитрий Лапшин (lapshin)

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

Owner: changed from Дмитрий Лапшин (lapshin) to anisimova.karina

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

  1. От того, что ты вынесла кусок init_list в main, ты не решила проблему. Сделала хуже. Не надо при инициализации списка выделять память: нужны фиктивные элементы — положи в структуру.
  2. scanf может выйти за пределы буфера.

Стиль:

  1. return; какую нагрузку несёт?
  2. Куча пустых строк в конце файла.
  3. Отступы.

8/12.

comment:4 Changed 5 years ago by anisimova.karina

Owner: changed from anisimova.karina to Дмитрий Лапшин (lapshin)

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

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

Стиль:

  1. Отступы разные в разных файлах. И в пределах одного.

11.

Note: See TracTickets for help on using tickets.