Opened 3 years ago

Last modified 3 years ago

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

HW #2 (Huffman) bukreev.roman

Reported by: Roman Bukreev Owned by: Roman Bukreev
Component: HW #3 (Huffman) Version:
Keywords: Cc:

Description


Change History (1)

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

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

Корректность: Если в файле всего один вид байт падает. А ещё периодически сегфолтит без сообщений почему. 8.

Тесты: Ошибка линковки!

Стиль:

  1. Разбор аргументов очень страшный. У тебя ж уже есть переменные под параметры, зачем какие-то индексы сохранять и ждать следующей итерации? Флаг -f: значит следующий должен быть и это входной файл, поехали! Кстати strcmp: string/string_view?
  2. Волшебные числа.
  3. Сырые владеющие указатели.
  4. А double-то тебе зачем? Простой подсчёт не подходит?) Вообще тут возникает вопрос какие поля являются локальным состоянием а какие заслуживают быть полями класса.
  5. Цикл до eof.
  6. bool operator<(TreeNode a); const? Ссылка?
  7. А что всё публично-то? А точно ли всё в заголовке нужно всем пользователям?
  8. Ну и скажу, что индексировать знаковыми или неизвестно знаковым ли типом массивы плохой план.

4.

Note: See TracTickets for help on using tickets.