Opened 3 years ago

Closed 3 years ago

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

HW_Huffman balaj.pavel hw_02

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

Description


Change History (6)

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

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

Корректность: пустой файл? На немаленьких тестах разжатый файл отличается от исходного причём сразу в первом байте. На больших тестах TL. 8.

Тесты: хде?

Стиль:

  1. shared_ptr — у нас разделяемое владение?
  2. Ты кажется хранишь кодируемые чарики в std::vector<bool>. Зочем.
  3. Позицию в стриме лучше задавать тем же типом, что в std::iostream.
  4. static const int bitsInByte = 8; а стандартные константы? Тем более дальше константой не пользуешься, хотя мог бы.
  5. На вещи не выставленные в заголовках хотелось бы static/namespace {.
  6. Увидел класс дерева, лайк. А где класс архиватора, просто в мейне код написан? Аккуратно всё разделено остальное, лайк.

6.

comment:2 Changed 3 years ago by balaj.pavel

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

comment:3 Changed 3 years ago by balaj.pavel

Добавил обработку исключений

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

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

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

Тесты: 5.

Стиль:

  1. У контейнеров которые хранят элементы подряд есть компараторы.

9.

comment:5 Changed 3 years ago by balaj.pavel

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

Изменил компаратор

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

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

+

Note: See TracTickets for help on using tickets.