Opened 3 years ago

Closed 3 years ago

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

WW #4

Reported by: ushakov.vladislav 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. Выводишь лишние пробелы
  2. Если ввести длинную неизвестную команду, то вывод не соответствует ожидаемому.

Стиль -- 3.5/4

  1. Макрос container_of зависит от stddef.h, нужно включить этот заголовочный файл перед объявление макроса
  2. Так лучше не писать: 
    struct intrusive_list * list

При объявлении указателя пробел ставят либо до либо после *, но не вокруг.

  1. В дереве if-ов, которое в функции remove_node, кажется что 1-я и 3-я ветка ничем друг от друга не отличаются.
  2. При вызове функций, пробел между именем функции и открывающейся скобкой не ставится.

comment:2 Changed 3 years ago by ushakov.vladislav

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

Все исправил.

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

Type: ожидается проверка → ожидаются исправления
  1. Не надо выносить заголовочные файлы за гарды. Перед гардами ничего быть не должно.
  2. Зачем переменную act делать глобальной, если она используется только в функции main? Достаточно её в main и объявить. Старайся стремиться к большей локальности, переменные должны объявляться в том скоупе, в котором будут использоваться.

11.5/12

comment:4 Changed 3 years ago by ushakov.vladislav

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

Починил.

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

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

Зачем ты act начал выделять на куче?
Её прекрасно можно оставить на стекфрейме main, ты же не планируешь её использовать после завершения функции main.

Инклюды от макросов и гардов лучше бы отделять пустой строкой.

11.8/12

Note: See TracTickets for help on using tickets.