Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

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

For check: Home assignment 1 by Kiryushkina Valentina

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

Description


Change History (5)

comment:1 Changed 8 years ago by Vladimir Rutsky

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

Замечания:

  1. Заголовочный файл в данной задаче должен называться matrices.hpp.
  1. Страж включения должен оборачивать весь файл:
#include <iostream>

#ifndef HA1_MATRICIES_H
#define HA1_MATRICIES_H

...
  1. В программе наблюдаются утечки памяти. Например, вы не освобождаете память результирующей матрицы, которую выводите на экран; вы не освобождаете память при возникновении ошибочной ситуации (например, при несоответствии размеров матриц).
  1. Вы некорректно освобождаете память в free_matrix:
void free_matrix(const size_t row_num, double **matrix)
{
    for (size_t i = 0; i != row_num; ++i) {
        delete [] matrix[i];
    }
    delete matrix;
}

matrix был выделен как new [], значит и освобожден должен быть с помощью delete[].

comment:2 Changed 8 years ago by Valentina Kiryushkina

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

comment:3 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. Если второй или последующий файл не удаётся открыть, то вы бросате исключение, но не ловите его, что приводит к аварийному завершению программы.

Это критическая ошибка, исправьте её, пожалуйста, в ближайшее время.

Также небольшое замечание: выводите в конце сообщений об ошибках перевод строки (вы делаете это не всегда).

comment:4 Changed 7 years ago by Vladimir Rutsky

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

Исправления не внесены, решение не зачтено.

comment:5 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-deadline

Milestone ha1-deadline deleted

Note: See TracTickets for help on using tickets.