Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

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

Отправил решение, но не создал тикет

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

Description

Доброго времени суток, Владимир!

К сожалению, где-то пропустил информацию насчёт необходимости создавать тикет после публикации решения, хотя само решение выложил ещё до дедлайна и сразу же отписался об этом в комментарии к решению в личном кабинете compscicenter.ru.

Надеюсь на ваше понимание.

С уважением, Каминский Василий.

Change History (2)

comment:1 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. Ловите исключения по константной ссылке, если не собираетесь их модифицировать:
	catch (std::ifstream::failure const & e) {
  1. Выделение и освобождение памяти под матрицы лучше вынести в отдельные функции.
  1. Решение должно быть в подпапке ha1.
  1. Матрица, созданная в mult(), никем не освобождается, что приводит к утечке памяти.

В целом это очень подозрительно, когда метод возвращает ссылку на объект --- это обычно означает, что класс владеет какой-то коллекцией объектов внутри себя и позволяет их модифицировать снаружи.

В вашем случае стоило создавать matrix tmp; на стеке и вернуть копию этой матрицы из mult().

К сожалению, я не могу зачесть ваше решение с данными ошибками.

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

comment:2 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-deadline

Milestone ha1-deadline deleted

Note: See TracTickets for help on using tickets.