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
Owner: | changed from Дмитрий Лапшин (lapshin) to Ivan Volkov |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 3 years ago by
Owner: | changed from Ivan Volkov to Дмитрий Лапшин (lapshin) |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Тесты
- А как правильно? (Локально у меня вроде все нормально) ((( Добавил
hw_02_test
в цель по дефолту, если Вы это имели в виду )))
- Fixed
Стиль
- Постарался константы убрать / вынести (кроме 0/1, без них совсем чума какая-то получается)
- Fixed
- А вот это неожиданно. Fixed
shared_ptr
переписал на сырые указатели, вроде утечек не должно быть.std::string
переписал наstd::vector<bool>
(правда без-O2
это пока работает чуть дольше на архивации, а еще пришлось вTree::get_code()
убрать проверку на то, что символ к которому запрашивают код встречался >0 раз, потому что== "?"
уже не прокатит)- Fixed, так и сделал.
- Fixed
comment:3 Changed 3 years ago by
А, про первый пункт я еще видимо понял, что у меня тесты каждый раз пересобираются потому что они помечены как .PHONY
(чтобы make
не смотрел на одноименную папку), но что-то ничего лучше чем переименовать цель / папку, я не придумал (или так можно?)
comment:4 Changed 3 years ago by
Owner: | changed from Дмитрий Лапшин (lapshin) to Ivan Volkov |
---|---|
Type: | ожидается проверка → ожидаются исправления |
У тебя цель test
не собирает файл test
. Я бы цель переименовал.
Корректность: 15.
Тесты: 5.
Стиль:
- Сырые указатели сразу вызывают вопросы о правиле 3/4/5 и прочем аду)
- В паре мест пробелы.
- Глобальные переменные в разборе аргументов не то чтобы хорошо, но в мейне приемлемо.
8.
Время на исправление есть.
comment:5 Changed 3 years ago by
Owner: | changed from Ivan Volkov to Дмитрий Лапшин (lapshin) |
---|---|
Type: | ожидаются исправления → ожидается проверка |
переименовал в hw_02_test
- Переделал, теперь все хранится в
unique_ptr
(и используюNode*
во временных местах) - Что нашел -- пофиксил
- fixed
Note: See
TracTickets for help on using
tickets.
Корректность: вах. 15.
Тесты:
make
и тесты собирал, и делал это правильно?CHECK_THROWS
.4.
Стиль:
shared_ptr
не самый оптимальный зверь. Хранить битовые строчками в видеstd::string
меня тоже смущает.print_output_flag
: не очень понятно почему_flag
в конце. Кстати, ИМХО было бы логичнее, если бы мейн как интерактивный с пользователем чувак сам выводил статистику.7.