Opened 4 years ago

Closed 4 years ago

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

HW_03 lopatin.mikhail

Reported by: lopatin.mikhail Owned by: Дмитрий Лапшин (lapshin)
Component: HW #3 (Huffman) Version: 3.0
Keywords: Cc:

Description


Change History (3)

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

Owner: changed from Дмитрий Лапшин (lapshin) to lopatin.mikhail
Summary: HW_03 → HW_03 lopatin.mikhail
Type: ожидается проверка → ожидаются исправления
Version: 1.0 → 2.0

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

  1. Валгриииинд.
  2. Пару раз тайм лимит (реально долго).
  3. Но работает! 6/9.

Тесты: тестируют. Но только разбор аргументов. 3/8.

Стиль:

  1. Пространства имён.
  2. static переменные. Ну в мейне-то зачем, он всегда один?!
  3. Константы енамами фу.
  4. HALF_BAIT чта. Ни имя, ни значение как-то не подсказывают смысла.
  5. Почему ZipException?

4/8.

Архитектура:

  1. CLI: так конструктор или публичный метод?

1 Почему архиватор сам печатает?

  1. shared_ptr: неплохо, но кажется разделённого владения нет...
  2. Цикл shared_ptr.

3/5.

comment:2 Changed 4 years ago by lopatin.mikhail

Owner: changed from lopatin.mikhail to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 2.0 → 3.0

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

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

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

  1. Валгринд на всех тестах.
  2. Медленно целиком.

Но работает... 6/9.

Тесты: их мало. Собственно алгоритм тестриуется одним примитивным тестом. 3/8.

Стиль:

  1. Неймспейс.
  2. Пробелы.
  3. strcmp, когда есть string/string_view.
  4. Константы. То что ты запихал каждую 8 в CHAR_BIT не даёт тебе логического права от неё считать размер инта (не-всегда-4, уж лучше sizeof(uint32_t)) и прочее.

4/8.

Архитектура:

  1. shared_ptr это хорошо, но раздельного владения нет, а ещё у тебя цикл родитель—ребёнок.

4/5.

Note: See TracTickets for help on using tickets.