Opened 5 years ago

Closed 4 years ago

#484 closed ожидаются исправления (задача сдана)

HW #1 (BMP) shein.andrey

Reported by: Андрей Шеин Owned by: Андрей Шеин
Component: HW #1 (BMP) Version: 2.0
Keywords: Cc:

Description


Change History (4)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Андрей Шеин
Summary: HW #1 (BMP) → HW #1 (BMP) shein.andrey
Type: ожидается проверка → ожидаются исправления

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

  1. По часовой это в какую сторону?
  2. А кроме этого в какой-то момент начинает творить дичь, что valgrind орёт.
  3. И в заголовках лажает.

4.

Стиль:

  1. Упаковывать структуру char-ов бесполезно, и так упакована.
  2. А вот хранить заголовки через char** и прыгать по ним руками это жестоко и бессмысленно. Давай структурки.
  3. Каким-то функциям просится static.

8.

comment:2 Changed 5 years ago by Андрей Шеин

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

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

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

Корректность: 20. Ура!

Стиль:

  1. А вот хранить заголовки через char** и прыгать по ним руками это жестоко и бессмысленно. Давай структурки. Я серьёзно, это очень трудно читать.
  2. Из-за пункта выше много магических констант.
  3. Местами, как мне кажется, слоёв указателей многовато. (void create_table(data ***d) → data **create_table(...)?).

6.

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

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.