Home Assignment #1

[Филипп]

Привет! Пара вещей на исправления:

1. std::invalid_exception живет в отдельном хедере, нужно его подключать
2. Замечания 4, 6, 9, 24 из частых ошибок
3. В file.open в некоторых ситуациях передаются некорректный аргумент

И в общем по решению:

1. Из операторов лучше возвращать не указатели, а объекты (или ссылки, если это возможно -- если создан не временный объект). И внутри операторов в данном случае предпочтительно использовать не поинтеры: разыменование указателя занимает процессорное время.
2. Логику чтения из/вывода в файлы лучше отделить от класса матриц. Для этих целей можно написать get(), возвращающий double&.

[Иван]

Добрый день! У меня нет идей исправления пункта 3. Почитал решения аналогичной проблемы на форумах, но они либо платформозависимые, либо используют конструктор (если я правильно понял) std::ifstream(file) != 0, что на мой взгляд является тем же самым действием, что я использую при создании file. Поэтому я проверяю существование файла через is_open(). Я могу рассчитывать на подсказку? =)

[Филипп]

Valgrind находит обращение к невыделенной памяти в кейсах smoke-теста, где число аргументов четно: код все-равно пытается открыть файл по имени последнего отсутствующего аргумента. Так не надо делать :)

И еще про важные замечания: в Matrix.hpp включен <exception> и не включен напрямую заголовочный файл с (std::)size_t. И можно смело добавить константности некоторым методам.

[Филипп]

В matrices.hpp остались включенными заголовочные файлы про стримы.

И косметические вещи/улучшения производительности (факультативно):

1. В answer.get(i,j) = array_[i][k] * factor.get(k, j) + answer.get(i, j) можно заиспользовать +=. А лучше каждую сумму складывать в локальную переменную, чтобы обращений по указателям было меньше.
2. Лучше делать array_ = std::nullptr, чем array_ = new double*[0] -- первое легче отловить и память на хранение размера массива не выделяется.

[Иван]

Исправил Ваши замечания. Перенес iostream и fstream в matrices.cpp.