HA-1

[Волков Даниил]

Please check my homework

[Vladimir Rutsky]

Замечания:

1. В заголовочном файле отсутствует страж включения.

2. Заголовочный файл должен быть самодостаточным, например, в matrices.hpp используется std::string, но <string> не включается.

3. Зачем вы используете метки?

	if (!fin.is_open())
	{
		clean:
		eraseMatrix_(prevResult);
		eraseStr_(s);
		assert(fin.is_open());
	}

4. Почему вы называете функции с подчеркиванием на конце (inputStr_, printMatrix_, calculate_)? Приватные функции/переменные для единицы трансляции необходимо пометить как static, либо поместить в анонимную область видимости, но здесь не тот случай. В целом, в C++ для приватных функций/переменных есть специальный синтаксис (в отличие, например, от условно приватных атрибутов в языках программирования с полной интроспекцией, вроде Python).

5. Почему вы дописываете в конец пустой строки, а не перезаписываете её здесь?

	std::string* s;
	s = new std::string[argc];
	for (size_t i = 0; i < argc; ++i)
	{
		s[i] += argv[i]; // s[i] = argv[i];
	}

6. Не включайте лишние файлы, например <cassert> в main.cpp.

7. assert(0); это плохой способ обрабатывать ошибки. Невыполнение assert приводит к ненормальному, "мгновенному", завершению программы, при этом не будет раскручен стек и вызваны деструкторы объектов, что недопустимо в C++.

assert стоит использовать для проверки внутренних инвариантов, нарушение которых является неожиданным и критическим, и когда продолжение программы бессмысленно. Плюс падание через assert отличается от обычного завершения с кодом ошибки (возвращение ненулевого кода в main()): обычно делается дамп памяти и ОС может логировать такие ошибки и предлагать отправить "отчет об ошибке".

В данный задаче некорректный размер входных матриц это один из ожидаемых вариантов входа, который должен быть аккуратно обработан.

Такого рода ошибки стоит обрабатывать либо используя механизм исключений (что не требуется в данной задаче), либо передавая из функции код ошибки.
Добавив код ошибки в matadd_, matmul_ также позволит не передавать внутрь const std::string *s (что сейчас выглядит странным: почему функция перемножения матриц должна делать какую-то дополнительную работу, кроме перемножения матриц?)

8. При запуске без аргументов программа падает.

9. При запуске ./matrices 5.txt --a 2.txt программа падает с assert-ом Assertion fin.is_open()' failed.`

[Волков Даниил]

Доброго времени суток!
Замечания 1-9 пофиксил.

[Vladimir Rutsky]

Замечания:

1. prevResult в readMatrix не используется.

matbool readMatrix (const std::string &nameFile, double* prevResult)

2. Данная строка выглядит подозрительно:

matbool result = readMatrix(s[0], result.first.p_Matrix); 

вы в инициализации используете поле переменной, которую инициализируете.

3. Было бы разумней убрать из printMatrix обработку ошибок и освобождение памяти для матрицы: пусть эта функция только выводит на экран матрицу, не меняя её. Это расширит способы её использования.

4. В случае возникновения ошибки вы в некоторых случаях вызываете eraseMatrix от неинициализированной матрицы, что приводит к undefined behavior. Например, при вызове ./matrices A_3x7.txt --add B_7x13.txt.

[Волков Даниил]

Доброго времени суток!
Поправил код согласно замечаниям 1-4.

[Vladimir Rutsky]

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

[Vladimir Rutsky]

Milestone ha1-deadline deleted