Opened 4 years ago

Closed 4 years ago

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

HW #3 chaykova.anastasiya

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

Description


Change History (3)

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

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

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

  1. То падаешь на сжатии, то расжимаешь не то. Не сломалось лишь на малой части.

3/9.

Тесты: не собираются.

Стиль:

  1. Магические константы!
  2. throw() такой конструкции в языке уже нет.
  3. Вызов конструктора, потом заполнение полей: а может конструктор неправильный?
  4. Предложения через запятую. Даже больше: они бесполезны.

5/8.

Архитектура:

  1. Столько публичных методов!
  2. Сырые указатели.

4/5.

comment:2 Changed 4 years ago by chaykova.anastasiya

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

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

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

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

  1. Если всего 1 вид байт, разжимаешь всегда не то.
  2. Медленно.

6/9.

Тесты: тестов на построение дерева не видно. 6/8.

Стиль:

  1. Пробелы. Отступы.
  2. Пространства имён.
  3. ASCII_CODE какой код? какой ASCII? При чём это всё тут?
  4. std::string(argv[i]) дублирование нетривиального выражения.

5/8.

Архитектура:

  1. write_statistics, for testing: get_statistics. Кажется правильно было бы оставить лишь второй.
  2. Вообще, для тестов выставлять публичное кажется плохим планом. Или оно честно публичное, или делай тест другом. А так получается, что как бы выставлена ручка, но не пользователю.

4/5.

Note: See TracTickets for help on using tickets.