Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#21 closed ожидается проверка (задача сдана)

Ha1

Reported by: Irina Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#1 matrices Version: 1.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 8 years ago by Vladimir Rutsky

Milestone: ha1-milestone1ha1-milestone2
Type: ожидается проверкаожидаются исправления

Замечания:

  1. Заголовочный файл в данной задаче должен называться matrices.hpp.
  1. Ловите объекты-исключения по константной ссылке, если не собираетесь их модифицировать:
catch (std::exception const &e)
  1. Передача по ссылке предполагает, что объект может быть изменён, но не удалён:
void throwSizeException(Matrix &base, Matrix &second, std::string action) {
...
	delete &second;
	delete &base;

Если предполагается передача владения объекта внутрь функции (и объект создан в куче), передавайте его по указателю.

  1. Зачем вы включаете <sys/types.h>? Этот заголовочный файл не входит в стандарт С++ (это часть POSIX стандарта), и вам не требуется. Например, тип std::size_t определён (в том числе) в <cstddef>.
  1. Вынесите код выделения и освобождения памяти в отдельные функции.
  1. При перемножении матриц происходит утечка памяти.
  1. При обработки ошибки открытия файла (например, когда файл не существует), память, выделенная под матрицы не освобождается.

comment:2 Changed 8 years ago by Irina

Type: ожидаются исправленияожидается проверка

comment:3 Changed 7 years ago by Vladimir Rutsky

Resolution: задача сдана
Status: newclosed

Замечания:

  1. В ~Matrix() следовало вызвать Matrix::removeData().

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

comment:4 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-milestone2

Milestone ha1-milestone2 deleted

Note: See TracTickets for help on using tickets.