Opened 3 years ago

Closed 3 years ago

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

WW #7

Reported by: Kuznetsov Kirill Owned by: Святослав Власов
Component: WW Matrix Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления
  1. Matrix::get должен возвращать значение, а не писать его в консоль
  2. Память под матрицу можно выделить за меньшее количество вызовов new, на прошлом семестре я вам объяснял как
  3. Вместо поэлементного копирования или заполнения матрицы можно использовать memcpy или memset
  4. В операторе *= лишний раз выделяешь память под временный объект, который потом копируешь в *this. Это лишняя аллокация и копирование, можно её избежать, лучше всего используя swap trick.
  5. В операторе присваивания swap trick тоже помог бы убрать копи-пасту

Пока 6/10

comment:2 Changed 3 years ago by Kuznetsov Kirill

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

Забыл обновить тикет. Вроде все поправил

comment:3 Changed 3 years ago by Святослав Власов

Resolution: задача сдана
Status: assignedclosed
  1. memset перед memcpy в конструкторе копирования избыточный

В остальном ок, 10/10

Note: See TracTickets for help on using tickets.