Opened 3 years ago

Last modified 3 years ago

#696 assigned ожидаются исправления

HW #3 (Huffman) milshin.maksim hw_02

Reported by: milshin.maksim Owned by: milshin.maksim
Component: HW #3 (Huffman) Version: 1.0
Keywords: Cc:

Description

Пока есть только рабочий архиватор и деархиватор

Change History (2)

comment:1 Changed 3 years ago by milshin.maksim

Component: HW #2 (X0)HW #3 (Huffman)
Summary: HW #2 (X0) milshin.maksim hw_02HW #3 (Huffman) milshin.maksim hw_02

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

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

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

Стиль:

  1. Методы лучше называть от глаголов, read/performReading.
  2. std::map: лучше уж std::uordered_map, а ещё лучше std::vector/массив.
  3. Мне не нравится что архиватор сам выводит статистику: откуда он знает куда это делать?
  4. Волшебные числа.
  5. while (!eof)
  6. Что-то я не понял зачем класс HuffmanTree ради одно статик метода, который возвращает результат через параметр...
  7. for (auto x): ты очень часто хочешь for (auto const &x), не будет копии, const иногда можно забывать бесплатно. Хотя строить дерево не строя его как задумка кажется интресной.

5.

Note: See TracTickets for help on using tickets.