Opened 3 years ago

Closed 3 years ago

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

WW_Matrix Kozyrev (lab_07)

Reported by: Andrei Kozyrev Owned by: Антон Филатов
Component: WW Matrix Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Антон Филатов

Type: ожидается проверкаожидаются исправления

тесты пройдены
Стиль:

  • нужно больше констов богу констов. Константные методы и аргументы!
  • сделайте MINIMUM_MATRIX_SIZE константным приватным полем. Дефайны в плюсах не так хорошо смотрятся

лайк за operator_plus_minus

  • из деструктора и оператора присваивания можно выделить копипасту в отдельную функцию

8,5/10

comment:2 Changed 3 years ago by Andrei Kozyrev

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 3 years ago by Антон Филатов

Type: ожидается проверкаожидаются исправления

Вот это запрещённое действие

this->~Matrix();

Так можно делать только в очень ограниченном наборе случаев.
В вашем случае раз вы делаете init сразу после деструктора и раз у нас не параллельная программа, то так делать можно. Но я буду настаивать на том, чтобы вы к такому коду не привыкали и переделали - выделите отдельную функцию.
9,8/10

comment:4 Changed 3 years ago by Andrei Kozyrev

Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:5 Changed 3 years ago by Антон Филатов

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

10/10

Note: See TracTickets for help on using tickets.