HA#1

[Vladimir Rutsky]

Замечания:

1. Вы включаете cpp-файл в hpp-файл --- так делать нельзя.

#pragma once

#include "matrices.cpp"

В hpp-файле должны находиться *объявления* переменных, классов, функций, которые представляют внешний интерфейс к функциональности, реализованной в cpp-файле.
В cpp-файле должны находиться *определения* (и реализация).

В случае, когда вся функциональность представима в виде класса, можно этот класс целиком поместить в заголовочном файле --- так поступают при написании header-only библиотек (т.е. библиотек, состоящих только из зголовочных файлов).
У этого подхода есть ряд плюсов, например, обьявления и определения в одном месте, и для использования такой библиотеки, достаточно просто включить её (не требуется компиляция и линковка дополнительных единиц трансляции).
Но есть и минусы, например, если хоть как-то поменять заголовочный файл header-only библиотеки, нужно пересобирать все единицы трансляции, транзитивно включающие его.

Вам необходимо оставить в заголовочном файле matrices.hpp объявление класса (включая его методы), а реализовать методы в matrices.cpp (если не разберетесь как это сделать --- пишите, я объясню).

2. В auto i = 0, тип i оказывается int, а его диапазона значений может не хватить. Используйте явно size_t.

[Григорий Каргин]

Учел замечания

[Vladimir Rutsky]

Замечания:

1. Заголовочный файл должен быть самодостаточным: при его включении в любой cpp файл (даже пустой), не должно быть ошибок компиляции. Сейчас в заголовочном файле, например, используется тип size_t, но <cstddef> не подключен.

2. Сделайте метод print() константным, т.к. он не должен менять состояние класса: print() const.

[Григорий Каргин]

Учел замечания

[Vladimir Rutsky]

Решение зачтено.