WW #14

[Sokolov Viacheslav]

nevermind

[Sokolov Viacheslav]

промахнулся tab-ом

[Sokolov Viacheslav]

8 class ExitHandler? : public cli::CommandHandler?

final?

26 UndefinedHandler?(const std::string &cmd)

explicit?

85 id1 = std::stoi(i.substr(1));
получение номера регистра таким способом многократно дублируется

67 const char *what() const throw();
noexcept. throw() - deprecated. Кроме того, override.

71 exType getType() const;
можно noexcept

11 return exDetails[_id].c_str();
здесь предполагается, что нумерация непрерывная и начинается с 0. Лучше поставить assert.

7 enum exId
какая мотивация использовать enum, а не enum class?

32 std::shared_ptr<mtxStorage> _mtx;
какая мотивация использовать здесь shared_ptr (а не unique_ptr)?

11 class Matrix
final?

По условию сообщение в случае out_of_memory должно быть в точности "Unable to allocate memory."

24 using cliHandlersMap = std::unordered_map<std::string, std::vector<std::shared_ptr<CommandHandler>>>;
какая мотивация использовать здесь shared_ptr?

37 CommandLineInterface?(const std::vector<std::shared_ptr<CommandHandler>> &handlers);

explicit?

18 std::string getCommand() const;
const& ?

CommandHandler?:
что с деструктором? Либо public: virtual, либо protected:

47 void printPrettyPrefix();
const?

43 std::string getCommand();

const?

67 if (_handlers[command].size() == 0)
здесь произойдет добавление пустого вектора в хранилище. Непонятно, зачем - лучше использовать find / count.

69 command = "";

continue?

71 for (auto handler : _handlers[command])
const auto&?

112 if (_pretty)
113 {
114 printPrettyPrefix();
115 }
зачем?

5 class MatrixCore?

final?

8 MatrixCore?(size_t registersNumber = 10);
explicit?

название MatrixCore? не очень-то понятное: что делает этот класс, за что отвечает?
Ядро - это все, что угодно. Лучше именовать более конкретно.

76 Matrix operator+(const Matrix &first, const Matrix &second)
77 {
78 Matrix result(first);
79 return result += second;
80 }

здесь лишнее копирование на выходе из функции.