Opened 4 years ago

Closed 4 years ago

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

WW #14 Golovin Valery

Reported by: golovin.valeriy Owned by: Дмитрий Свиридкин
Component: WW_exceptions Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 4 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to golovin.valeriy
Type: ожидается проверкаожидаются исправления
src/matrix.cpp: In constructor ‘Matrix::Matrix(int, int)’:
src/matrix.cpp:8:19: error: catching polymorphic type ‘class std::bad_alloc’ by value [-Werror=catch-value=]
    8 |     } catch (std::bad_alloc) {
      |                   ^~~~~~~~~
src/matrix.cpp: In member function ‘void Matrix::readMatrix(const string&)’:
src/matrix.cpp:28:19: error: catching polymorphic type ‘class std::bad_alloc’ by value [-Werror=catch-value=]
   28 |     } catch (std::bad_alloc) {


src/main.cpp:44:34: error: catching polymorphic type ‘class MatrixException’ by value [-Werror=catch-value=]
   44 |         } catch (MatrixException except) {
      |                     

Сообщение об ошибке требовалось выводить в std::cout, а не cerr

Поле message в классе исключения должно быть приватным.

   std::ifstream instr;
    instr.open(filename);

Не стоит разделять инициализацию и создание переменной. У *fstream есть конструктор, принимающий путь к файлу и сразу же открывающий его. Метод close также явно можно не вызывать -- деструктор освободит ресурсы. Используйте преимущества RAII.


7.5

comment:2 Changed 4 years ago by golovin.valeriy

Owner: changed from golovin.valeriy to Дмитрий Свиридкин
Version: 1.02.0

comment:3 Changed 4 years ago by Дмитрий Свиридкин

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

10

Note: See TracTickets for help on using tickets.