Opened 3 years ago

Closed 3 years ago

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

WW_Matrix, Garaev Timur, lab_07

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

Description

Почему-то меня спасло удаление конструктора без параметров. но я обещаю разобраться (или спросить у вас в будущем), почему так произошло.

Change History (6)

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

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

Странно работает на вычитание, а также с сегфолтом падает на умножение.
Про вычитание
init a 2 3

  • a a

выводит
0 0
0 0
0 0

а должен был
0 0 0
0 0 0

По умножению:
init a 2 1
init b 1 2

  • a b - сегфолт

По стилю:

  • богу констов не хватает констов: константные методы, константные аргументы. Везде, куда сможете поставить конст - ставьте
  • кажется, что вызывать копирование в операторах += избыточно. Можно же просто выразить его напрямую через +. Типа *this = *this + m

5/10

comment:2 Changed 3 years ago by Garaev Timur

Version: 1.02.0

Бог констов будет доволен, думаю.

Поправил еще size_t в цикликах, теперь вообще красота, кажется.
Уможение тоже в порядке!

comment:3 Changed 3 years ago by Garaev Timur

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

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

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

по стилю да, теперь всё пучком. А вот оператор - у вас немного не туда воюет.

- a b

всё же это обозначает а - b, а не b - a

9/10

Last edited 3 years ago by Антон Филатов (previous) (diff)

comment:5 Changed 3 years ago by Garaev Timur

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

Не видел обновление тикета почему-то :с
Вроде все поправил и работает!

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

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

10/10

Note: See TracTickets for help on using tickets.