Change History (3)

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

Type: ожидается проверкаожидаются исправления
  1. Валгринд сильно ругается на операторах умножения
  2. Служебные функции mem_alloc, fill_zero, ... нужно спрятать в приватную область видимости
  3. Память под матрицу можно выделить одним-двумя вызовами new, и я уверен, что ты даже знаешь как :)
  4. Давай сделаем swap-trick в операторе присваивания и избавимся от копи-пасты в нем
  5. В операторе *= у тебя происходит лишнее копирование в строчке *this = c. От него можно избавиться, если и тут заюзать swap-trick
  6. Неприсваивающие операторы +,-,* можно реализовать через присваивающие и избавиться от копи-пасты.

Пока 6/10

comment:2 Changed 3 years ago by Наталья Марцинковская

Type: ожидаются исправленияожидается проверка
Version: 1.02.0
  1. уже не ругается
  2. done
  3. done (2)
  4. done
  5. done
  6. done

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

Resolution: задача сдана
Status: assignedclosed
  1. Оператор == работает неправильно. Матрицы с разным размером должны возвращать false, а у тебя падает по ассерту.
  2. Умножение работает неправильно. У тебя ошибка в строчке for (size_t col = 0; col < _cols; col++), должно быть col < m._cols. Проверяй программы внимательнее!

8/10

Note: See TracTickets for help on using tickets.