Opened 4 years ago

Closed 4 years ago

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

HW_03 kozubaev.nurmukhammad

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

Description


Change History (3)

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

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

Корректность: Почти работает: не умеешь в произвольный порядок аргументов. 7/9.

Тесты: один тест это богато! И тестирует он занятно. 0.

Стиль:

  1. Пробелы.
  2. Пространства имён.
  3. Магические константы.

6/8.

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

  1. Сырые указатели. Более того, если контейнер содержит какие-либо указатели, возникает вопрос: а надо ли ему?
  2. Почему архиватор сам выводит свою статистику?

4/5.

comment:2 Changed 4 years ago by kozubaev.nurmukhammad

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

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

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

Корректность: 9/9.

Тесты: сами падают. Учитывая пункт выше. Сами тесты это три примерно случайных теста, которые мало что проверяют. 3/8.

Стиль:

  1. Пространства имён.
  2. Магические константы всё ещё магичны.
  3. А зачем глобальные переменные в main?

6/8.

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

  1. Всё публично?
  2. Сырые указатели всё ещё сырые.
  3. Исключения не относятся к std::exception.

4/5.

Note: See TracTickets for help on using tickets.