Code review from Nastya aka BimodalCape O_o

[BimodaCape]

don't use absolute path, use relative one

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 3 in dbe072c

#include "D:\ASD\doctest.h"

don't use default values in public invocations, move fishy assignment to body, nullptr is strange why not empty array, "{" to new line

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 12 in dbe072c

Matrix(int** arr = nullptr, size_t n = 0, size_t m = 0) :n_(n), m_(m) {

rows are not initialized, is it supposed to be so?

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 15 in dbe072c

rows_.push_back(pair<size_t, list<pair<size_t, int>>>(i, list<pair<size_t, int>>()));

instead of size_t use auto keyword (will be automaticaly adjust to int type)

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 16 in dbe072c

for (size_t j = 0; j < m_; j++)

"{" to new line

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 18 in dbe072c

if (arr[i][j] != 0) {

"{" to new line

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 22 in dbe072c

if (rows_.back().second.empty()) {

remove if not used

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 27 in dbe072c

~Matrix() {}

add brackets

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 27 in dbe072c

~Matrix() {}

logic with that one is too complex, it can be simplified

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 47 in dbe072c

int* element = nullptr;

just return second element here, at the end return 0, int* element can be removed

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 96 in dbe072c

element = &it_j->second;

prefer to use _n, _m

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 189 in dbe072c

size_t n_, m_;

Unit test should test 1 item at a time, divide those to separate tests

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 194 in dbe072c

TEST_CASE("Testing Matrix") {

bad practice to assign elemnts inside function invocation

ASD_Labs/Matrix/Matrix/Matrix.cpp

Line 287 in dbe072c

CHECK_NOTHROW(t = m.Add(&m1));

[BimodaCape]

decided approach is too complex *(