Opened 4 years ago

Closed 4 years ago

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

ДЗ №1. Матрицы

Reported by: kaminskij.vasilij Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha1-milestone2
Component: HA#1 matrices Version: 1.0
Keywords: Cc:

Description

Здравствуйте!

Прошу проверить моё решение.

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

Change History (5)

comment:1 Changed 4 years ago by Vladimir Rutsky

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

Замечания:

  1. В вашем решений присутствуют ошибки из списка из условия задачи "часто возникающие ошибки":
  • № 9 (matrix::print()),
  • 18.c) (std::ifstream::failure),
  • 13 (цикл для освобождения памяти реализован два раза в operator= и ~matrix).
  1. В matrix::mult происходит утечка памяти: вы создаёте matrix в динамической памяти (matrix* tmp = new matrix(N_, m.M_);), но никогда не освобождаете память, выделенную для объекта matrix (т.е. где-то должен быть аналог delete tmp).
  1. В operator= вы делаете value_ = matrix.value_, что приводит к тому, что один и тот же буфер используется в нескольких экземплярах класса matrix, что скорее всего приведёт к двойному освобождению памяти буфера.

comment:2 Changed 4 years ago by kaminskij.vasilij

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

Здравствуйте!

Внёс изменения в соответствии с вашими замечаниями. Прошу проверить.

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

comment:3 Changed 4 years ago by Vladimir Rutsky

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

Василий, вы закоммитили исправления в Subversion? Последний раз ваше решение менялось в Subversion 20 сентября.

comment:4 Changed 4 years ago by kaminskij.vasilij

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

Приношу свои извинения. Действительно, не закоммитил исправления. Теперь всё должно быть хорошо.

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

comment:5 Changed 4 years ago by Vladimir Rutsky

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

Замечания:

  1. В аргументах matrix::allocation(int N_, int M_) стоит использовать тип size_t, вместо int. Плюс аргументы стоит назвать без подчеркивания.

Решение зачтено (уверен, что вы бы исправили замечание выше в оставшуюся попытку сдачи).

Note: See TracTickets for help on using tickets.