Opened 7 years ago

Closed 7 years ago

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

HA1

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

Description

Здравствуйте! Проверьте, пожалуйста, домашнее задание.

Change History (1)

comment:1 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. В вашем решений присутствуют ошибки из списка из условия задачи "часто возникающие ошибки":
    • 26 (используйте тип size_t вместо int для индексации и размеров: print_matrix(double ** matrix, int rows_number, int columns_nomber)),
    • 10 (read_matrix(std::string const & file_name, ...)),
    • 12 (при возникновении ошибок вы не освобождаете память от результирующей матрицы и других объектов).
  1. В matrices.cpp стоит включить matrices.hpp.
  1. Не стоит выделять в динамической памяти то, что можно использовать в памяти на стеке. Вместо:
int * res_rows_number = new int(0);
int * res_columns_number = new int(0);

...
result_matrix = read_matrix(file_matrix, res_rows_number, res_columns_number);

Стоило использовать res_rows_number, res_columns_number на стеке:

int res_rows_number = 0;
int res_columns_number = 0;

...
result_matrix = read_matrix(file_matrix, &res_rows_number, &res_columns_number);
  1. Вы некорректно освобождаете память матрицы:
void free_matrix(double ** matrix) {
    delete[] matrix[0];
    //delete matrix; // некорректно
    delete[] matrix; // корректно
}

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

Note: See TracTickets for help on using tickets.