gitluup
commented
5 years ago Абсолютно все замеры времени работы оператора* должны быть в main'e. Это объясняется тем, что мы хотим измерять всё время его работы, а не только какую-то его часть.
У тебя по-прежнему табы вместо пробелов - это нужно поправить
https://github.com/Ivanova475/cplusplus-tasks/blob/c3b3d52a222491f9e60fc4f4e74058864f4b9727/Matrix/Matrix/matrix.cpp#L148 Не совсем понятно, зачем нужна эта строчка, если потом сразу же get вызывается. Ведь get сначала ждёт, а потом выдаёт значение.
Табы надо заменить на пробелы!
https://github.com/Ivanova475/cplusplus-tasks/blob/7b4f4144c977c293705d8dda703fb8a5318d29ab/Matrix/Matrix/matrix.cpp#L5 В эту функцию в качетсве аргумента rhs_part передаётся на самом деле не кусочек rhs, а транспонированный кусочек rhs. Я бы это как-то отразил в названии аргумента. Например, Rhs_part_transposed. А то так люди с первого взгляда могут подумать, что ты матрицы перемножать не умеешь)
https://github.com/Ivanova475/cplusplus-tasks/blob/7b4f4144c977c293705d8dda703fb8a5318d29ab/Matrix/Matrix/main.cpp#L257 Когда тестов становится всё больше и больше, имеет смысл вынести запуск всех тестов в отдельную функцию, например, TestAll, и запускать её и только её в main'е.