Opened 3 years ago

Closed 3 years ago

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

HW #2 (Huffman) volkov.ivan

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

Description


Change History (6)

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

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

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

Тесты:

  1. А можно сделать чтобы make и тесты собирал, и делал это правильно?
  2. CHECK_THROWS.

4.

Стиль:

  1. Ох сколько констант магических.
  2. Если уж у тебя есть отдельно битовый ввод/вывод может он достоин отдельного файла?
  3. Массивы тоже можно инициализировать в списках инициализации.
  4. shared_ptr не самый оптимальный зверь. Хранить битовые строчками в виде std::string меня тоже смущает.
  5. print_output_flag: не очень понятно почему _flag в конце. Кстати, ИМХО было бы логичнее, если бы мейн как интерактивный с пользователем чувак сам выводил статистику.
  6. Зачем закрывать-открывать файл, когда можно его перемотать.

7.

comment:2 Changed 3 years ago by Ivan Volkov

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

Тесты

  1. А как правильно? (Локально у меня вроде все нормально) ((( Добавил hw_02_test в цель по дефолту, если Вы это имели в виду )))
  1. Fixed

Стиль

  1. Постарался константы убрать / вынести (кроме 0/1, без них совсем чума какая-то получается)
  2. Fixed
  3. А вот это неожиданно. Fixed
  4. shared_ptr переписал на сырые указатели, вроде утечек не должно быть. std::string переписал на std::vector<bool> (правда без -O2 это пока работает чуть дольше на архивации, а еще пришлось в Tree::get_code() убрать проверку на то, что символ к которому запрашивают код встречался >0 раз, потому что == "?" уже не прокатит)
  5. Fixed, так и сделал.
  6. Fixed

comment:3 Changed 3 years ago by Ivan Volkov

А, про первый пункт я еще видимо понял, что у меня тесты каждый раз пересобираются потому что они помечены как .PHONY (чтобы make не смотрел на одноименную папку), но что-то ничего лучше чем переименовать цель / папку, я не придумал (или так можно?)

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

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

У тебя цель test не собирает файл test. Я бы цель переименовал.

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

Тесты: 5.

Стиль:

  1. Сырые указатели сразу вызывают вопросы о правиле 3/4/5 и прочем аду)
  2. В паре мест пробелы.
  3. Глобальные переменные в разборе аргументов не то чтобы хорошо, но в мейне приемлемо.

8.

Время на исправление есть.

comment:5 Changed 3 years ago by Ivan Volkov

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

переименовал в hw_02_test

  1. Переделал, теперь все хранится в unique_ptr (и использую Node* во временных местах)
  2. Что нашел -- пофиксил
  3. fixed

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

Resolution: задача сдана
Status: assignedclosed

Ок.

Note: See TracTickets for help on using tickets.