c++ HA1 dudina.olga

[Vladimir Rutsky]

Замечания:

1. Решение разбито на файлы не так, как требовалось в условии.

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

3. Не следуюет включать явно не требующиеся заголовочные файлы. Например в Main.cpp не требуется <vector>.

4. В данной задаче запрещено использовать std::vector, вся работа с динамической памятью для элементов матрицы должна быть выполнена вручную (используя new[]/delete[], и чем проще будет решение, тем меньше будет работы с динамической памятью).

5. Используйте тип size_t для индексов и размеров.

6. Передавайте объекты, которые не должны быть модифицированы, по константной ссылке:

	Matrix calculate(const std::vector<Command> & commands) const;
	Matrix readFromFile(const std::string & filename) const;

7. Вы можете открыть файл в конструкторе:

	ifstream myfile(filename);

8. В программе наблюдаются утечки памяти. Например, вы почти всегда не освобождаете память, выделенную я матрицы (не вызываете Matrix.deallocate()).

[dudina.olga]

Попыталась исправить разбиение на файлы так, как это требовалось. Оказалось, это уже достаточно проблемно сделать. Надеюсь, это не такая большая проблема.

И не поняла, как можно открыть файл в конструкторе

ifstream myfile(filename);

Там же в цикле открываются разные файлы.

[Vladimir Rutsky]

Replying to dudina.olga:

Попыталась исправить разбиение на файлы так, как это требовалось. Оказалось, это уже достаточно проблемно сделать. Надеюсь, это не такая большая проблема.

На первый раз пусть будет так, но на будущее учтите, что нужно строго следовать вопросам оформления задач --- большую часть задач я проверяю скриптами, и такие вещи как имена файлов решения или имена классов (если они указаны в условии задачи) являются строго зафиксированными.

И не поняла, как можно открыть файл в конструкторе

ifstream myfile(filename);

Там же в цикле открываются разные файлы.

Замечания относится к следующей функции:

Matrix MatrixCalculator::readFromFile(const std::string &filename) const
{
	ifstream myfile;
	myfile.open(filename);

	if (myfile.is_open())
	{
...

здесь вы читаете только один файл.

[dudina.olga]

Окей, учту четкое разбиение на файлы в следующих работах.

А, поняла с конструктором. Я подумала про конструктор MatrixCalculator?.

[Vladimir Rutsky]

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

[Vladimir Rutsky]

Milestone ha1-deadline deleted