Opened 3 years ago

Closed 3 years ago

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

WW_Matrix Kiselev Igor lab_07

Reported by: kiselev.igor Owned by: Антон Филатов
Component: WW Matrix Version: 2.0
Keywords: Cc:

Description


Change History (3)

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

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

неправильно назван исполняемый файл

По тестам. Проверьте, как вы выводите матрицу на экран. Иногда вместо

1 1
1 1

вы выводите

11
11

и в умножении есть баг

init a 2 1
init c 2 2
* c a // или *= c a

Вылезает сообщение о несовпадении размерностей, но умножение происходит

Стиль:

  • больше констов богу констов. Некоторые функции могут быть константными
  • в операторе присваивания и деструкторе есть общий код, который можно вынести в отдельную функцию (только не вызывайте деструктор напрямую - это плохой стиль)
  • по поводу mode в add и mull. Плохо, когда есть безымянные константны. Лучше сделайте enum. А ещё кажется, что можно operator += выразить через оператор + (или наоборот)
  • уберите комментарии из функции print

пока 4/10

comment:2 Changed 3 years ago by kiselev.igor

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

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

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

Тесты проходят
Стиль:

  • оператор +, -, * могут быть константными. Они же не меняют this. Правда, с вашей реализацией через неконстантные фукнции add, sub и mul, это невозможно.

9,5/10

Note: See TracTickets for help on using tickets.