Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

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

WW #7

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

Description


Change History (4)

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

Type: ожидается проверкаожидаются исправления
  1. Умножение работает неправильно
    init a 2 1
    init b 1 2
    set a 0 0 1
    set a 1 0 2
    set b 0 0 3
    set b 0 1 4
    
    * a b
    3 0
    6 0
    
  2. Служебные функции нужно убрать в приватную область видимости
  3. Память под матрицу можно выделить одним-двумя вызовами new, вместо цикла
  4. memset и memcpy позволяют быстрее скопировать или заполнить элементы
  5. В присваивающих операторах +=,-=,*= у тебя получается лишняя аллокация и копирование данных, его можно избежать.
  6. В Matrix::arithm не нужно использовать magic-numbers для определения типа операции, лучше завести enum в приватной области видимости и использовать константы оттуда.

Пока 6.5/10

comment:2 Changed 3 years ago by em.vladislv

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

все исправил

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

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

Оператор сравнения при попытке сравнить матрицы разного размера падает в сегфолт

9/10

comment:4 Changed 3 years ago by em.vladislv

"Можно считать, что класс используется корректно (например, сложение производится для матриц с одинаковыми размерностями)." - условие лабораторной.

Мне несложно было добавить одну строчку, но судя по условию, не нужно было учитывать, что пользователь будет сравнивать матрицы размерами 2x5 и 7x7

Note: See TracTickets for help on using tickets.